Fscanf c ошибка

Код считывает массив структур из файла. Но в функции main, почему то выводит мусор. Как это можно исправить? (В файле данные расположены так: «Солнечный,Самара,Москва,пеший,4,29»):

#include <stdio.h>
#include <stdlib.h>
#include <locale>
#include <conio.h>


struct Routes
{
	char title[32];
	char start[32];
	char finish[32];
	char type[32];
	int complexity;
	int duration;
}; // 6 записей

int size = 0;
Routes* arr = (Routes*)malloc(size * sizeof(Routes));

// Загрузить данные из файла
void get_datas(Routes *arr, int* size)
{
	FILE* file;
	file = fopen("datas.txt", "r");
	if (file == NULL) { printf("Ошибка открытия файла"); return; }
	int count = *size;
	int record = *size;
	int read; // Кол-во записей
	do
	{
		count++;
		arr = (Routes*)realloc(arr, count * sizeof(Routes));

		read = fscanf(file, "%32[^,],%32[^,],%32[^,],%32[^,],%d,%d\n",
			arr[record].title,
			arr[record].start,
			arr[record].finish,
			arr[record].type,
			&arr[record].complexity,
			&arr[record].duration
			);

		if (read == 6) record++;
		if (read != 6 && !feof(file)) return; // Неправильный формат файла
		if (ferror(file)) return; // Если произойдет ошибка
	} while (!feof(file));
	fclose(file);
	*size = count;
}

int main()
{
	setlocale(LC_ALL, "ru");
	printf("%d\n", size);
	get_datas(arr, &size);

	printf("%d\n", size);

	for (int i = 0; i < size; i++)
	{
		printf("%s %s %s %s %d %d", arr[i].title,
			arr[i].start,
			arr[i].finish,
			arr[i].type,
			arr[i].complexity,
			arr[i].duration);
	}

	free(arr);
}

У вашего цикла есть несколько проблем. Вы написали:

while( fscanf( f, "%[^\n\r]s", cLine ) != EOF ) 
    /* do something */;

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

1. fscanf () возвращает количество сохраненных элементов. Он может вернуть EOF, если он читает после конца файла или если дескриптор файла содержит ошибку. Вам нужно отличать действительный возврат нуля, когда в буфере cLine нет нового содержимого, от успешного чтения.

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

3. Написанный вами шаблон, вероятно, не соответствует вашим задумкам. Он сопоставляет любое количество символов, не являющихся CR или LF, а затем ожидает найти литерал s.

4. Вы не защитили свой буфер от переполнения. Любое количество символов может быть прочитано из файла и записано в буфер, независимо от размера, выделенного для этого буфера. Это, к сожалению, распространенная ошибка, которая во многих случаях может быть использована злоумышленником для запуска произвольного кода по выбору злоумышленников.

5. Если вы специально не запросили, чтобы f был открыт в двоичном режиме, перевод концов строки будет происходить в библиотеке, и вы, как правило, никогда не увидите символы CR, и обычно не в текстовых файлах.

Вероятно, вам нужен цикл, похожий на следующий:

while(fgets(cLine, N_CLINE, f)) {
    /* do something */ ;
}

где N_CLINE — количество байтов, доступных в буфере, начиная с cLine.

Функция fgets() — наиболее предпочтительный способ чтения строки из файла. Его второй параметр — это размер буфера, и он читает из файла в буфер на 1 байт меньше этого размера. Он всегда завершает буфер символом nul, чтобы его можно было безопасно передать другим строковым функциям C.

Он останавливается при чтении первого из конца файла, новой строки или buffer_size-1 байтов.

Он оставляет символ новой строки в буфере, и этот факт позволяет вам отличить одну строку длиннее, чем ваш буфер, от строки, короче, чем буфер.

Он возвращает NULL, если байты не были скопированы из-за конца файла или ошибки, а в противном случае — указатель на буфер. Вы можете использовать feof() и / или ferror(), чтобы различать эти случаи.

person
RBerteig
  
schedule
14.05.2009