[C++]Esercizio sui file

[TheBestNapoli]

Salve a tutti, avrei questo programma da implementare sulla gestione di una ferramenta: in pratica il programma deve aprire un file "hardware.dat", mi deve permettere di inserire gli attrezzi, di visualizzarli tutti, di eliminarne quelli con quantità = 0, e di aggiornare qualsiasi campo del record Attrezzo.
Le funzioni eliminaAttrezzo e aggiornaAttrezzo le ho implementate e con numerose prove fatte ho verificato che funzionano bene... il problema sta nell'inserimento e nella stampa... vi posto il codice che ho scritto (evitanto di inserire le funzioni di elimina e aggiorna):

header.h

#ifndef HEADER_H_
#define HEADER_H_
#include <iostream>
#include <fstream>
#include <cstdlib>
#include <cstring>
#include <iomanip>

using namespace std;

struct Record{
	int codice;
	char nome[50];
	int quantita;
	float costo;
	char note[100];
};

void inserisciAttrezzo();
void stampaMagazzino();
void stampaAttrezzo(ostream &, const Record &);

#endif /* HEADER_H_ */

header.cpp

#include "header.h"

void inserisciAttrezzo()
{
	Record attrezzo;
	cout<<"Inserire codice attrezzo: ";
	cin>>attrezzo.codice;
	cout<<"Inserire nome attrezzo: ";
	cin.sync();
	cin.getline(attrezzo.nome, 50);
	cout<<"Inserire quantita' disponibile: ";
	cin>>attrezzo.quantita;
	cout<<"Inserire costo attrezzo: ";
	cin>>attrezzo.costo;
	if(attrezzo.quantita == 0)
	{
		strcpy(attrezzo.note,"Attrezzo esaurito in magazzino");
	}
	if(attrezzo.quantita >0 && attrezzo.quantita <= 20)
	{
		strcpy(attrezzo.note,"Attrezzo in quantita' sotto la soglia di magazzino");
	}
	if(attrezzo.quantita > 20 && attrezzo.quantita <= 50)
	{
		strcpy(attrezzo.note,"Attrezzo in quantita' sufficiente");
	}
	if(attrezzo.quantita > 50)
	{
		strcpy(attrezzo.note,"Attrezzo in quantita' piu' che sufficiente");
	}
	fstream scriviSuFile("hardware.dat");
	if(!scriviSuFile)
	{
		cerr<<"Errore\n";
	}
	scriviSuFile.seekp((attrezzo.codice-1)*sizeof(Record));
	scriviSuFile.write(reinterpret_cast<const char*>(&attrezzo), sizeof(Record));
}

void stampaMagazzino()
{
	fstream leggiDaFile("hardware.dat");
	if(!leggiDaFile)
	{
		cerr<<"Errore\n";
	}
	cout<<"\n"<<setiosflags(ios::left)<<setw(10)<<"N°"<<setw(15)<<"Nome"<<setw(10)<<"Quantita'"<<setw(10)<<"Costo"<<setw(50)<<"Note"<<endl;
	leggiDaFile.seekg(0);
	Record attrezzo;
	leggiDaFile.read(reinterpret_cast<char*>(&attrezzo), sizeof(Record));
	while(!leggiDaFile.eof())
	{
		if(attrezzo.codice != 0)
		{
			stampaAttrezzo(cout, attrezzo);
		}
		leggiDaFile.read(reinterpret_cast<char*>(&attrezzo), sizeof(Record));
	}
}

void stampaAttrezzo(ostream &output, const Record &r)
{
	output<<setiosflags(ios::left)<<setw(10)<<r.codice<<setw(15)<<r.nome<<setw(10)<<r.quantita<<setw(10)<<r.costo<<setw(50)<<r.note<<endl;
}

main.cpp

#include "header.h"

int main()
{
	int x;
	char s;
	do
	{
		cout<<"\nScegli un'opzione dal seguente menu':\n\n"
			<<"1) Inserisci un attrezzo nel magazzino\n"
			<<"2) Elenca gli attrezzi presenti in magazzino\n"
		cout<<"Inserisci opzione: ";
		cin>>x;
		cin.sync();
		switch(x)
		{
			case 1:
			{
				inserisciAttrezzo();
				break;
			}
			case 2:
			{
				stampaMagazzino();
				break;
			}
			default:
			{
				cout<<"Valore errato\n";
				break;
			}
		}
		cout<<"\nVuoi effettuare un'altra operazione? (S/N): ";
		cin>>s;
	}
	while(s=='s' || s=='S');
	cout<<"Fine Programma\n";
	return 0;
}

Allora gli errori che mi dà in output sono strani infatti:
1) inserisco il primo attrezzo, poi stampo e va tutto bene; inserisco il secondo attrezzo, poi stampo e va ancora bene; se invece inserisco il primo e poi subito il secondo senza stampare in mezzo, non mi stampa niente più
2) dopo avere inserito due attrezzi (con la stampa in mezzo), se ne inserisco un'altro succede il finimondo al momento della stampa
vi posto un output come esempio:

Scegli un'opzione dal seguente menu':

1) Inserisci un attrezzo nel magazzino
2) Elenca gli attrezzi presenti in magazzino

Inserisci opzione: 2
N°       Nome          Quantità   Costo     Note
16       Cacciavite    0          1.99      Attrezzo esaurito in magazzino
17       Martello      76         11.99     Attrezzo in quantita' piu' che sufficiente

