[SOLUCIONADO] problema recorriendo un archivo en c

[sfermat]

Hola gente, tengo el siguiente problema. Este es mi codigo para procesar las palabras de un archivo

Código:

FILE *archivo;
    archivo=fopen(nombrecompleto,"r");
    char leido[MAX_WORD];

    while(feof(archivo)==0)
    {
        fscanf(archivo,"%s",leido);
        insertarM(toReturn,leido);//inseta en un TDA 
        printf("inserte %s en un multiset\n",leido);
        insertarM(todos,leido);//inserta en otro multiset
        printf("inserte %s en todos\n",leido);
    }

    fclose(archivo);

Mi problema es que lee dos veces la ultima palabra del archivo. Como puedo hacer para mejorar eso?

[Fw190]

Cuando lees la última entrada del archivo, aún no se ha leído la marca de final de archivo. Por eso, en el bucle, intenta hacer una lectura más, que es cuando ya lee la marca de final de archivo. Cuando ocurre esto, lo que te devuelve fscanf() es lo último que había leído antes de leer el fnal de archivo. Para resolver esta aparente repetición de la última lectura, basta con hacer una comprobación justo después de fscanf():

Código C:

while(feof(archivo)==0)
{
    fscanf(archivo,"%s",leido);
    /* Comprobar aquí si lo que se ha leído es el final del archivo */
    if (feof(archivo))
        break;
    insertarM(toReturn,leido);//inseta en un TDA 
    printf("inserte %s en un multiset\n",leido);
    insertarM(todos,leido);//inserta en otro multiset
    printf("inserte %s en todos\n",leido);
}
 

Saludos,

[sfermat]

muchas gracias. Pero ese codigo no solucionara mi problema. En un archivo con dos palabras: Lee la priemera, el if da falso, ejecuto hasta el final. Lee la segunda, el if da falso, ejecuto hasta el final. Lee erroneamente la ultima palabra de nuevo, el if da falso, ejecuto hasta el final. Lee eof, termina.
Si agrego un sacanf luego de la ultima instruccion(linea 10 por ejemplo) voy a perder palabras :s

[vosk]

Una cosa, el codigo de Fw190 funciona porque sale del bucle cuando aun habiendo entrado en el ciclo se llega a la condicion de eof.

Propongo otra cosa, usa las comprovaciones de estado/error para saber que está haciendo tu ciclo: o se habrá leido correctamente, o se habrá llegado al eof, o se habrá producido un error; sabiendo esto solo tienes que aplicar las tres condiciones

Código C:

//ciclo infinito
while(1) {
    //lees y guardas el nº de elementos leidos
    ncr = fscanf(archivo, "%s", leido);
    
    //comprovacion de estado
    if(ncr > 0) {//ok
        printf("%s\n", leido);
    }
    else if(ncr == EOF) {//eof
        printf("EOF\n");
        break;
    }
    else {//error
        printf("Error %d\n", ferror(archivo));
        break;
    }
}
 

El 'ncr' es un entero. Ten en cuenta que el valor de retorno de fscanf en caso de exito depende del numero de entidades a escanear que le envias como formato, en este caso quieres escanear un %s entonces en la condicion de exito retorna 1. En el codigo he puesto que la condicion de exito sea cuando el nº de entidades sea >0, pero puedes encontrarte casos con una lectura correcta y que el nº de entidades leidas sea 0 (p.ej. si quieres escanear un entero y en el texto solo hay letras).

Si no te fias de controlar correctamente el ciclo infinito puedes regular el ciclo con el feof(archivo) tal como tenias.

Saludos
vosk

[sfermat]

Iniciado por vosk

Una cosa, el codigo de Fw190 funciona porque sale del bucle cuando aun habiendo entrado en el ciclo se llega a la condicion de eof.

Propongo otra cosa, usa las comprovaciones de estado/error para saber que está haciendo tu ciclo: o se habrá leido correctamente, o se habrá llegado al eof, o se habrá producido un error; sabiendo esto solo tienes que aplicar las tres condiciones

Código C:

//ciclo infinito
while(1) {
    //lees y guardas el nº de elementos leidos
    ncr = fscanf(archivo, "%s", leido);
    
    //comprovacion de estado
    if(ncr > 0) {//ok
        printf("%s\n", leido);
    }
    else if(ncr == EOF) {//eof
        printf("EOF\n");
        break;
    }
    else {//error
        printf("Error %d\n", ferror(archivo));
        break;
    }
}
 

El 'ncr' es un entero. Ten en cuenta que el valor de retorno de fscanf en caso de exito depende del numero de entidades a escanear que le envias como formato, en este caso quieres escanear un %s entonces en la condicion de exito retorna 1. En el codigo he puesto que la condicion de exito sea cuando el nº de entidades sea >0, pero puedes encontrarte casos con una lectura correcta y que el nº de entidades leidas sea 0 (p.ej. si quieres escanear un entero y en el texto solo hay letras).

Si no te fias de controlar correctamente el ciclo infinito puedes regular el ciclo con el feof(archivo) tal como tenias.

Saludos
vosk

voy a intentar esto, en cuanto al código anterior de Fw160. El problema es que no llega a eof, sino que lee la última palabra dos veces antes del eof. Según el debug en un txt con "uno" "dos" las lecturas de fscan son: uno, dos, dos, eof... Por eso no serviría un control extra... Otra cuestión, estaba programando en linux, al pasar a windows ese mismo bucle pasa a ser infinito... Tiene explicación?

[vosk]

"...al pasar a windows ese mismo bucle pasa a ser infinito... Tiene explicación?..."

No se, siento no ser de mas ayuda; tal vez alguien mas pueda aclarar este punto.

Saludos
vosk

[sfermat]

dado que los txt tienen solo palabras y para evitar el uso del break, finalmente con este codigo logre lo que queria. Solo me falta probarlo en windows. Muchas gracias

Código C:

int ncr =1;
        while(ncr) {
            ncr = fscanf(archivo, "%s", leido);
            //comprovacion de estado
            if(ncr > 0) {
                insertarM(toReturn,leido);//inseta en un TDA
                insertarM(todos,leido);//inserta en otro multiset
            }else if(ncr == EOF) {//eof
                    ncr=0;
                    }
                }
 

[vosk]

Pero cuando se produce un error tambien tienes que notificarlo, que para eso esta el control de errores. Es una ayuda extra no solo durante las pruebas sino tambien durante el uso normal de las aplicaciones.

Saludos
vosk

[sfermat]

Lo probé y cuando escanea cualquier cosa que no sea un Sting me retorna - 1 al igual que eof, igual lo agregué para darle más prolijidad al código