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 ?

[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() ?

[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