Etrange problème avec un fprintf

Bonsoir à tous,

J'ai developpé un petit programme de traitement d'image ppm/pgm sous linux. Ca fonctionne sans problème sous linux mais lorsque je passe sous windows je rencontre des erreurs etranges. J'ai tenté d'isoler le problème et il semble que le bug vienne d'un simple fprintf. Pour être plus précis voici le morceau de code, très simple je vous l'accorde, incriminé.

Code:

---

1 2 3 4

FILE *fichier = NULL; fichier = fopen( nomFichier , "wb" ); fprintf(fichier,"P5\n256 256\n255\n"); fclose(fichier);

---

Je code sous code::blocks avec le toolkit mingw et lorsque je place mon curseur juste avant le fprintf, que j'execute en mode debug, puis j'execute la suite du programme (tjs en mode debug) ça marche.

Autre observation : J'avais des accents et des espaces dans le chemin du dossier contenant le programme et bien que je ne les utilisais pas directement dans le programme (je n'utilise que des chemins relatif qui ne sortent pas du dossier), le fait de les enlever a fait marcher le fprintf. J'ai donc ajouté le fwrite initialement présent à la suite du fprintf et là rebug.

J'avoue que là, je suis un peu dépassé

ps:le fichier est bien créé.

edit : ajout du flux dans le fprintf (erreur de frappe)

Bien le bonsoir,

sortant tout droit du man,
int fprintf (FILE *stream, const char *format, ...);
le premier argument est un FILE* et pas directement le contenu à y mettre ;)

A oups je m'excuse pour la faute de frappe.

Mais hélas pour moi j'ai bien respecté la synthaxe du fprintf et le problème semble plus sournois. Par ailleur je rappelle que le code compile et que le programme fonctionne sous linux

Citation:

---

Envoyé par f56bre

je rencontre des erreurs etranges. J'ai tenté d'isoler le

---

Ce code est incomplet. On ne sait pas ce que tu as oublié. <stdio.h>est obligatoire ici.
Il faut tester l'ouverture avant d'appeler les autres fonction f*().

Code:

---

1 2 3 4

