probleme sur fonction fwrite

**gijy** · 10/01/2008, 11h46

J'utilise la fonction fwrite pour faire quelques tests notament programmation sous linux.
ci joint 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
#include <stdio.h>
#include <stdlib.h>
 
main()
{
 char text[20];
 int i;
 FILE *file;
 
 
 file=fopen("enreg.txt","w+");
 i=20;
 sprintf(text,"enregistrement-%d",i);
 fwrite(text,sizeof(text),1,file);
 i=30;
 sprintf(text,"enregistrement-%d",i);
 fwrite(text,sizeof(text),1,file);
 fclose(file);
 return(0);
 }

rien de bien mechant
he bien sous win le fichier est lu correctement par l'editeur IDE de borland mais pas par le bloc note:
BC: enregistrement-20 Í!enregistrement-30
bloc note: 湥敲楧瑳敲敭瑮㈭0⇍湥敲楧瑳敲敭瑮㌭0⇍
bizzard!!
Par contre ci je fini ma chaine de text avec \n pour un retour chariot ca va.

Maintenant le meme code compiler sous linux (distri DEBIAN) avec gcc
me donne que enregistrement-20
pourquoi un fonctionnement si erratique!!
L'exemple provient d'un tuto de programmation LINUX

**homeostasie** · 10/01/2008, 12h15

En plus de ne pas tester le retour des fonctions fopen et fwrite, il manque le caractère de fin de chaine '\0' dans ton tableau de char text.

Tu peux déjà initialisé ton tableau lors de sa définition:

Code :

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

1
2
 
char text[20] = {0};

Maintenant le meme code compiler sous linux (distri DEBIAN) avec gcc
me donne que enregistrement-20
pourquoi un fonctionnement si erratique!!

Je dirais que surement le compilateur initialise par défaut à 0 tes variables...

**Melem** · 10/01/2008, 12h36

