[SOLUCIONADO] Archivos

[Ch3Ch0]

Buenas, tengo un problema con este codigo y es que el While no me muestra la cadena. A ver si alguien me puede ayudar...(sin modificar todo el codigo claro :D)

Código C++:

#include <iostream>
#include <fstream>
#include <string.h>
 
using namespace std;
 
int main(int argc, char *argv[]) {
    char cadena[80];
    
    cout<<"Introduzca la cadena: ";
    cin>>cadena;
    
    int len = strlen(cadena);
    fstream archivo;                        //Flujo de entrada y salida
    archivo.open("TEXTO", ios::in | ios::out);
    
    for(int i=0; i<len; i++)
        archivo.put(cadena[i]);             //Pone un caracter en archivo
    
    archivo.seekg(0);                       //Va al principio
    
    char ch;
    while(archivo)
    {
        archivo.get(ch);                    //Obtiene un caracter del archivo
        cout<<ch;                           //Lo muestra en la pantalla
    }
    return 0;
}
 

[Henkka]

No podés usar std::ios::in en un archivo que no existe.

Podés usar app, que crea el archivo o escribe al final si existe:

Código C++:

archivo.open("TEXTO", ios::in | ios::out | ios::app);
 

O trunc, crea el archivo o borra su contenido si existe:

Código C++:

archivo.open("TEXTO", ios::in | ios::out | ios::trunc);
 

[Ch3Ch0]

Hola, antes que nada gracias por tu respuesta que me soluciono esa duda que tenia salvo una cosita, cuando escribo una cadena me imprime la cadena con el ultimo caracter repetido. Por ej, si escribo "Casa" lo que me imprime es "Casaa", que puede ser?
Yo lo solucione agregandole una condicion a While de esta forma:

Código C++:

char ch;
    int cont = 0;
    while(archivo && cont != len)
    {
        archivo.get(ch);                    //Obtiene un caracter del archivo
        cout<<ch;                           //Lo muestra en la pantalla
        cont++;
    }
 

pero no se si esta bien evitar el error de esa forma jaja, me gustaria saber porque me agregaba ese caracter de mas

[Henkka]

Porque usar while(fstream) no es correcto. Lo que hace es comprobar si el descriptor está en un estado válido, y el fin de archivo lo es.

En la última iteración, while(archivo) evalua a true, pero archivo.get(ch) no es válido, entonces se imprime dos veces el mismo ch.

Mejor sería poner el get() en el while, como while(archivo.get(ch)), que casi seguro funciona.

[Ch3Ch0]

Iniciado por Henkka

Porque usar while(fstream) no es correcto. Lo que hace es comprobar si el descriptor está en un estado válido, y el fin de archivo lo es.

En la última iteración, while(archivo) evalua a true, pero archivo.get(ch) no es válido, entonces se imprime dos veces el mismo ch.

Mejor sería poner el get() en el while, como while(archivo.get(ch)), que casi seguro funciona.

Muchas gracias, me funciono como dijiste. Eso quiere decir que la solucion que le busque yo con el contador y agregarle esa condicion no esta bien hacerlo?

[anrodse]

Podrás usar el contador que has usado, aunque yo pondria la condicion cont < len por lo que pueda pasar.
Tambien puedes comprobar ch!='EOF', y alguna cosa mas.
Otra es while(archivo.get(ch)!='EOF'), porque el caracter 'EOF' no querras imprimirlo. El contador funciona, pero tienes que mirar el tamaño del archivo. En este caso funcionará, pero si por lo que sea, ademas de leer, tambien escribes en el fichero, tienes que revisar el tamaño del fichero por si ha cambiado. Ten esto en cuenta o usa la forma de Henkka que funciona bien.

[Ch3Ch0]

Iniciado por anrodse

Podrás usar el contador que has usado, aunque yo pondria la condicion cont < len por lo que pueda pasar.
Tambien puedes comprobar ch!='EOF', y alguna cosa mas.
Otra es while(archivo.get(ch)!='EOF'), porque el caracter 'EOF' no querras imprimirlo. El contador funciona, pero tienes que mirar el tamaño del archivo. En este caso funcionará, pero si por lo que sea, ademas de leer, tambien escribes en el fichero, tienes que revisar el tamaño del fichero por si ha cambiado. Ten esto en cuenta o usa la forma de Henkka que funciona bien.

Okey, gracias por responder. Un abrazo!