Vuoi effettuare un'altra operazione? (S/N): s

Scegli un'opzione dal seguente menu':

1) Inserisci un attrezzo nel magazzino
2) Elenca gli attrezzi presenti in magazzino

Inserisci opzione: 1
Inserire codice attrezzo: 15
Inserire nome attrezzo: Trapano
Inserire quantità disponibile: 10
Inserire costo attrezzo: 21.99

Vuoi effettuare un'altra operazione? (S/N): s

Scegli un'opzione dal seguente menu':

1) Inserisci un attrezzo nel magazzino
2) Elenca gli attrezzi presenti in magazzino

Inserisci opzione: 2
N°        Nome           Quantità  Costo    Note
15        Trapano        10        21.99    Attrezzo in quantita' sotto la soglia di magazzino
1124073472acciavite      137573171211.997   ttrezzo esaurito in magazzino

Forse il problema sta nell'aver dichiarato oggetti per gestire il file con fstream("hardware.dat") senza specificare con ios::in o ios::out o qualcos'altro...
Qualcuno potrebbe aiutarmi? Grazie mille a tutti coloro che mi risponderanno

[Rggb1]

Beh, effettui una seek() per l'inserimento sulla base del codice, ma devi considerare che (forse) questa non va a buon fine.

[TheBestNapoli]

Intendi una seekp() in fase di scrittura? Quella serve per scrivere sul file in modo ordinato e devo ammettere che l'ho copiata uguale da un altro esercizio da cui ho preso spunto per comprendere bene l'uso dei file ad accesso sequenziale e casuale.
In realtà ho provato pure in un modo alternativo (ancora più simile all'esercizio da cui ho preso spunto) ma è sempre la stessa cosa:

void inserisciAttrezzo()
{
	Record attrezzo;
	int cod;
	cout<<"Inserire codice attrezzo: ";
	cin>>cod;
	cout<<"Inserire nome attrezzo: ";
	cin.sync();
	cin.getline(attrezzo.nome, 50);
	cout<<"Inserire quantita' disponibile: ";
	cin>>attrezzo.quantita;
	cout<<"Inserire costo attrezzo: ";
	cin>>attrezzo.costo;
	if(attrezzo.quantita == 0)
	{
		strcpy(attrezzo.note,"Attrezzo esaurito in magazzino");
	}
	if(attrezzo.quantita >0 && attrezzo.quantita <= 20)
	{
		strcpy(attrezzo.note,"Attrezzo in quantita' sotto la soglia di magazzino");
	}
	if(attrezzo.quantita > 20 && attrezzo.quantita <= 50)
	{
		strcpy(attrezzo.note,"Attrezzo in quantita' sufficiente");
	}
	if(attrezzo.quantita > 50)
	{
		strcpy(attrezzo.note,"Attrezzo in quantita' piu' che sufficiente");
	}
	attrezzo.codice = cod;
	fstream scriviSuFile("hardware.dat");
	if(!scriviSuFile)
	{
		cerr<<"Errore\n";
	}
	scriviSuFile.seekp((cod-1)*sizeof(Record));
	scriviSuFile.write(reinterpret_cast<const char*>(&attrezzo), sizeof(Record));
}

Potresti darmi una mano? Grazie mille

[Rggb1]

Ho fatto copiancolla del tuo codice e ricompilato, ma non riesco a riprodurre l'errore... Traduzione: a me funziona.

Magari stai lavorando sotto \$finestre\$? Potrebbe essere un problema (sic) del sistema operativo.

[apatriarca]

Non ho testato il codice, ma prova a scrivere

fstream leggiDaFile("hardware.dat", fstream::in | fstream::binary);

e in modo simile per il file in scrittura. fstream lavora infatti di default in modalità testuale (ma non in linux in cui utilizza la modalità binaria sempre) e quindi potrebbe fare alcune conversioni durante la lettura. Quando si lavora con file binari conviene SEMPRE inserire l'opzione in modo esplicito in quanto in caso contrario il comportamento dipenderà dalle convenzioni del sistema operativo.

[Rggb1]

@apatriarca
Hai probabilmente identificato il problema. In effetti davo per scontato utilizzasse la modalità binaria.

[TheBestNapoli]

Grazie mille ad entrambi, ho capito cosa intendete però avrei ancora un problema e un dubbio:
il dubbio è: fstream::in oppure fstream::binary equivalgono a ios::in o ios::binary?
Per quanto riguarda il problema, nella funzione stampaMagazzino() al posto di

fstream leggiDaFile("hardware.dat");

ho messo

fstream leggiDaFile("hardware.dat", fstream::in | fstream::binary);

ora però nella funzione inserisciAttrezzo() cosa metto affianco al nome del file? Ho provato con

fstream scriviSuFile("hardware.dat", fstream::out | fstream::binary);

ma ogni volta che inserisco un nuovo attrezzo mi cancella quello precedente... ho provato anche a sostituire out con ate o con app ma non funzionano proprio...
Mi dareste una mano? Grazie ancora.

[Rggb1]

Io direi di mettere:

fstream scriviSuFile("hardware.dat", fstream::in | fstream::out | fstream::binary);

Riprova e facci sapere.

[TheBestNapoli]

Grazieeee!!! Sei un grande! Ho sostituito la tua linea di codice anche nelle funzioni di elimina e aggiorna attrezzi e ora funziona tutto alla perfezione! Grazie ancora ciaoo