Шифр Цезаря: как рассчитать при сдвиге значения> 10 (или больше)? - PullRequest
Новая
       12

Шифр Цезаря: как рассчитать при сдвиге значения> 10 (или больше)?

0 голосов
/ 22 февраля 2019

Как я знаю, "формула" Смещения Цезаря - это (x + k)% 26, где k - значение сдвига, а расшифровка просто заменяет "+" на "-".

, но мой кодне работает, когда k> 10 (после того, как я проверил k = 10, я обнаружил, что «сдвиг» первых нескольких символов неправильный, поэтому я считаю, что k> 10 также будет неправильным (количество неправильных символов увеличивается).)Сначала я изменяю символы на ASCII, а затем делаю вычисления.Наконец, измените его обратно на символы.

Вот мой код. 

#include <iostream>
#include <string>
using namespace std;
int main() {
    string target;
    char s;
    int k, i, num, length, j;
    cin >> s >> k;
    getline(cin, target);

    for (j = 0; j <= (int)target.length(); j++) {
        if ((target[j]) = ' ') {
            target.erase(j, 1);
        }
    }

    length = (int)target.length();

    if (s == 'e') {
        for (num = 0; num <= length; num++) {

            if (isupper(target[num]))
                target[num] = tolower(char(int(target[num] + k - 65) % 26 + 65));
            else if (islower(target[num]))
                target[num] = toupper(char(int(target[num] + k - 97) % 26 + 97));
        }
    }
    else if (s == 'd') {
        for (num = 0; num <= length; num++) {
            if (isupper(target[num]))
                target[num] = tolower(char(int(target[num] - k - 65) % 26 + 65));
            else if (islower(target[num]))
                target[num] = toupper(char(int(target[num] - k - 97) % 26 + 97));
            }
        }
    cout << target;
    return 0;
}

Позвольте мне записать случай, который мне не удалось запустить.

input:

d 10 n 3 V 3 D 3 N _ M Y N 3 _ S C _ N 3 L E (сначала введите значение d / e, затем значение смещения, наконец, последовательность строк, которую необходимо «изменить», необходимо удалить пробел.)

ожидаемый результат:

D3l3t3d_cod3_is_d3bu

мой вывод:

D3l3:3d_cod3_i9_d3b; Спасибо!

1 Ответ

0 голосов
/ 22 февраля 2019

Ваша проблема в том, что при декодировании вы получаете отрицательные числа.С k == 13 выражение 'T' - k - 65 дает -7.-7 % 26 по-прежнему -7.-7+65 - это 58, что не является буквой.

Вы можете избежать отрицательных чисел, просто установив k в 26 - k при декодировании.

Ваш код затем упрощается до: 

if (s == 'd') {
    k = 26 - k;
}
for (num = 0; num <= length; num++) {

    if (isupper(target[num]))
        target[num] = tolower(char(int(target[num] + k - 'A') % 26 + 'A'));
    else if (islower(target[num]))
        target[num] = toupper(char(int(target[num] + k - 'a') % 26 + 'a'));
}

Примечание. Я заменил ваши целочисленные константы их эквивалентными символами, что значительно облегчает понимание кода.Обратите внимание, что у вас также есть ошибка в вашем первом цикле (target[j]) = ' ' должно быть (target[j]) == ' '.

Используя все предложения c ++, вы можете уменьшить свой код до: 

#include <iostream>
#include <string>
#include <algorithm>

int main() {
    std::string target = "mXLM";
    char s = 'e';
    int k = 7;

    target.erase(std::remove(target.begin(), target.end(), ' '), target.end());

    if (s == 'd') {
        k = 26 - k;
    }
    std::string result;
    std::transform(target.begin(), target.end(), std::back_inserter(result), [k](char in) {
        if (isalpha(in)) {
            char inputOffset = isupper(in) ? 'A' : 'a';
            char outputOffset = isupper(in) ? 'a' : 'A';
            return char(int(in + k - inputOffset) % 26 + outputOffset);
        }
        return in;
    });
    std::cout << result;
    return 0;
}
...