Discussion :

[nadia_inf]

Bonjour,

J'ai un exercice qui demande d'écrire un programme en C qui accepte un nombre arbitraire de nom de fichiers comme paramètres, ouvre les fichiers, compte le nombre de leurs caractères et affiche les noms des fichiers et leurs tailles (exprimes en nombre de caractères) respectives sur la sortie standard.

Le programme doit traiter les erreurs dues à l'ouverture des fichiers et afficher les messages d'erreur adéquats.

Mon problème est lors de la phase de compilation, il m'affiche au maximum le nom et la taille de deux fichiers seulement. Voila mon code source et j'espère que vous pourrez m'aider à résoudre ce problème. Merci d'avance

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
#include <stdio.h> 
#include <stdlib.h> 
#include <fcntl.h> 
#include <unistd.h> 
#define w 1024 
int main(int argc,char * argv[]){ 
int fd, nb; 
char buf; 
int i=1; 
if (argc==1) printf("manque d'arguments\n"); 
else{ 
while (argv[i]!=NULL){ 
fd=open(argv[i],O_RDONLY); 
if (fd==-1) printf("le fichier %s n'existe pas",argv[i]); 
printf("==> le nom du fichier est: %s\n",argv[i]); 
while ((nb=read(fd,&buf,w))>0) 
printf(" sa taille est : %d caracteres\n",nb); 
printf("\n"); 
close(fd); 
i++; 
} 
} 
}
}

[nikobordx]

Bonjour,

Tu peux utiliser ce 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
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
 
#include <stdio.h>
#include <stdlib.h>
 
int main(int argc, char *argv[])
{
    int count = 0;
    FILE *in_file;
    int ch;
 
    if (argc==1)
    {
        printf("Manque d'arguments !\n");
    } else {
        for(int i = 1; i < argc; i++)
        {
 
            in_file = fopen(argv[i], "r");
            if (in_file == NULL)
            {
                printf("Le fichier %s n'existe pas !\n", argv[i]);
                exit(0);
            } else {
                while (1)
                {
                    ch = fgetc(in_file);
                    if (ch == EOF)
                    {
                        break;
                    }
                    ++count;
                }
                printf("Le fichier %s a %d caractère(s)\n", argv[i], count);
                fclose(in_file);
            }
        }
    }
    return (0);
}

Cordialement,
Nicolas.

[LittleWhite]

Bonjour,

Tu peux utiliser ce 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
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
 
#include <stdio.h>
#include <stdlib.h>
 
int main(int argc, char *argv[])
{
    int count = 0;
    FILE *in_file;
    int ch;
 
    if (argc==1)
    {
        printf("Manque d'arguments !\n");
    } else {
        for(int i = 1; i < argc; i++)
        {
 
            in_file = fopen(argv[i], "r");
            if (in_file == NULL)
            {
                printf("Le fichier %s n'existe pas !\n", argv[i]);
                exit(0);
            } else {
                while (1)
                {
                    ch = fgetc(in_file);
                    if (ch == EOF)
                    {
                        break;
                    }
                    ++count;
                }
                printf("Le fichier %s a %d caractère(s)\n", argv[i], count);
                fclose(in_file);
            }
        }
    }
    return (0);
}

Cordialement,
Nicolas.

Le return étant un mot clé, n'a pas besoin de parenthèse autour de la valeur retournée.

[nikobordx]

Bonjour LittleWhite,

En fait, j'ai pris le code sur le net et je l'ai rapidement adapté pour les besoins de nadia_inf, effectivement, les parenthèses sont complètement inutiles !

Nicolas.

[LittleWhite]

Bonjour,

Dans le code que vous avez posté, il y a un '}' en trop, à la fin.

Pour connaitre la taille d'un fichier, il est peut être plus judicieux d'utiliser

Code :

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

1
2
fseek(file,0,SEEK_END); 
ftell();

Et votre programme crashe. Vous devriez faire une boucle 'for' de 1 à argc, au lieu du while().