Проблема с работой с файлами Си

Question

[Евгений Игнатик]

Товарищи! Есть просто очень большой вопрос:
В общем, задание "Разработать программу для заполнения двоичного файла целыми числами в интервале [-150; 150]. Количество компонент файла вводится с клавиатуры. Обнулить компоненты файла (прямо в файле) с максимальным значением (таких может быть несколько)." Написал, но почему-то, когда хочу заменить числа совпадающие с максимумом цикл уходит в бесконечность. Продолжает читать файл, причем сначала и пишет его, добавляя все новые порции, а EOF никогда не достигает (поставил комментарий возле проблемного места). Самое интересное, что как я не парюсь, всегда либо при сравнении он перепрыгнет через число, либо наоборот зациклится. В общем, ломаю голову уже долго... Буду очень признателен, если поможете разобраться, что за злопакость творится :)

p.s: может, это какие-то особенности режима доступа к файлу, о которых я не знаю?

#include "stdafx.h"
#include <stdlib.h>
#include <stdio.h>
#include <time.h>
#include <iostream>
using namespace std;

int main()
{
	FILE *f;
	int i, size, max, iMax, val, maxIndex, datVar;
	setlocale(LC_CTYPE, "Russian");
	srand(time(NULL));
	for (;;){
		cout << "Введите количество компонент: ";
		scanf("%d", &size);
		if (size>0) break;
		else cout << "Неправильное количество компонент. Повторите ввод: ";
	}
	f = fopen("file1.bin", "wb");
	for (i = 0; i < size; i++){ //запись в файл рандомных чисел
		datVar = rand() % 5;
		fwrite(&datVar, sizeof(int), 1, f);
		cout << datVar << " ";
	}
	
	cout << "\n";
	fclose(f);
	
	f = fopen("file1.bin", "r+b");
	fseek(f, 0, SEEK_SET);
	fread(&val, sizeof(int), 1, f); //считываем в файле int'ы
	max = val;
	while (fread(&val, sizeof(int), 1, f)!=0){
		if (val >= max) { 
			max = val; 
		}	
	}
	int nulVar = 0;
	
	int ft = 0;
	fseek(f, 0, SEEK_END);
	int sizeF = ftell(f);
	fseek(f, 0, SEEK_SET);
	while (fread(&val, sizeof(int), 1, f) > 0)
	{
		if (val == max)
		{
			fseek(f, ftell(f) - sizeof(int), SEEK_SET);
			fwrite(&nulVar, sizeof(int), 1, f);
			fflush(f);//ВОТ РЕШЕНИЕ
		}
		
		if (ftell(f) == sizeF) break;
	}
	fseek(f, 0, SEEK_SET);
	while (fread(&val, sizeof(int), 1, f) > 0)printf("%d ", val);
	fclose(f);
	cout << "\n";
	system("pause");
	return 1;
}

Answer 1

[Евгений Игнатик]

Проблема решилась добавление fflush(f); после fwrite. Просто fwrite не переводил курсор в файле и из-за этого начинались проблемы при последующих манипуляциях с курсором. А fflush(); сразу скидывает буфер в файл и перемещает курсор. В общем, всё оказалось проще.

Answer 2

[Sergey Zabodalov]

я бы сделал так:

while (fread(&val, sizeof(int), 1, f) > 0) // EOF == -1
{
if (val == max)
{
ft= ftell(f);
fseek(f, ft-sizeof(int), SEEK_SET); // отмотали на int назад
fwrite(&nulVar, sizeof(int), 1, f); // записали - теперь мы там же, где до ифа
}

Comments

[Евгений Игнатик]

Не прокатило, всё тоже самое. Проверял fread в конце файла возвращает 0, а не -1, как ни странно...

[Лолшто]

The fread function returns the number of elements successfully read, which may be less than nmemb if a read error or end-of-file is encountered. Так что 0 тоже означает EOF (www.open-std.org/jtc1/sc22/wg14/www/docs/n1124.pdf)

[Sergey Zabodalov]

тогда попробуй вместо while (fread) поделить длину файла на sizeof(int) и сделать for. Он точно не залипнет.

Answer 3

[Миша Кринкин]

1. Вам отступы расставлять религия не позволяет?
2. У меня программа нормально работает и я не вижу причин почему не должна. Попробуйте проверить что возвращают fseek, ftell и fwrite, теоретически они тоже могут закончиться с ошибкой.
3. Если код на C, то пишите на C, зачем вы подключаете C++ заголовки?

Comments

[Евгений Игнатик]

1. форматирование почему-то не сохранилось
2. программа заработала после добавления fflush(f) после write
3. требования на практике в уч.заведении настолько суровые...

[Миша Кринкин]

@eignatik догадываюсь почему оно не сохранилось, но поправить его руками дело нескольких секунд. По-поводу, вашего решения, если дело действительно в этом, то, по-моему, это повод сделать багрепорт, в стандарте четко и ясно говорится о том, что fwrite при удачном завершении должен передвинуть курсор файла.

[Миша Кринкин]

@eignatik в измененном коде кроме fflush добавилась еще проверка, так что подозреваю, что не в fflush дело вовсе.

[Евгений Игнатик]

@kmu1990 fflush добавил после того, как добавил проверку, проверку просто убрать забыл, а боьше ничего не менял