Discussion :

[scary]

Bonjour,

Voila j'ai un petit soucis avec mon code qui doit lire le contenu d'un fichier l'afficher sur la sortie standard mais seulement je ne dois utiliser que des appels systèmes. Mon code est le suivant:

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
#include<stdio.h>
#include<fcntl.h>
#include<stdlib.h>
#include<sys/stat.h>
 
int main (void)
{
    int tab[256] = {0}, fd = open ("toto.txt", O_RDONLY);
    char *c;
    ssize_t taille = 0;
    struct stat s;
 
    if (fd == -1)
    {
        printf ("Erreur lors de l'ouverture du fichier\n");
        exit (1);
    }
    fstat (fd, &s);
    c = malloc (s.st_size * sizeof (char));
    taille = read (fd, &c, s.st_size);
    if (taille == -1)
    {
        printf ("Erreur lors de la lecture du fichier\n");
        exit (1);
    }
    printf ("%s\n", c-1);
    close (fd);
 
    return 0;
}

Voila alors mon problème est que lorsque j'utilise "fstat ()" pour connaitre la taille de mon fichier et bien je n'ai plus rien du tout qui s'affiche, alors que si je l'enlève et que je remplace "s.st_size" par son contenu en dure et bien ça marche et je n'ai aucune idée du pourquoi... Si quelqu'un pouvais bien m'éclairer Merci d'avance

[souviron34]

Plusieurs remarques :

• 
  

  Code :

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

  #include<sys/stat

  il faut mettre un espace après "include"
  
  
• 
  

  Code :

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

      c = malloc (s.st_size * sizeof (char));

  Avant de continuer il faut vérifier que l'allocation s'est bien passée
  
  
• 
  

  Code :

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

  1
  2
   
      printf ("%s\n", c-1);

  C est un pointeur (allocaton). c-1 est non défini (pointe à la position -1 du pointeur défini).
  Si ton fichier est un fichier texte, alors il faut plutôt allouer size+1, et forcer c[size]='\0', et là tu peux simplement faire :
  

  Code :

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

  1
  2
   
      printf ("%s\n", c);

[scary]

Fait mais cela ne change rien il n'y a toujours rien qui s'affiche

[souviron34]

montre ta version du code maintenant, stp..

Aussi, pourquoi utiliser open et non fopen ?

[gl]

Tu modifies le pointeur c lors de la lecture.

Ce n'est pas

Code :

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

taille = read (fd, &c, s.st_size);

mais

Code :

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

taille = read (fd, c, s.st_size);

qu'il faut utiliser.

Au passage pourquoi ne pas utiliser les fonctions standard fopen(), fread(), etc. ?

[souviron34]

Au passage pourquoi ne pas utiliser les fonctions standard fopen(), fread(), etc. ?

on s'est croisé

Pour le reste effectivement j'avais pas vu

[scary]

Voici:

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
#include <stdio.h>
#include <fcntl.h>
#include <stdlib.h>
#include <sys/stat.h>
 
int main (void)
{
    int tab[256] = {0}, fd = open ("toto.txt", O_RDONLY);
    char *c;
    ssize_t taille = 0;
    struct stat s;
 
    if (fd == -1)
    {
        printf ("Erreur lors de l'ouverture du fichier\n");
        exit (1);
    }
    fstat (fd, &s);
    c = malloc (s.st_size+1 * sizeof (char));
    if (c != NULL)
    {
         taille = read (fd, &c, s.st_size);
         if (taille == -1)
         {
             printf ("Erreur lors de la lecture du fichier\n");
             exit (1);
         }
         c[s.st_size] = '\0';
         printf ("%s\n", c);
         close (fd);
    }
    else
    {
         printf ("Erreur lors de l'allocation");
         exit (1);
    }
    return 0;
}

Ensuite je n'utilise pas fopen car j'ai envie de faire un code qui n'utilise que des appels systèmes tout simplement pour pouvoir m'y habitué et les comprendre

[diogene]

Code :

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

    c = malloc (s.st_size+1 * sizeof (char));

Tu devrais avoir :

Code :

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

    c = malloc ((s.st_size+1) * sizeof (char));

Toutefois tu t'en tire bien parce que sizeof(char) vaut toujours 1 et que ces expressions équivalent à

Code :

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

    c = malloc (s.st_size+1);

Code :

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

 taille = read (fd, &c, s.st_size);

Pourquoi &c ?

Ensuite je n'utilise pas fopen car j'ai envie de faire un code qui n'utilise que des appels systèmes tout simplement pour pouvoir m'y habitué et les comprendre

En quoi open() est-il plus "appels systèmes" que fopen() ?

[scary]

Code :

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

    c = malloc (s.st_size+1 * sizeof (char));

Tu devrais avoir :

Code :

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

    c = malloc ((s.st_size+1) * sizeof (char));

Toutefois tu t'en tire bien parce que sizeof(char) vaut toujours 1 et que ces expressions équivalent à

Code :

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

    c = malloc (s.st_size+1);

Ok c'est corrigé merci de la précision

Code :

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

 taille = read (fd, &c, s.st_size);

Pourquoi &c ?

La signature de la fonction "read ()" est

Code :

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

ssize_t read(int fd, void *buf, size_t count);

Et donc il faut bien que je lui passe l'adresse du mon tableau de caractère et il est vrai qu'un simple "c" suffit autant pour moi

En quoi open() est-il plus "appels systèmes" que fopen() ?

Et bien tout simplement parce que "fopen ()" utilise "open()" et qu'au lieu de renvoyer un descripteur de fichier il renvoie un pointeur sur un type FILE. Et que quasiment toutes les fonctions systèmes sont dans "fcntl.h" et pas dans "stdlib.h". Enfin tout ceci d'après un petit cours que j'ai pu lire

[scary]

C'est bon j'ai trouvé:

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
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
#include <stdio.h>
#include <fcntl.h>
#include <stdlib.h>
#include <sys/stat.h>
 
int main (int argc, char **argv)
{
    int tab[256] = {0}, cmpt = 0, i, fd = open (argv[1], O_RDONLY);
    char *c, *x;
    ssize_t taille = 0;
    struct stat s;
 
    if (fd == -1)
    {
        printf ("Erreur lors de l'ouverture du fichier %s\n", argv[1]);
        exit (1);
    }
    if (fstat (fd, &s) == -1)
    {
        printf ("Erreur lors de l'etude statistique\n");
        exit (1);
    }
    c = malloc ((s.st_size+1) * sizeof (char));
    x = malloc (cmpt * sizeof (int));
    if (c != NULL && x != NULL)
    {
        taille = read (fd, c, s.st_size);
        if (taille == -1)
        {
            printf ("Erreur lors de la lecture du fichier %s\n", argv[1]);
            exit (1);
        }
        c[s.st_size] = '\0';
        for (i=0;i<s.st_size+1;i++)
        {
            if (c[i] != '\0' && tab[c[i]] != 1)
            {
                tab[c[i]] = 1;
                x[cmpt] = c[i];
                cmpt++;
                if (realloc (x, cmpt) == NULL)
                {
                    printf ("Erreur lors de la reallocation memoire\n");
                    exit (1);
                }
            }
        }
        printf ("compte %s\n%d caracteres differents : ", argv[1], cmpt);
        for (i=0;i<cmpt;i++)
        {
            printf ("%c, ", x[i]);
        }
        printf ("\n");
        close (fd);
    }
    else
    {
        printf ("Erreur lors de l'allocation memoire");
        exit (1);
    }
 
    return 0;
}

Merci à tous pour votre aide