Проблема с программой чтения строк файла.

[CoLT]

Здравствуйте. Я недавно начал изучать программирование C++ по книге.
Там приводится листинг программы, которая должна открыть файл, считать его строки и вывести их на экран. Вот листинг

#include <fstream>
#include <vector>
using namespace std;
int main() {
	char buff[4];
	vector<string> s;
	ifstream infile;
	infile.open("Text");
	while(1) {
		infile.getline(buff, sizeof(buff));
		if(infile.eof()) break;
		s.push_back(buff);
	}
	sort(s.begin(),s.end());
	for(int i=0; i<s.size(); i++)
		cout << s[i] << endl;
	infile.close();
	system("pause");
}

Но почему-то цикл while не прекращается, а продолжается бесконечно (проверил вставкой в цикл cout << 1; так он эту единицу мне писал, пока я программу не закрыл). Я подозреваю, что что-то не так с функцией eof() (книжка 2005 года, мог измениться язык).
Вобщем подскажите пожалуйста)))

В файле Text записано "text"

[Stilet]

	while(!infile.eof()) {
		infile.getline(buff, sizeof(buff));
		s.push_back(buff);
	}

Не?

[CoLT]

Нет, всё равно не выходит, цикл не завершается

[Stilet]

Файл можешь приложить?

[CoLT]

Прилагаю файл

[Stilet]

infile.open("Text");

А файл у тебя "Text.txt"
Ничего не смущает?

[CoLT]

Ок, я дописал расширение, но всё равно глухо... Может дело в кодировке?

[Stilet]

Не в курсе что ты там делаешь, но у меня вот такое работает:

	char buff[4];
	vector<string> s;
	ifstream infile("d:\\1\\Text.txt",ios::binary);
	while(!infile.eof()) {
		infile.read(buff,4);
		s.push_back(buff);
	}
	infile.close();
	system("pause");
	return 0;

Рули отладчиком, где-то ты еще что-то не учел.

[CoLT]

У меня заработало вот такое

infile.open ("Text.txt");
while(1) {
		infile.read(buff, sizeof(buff));
		if(infile.eof()) break;
		s.push_back(buff);
	}
	sort (s.begin(), s.end());
	for(int i=0; i<s.size(); i++)
		cout << s[i] << endl;
	infile.close();
	system("pause");
	return 0;

Но после "text" пишутся всякие кракозябры. В чём может быть проблема?
И почему не работает функция getline() ?

[kaljan775]

кракозябры потому, что вы выделили массив символов на 4, и когда вы выводите через cout<<, cout ищет завершающий символ ( который \0 ), не находит, и идет дальше по памяти. то есть по сути у вас тут выход за границы массива. Для того чтобы такой дребедени не происходило, Вам надо выделить на 1 символ больше и явно сделать его \0 вот так

    buff[strlen(buff)-1] = '\0'; // или  buff[strlen(buff)-1] = 0;

и при чтении выдавать на 1 символ меньше

infile.getline(buff, strlen(buff)-1);

Разумеется, я могу быть не прав, потому что на С++ не программировал уже полгода, но, все-таки, проверьте, мне самому интересно