ayuda, leer archivos en c++

[thxly]

Hola,
bueno lo que estoy haciendo es leer un archivo y dividirlo en dos, pues hasta aquí todo bien, tengo los archivos creados con sus respectivos registros.
Pero al momento de leer uno de los archivos que cree antes y poner los datos en un vector, solo ingresa la primera linea del archivo al vector. Si mando a imprimir lo que devuelve de la función getline me muestra los demás registros.No se porque solo se ingresa la primera linea del archivo.

Este es el código que estoy utilizando, tal vez algún consejo?

Código [Seleccionar] Expandir

vector<string> archivo::cDatos(string ruta){
vector<string> datos;
fstream fichero;
fichero.open(ruta.c_str(),ifstream::in);
string linea="";
while(getline(fichero,linea,'\n')!=NULL){
cout<<"linea: "<<linea<<endl;
datos.push_back(linea);
}
fichero.close();
return datos;
}

Y en consola muestra(por ejemplo):

linea: 4.8,3,1.4,0.1,Iris-setosa
4.7,3.2,1.6,0.2,Iris-setosa
4.6,3.4,1.4,0.3,Iris-setosa

[Orubatosu]

Sin mirar mucho en profundidad, veo que usas

Código (cpp) [Seleccionar] Expandir

while(getline(fichero,linea,'\n')!=NULL){

¿Porque no usar algo con menos problemas potenciales como?

Código (cpp) [Seleccionar] Expandir

while(!fichero.eof()){

Simplemente: "Mientras el objeto "fichero" no llegue al final..."

Asi te aseguras de que el while pase por todas las líneas hasta el final

[rir3760]

Pero al momento de leer uno de los archivos que cree antes y poner los datos en un vector, solo ingresa la primera linea del archivo al vector. Si mando a imprimir lo que devuelve de la función getline me muestra los demás registros.No se porque solo se ingresa la primera linea del archivo.

Este es el código que estoy utilizando, tal vez algún consejo?

Si agregamos esa función (ligeramente modificada) a un programa de prueba funciona correctamente así que el error que mencionas se debe generar en alguna otra parte del programa, por favor publica su código fuente completo (por supuesto en la forma mas pequeña posible).

Dos sugerencias en relación a esa función son 1) Los objetos de tipo string se inicializan con una cadena vacía, no es necesaria la asignación y 2) getline retorna el objeto de tipo istream utilizado como su primer argumento, no hay porque compararlo contra NUll, quita esa comparación.

----

¿Porque no usar algo con menos problemas potenciales como?

Código (cpp) [Seleccionar] Expandir

while(!fichero.eof()){
Simplemente: "Mientras el objeto "fichero" no llegue al final..."

No porque en realidad mete mas problemas.

Las funciones feof/eof de C/C++ retornan verdadero en base a la ultima operación de lectura, por ello un bucle controlado por esas funciones tendrá una iteración de mas (a eso hay que agregar que la gran mayoría de las funciones retornan el resultado de la operación de E/S) por lo que no se recomienda su uso mas que en contados escenarios.

Un saludo

[_Enko]

Y creo que el NULL estaría de mas no?

Código [Seleccionar] Expandir

while(getline(fichero,linea,'\n')){

Si la funcion retorna NULL, la sentencia del while se evaluaria como falsa?

http://www.parashift.com/c++-faq/istream-and-eof.html

[thxly]

Gracias por sus respuestas. el archivo original lo puse en un struct, no se si tal vez estoy ingresando allí algo mal.
Mi código para el struct:

Código [Seleccionar] Expandir

struct IRIS{
double sl;
double sw;
double pl;
double pw;
string clase;

void llenarDatos(string linea){
stringstream ss;
ss << linea;
string token="";
for(int i=0;i<COLS;i++){
getline(ss,token,',');
stringstream aux;
aux << token;
switch(i){
case 0:
aux >> sl;
break;
case 1:
aux >> sw;
break;
case 2:
aux >> pl;
break;
case 3:
aux >> pw;
break;
case 4:
clase=token;
break;
}
}
}

string toString(){
stringstream ss;
ss << sl << "," << sw <<","<< pl << "," << pw << "," << clase;
return ss.str();
}
};


Mi codigo para leer el archivo:

Código [Seleccionar] Expandir

vector<IRIS> archivo::cargarDatos(string ruta){
IRIS muestra;
string linea="";
ifstream fichero;
vector<IRIS> datos;
fichero.open(ruta.c_str(),ios::in);
if(fichero.is_open()){
cout<<"ok"<<endl;
                while((getline(fichero,linea,'\n'))!=NULL){
  muestra.llenarDatos(linea);
datos.push_back(muestra);
        }
}else{
cout<<"error al abrir el fichero"<<endl;
}


return datos;
}