if (fichier != NULL) { ...

---

Je sais que je propose ici un code quelque peu simplifié. Cependant j'ai bien inclut le stdio.h et je teste bien l'ouverture du fichier avec une sortie de la fonction en cas d'erreur. Voici le code complet de la fonction. Comme vous pouvez le voir je n'utilise pas le paramètre, je créé un nom de fichier dans la fonction, et comme vous pouvez le voir cette fonction semble sortie tout droit d'un exemple de cours de C.

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18

char * RempliFichierBinaire( const ContenuFichier * contenuFichier ) { const char * nomFichier = "fichierbinaire.ppm"; FILE * fichier = NULL;   fichier = fopen( nomFichier , "wb" ); if( fichier == NULL ) return NULL;     fprintf( fichier , "P5\n256 256\n255\n" );   //fwrite( contenuFichier->image->t , sizeof(unsigned char) , contenuFichier->image->hauteur*contenuFichier->image->largeur , fichier );   fclose(fichier);   return nomFichier; }

---

Citation:

---

Envoyé par f56bre

Je sais que je propose ici un code quelque peu simplifié. Cependant j'ai bien inclut le stdio.h et je teste bien l'ouverture du fichier avec une sortie de la fonction en cas d'erreur. Voici le code complet de la fonction. Comme vous pouvez le voir je n'utilise pas le paramètre, je créé un nom de fichier dans la fonction, et comme vous pouvez le voir cette fonction semble sortie tout droit d'un exemple de cours de C.

---

Ceci fonctionne (je ne sais pas pourquoi il est si difficile d'obtenir du code compilable et testable... personne ne sait donc ce qu'est un test unitaire ?):

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

#include <stdio.h> #include <stdlib.h> typedef struct { int dummy; } ContenuFichier;   int RempliFichierBinaire (const ContenuFichier * contenuFichier) { const char *nomFichier = "fichierbinaire.ppm"; FILE *fichier = NULL; fichier = fopen (nomFichier, "wb"); if (fichier == NULL) return 1; fprintf (fichier, "P5\n256 256\n255\n"); fclose (fichier); return 0; }   int main (void) { const contenuFichier = { 0 }; int err = RempliFichierBinaire (contenuFichier);   if (err) { puts ("erreur"); } return 0; }

---

et j'obtiens un fichier contenant ceci :

Code:

---

1 2 3 4

P5 256 256 255

---

soit en hexa :

Code:

---

00000000h: 50 35 0A 32 35 36 20 32 35 36 0A 32 35 35 0A ; P5.256 256.255.

---

Ce qui me parait tout à fait conforme aux attentes.

Est il possible qu'à la suite d'une allocation dynamique on rencontre ce genre de problèmes. En effet j'ai chargé une image à partir d'un fichier ppm texte dans une structure par allocation dynamique, juste avant de lancer cette fonction. Ce qui est byzarre c'est que si j'enlève l'appel à la fonction ci-dessus et ajoute un affichage du contenu de la structure ce que j'obtient est nickel.

De plus, encore une fois, tout fonctionne nickel sous linux.

Et enfin en enlevant la fonction de chargement d'image et en ne lançant que la fonction ci-dessus elle marche nickel (le résultat est conforme aux attentes comme dirait Mr Delahaye;) )

Je vais voir du côté de l'allocation mais cela ressemble t'il à quelque chose que vous avez déjà rencontré?

Une dernière question. Lorsque l'on alloue dynamiquement de la memoire pour une structure les membres de celle-ci sont ils initialisés à 0 (ou NULL pour les pointeurs). Il semble que ce soit le cas sous linux mais de toute façon je le fait par sécurité.

Citation:

---

Envoyé par f56bre

Est il possible qu'à la suite d'une allocation dynamique on rencontre ce genre de problèmes.

---

Si un comportement indéterminé s'est produit, tout peut arriver. Les deux problèmes principaux sont :

• utilisation d'un pointeur non initialisé
• débordement de tableau.

Citation:

---

De plus, encore une fois, tout fonctionne nickel sous linux.

---

Ce qui ne prouve rien.

Citation:

---

Une dernière question. Lorsque l'on alloue dynamiquement de la memoire pour une structure les membres de celle-ci sont ils initialisés à 0 (ou NULL pour les pointeurs). Il semble que ce soit le cas sous linux mais de toute façon je le fait par sécurité.

---

Non, pas du tout. Et all-bits-to-0 n'est pas forcément ce qu'il faut pour obtenir des pointeurs NULL. Il faut soit initialiser 'à la main', soit utiliser des init statiques et faire une copie...

Code:

---

1 2 3 4 5 6 7

T *p = malloc (sizeof *T); if (p != NULL) { static const T z = {0}; *p = z; /* ... */

---

J'ai un début de réponse.

Un fichier ppm commence par un code de 2 caractères (P2,P5, etc...). J'ai donc un char* de taille 3 dans lequel je fais un gets de 3 char. Si je passe la taille du char* de 3 à 2 le bug expliqué ci dessus disparait. Par contre pas moyen de scanner les 2 caractères avec un gets de 2 char 8O. Ou en tout cas lorsque j'affiche la chaine scannée je n'ai que le "P".

Pourtant un gets scanne le nombre de char scécifié sauf s'il rencontre avant un '\0' ou '\n'. Dans mon cas après le chiffre il doit y avoir un '\n'.

Citation:

---

Envoyé par f56bre

J'ai un début de réponse.

Un fichier ppm commence par un code de 2 caractères (P2,P5, etc...). J'ai donc un char* de taille 3 dans lequel je fais un gets

---

:!: STOP :!:

gets() est un bug potentiel qu'il ne faut pas utiliser dans les nouveaux développements. Apprendre à utiliser fgets() correctement.

http://emmanuel-delahaye.developpez....tes.htm#saisie
http://emmanuel-delahaye.developpez....s.htm#fichiers

D'autre part, c'est quoi un "char * de taille 3" ? char *tab[3]; ?

oups je me suis encore craqué, c'est des fgets que j'utilise pour lire dans le fichier et les char* de taille 3 sont des tableau de char de 3 éléments.

merci pour les liens je vais lire ça.

Citation:

---

Envoyé par f56bre

oups je me suis encore craqué, c'est des fgets que j'utilise pour lire dans le fichier et les char* de taille 3 sont des tableau de char de 3 éléments.

merci pour les liens je vais lire ça.

---

Un tableau de caractères et un char *, c'est pas tout à fait la même chose.

Thierry

Pour moi c'est exactement la même chose. Dans tous les cas c'est un pointeur vers un char. Seul peut différer la façon dont la mémoire peut être allouée derrière.

Citation:

---

Envoyé par f56bre

Pour moi c'est exactement la même chose. Dans tous les cas c'est un pointeur vers un char. Seul peut différer la façon dont la mémoire peut être allouée derrière.

---

deux contre exemples:

* sizeof : sizeof applique a un pointeur, donnera toujours la meme taille (celle d'un pointeur) alors que sizeof applique a un tableau fournit la taille reelle du tableau.
* affectation:

Code:

---

1 2 3 4 5 6

char a; char* ptr; char tab[10];   ptr = &a; // Correct tab = &a; // Invalide

---

Citation:

---

Envoyé par f56bre

Pour moi c'est exactement la même chose. Dans tous les cas c'est un pointeur vers un char. Seul peut différer la façon dont la mémoire peut être allouée derrière.

---

Bah non... Un tableau n'est pas un pointeur... Et le type d'un tableau de char n'est certainement pas char *. Ca se saurait... Par contre, char * est le type de l'adresse d'un élément du tableau, oui.

http://emmanuel-delahaye.developpez.....htm#char_star