Как редактировать бинарный файл?

Question

[Vlad_beg]

Требуется отредактировать записанные в бинарный файл значения, знаю что нужно использовать seek, но как именно это организовать не могу понять.

Нужно отредактировать "flight_destination"

<code lang="cpp">
struct Flight_Details {
	char destination[99];
	char departure[99];
	char time_depart[80];
	char time_arrive[80];
	int flight_number;
};


int menu;
	
	do {
		cout << "***MENU***\n\n1. New Flight Option\n2. Output Flight Option\n3. Edit\n4. Exit\n\n";
		cout << "Enter menu element: ";
		cin >> menu;

		switch (menu)
		{
			case MENU::NEW_FLIGHT: {
				Flight_Details flight_d;
				cout << "Enter Departure: ";
				cin >> flight_d.destination;
				cout << "Enter Destination: ";
				cin >> flight_d.departure;
				cout << "Enter Departure Time: ";
				cin >> flight_d.time_depart;
				cout << "Please enter arriving time: ";
				cin >> flight_d.time_arrive;
				cout << "Flight Number: ";
				cin >> flight_d.flight_number;

				ofstream file;
				file.open("Flgiht_Details.data", ios::binary);

				if (!file) cout << "Could create/open file";
				else {
					file.write((char*)&flight_d, sizeof(flight_d));
					file.close();
				}
				break;
			}
			case MENU::OUTPUT_FILE: {

				ifstream readFile;
				readFile.open("Flgiht_Details.data");
				if (!readFile) cout << "Couldn't open file";
				else {
					readFile.seekg(0, ios::end);
					int fileSize = readFile.tellg();
					int countOfFlights = fileSize / sizeof(Flight_Details);
					readFile.seekg(0, ios::beg);

					Flight_Details* flight_d = new Flight_Details[countOfFlights];
					readFile.read((char*)flight_d, countOfFlights *sizeof(Flight_Details));
					readFile.close();

					for (int i = 0; i < countOfFlights; i++)
					{
						cout << flight_d[i].destination << "\n" << flight_d[i].departure << "\n" << flight_d[i].time_depart << "\n" << flight_d[i].time_arrive << "\n" << flight_d[i].flight_number << "\n\n";
					}
					break;
				}
			}
			
			case MENU::EDIT: {

				ifstream readFile;
				readFile.open("Flgiht_Details.data", ios::binary | ios::in || ios::out);
				if (!readFile) cout << "Couldn't open file";
				else {
					readFile.seekg(0, ios::end);
					int fileSize = readFile.tellg();
					int countOfFlights = fileSize / sizeof(Flight_Details);
					readFile.seekg(0, ios::beg);

					Flight_Details* flight_d = new Flight_Details[countOfFlights];
					readFile.read((char*)flight_d, countOfFlights * sizeof(Flight_Details));
					readFile.close();
					
					char edit[50];
					cin.ignore();
					cout << "Edit: ";
					cin.getline(edit, 50);
					
					
					ofstream file;
					file.open("Flgiht_Details.data", ios::binary | ios::in | ios::out); 
					
					for (int i = 0; i < countOfFlights; i++)
					{
						if (strcmp(flight_d[i].destination, edit) == 0)
						{
							
							strcpy_s(flight_d[i].destination, edit);
							readFile.seekg(i* sizeof(Flight_Details));
							file.write(reinterpret_cast<const char*>(&flight_d[i]),sizeof(Flight_Details));
							file.close();
							break;
						}
					}
				}
				break;
			}
</code>

Comments

[Adamos]

Просто замечу: в структуре заложена бомба, которая рванет при переходе с x32 на x64 или наоборот. Поскольку размер int на этих платформах разный.

[Дмитрий]

Adamos: в 99% архитектур размер int 32 бита

[Adamos]

Дмитрий: стандарт гарантирует "минимум 32 бита".
Но вы правы, конечно, бомба тут не так вероятна, как если бы был long.

Answer 1

[Mercury13]

Допустим, надо перезаписать i-й полёт.
Надо перескочить на i*sizeof(…), и там читать-писать. То есть

if (strcmp(flight_d[i].destination, edit) == 0)
            {
                   flight_d[i] = …;
                   file.open(…);
                   file.seekg(i * sizeof(…));
                   file.write(reinterpret_cast<const char*>(&flight_d[i]),
                                         sizeof(Flight_Details));
                   break;
            }

Да, и вы путаете глобальную переменную flight_d и локальную flight.

Comments

[Vlad_beg]

можете написать пожалуйста, как это должно выглядеть

[Mercury13]

Vlad_beg: написал.

[Vlad_beg]

в этом месте flight_d[i] = …; что происходит? i-тому элементу полета присваиваем новый элемент?

Я написал так:

strcpy_s(flight_d[i].destination, edit);
readFile.seekg(i * sizeof(flight_d));
file.write(reinterpret_cast<const char*>(&flight_d[i]),sizeof(Flight_Details));

Но, при запуске файл не изменяется

[Mercury13]

1. i * sizeof(flight_d) — известная ошибка, второе равняется размеру указателя.
2. Повторяю, у вас код неполный, но на вид спутаны глобальная flight_d и локальная flight.

[Mercury13]

3. Открыть надо не просто для записи, а для изменения.

[Mercury13]

Vlad_beg:
> 3. Открыть надо не просто для записи, а для изменения.
То есть вторым параметров ios::read | ios::write.

[Vlad_beg]

Обновил код, не могли бы взглянуть, что не так

[Mercury13]

Vlad_beg:

Пока вижду это.

file.open("Flgiht_Details.data", ios::binary | ios::in | ios::out); 
…
readFile.seekp(i* sizeof(Flight_details));

[Vlad_beg]

а вместо ios::read | ios::write не должно быть так

file.open("Flgiht_Details.data", ios::binary | ios::in | ios::out);

[Mercury13]

Vlad_beg: Точно, in/out, а не read/write.

[Vlad_beg]

да вот только все равно никаких изменений