0

У меня стоит задача зашифровать текст. Я считываю исходный текст из файла и текст шифра из файла. Считываю текст я в переменную типа Sting^, мне нужно сравнить символ из первой строки с каждым символом из стоки шифра.У меня не получается ходить по строке типа String^ с шифром, я пишу cipher[0], cipher[1] и тд. но компилятор выдает мне сообщение об ошибке, как решить эту проблему? Вот код:

private: System::Void button4_Click(System::Object^  sender, System::EventArgs^  e) {
    StreamReader^ fileCipher = gcnew StreamReader("cipher.txt");
    String^ cipher;
    int count = 0;
    while(fileCipher->ReadLine()){
        cipher = fileCipher->ReadLine();
    }
    String^ temp = gcnew String("nbc");
    if(cipher[0] == temp[0])
        MessageBox::Show("Yes!!!");
}

При компиляции при нажатии на кнопку вылетает ошибка:

Необработанное исключение типа "System.NullReferenceException" в WindowsFormsApplication1.exe

2
  • Вы не создаёте экземпляр строки cipher. В цикле делаете что-то странное. Вы бы хоть поотлаживали код перед тем, как писать здесь вопрос. / Зачем вообще вам C++ CLI? У вас проблемы с базовыми вещами. "Безопаности" таким способом вы не добьётесь, если это вас интересует.
    – Kyubey
    1 апр 2015 в 16:11
  • 1

2 ответа 2

1

Код не запускал, но простой построчный просмотр кода показывает, что из всех перечисленных в этом фрагменте переменных значение null может иметь только cipher (входящие параметры подразумеваем корректными). Она не инициализируется при создании, присваивается только внутри тела while, который может и не выполниться ни разу.

Нужны как минимум два исправления: 1. изначально инициализировать строку cipher пустой строкой, 2. после чтения проверить, что она не пустая (а то вместо null reference получите out of range)

3
  • Советы помогли в решении проблемы, спасибо! 1 апр 2015 в 16:32
  • Не за что. Кроме того, @Athari прав в следующем: чтобы найти точную строку, в которой произошла ошибка, всегда достаточно запустить отладку/debug. При возникновении исключения отладчик остановится точно в том месте, а дальше можете спокойно смотреть значения всех переменных и выяснять причину.
    – NIA
    2 апр 2015 в 8:28
  • @NIA иницилизировать пустой строкой не нужно. ReadLine() возвращает указатель на нужную строку. Проверять на null, в принципе, хорошая привычка, но при этом нужно понимать, что вообще происходит.
    – atwice
    2 апр 2015 в 14:53
0

Проблема тут:

 while(fileCipher->ReadLine()){
     cipher = fileCipher->ReadLine();
 }

При проверке условия цикла Вы считываете строку и не сохраняете её:

(fileCipher->ReadLine()).

Но строка не пустая, поэтому выполняется тело цикла: cipher = fileCipher->ReadLine(); Второй строки уже нет (в тестовом фале всего одна строка), поэтому в cipher возвращается null.

Дальше обращение к строке cipher[0] и вызывает NullPointerException.

Перепишите в стиле C (присваивание и проверка в условии цикла)

String^ temp = gcnew String("nbc");

while( String^ line = fileCipher->ReadLine() ) {
    if (line->Equals( temp ) ) {
        Console::WriteLine( "Yes!!!" );
    }
}

Ваш ответ

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

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