Compter le nombre de ligne et de colonnes dans un fichier

Bonjour , je suis un grand novice dans le C et pour un de mes TP que je dois rendre je dois ouvrir un fichier et compter le nombre de colonne (le nombre de caractere) et le nombre de lignes

Pour cela j'ai écris un code mais je n'arrive pas à le compiler

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40

#include <stdio.h>   FILE *fp;   void main (void) { char file[80]; char line[255]; int nb_col = 0; int nb_ligne = 0;   printf("Fichier: "); scanf("%s", file);   fp = fopen(file, "r"); if (fp == NULL) printf("Fichier introuvable\n"); else { while (fgets(line, sizeof line, fp), !feof(fp)) {   printf("%s", line);   nb_ligne = nb_ligne +1 ;   nb_col = strlen(line); }   printf(" le nombre de ligne est %s , nb_ligne ");   printf(" le nombre de colonne est %s , nb_col ");   fclose(fp); } }

---

Vous serait-il possible de le tester pour moi et de m'indiquer les erreurs ( voir de me le corriger )

Déjà, on enlève les globales on inclu "stdlib.h", on met int main(void), on retourne une valeur, et on relit le code !

Code:

---

printf(" le nombre de ligne est %s , nb_ligne ");

---

Tu places mal tes "":

Code:

---

printf(" le nombre de ligne est %s", nb_ligne);

---

Déjà fait cela.

PS: Utilise fgets au lieu de scanf :)

combien de fois faudra-t-il le repeter 8O:roll:

FEOF NE FAIT PAS CE QUE TU CROIS .....

Si tu avais regarde attentivement le documentaiton, tu verrais que il faut faire :

Code:

---

while (fgets(line, sizeof line, fp) != NULL )

---

Ensuite, pour compter la plus grande longueur, il te faut (pour eviter une operation en plus) une variable en plus :

Code:

---

1 2 3 4 5 6 7

int nb_col0 ;   .....   nb_col = strlen(line); if ( nb_col > nb_col0 ) nb_col0 = nb_col ;

---

Si maintenant tu veux compter le nombre total de caracteres, alors c'est mieux de faire :

Code:

---

nb_col = nb_col + strlen(line);

---

Ta variable FILE n'a rien a faire en global.

Enfin, je suppose qu'il doit te manquer

Code:

---

#include <stdlib.h>

---

Et enfin pour voir les erreurs, au minimum met -Wall -pedantic dans les flags de compilation...

Citation:

---

Envoyé par _SamSoft_

Code:

---

printf(" le nombre de ligne est %s", nb_ligne);

---

---

%s pour afficher de l'int => SUPAIRE !!!:lahola:

Citation:

---

Envoyé par _SamSoft_

...et on relit le code !

---

No comment...

Citation:

---

Envoyé par _SamSoft_

Déjà, on enlève les globales on inclu "stdlib.h",

---

Euh, pourquoi ajouter stdlib.h?

Thierry

Voici quelques corrections:

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 46 47 48 49 50 51 52 53 54 55 56 57 58 59 60 61 62 63 64 65 66 67 68 69 70 71 72 73 74 75 76

#include <stdio.h> #include <string.h>   /* -tc- elimine le caractere de fin de ligne saisi par fgets() et fait le menage   dans le tampon du flux entrant en cas de saisie tronquee */ static void fclean(char *buffer, FILE *fp) { if (buffer != NULL && fp != NULL) { char *pc = strchr(buffer, '\n');   if (pc != NULL) { *pc = 0; } else { int c;   while ((c = fgetc(fp)) != '\n' && c != EOF) { } } } }   /* -tc- main retourne un entier de type int, TOUJOURS! */ int main (void) { /* -tc- c'est une bonne pratique d'initialiser les tableaux de caracteres */ char file[80] = ""; char line[255] = ""; FILE *fp = NULL; int nb_col = 0; int nb_col_max = 0; int nb_ligne = 0;   printf("Fichier: "); /* -tc- il est necessaire ici de forcer l'affichage avec fflush() */ fflush(stdout); fgets(file, sizeof file, stdin); fclean(file, stdin);   fp = fopen(file, "r"); if (fp == NULL) { printf("Fichier introuvable\n"); } else { /* -tc- lorsque la fin de fichier est rencontree ou qu'une erreur de   lecture intervient, fgets() renvoie NULL */ while (fgets(line, sizeof line, fp) != NULL) { fclean(line, fp); printf("%s\n", line); nb_ligne = nb_ligne + 1 ; nb_col = strlen(line); if (nb_col > nb_col_max) { nb_col_max = nb_col; } }   printf("Le nombre de ligne est %d\n", nb_ligne); printf("Le nombre de colonne est %d\n", nb_col_max);   /* -tc- c'est un bonne pratique d'affecter la valeur NULL a fp une fois   le fichier ferme */ fclose(fp), fp = NULL; }   /* -tc- les 3 valeurs portables que peut retourner main() sont 0,   EXIT_SUCCESS et EXIT_FAILURE (inclure stdlib.h pour les 2 dernières */ return 0; }

---

Thierry

Citation:

---

Envoyé par Sve@r

%s pour afficher de l'int => SUPAIRE !!!:lahola:


No comment...

---

C'est super ton orthographe !
J'ai dit à l'auteur du code, de relire son code ! Cette phrase ne m'est pas adressée sachant que je n'ai que faire de son code. Je lui est dit de modifier printf, qu'est ce que j'en sais moi si c'est un int ou un char, c'est basique ca, s'il ne sait pas faire la différence alors...

Pour ma défense : line et nb_ligne à 10heures du soir et après 8h de prog, c'est la même chose... De plus c'est étrange, un coup il écrit des variables en anglais, un coup en français ?!

Bonne prog !

PS: Au début de mes programmes, j'inclus les deux au moins on est sûr de pas utiliser des fonctions non déclarées (après si c'est un tout petit prog qui doit pas être un très grand mangeur de mémoire vive alors on choisit bien ses headers surtout pour l'embarqué par exemple).

Citation:

---

Envoyé par Thierry Chappuis

Euh, pourquoi ajouter stdlib.h?

Thierry

---

Pour NULL ?

Citation:

---

Envoyé par Tusbar

Pour NULL ?

---

Après un rapide coup d'oeil sur la norme, il me semble que la macro NULL est définie un peu partout :

Code:

---

1 2 3 4 5 6 7 8

<time.h> <string.h> <stdlib.h> <locale.h> <stddef.h> <stdio.h> <wchar.h>

---