0

Второй день пытаюсь сделать шифрование Xor'ом, но все никак не получается. Вот мой код. Что у меня не так?

#include "stdafx.h"
#include <iostream>
#include <Windows.h>
#include <string>

using namespace std;

int main()
{
SetConsoleCP(1251);
    SetConsoleOutputCP(1251);


int key;
char text[100];
string res;

cout << "Введите текст: " << endl; // ввожу текст
cin >> text;

cout << "Введите ключ: " << endl;
cin >> key; // ввожу ключ


for (int i = 0; i < strlen(text); i++) { // подгоняю размер
    res[i] = text[i] ^ key; //само шифрование

}
cout << "Зашифрованный текст: " << res << endl;

return 0;
}
6
  • Что именно не получается? Кроме проверки соответствия длины ключа и сообщения, криминала не вижу. Ну и под res память бы выделить 14 фев 2018 в 15:57
  • @AntonShchyrov Ну вот допустим если я введу любое слово и ключ, вылазит ошибка Line:2954, Expression: string subscript out of range
    – P1NG2WIN
    14 фев 2018 в 15:59
  • 2
    Я же сказал - память под string нужно выделить 14 фев 2018 в 16:03
  • Для начала расскажите, какую Вы задачу решаете? Какая у Вас длина ключа? Зачем вам тип string. И нужно ли Вам одновременно хранить исходную строку и зашифрованную? 14 фев 2018 в 16:14
  • @AntonShchyrov А как выделить динамическое кол во памяти? Я сделал string *res = new string; , но оно выдает ошибку "Вызвано исключение: нарушение доступа для записи. _Left было (адрес)"
    – P1NG2WIN
    14 фев 2018 в 16:14

2 ответа 2

2

Как уже сказали, у тебя строка res нулевой длины, т.е. строка есть, а память под симполы не выделено. Прежде чем обратиться в i_тому символу, надо чтоб этот символ был. Т.е. надо проинициализировать строку res, например, так:

....
char text[100];

cout << "Введите текст: " << endl; // ввожу текст
cin >> text;
string res(test);
...

Теперь у тебя строка res эта полная копия text. Но тут правильно заметили, если тебе не нужно хранить оригинальную строку, то можешь прямое ее и модифицировать вот так:

...
for (int i = 0; i < strlen(text); i++) { // подгоняю размер
    text[i] = text[i] ^ key; //само шифрование

}
cout << "Зашифрованный текст: " << text << endl;
...

Но и тут есть подводные камни. Если ты введешь ключ который совпадает с ASCII кодом символа из строки, то после операции XOR на месте это символа окажется \0, что является признаком конца троки. Тогда std::cout распечатает зашифрованную строку только до этого символа и у тебя будет навсегда потерям оригинальный размер строки.

1
  • Спасибо за объяснение! @ffk
    – P1NG2WIN
    16 фев 2018 в 16:17
1
std::string key;
std::string str;

std::cout << "Введите текст: " << std::endl;
std::getline(std::cin, str); // вводим текст. Память выделяется автоматически

std::cout << "Введите ключ: " << std::endl;
std::getline(std::cin, key);  // вводим ключ

int keyIdx = 0;  // текущий индекс байта ключа
for (int i = 0; i < str.size(); i++) {
  str[i] = str[i] ^ key[keyIdx++]; // шифруем строку и смещаем индекс ключа
  if (keyIdx == key.size())  // если достигли конца ключа - начинаем сначала
    keyIdx = 0;
}
std::cout << "Зашифрованный текст: " << str << std::endl;

Тип std::string может содержать \0 внутри строки

11
  • И за что минус? 15 фев 2018 в 12:53
  • Спасибо за пример! @Anton Shchyrov
    – P1NG2WIN
    16 фев 2018 в 16:17
  • А завершение шифрования раньше конца строки из за \0 возможно пофиксить?
    – P1NG2WIN
    18 фев 2018 в 12:19
  • @P1NG2WIN Тип string этой проблеме не подвержен. Там размер задается отдельно. Проблема актуальна только для char* 18 фев 2018 в 12:23
  • Но ты же сам пишешь что он может содержать \0 и вот даже с твоим кодом у меня что-то не получается [ imgur.com/a/iAKLs ] + там где английским у меня почему то не вставлялся текст
    – P1NG2WIN
    18 фев 2018 в 12:44

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.