Ошибка функции с EOF

Question

У меня есть функция, которая должна читать символ файла char и выводить на консоль, пока не достигнет EOF. Функция отлично работает, отображая все нужные мне символы, но затем она просто продолжает двигаться и никогда не останавливается в конце.

Ниже функция

void displayAllLines(ifstream &joke) 
{ 
    char ch; 
    joke.clear(); 
    ch = joke.get(); 
    while (ch != EOF) 
    { 
     cout << ch; 
     ch = joke.get(); 
    } 
} 

Содержимое файла перечислены ниже

"(пустая строка) Q: Что язык сказать зубы?"

Как я уже говорил, он отображает линию просто отлично, но цикл продолжает идти после вопросительного знака.

Любые мысли о том, почему это может быть? Кроме того, больше информации, необходимой для правильного ответа на мой вопрос?

Answer 1

канонический способ сделать это:

void displayAllLines(ifstream &joke) 
{ 
    char ch; 
    while (true) 
    { 
     ch = joke.get(); 
     if (joke) { 
      cout << ch; 
     } else { 
      break; 
     } 
    } 
} 

Выражение joke будет вычисляться правда, пока не достигнет EOF. Я не уверен, почему вы хотите сначала позвонить joke.clear().

Answer 2

@Phil ответил «как это сделать на C++». Мой ответ гласит: «Почему это происходит» и «как это сделать в C» (он все еще работает на C++).

---

Поскольку таблица ASCII проходит весь путь от 1 и мы имеем значение байта 0 ('\0') зарезервирован для терминации строки (языки говорить C и C++), то EOF должно быть особое значение вне из этого.

Именно поэтому getchar(), а также ifstream::get() return a int, а не char. Так что все, что вам нужно сделать, это изменить

char ch; 

в

int ch; 

и брось на символ для печати:

cout << (char) ch; 

Технически EOF обычно -1. Ваш char может иметь дальность 0 - 255, что означает, что он никогда не будет равен -1. Хороший компилятор с включенным -Wall должен дать вам хотя бы предупреждение.