C/C++ - Non legge While - Piero Tofy.it

Forum - C/C++ - Non legge While

oliverissimo (Normal User)
Newbie

Messaggi: 11
Iscritto: 14/05/2010

Postato alle 1:55
Venerdì, 18/06/2010

Ciao a tutti, eccomi di nuovo con uno dei miei problemi

Questo e' una funzione che stampa due file .txt e chiede se vuoi sovrascrivere "fileo" con "filed", in caso che la risposta sia affermativa creo un file di output "new_file" con il nome di imprse.txt (lo stesso di fileo) in modo da sostituirlo e poi sovrascrivere "new_file" con quello che c'e' dentro a "filed".
Non riesco pero' a capire perche' non entra dentro al while (vedi "<--").

Premetto anche che nei file CI SONO SICURAMENTE CARATTERI quindi il problema non sta' che in filed.get(c) non c'e' nulla da leggere creando una condizione del while falsa. :nono:

Ecco il codice:

Codice sorgente - presumibilmente C++

void stampa_dati ( char file_originale[11], char buffer_depurato[7] ) 
{
        char yon;
        char c;
        
        ifstream fileo;
        ifstream filed;
        
        fileo.open(file_originale);
        filed.open(buffer_depurato);
        
        cout<<"File originale:\n";
        
        while(fileo.get(c))
        {
                cout<<c;
                }
                
        cout<<"\n\nFile depurato:\n";
        
        while(filed.get(c))
        {
                cout<<c;
                }       
 
        do
        {
                cout<<"\n\nVuoi sovrascrivere [S/N]? "; 
                cin>>yon;
        
                if (yon=='S')   
                {
                        ofstream new_file;
                
                        new_file.open("imprese.txt");
                
                        while (filed.get(c))          // <-- problema
                        {
                                new_file<<c;
                                }
        
                        new_file.close();
                
                        cout<<"\nFile sovrascritto con successo.";
                        }
                
                if (yon=='N')
                {
                        cout<<"\nFile non sovrascritto.";
                        }
                if (yon!='S' && yon!='N')
                {
                        cout<<"\nCarattere non ammesso.";
                        }
                        
                }
                while (yon!='S' && yon!='N');
                
        fileo.close();
        filed.close();
        remove(buffer_depurato);
        }
 
//char OF[11]={'i','m','p','r','e','s','e','.','t','x','t'};
//char BD[7]={'d','e','p','.','t','x','t'};
//stampa_dati(OF,BD);

Grazie mille

640 Kb dovrebbero essere abbastanza per chiunque.

Un programma dieci volte più lungo è circa cento volte più complicato da scrivere.

nessuno (Normal User)
Guru^2

Messaggi: 5676
Iscritto: 03/01/2010

Postato alle 8:05
Venerdì, 18/06/2010

La get fallisce perché la open precedente era fallita.

E questo perché il nome del file non è passato correttamente.

In una stringa devi considerare il terminatore NUL (0 binario) e quindi un carattere in più alla fine della stessa.

Inoltre, non devi inizializzare le stringhe in quel modo e se sono costanti le puoi passare direttamente.

Quindi, per riassumere, devi fare queste modifiche

1) indicare più caratteri per i nomi dei file, anche se non sono tutti usati, ad esempio

void stampa_dati ( char file_originale[50], char buffer_depurato[50] )

2) usare delle costanti stringa quando richiami la funzione, ovvero

stampa_dati("imprese.txt", "dep.txt");

Inoltre non dovresti eseguire le open del file senza controllare che siano andate a buon fine ...

Ultima modifica effettuata da nessuno il 18/06/2010 alle 8:06

Ricorda che nessuno è obbligato a risponderti e che nessuno è perfetto ...