Discussion :

[f56bre]

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 :

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

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)

[khayyam90]

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

[f56bre]

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

[Emmanuel Delahaye]

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 :

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

1
2
3
4
 
   if (fichier != NULL)
   {
      ...

[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.

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
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;
}

[Emmanuel Delahaye]

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 :

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

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

1
2
3
4
 
P5
256 256
255

soit en hexa :

Code :

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

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.

[f56bre]

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é.

[Emmanuel Delahaye]

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.

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

Ce qui ne prouve rien.

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 :

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

1
2
3
4
5
6
7
 
   T *p = malloc (sizeof *T);
   if (p != NULL)
   {
      static const T z = {0};
      *p = z;
      /* ... */