fstream && struct

[Hacker19_90]

доброго времени суток!
Возникла проблемка при записи структур в файл
пишу так

ofstream fout(filename.c_str(), ofstream::binary);
	cout << "count of books?" << endl;
	cin >> count;
	if (fout)
		for(int i = 0; i < count; i++)
		{
			cout << "#" << i+1 << " book:" << endl;
			cout << "name of book: " << endl;
			cin >> book1.name;
			cout << "special of book" << endl;
			cin >> book1.special;
		  fout.write((char*)&book1, sizeof(book));
		  fout.flush();
		}
	fout.close();

читаю так

ifstream fin(filename.c_str(), ifstream::binary);
	fin.seekg(0, ifstream::beg);
	if (fin)
		while(!fin.eof())
		{	 
       fin.read((char*) &temp, sizeof(book));
		 cout << temp.name << "\t" << temp.special << endl;
		}
	fin.close();

структура

struct book
{
	char name[21];
	char special[11];
};

Вопрос:
Почему при считывании у меня последняя структура дважды выводится?
Я ошибся где то при вводе или выводе?
Что я не усмотрел?
Всем спасибо!

[Hacker19_90]

И так! я переписал проект и собствеено выкладываю его вам весь!
вопрос тот же почему не срабатывает eof
Хотя размер файла делёный на размер структуры равен количеству записанных мной структур!
код

#include <fstream>
#include <iostream>
#include <cstdlib>

using namespace std;

struct somestruct {
	char str[11];
	int i;
};

int main () {
	somestruct ss;
	char fname [] ="somefile.dat";
	ofstream fout(fname, ios::binary);
	cout << "input: " << endl;
	if (fout)
		for (int j = 0; j < 4; j++) {
			cout << "#" << j << endl;
			cin >> ss.str >> ss.i;
			fout.write((char*) &ss, sizeof(somestruct));
		}
	fout.close();
	cout << "end input " << endl;
	system("pause");
	system("cls");
	ifstream fin(fname, ios::binary);
	if (fin)
		while (!fin.eof()) {
			fin.read((char*) &ss, sizeof(somestruct));
			cout << ss.str << "\t" << ss.i << endl;
		}
	fin.close();
	cin.ignore();
	cin.get();
	return 0;
}

вложения
скрины и архив с ехе и исходником

[zvoronz]

Вот в этом месте

while (!fin.eof()) {
			fin.read((char*) &ss, sizeof(somestruct));
			cout << ss.str << "\t" << ss.i << endl;
		}

Всегда делал вот так

while (fin.read((char*) &ss, sizeof(somestruct)) && !fin.eof()) {			
			cout << ss.str << "\t" << ss.i << endl;
		}

и всё работало.

[Hacker19_90]

спасибо конечно!
Но более интересно почему именно eof сама по себе не срабатывает?
Просто как это обойти я знаю. Но вот почему dont working? i dont know!

[zvoronz]

Если с английским дружите

If the End-of-File is reached before n characters have been read, the array will contain all the elements read until it, and the failbit and eofbit will be set (which can be checked with members fail and eof respectively).

То есть пока вы не попытаетесь прочитать дальше конца файла флаг eofbit не будет установлен.

[Hacker19_90]

Господи родной! Да где же вы раньше то были?
+1 ВАМ ОДНОЗНАЧНО!
Теперь то всё понятно!
Получается после прочтения последней записи, eof true не возращает поэтому получается ещё один холостой проход цикла в котором собствено read устанавлиается флаг, ну а так как прочитать уже ничего не можем старая структура и выводится!
ХЕХЕ как всё просто! ЕЩЁ РАЗ СПАСИБО!
Тему можно закрывать!

[_Bers]

Один момент:

while (fin.read((char*) &ss, sizeof(somestruct)) && !fin.eof()) {

Булевые операции есть "ленивые". Рассмотрим следующую ситуацию:

if( Foo()==1 && Bar()==1) { /*что то делаем*/ }

Булевое "и" есть истина только и только в том случае, если и Foo()==1 и Bar()==1

Если же Foo() вернёт не_единицу, то условие уже априори false. Не зависимо от того, что вернет второй операнд. А посему, он даже и не будет вычисляться (а зачем делать лишьние движения?)

Теперь представим, что Foo() очень тяжелая функция, а Bar() выполняется почти мгновенно. В этом случае, если переставить в булевом выражении функции местами, то смысл не изменится, а производительность резко вырастит.

В действительности, если легкая Bar() вернёт false, то тяжелая Foo() даже не будет запущена, что сэкономит и время и память.

Итого: есть смысл ставить более легкие процедуры в левой части булевых выражений:

while (fin.read((char*) &ss, sizeof(somestruct)) && !fin.eof()) {

fin.eof() тупо возвращает флаг.
fin.read() обращается к системе, и задействует тяжелую операцию чтения из файла.

Если поменять их местами - смысл не изменится, но программулька будет меньше жрать памяти и быстрее бегать.

[asmars]

Берс, есть похожая уловка..
этот код будет менее эффективней работать

for(i=0; i<sizeof(str); i++){
/* do something */
}

чем этот

t=sizeof(str);
for(i=0; i<t; i++){
/* do something */
}

[_Bers]

Сообщение от asmars

Берс, есть похожая уловка..
этот код будет менее эффективней работать

for(i=0; i<sizeof(str); i++){
/* do something */
}

чем этот

t=sizeof(str);
for(i=0; i<t; i++){
/* do something */
}

Ты ошибаешься. sizeof вычисляет времени компиляции.
В обоих твоих случаях результат будет одинаковый.

[asmars]

хм..
а если вместо sizeof поставить strlen ?