Problème avec appels systèmes

Version imprimable

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:

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

24/10/2009, 13h48
souviron34
Plusieurs remarques :
- Code:
  
  #include<sys/stat
  
  il faut mettre un espace après "include"
- Code:
  
  c = malloc (s.st_size * sizeof (char));
  
  Avant de continuer il faut vérifier que l'allocation s'est bien passée
- Code:
  
  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:
  
  1 2 printf ("%s\n", c);
24/10/2009, 13h58
scary

Fait mais cela ne change rien :( il n'y a toujours rien qui s'affiche:cry:
24/10/2009, 14h05
souviron34

montre ta version du code maintenant, stp..

Aussi, pourquoi utiliser open et non fopen ?
24/10/2009, 14h09
gl

Tu modifies le pointeur c lors de la lecture.

Ce n'est pas

Code:

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

mais

Code:

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

qu'il faut utiliser.

Au passage pourquoi ne pas utiliser les fonctions standard fopen(), fread(), etc. ?
24/10/2009, 14h11
souviron34

Citation:

Envoyé par gl

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

on s'est croisé ;)

Pour le reste effectivement j'avais pas vu :oops:

Voici:

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

24/10/2009, 14h39
diogene

Code:

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

Tu devrais avoir :

Code:

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:

c = malloc (s.st_size+1);

Code:

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

Pourquoi &c ?

Citation:

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() ?
24/10/2009, 14h49
scary

Citation:

Envoyé par diogene

Code:

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

Tu devrais avoir :

Code:

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:

c = malloc (s.st_size+1);

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

Citation:

Envoyé par diogene

Code:

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

Pourquoi &c ?

La signature de la fonction "read ()" est

Code:

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

Citation:

Envoyé par diogene

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

C'est bon j'ai trouvé:

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