[c++] input file

[PeppePes88]

ciao...

Ho scritto un programma in c++ per prendere in input delle stringhe da un file, ma nell'esecuzione mi a l'erroe segment faileur, programmo sotto linux!
Posto il codice dategli uno sguardo...

#include <iostream>
#include <fstream> 
#include <string> 

using namespace std; 

struct elem {
	string frase; 
	elem *ptr; 
};

typedef struct elem * elem_ptr; 

bool inserisci(elem_ptr& lista, string f); 

int main ( int argc, char *argv[] ) {
	if ( argc != 2) { 
		cerr << "Numero parametri non corretto" << endl; 
		return 0;
	}
	ifstream file;
	cout << argv[1]<<endl; 
	file.open(argv[1], ios::in); // contrallare se il file è stato aperto
	if (!file) { 
		cerr<<"Apertura file non riuscita\n"; 
		exit(1); 
	}
	string  frase; 
	elem_ptr lista = NULL; 
	while (!file.eof() ) {
			cout << "ciao" << endl; 
			getline(file,frase, '\n'); 
			if (!inserisci(lista, frase)){
			cerr << "Errore lettura del file\n" << endl; 
			exit (1); 
		}
		cout << lista->frase << endl;    
	}
	file.close();
	while ( lista != NULL) {
	cout << lista->frase << endl; 
	lista = lista->ptr; 
	}	
	return 0; 
}


bool inserisci ( elem_ptr& lista, string f) {
	cout << "fun"<<endl;
	if ( lista == NULL) {
		lista  = new elem; 
		lista->frase = f; 
		lista->ptr = NULL; 
	} else {
		elem_ptr temp = lista, prec_temp  = NULL; 
		elem_ptr nuovo = new elem;
	        nuovo->frase = f; 	
		while (lista != NULL && lista->frase > f) {
			prec_temp = temp; 
			temp = temp->ptr; 
		} 
		if ( temp == lista) lista = nuovo; 
		else prec_temp->ptr = nuovo; 
		nuovo->ptr = temp; 
         }
return true; 
}
[

[oregon]

Il problema si ha quando la variabile

frase

e' vuota (perche' ha letto dal file una riga vuota) e tenta di inserirla con

if (!inserisci(lista, frase)){

[PeppePes88]

scusa non si dovrebbe fermare????
spiegati meglio...

[oregon]

Fermare ? E perche' ?

Se c'e' una riga vuota nel file, questa viene letta ...

[PeppePes88]

Si è quindi come faccio a dirle di fermarsi??

Un idea sarebbe se la stringa è vuota fermati...

[oregon]

Originariamente inviato da PeppePes88
Un idea sarebbe se la stringa è vuota fermati...

Ottima idea ...

[shodan]

In realtà il problema nasce dal fatto che il controllo deve essere fatto non su lista, ma su temp.

		while (temp != NULL && lista->frase > f) {
			prec_temp = temp; 
			temp = temp->ptr; 
		}

p.s.
Non puoi usare una std::list?

[oregon]

Quello che intendevo era fare un controllo prima di

if (!inserisci(lista, frase)){

ed uscire dal ciclo se frase non contiene caratteri ...

[shodan]

Si, avevo capito. Però facendo il debug mi sono accorto che quando la stringa è vuota, il while indicato viene eseguito e non essendoci il controllo del puntatore temp, il programma va in crash in quella condizione.