Discussion :

[SPACHFR]

Bonjour,

Je continu mon auto formation avec l'ANSI_C, mais là je bloque sur un petit pgm (inutile mais formateur)
L'objectif de ce pgm est d'ouvrir un fichier, de lire le contenu ligne par ligne est d'afficher chaque ligne.
Voici mon code

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
 
#include <stdio.h>                 
#define LRECL 80                   
FILE *fp;                          
char line[LRECL];                  
void read_fic(void){               
  printf("fonction read_fic");     
  while (fgets(line,LRECL,fp)!=EOF)
      {                            
      puts(line);                  
      }/*fin while*/               
}/*fin read_fic*/                  
int main() {                                           
if ((fp = fopen("ADCDB.MIB.HMC","r")) == NULL)         
   fprintf(stderr,"Impossible d'ouvrir le fichier \n");
 else                                                  
   {                                                   
   printf(" Open OK ");                                
   read_fic();                                         
   fclose(fp);                                         
   }                                                   
} /* fin du main */

A la compilation j'ai systematiquement ce message qui concerne le while
No common type found for operands with type "char *" and "int".
J'ai regardé qqles exemples mais je vois pas d'erreur. J'ai passé la déclaration de mon tableau en globale mais idem....
Quelqu'un a t'il un piste à me donner ?

[fearyourself]

Voilà ce que mon compilateur répond :

ex.c: In function ‘read_fic’:
ex.c:8: warning: comparison between pointer and integer
ex.c: In function ‘main’:
ex.c:23: warning: control reaches end of non-void function

La première erreur est ici :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

    while (fgets(line,LRECL,fp)!=EOF)

puisque fgets retourne un pointeur...

Il faut tester avec un pointeur et la valeur à tester est NULL pour dire si la lecture a eu un problème ou est finie...

La deuxième erreur est le manque de retour dans le main, il faut rajouter un :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

return 0

à la fin du main ou mieux :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
 
#include <stdlib.h>
...
return EXIT_SUCCESS;

Jc

[TheGzD]

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

fgets(line,LRECL,fp)!=EOF

cette instruction pose évidemment problème :
à droite ta fonction renvoie en "char*" alors que EOF est de type "int" (ou "char "si tu préféres)

Il faut donc que tu revoies ton test qui n'est pas correct.

[SPACHFR]

Avec un test sur NULL ça marche toute suite mieux.....
je vais ajouter le RETURN

Merci pour cette aide rapide et efficace !!!!