Foro de elhacker.net

Hola, tengo el siguiente codigo:

Basicamente lee numeros desde "archivo2" y los muestra uno abajo del otro.

El motivo de mi post radica en que si yo al archivo le pongo algun caracter se genera un bucle que muestra infinitas veces el ultimo numero. Y por ende yo quisiera que no lea los caracteres del archivo que no sean numeros para que esto no suceda.

Como puedo solucionarlo y porque pasa esto? Gracias de antemano

http://www.cplusplus.com/reference/cstdio/scanf/

Si encuentra un caracter que no cuadra con el formato eue le has pasado, no modificará el stream (el archivo al usar fsanf), y ferror() retornará un valor distinto de 0.

Puedes comprobar ferror() para saber si hay algún caracter erróneo. En cualquier caso, si hay caracteres aleatorios en el archivo, es difícil que lo puedas leer con el programa. El archivo ha de tener un formato lógico que tu programa pueda leer.

Lo mejor por sencillo es seguir la recomendación de ivancea96.

Si aun así quieres leer los números en el archivo puedes utilizar:
* fscanf para leerlos.
* fgetc para leer y descartar los caracteres que no sean dígitos (isdigit).
* ungetc para retornar el ultimo carácter leído al stream.

De esta forma:

void leerarchivo(char *path)
{ 
   FILE *in;
   int rv;
   int num;
 
   if ((in = fopen(path, "r")) != NULL){
      while((rv = fscanf(in, "%d", &num)) != EOF){
         if (rv == 1)
            printf("Numero: %d\n", num);
         else {
            int ch;
 
            while ((ch = fgetc(in)) != EOF && !isdigit(ch))
               ;
            ungetc(ch, in);
         }
      }
 
      fclose(in); 
   }
}

Un saludo

Muchas gracias a ambos. Me estoy metiendo de lleno en C y me alegra saber que hay gente del otro lado queriendote ayudar.

PD: Estoy analizando el codigo y hay dos dudas que se me generaron:

1- Al poner en el while "... && !isdigit(ch)..." estamos diciendo que el while de verdadero si isdigit retorna 0, no?

2- Porque se usa el ultimo while con ungetc? busque sobre esta funcion pero las explicaciones no las pude entender.

3- fscanf va a retornar algo diferente a 1 si por ej: le especificamos un solo %d y este dato no es un digito?

Correcto. El bucle se ejecutara mientras el carácter leído no sea EOF (error o fin de archivo) y no sea un dígito.


Porque el bucle puede terminar cuando se lee un digito. Por ejemplo supongamos que el contenido del archivo es:
Todos los caracteres para los cuales isdigit retorna cero (falso) se descartan mediante el mentado bucle, se lee el carácter '1' el cual es un dígito y causa la salida de este. El detalle es que ese carácter se debe procesar con los siguientes para forma el numero 123, eso se logra retornando el carácter '1' al stream mediante la función ungetc.

Si. Cuando se utiliza "%d" con fscanf los posibles valores de retorno son:

A) 1 si se lee con éxito el entero.
B) 0 si falla por un matching failure, por ejemplo la entrada es "JKL".
C) EOF si falla por un input failure, por ejemplo el gato destrozo el HD.

Un saludo