Discussion :

[coco.cohen]

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 :

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
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 )

[_SamSoft_]

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 :

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

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

Tu places mal tes "":

Code :

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

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

Déjà fait cela.

PS: Utilise fgets au lieu de scanf

[souviron34]

combien de fois faudra-t-il le repeter

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

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

Code :

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

 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 :

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

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 :

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

			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 :

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

#include <stdlib.h>

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

[Sve@r]

Code :

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

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

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

...et on relit le code !

No comment...

[Thierry Chappuis]

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

Euh, pourquoi ajouter stdlib.h?

Thierry

[Thierry Chappuis]

Voici quelques corrections:

Code C :

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
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

[_SamSoft_]

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


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).

[Tusbar]

Euh, pourquoi ajouter stdlib.h?

Thierry

Pour NULL ?

[orfix]

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 :

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

1
2
3
4
5
6
7
8
 
<time.h>
<string.h>
<stdlib.h>
<locale.h>
<stddef.h>
<stdio.h>
<wchar.h>