Функции и файлы (С++)

[Артём Волжанкин]

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

Для хранения данных описать структуру (описана в соответствии с заданием).
Написать функцию для записи в бинаный файл данных из структуры.
Структура файла:
в первых двух байтах значение типа int, определяющее количество сделанных записей.

Написать функцию, которая извлекает данный из файла.

#include<fstream>
#include<iostream>
#include<string.h>
#include<string>
#include<stdio.h>
#include<stdlib.h>
#include<fstream>
using namespace std;

struct scan_info{
		char model[25]; 
		int price;
		double x_size;
		double y_size;
		int optr;
		int grey;
	};

int zapolnenie(int* l, scan_info mas[]){
	int k=*l;
	cout << "Заполнение структуры\n";
	cout << "Введите название модели\n";
	cin >> mas[*l].model;
	cout << "Введите цену\n";
	cin >> mas[*l].price;
	cout << "Горизонтальный размер области сканирования\n";
	cin >> mas[*l].x_size;
	cout << "Вертикальный размер области сканирования\n";
	cin >> mas[*l].y_size;
	cout << "Оптическое разрешение\n";
	cin >> mas[*l].optr;
	cout << "Число градаций серого\n";
	cin >> mas[*l].grey;
	cout << "Введите 1 для выхода или 0 - для продожения \n";
	int par;
	k++;
	cin >> par;
	if (par == 1) 
		return k; 
	else { 
		zapolnenie(&k, mas);
	}

}

int zapisVFail(int kol, scan_info mas[]){
	cout << "Зашли в функцию zapisVFail\n";
	cout << "Количество проходов\n" << kol << "\n";
	fstream f("C:\\cpp\\text1.txt", ios_base::in);
	if (!f.is_open()) {
		cout << "Невозможно открыть файл";
		return -1;
	}
		f << kol;

		for (int i=0; i<kol; i++){
		f << mas[i].model;
		f << mas[i].price;
		f << mas[i].x_size;
		f << mas[i].y_size;
		f << mas[i].optr;
		f << mas[i].grey;
	}
		f.close();
		return 0;
}

int shtenieIzFaila(scan_info mas[]){
	cout << "Зашли в функцию shtenieIzFaila\n";
	fstream f("C:\\cpp\\text1.txt", ios_base::out);
	if (!f.is_open()) {
		cout << "Невозможно открыть файл\n";
		return -1;
	}
		int kol;
		f >> kol;
		for (int i=0; i<kol; i++){
		f >> mas[i].model;
	
		f >> mas[i].price;

		f >> mas[i].x_size;
	
		f >> mas[i].y_size;
		
		f >> mas[i].optr;
		
		f >> mas[i].grey;
		
		}
	
	f.close();
	return 0;

}

void main(){

	int m=0;					// первая запись для функции
	int *p=&m;					// создаем указатель для передачи в функцию
	scan_info mas[8];			// массив структур по заданию
	scan_info *pa=mas;			// указатель на массив для передачи в функцию
	m=zapolnenie(&m, pa);		// вызываем функцию заполнения и сохраняем в м количество заполнений
	cout << "Записываем в файл массив\n";
	zapisVFail(m, pa);			// записываем в файл массив
	cout << "Чтение из файла в структуру\n";
	shtenieIzFaila(pa);			//чтение из файла в структуру
	cout << "Вывод структуры на экран\n";
	for (int i=0; i<m; i++){
		cout << "-----------------------------------\n";
		cout << "         " << i+1 << "\n";
		cout << mas[i].model << "\n";
		cout << mas[i].price << "\n";
		cout << mas[i].x_size << "\n";
		cout << mas[i].y_size << "\n";
		cout << mas[i].optr << "\n";
		cout << mas[i].grey << "\n";
	}



	system("PAUSE");
}

[Stilet]

int* l

Зачем указатель? Тут без него можно.

zapisVFail(m, pa); // записываем в файл массив

А где проверка на ошибку? У тебя предусмотренно в случае ошибки результат -1, почему не проверяешь?

shtenieIzFaila(pa); //чтение из файла в структуру

Аналогично. Зачем делать обработчики ошибок, если не используешь их?

else {
zapolnenie(&k, mas);
}

Почему рекурсией? Чем простой цикл не устроил?

[Артём Волжанкин]

Спасибо за ответ.

Зачем указатель? Тут без него можно.

Мне надо было чтобы функция меняла значение m.

Проверки действительно не организовал.

Почему рекурсией? Чем простой цикл не устроил?

Что-то в голову стукнуло.
Если циклом, то нужно передавать в функцию колличество элементов массива

[Last]

Сообщение от Артём Волжанкин

Мне надо было чтобы функция меняла значение m.

Можно передавать значение по ссылке:

func(&m);

[waleri]

Сообщение от Артём Волжанкин

Структура файла:
в первых двух байтах значение типа int, определяющее количество сделанных записей.

Мне чисто любопытно - структура файла тоже описана в задании или каждый придумывает сам?

[Артём Волжанкин]

Мне чисто любопытно - структура файла тоже описана в задании или каждый придумывает сам?

В задание.