Parce que tu écris tout le contenu de ton tableau dans le fichier donc Dieu seul sait à priori ce que ton programme a bien pu écrire (puisque le contenu d'une mémoire non initialisée est à priori inconnu, en d'autres termes elle peut contenir ... n'importe quoi!), il ne faut écrire que ce qu'il faut (tous les caractères avant le '\0'). Au moins tu aurais donc du écrire :

Code :

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

fwrite(text, sizeof(char), strlen(text), file);

Mais en réalité, tu aurais plutôt du utiliser tout simplement fprintf au lieu de sprintf suivi de fwrite.

Envoyé par homeostasie

En plus de ne pas tester le retour des fonctions fopen et fwrite, il manque le caractère de fin de chaine '\0' dans ton tableau de char text.

Hein?

Envoyé par gijy

Code :

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

sprintf(text, "enregistrement-%d", i);

**Médinoc** · 10/01/2008, 13h04

Le problème ne vient pas de ton programme, mais de notepad:
http://en.wikipedia.org/wiki/Bush_hid_the_facts

**Melem** · 10/01/2008, 13h27

Envoyé par Médinoc

Le problème ne vient pas de ton programme, mais de notepad:
http://en.wikipedia.org/wiki/Bush_hid_the_facts

Bien sûr que le programme y est pour quelque chose (et dans notre cas notepad aussi). Qui sait ce que contient la partie non initialisée du tableau, ca peut représenter des séquences spéciaux pour le logiciel (comme il a été le cas ici), notepad ou pas notepad ...

**Médinoc** · 10/01/2008, 15h13

Ah, en effet, je n'avais bien lu que la première erreur:

Envoyé par gijy

he bien sous win le fichier est lu correctement par l'editeur IDE de borland mais pas par le bloc note:
BC: enregistrement-20 Í!enregistrement-30
bloc note: 湥敲楧瑳敲敭瑮㈭0⇍湥敲楧瑳敲敭瑮㌭0⇍

Cette erreur là est bien due aux "cachotteries de Bush".

La suivante par contre, indique bien un bug du programme:

Maintenant le meme code compiler sous linux (distri DEBIAN) avec gcc
me donne que enregistrement-20

**gijy** · 11/01/2008, 08h12

Merci pour toutes vos reponse.
- il n'y a pas de test sur les retours des fonctions car le but est de jouez avec la fonction fwrite. Personnellement j'utilise plus souvent fprintf ou _write en bas niveau.
- Sous win apres un memset de text à 0 la lecture sous notepad est correcte sauf evidement les carateres non utilisé du tampon.
- Sous DEBIAN memset qui est une fonction ANSI n'est pas reconnue meme avec l'include mem.h

Je ne suis pas pret pour la portabilité !!!

**Melem** · 11/01/2008, 08h46

Envoyé par gijy

il n'y a pas de test sur les retours des fonctions car le but est de jouez avec la fonction fwrite. Personnellement j'utilise plus souvent fprintf ou write en bas niveau.

C'est quoi au fait l'avantage des fonctions de bas niveau par rapport aux fonctions de haut niveau?

Envoyé par gijy

Sous win apres un memset de text à 0 la lecture sous notepad est correcte sauf evidement les carateres non utilisé du tampon.

memset ne change rien, si tu utilises fwrite tu n'as même pas besoin de '\0' puisqu'on spécifie le nombre de caractères à écrire, mais terminer une chaîne par '\0' permet d'utiliser strlen pour obtenir la longueur du chaîne. Et je répète qu'on n'écrit pas le '\0' dans un fichier texte.

Envoyé par gijy

Sous DEBIAN memset qui est une fonction ANSI n'est pas reconnue meme avec l'include mem.h

parce que mem.h n'est pas un en-tête standard, memset est déclarée dans string.h.

Envoyé par gijy

Je ne suis pas pret pour la portabilité !!!

Normal, tu ne sais même pas ce que c'est.

**gijy** · 11/01/2008, 09h58

Et toi quelle est ta definition de la portatilité?

**Melem** · 11/01/2008, 10h21

Je n'ai pas ma définition de la portabilité, mais je connais la définition de la portabilité. Si tu veux, un programme portable est un un programme qui peut être porté vers d'autres plateformes sans aucune modification du code source. Pour arriver à écrire du code correct et portable en langage C il faut un bon cours, beaucoup d'entraînement et beaucoup de lecture. Voici de bons liens :

http://c.developpez.com/cours/poly-c
http://www.open-std.org/jtc1/sc22/wg14/www/docs/n1124.pdf

**Emmanuel Delahaye** · 11/01/2008, 10h39

Envoyé par gijy

Code :

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

1
2
3
 char text[20];
 fwrite(text,sizeof(text),1,file);
 fwrite(text,sizeof(text),1,file);

Problème classique. C'est pas sizeof, mais strlen(). ici, sizeof prend le 0 final (et des caractères non initialisés après). On enregistre pas de 0 dans un fichier texte.

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
 
#include <stdio.h>
#include <string.h>
 
int main (void)
{
   FILE *file = fopen ("enreg.txt", "w");
   if (file != NULL)
   {
      char text[20];
      int i = 20;
      sprintf (text, "enregistrement-%d", i);
      fwrite (text, strlen (text), 1, file);
      i = 30;
      sprintf (text, "enregistrement-%d", i);
      fwrite (text, strlen (text), 1, file);
      fclose (file);
   }
   return 0;
}

Code :

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

enregistrement-20enregistrement-30

D'autre part, il est recommandé de respecter la structure 'texte' dans un fichier ouvert en mode texte. La ligne devrait être terminée par un \n.

Tu veux probablement ceci :

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
 
#include <stdio.h>
#include <string.h>
 
int main (void)
{
   FILE *file = fopen ("enreg.txt", "w");
   if (file != NULL)
   {
      char text[20];
      int i = 20;
      sprintf (text, "enregistrement-%d\n", i);
      fwrite (text, strlen (text), 1, file);
      i = 30;
      sprintf (text, "enregistrement-%d\n", i);
      fwrite (text, strlen (text), 1, file);
      fclose (file);
   }
   return 0;
}

Code :

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

1
2
3
 
enregistrement-20
enregistrement-30

Nota la taille de 'text' est correcte, mais limite...

**Emmanuel Delahaye** · 11/01/2008, 10h42

Envoyé par Melem

Mais en réalité, tu aurais plutôt du utiliser tout simplement fprintf au lieu de sprintf suivi de fwrite.

+1, mais c'est peut être une simulation de chaussettes...

**Emmanuel Delahaye** · 11/01/2008, 10h44

Envoyé par Médinoc

Le problème ne vient pas de ton programme, mais de notepad:
http://en.wikipedia.org/wiki/Bush_hid_the_facts

Rien à voir ici. La chaine enregistrée n'est pas claire, tout peut arriver. C'est bien le programme qui est buggé.

**Emmanuel Delahaye** · 11/01/2008, 10h46

Envoyé par gijy

- Sous DEBIAN memset qui est une fonction ANSI n'est pas reconnue meme avec l'include mem.h

Je ne suis pas pret pour la portabilité !!!

Tu devrais surtout commencer par relire ton livre de C.

Pour memset(), il faut <string.h> et c'est parfaitement portable. mem.h, vu du langage C, ça n'existe pas...

**gijy** · 11/01/2008, 11h07

Effectivement

Code :

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

1
2
 
fwrite(text,sizeof(char),strlen(text),fp);

resoud tres bien le probleme sous linux, win et avec n'importe quel editeur de texte.

**Emmanuel Delahaye** · 11/01/2008, 11h12

Envoyé par gijy

Effectivement

Code :

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

1
2
 
fwrite(text,sizeof(char),strlen(text),fp);

resoud tres bien le probleme sous linux, win et avec n'importe quel editeur de texte.

Ou tout simplement :

Code :

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

1
2
 
fwrite(text, 1, strlen(text), fp);

car sizeof(char) vaut 1 par définition (char est l'unité de compte pour la taille des objets. Comme toutes les unités, elle vaut 1... Simple bon sens)

**Médinoc** · 11/01/2008, 11h16

Alors sizeof text[0], non ?
Au moins, ce sera ça de moins à modifier dans le code en cas de passage en wchar_t...

**Emmanuel Delahaye** · 11/01/2008, 11h23

Envoyé par Médinoc

Alors sizeof text[0], non ?
Au moins, ce sera ça de moins à modifier dans le code en cas de passage en wchar_t...

Certes, mais beaucoup de choses changent dans ce cas, à commencer par strlen().

**Melem** · 11/01/2008, 11h23

Envoyé par Emmanuel Delahaye

Ou tout simplement :

Code :

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

1
2
 
fwrite(text, 1, strlen(text), fp);

car sizeof(char) vaut 1 par définition (char est l'unité de compte pour la taille des objets. Comme toutes les unités, elle vaut 1... Simple bon sens)

Ouais mais c'était surtout pour mettre en évidence d'où vient le 1.

Envoyé par Médinoc

Alors sizeof text[0], non ?
Au moins, ce sera ça de moins à modifier dans le code en cas de passage en wchar_t...

D'habitude c'est bien. Mais ici si on passe en wchar_t il y a donc aussi strlen à modifier (strlen => wcslen) ...

**gijy** · 11/01/2008, 11h30

Pour la petite histoire voici le code issu du livre programmation LINUX de Neil MAtthew, sur lequel je me suis basé pour l'utilisation à l'origine de la fonction mmap .

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
/*  We start by defining a RECORD structure
    and then create NRECORDS versions each recording their number.
    These are appended to the file records.dat.  */
 
#include <unistd.h>
#include <stdio.h>
#include <sys/mman.h>
#include <fcntl.h>
 
typedef struct {
    int integer;
    char string[24];
} RECORD;
 
#define NRECORDS (100)
 
int main()
{
    RECORD record, *mapped;
    int i, f;
    FILE *fp;
 
    fp = fopen("records.dat","w+");
    for(i=0; i<NRECORDS; i++) {
        record.integer = i;
        sprintf(record.string,"RECORD-%d",i);
        fwrite(&record,sizeof(record),1,fp);
    }
    fclose(fp);
 
/*  We now change the integer value of record 43 to 143
    and write this to the 43rd record's string.  */
 
    fp = fopen("records.dat","r+");
    fseek(fp,43*sizeof(record),SEEK_SET);
    fread(&record,sizeof(record),1,fp);
 
    record.integer = 143;
    sprintf(record.string,"RECORD-%d",record.integer);
 
    fseek(fp,43*sizeof(record),SEEK_SET);
    fwrite(&record,sizeof(record),1,fp);
    fclose(fp);
 
/*  We now map the records into memory
    and access the 43rd record in order to change the integer to 243
    (and update the record string), again using memory mapping.  */
 
    f = open("records.dat",O_RDWR);
    mapped = (RECORD *)mmap(0, NRECORDS*sizeof(record), 
                          PROT_READ|PROT_WRITE, MAP_SHARED, f, 0);
 
    mapped[43].integer = 243;
    sprintf(mapped[43].string,"RECORD-%d",mapped[43].integer);
 
    msync((void *)mapped, NRECORDS*sizeof(record), MS_ASYNC);
    munmap((void *)mapped, NRECORDS*sizeof(record));
    close(f);
 
    exit(0);
}

Salut.

probleme sur fonction fwrite

Bibliothèque standard C

Discussions similaires

Partager

Partager