Foro de elhacker.net

hola amigos bueno a sucedido algo raro mientras trataba de ahcer algo con el manejo de archivos txt.
tengo un txt que tiene nombres de ciudades y finaliza con un 1 deberia darme solo ese trozo desde el nombre hasta el 1
compila de diez, pero ejecuto y me sale un mensaje...
aca el codigo

ifstream miarchivo;
miarchivo.open(nombre);

La librería fstream no funciona con strings (de hecho, no se como te ha llegado a compilar).

Tienes que usar la función c_str() para convertir el string a char*

ifstream miarchivo;
miarchivo.open(nombre.c_str());

Por otro lado:

	if(!miarchivo.good())
	{
		cout << "Error abriendo archivo";
		return "ERROR";
	}

No necesitas comprobar el booleano "good" para ver si se abre el fichero, hay una forma en la que escribes menos:

if(!miarchivo)
	{
		cout << "Error abriendo archivo";
		return "ERROR";
	}

Por otro lado, ese código tiene una ambiguedad. Y es que es posible que en algun momento se lea el texto ERROR, lo cual puede llevar a equivocaciones y a problemas.

Una forma de resolverlo sería pasar el string por referencia y devolver verdadero y falso si la función acabo perfectamente:

bool carga_archivo (string nombre,string &linea)
{
	ifstream miarchivo;
	miarchivo.open(nombre.c_str());
	if(!miarchivo.good())
	{
		cout << "Error abriendo archivo";
		return false;
	}
 
	while(!miarchivo.eof())
	{
		getline(miarchivo, linea);
	}
	miarchivo.close();
 
	return true;
}

Si no tienes un libro o curso de calidad sobre C++ te recomiendo consigas uno, puedes encontrar recomendaciones mediante el motor de búsqueda de los foros.

Ademas de los problemas ya mencionados por amchacon hay otros dos en la logica del programa.
1) Utilizas la función miembro eof para controlar el bucle, usualmente no es una buena idea, en su lugar basta con verificar el estado del stream (el valor de retorno de la función getline).

2) Lees cada linea y las almacenas en la variable linea, ello tiene el efecto de descartar todas menos la ultima pero como el bucle para empezar esta mal (por el uso de eof()) las pierdes todas.

Debes cambiar el bucle y utilizar un vector para almacenar todas las lineas del archivo. Mas o menos así:

vector<string>& leer_lineas(const char *nombre, vector<string>& linea)
{
   ifstream entrada(nombre);
 
   if (entrada.good()){
      for (string temp; getline(entrada, temp); linea.push_back(temp))
         ;
 
      entrada.close();
   }
 
   return linea;
}

Un saludo

gracias rir3760 por el consejo en realidad si estoy leyendo uno, que se llama c++ como programar de deitel, en definitiva si cada libro que leo explicara detalladamente cada tema estos foros no tendrían razón de ser, deberás ver que es la aplicación practica a un caso particular de lo aprendido lo que esta mal(o sea yo), y no los conocimientos o no que tenga...si nos lo tuviera no habría echo ni una linea del código , en esencia para aprender a sumar no solo hace falta aprenderlo sino practicarlo...a cuantos no nos dolía la cabeza la regla de los signos en ejercicios combinados :rolleyes: :rolleyes:
pasando a tu aclaración mil gracias la pondré en practica y les aviso.
desde ya agradezco el tiempo puesto para a todos los que leyeron y mas gracias a los que respondieron  ;-) ;-)