Discussion :

[kevine2710]

Bonjour, je souhaite faire un programme qui permet d'inverser une image PPM.
Pour cela je recopie l'entête et je stock la partie Ascii dans un tableau :

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
#include <stdio.h>
 
main(int arg , char **argv[]) {
	FILE *f,*g;
	printf("1er argument : %s\n" ,argv[1]);
	printf("2eme argument : %s\n" ,argv[2]);
	f = (FILE *) fopen (argv[1], "rb");
	g = (FILE *) fopen (argv[2], "wb");
	int hor, vert,i,x ;
	int cpt = 0;
        while (i != 10 ) { //1ere ligne de l'entete
            i=fgetc(f);
            fputc(i,g);
        }
    fscanf(f, "%d", &hor); //2eme ligne
	fprintf(g,"%d",hor);
	i=fgetc(f);
    fputc(i,g);
    fscanf(f, "%d", &vert);
	fprintf(g,"%d",vert);
    i=fgetc(f);
    int mat[vert*hor];
        while (!feof(f)) {
            while (cpt < 2) { //3eme ligne
                fputc(i,g);
                    if (i==10) {
                    cpt ++ ;
                }
            i=fgetc(f);
            }
        mat[x] = i;
        i=fgetc(f);
        x++;
        }
    fclose(f);
	fclose(g);
        }

Le script compile parfaitement sous Code::Blocks avec Windows mais a l’exécution le programme plante .
Si quelqu'un a la réponse à mon problème.
Merci,

[Joker-eph]

C'est un peu difficile de répondre comme ça, tu pourrais simplifier l'effort que demande de comprendre tout le code en disant à quelle ligne "ça plante" ? Avec quoi comme erreur (segfault?) ?

[Invité]

Bonjour,

Le script compile parfaitement sous Code::Blocks

Moi j'appelle pas ça parfaitement

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
main.c:3:1: warning: return type defaults to 'int' [enabled by default]
main.c:3:1: warning: second argument of 'main' should be 'char **' [-Wmain]
main.c: In function 'main':
main.c:5:2: warning: format '%s' expects argument of type 'char *', but argument 2 has type 'char **' [-Wformat]
main.c:6:2: warning: format '%s' expects argument of type 'char *', but argument 2 has type 'char **' [-Wformat]
main.c:7:2: warning: passing argument 1 of 'fopen' from incompatible pointer type [enabled by default]
In file included from main.c:1:0:
c:\gcc\bin\../lib/gcc/i686-pc-mingw32/4.7.0/../../../../i686-pc-mingw32/include/
stdio.h:434:17: note: expected 'const char * restrict' but argument is of type 'char **'
main.c:8:2: warning: passing argument 1 of 'fopen' from incompatible pointer type [enabled by default]
In file included from main.c:1:0:
c:\gcc\bin\../lib/gcc/i686-pc-mingw32/4.7.0/../../../../i686-pc-mingw32/include/
stdio.h:434:17: note: expected 'const char * restrict' but argument is of type 'char **'
main.c:22:9: warning: variable 'mat' set but not used [-Wunused-but-set-variable]
main.c:3:10: warning: unused parameter 'arg' [-Wunused-parameter]

Donc commence par corriger ces erreurs.

[kevine2710]

Bonjour merci pour les réponse, effectivement je n'ai pas apporté assez de précision, quand je lance mon programme je vois mes deux printf s'afficher en console et après [nom du programme].exe à cesser de fonctionner.

Pour l'erreur des Warning je l'ai résolu mais mon prof m'a dit que c'était normal d'avoir de Warning, le 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
#include <stdio.h>
 
main(int arg , char *argv[]) {
	FILE *f,*g;
	printf("1er argument : %s\n" ,argv[1]);
	printf("2eme argument : %s\n" ,argv[2]);
	f = (FILE *) fopen (argv[1], "rb");
	g = (FILE *) fopen (argv[2], "wb");
	int hor, vert,i,x ;
	int cpt = 0;
        while (i != 10 ) { //1ere ligne de l'entete
            i=fgetc(f);
            fputc(i,g);
        }
    fscanf(f, "%d", &hor); //2eme ligne
	fprintf(g,"%d",hor);
	i=fgetc(f);
    fputc(i,g);
    fscanf(f, "%d", &vert);
	fprintf(g,"%d",vert);
    i=fgetc(f);
    int mat[vert*hor];
        while (!feof(f)) {
            while (cpt < 2) { //3eme ligne
                fputc(i,g);
                    if (i==10) {
                    cpt ++ ;
                }
            i=fgetc(f);
            }
        mat[x] = i;
        i=fgetc(f);
        x++;
        }
    fclose(f);
	fclose(g);
        }

On m'a dit qu'il fallait absolument en double pointeur au niveau des arguments.

[transgohan]

Suivant les données de ton fichier tu as une boucle infinie dans ton code au niveau de la seconde boucle while.
Si on ne trouve pas de "10" tu n'incrémentes jamais et tu ne détecteras jamais la fin de fichier (car pas de remontée à la boucle du dessus).

[ternel]

Bonjour,

Pour l'erreur des Warning je l'ai résolu mais mon prof m'a dit que c'était normal d'avoir de Warning

Et quand quelqu'un te crie "attention!", tu l'ignores? ou regardes-tu d'où vient le danger?

Les warnings sont des signaux d'erreurs de développement.
Une variable inutilisée, c'est souvent qu'on s'est trompé dans les arguments d'une fonction…
"incompatible pointer type" est une cause probable de segmentation fault…

[kevine2710]

Les warnings sont des signaux d'erreurs de développement.
Une variable inutilisée, c'est souvent qu'on s'est trompé dans les arguments d'une fonction…
"incompatible pointer type" est une cause probable de segmentation fault…

Tu as raison je ne savais pas qu'il fallait régler les warnings car justement j'avais posé la question et m'avais dit d'ignorer le warning.
Cependant j'ai réglé ce problème (le deuxième script que j'ai posté) mais le problème persiste j'ai toujours [mon programme].exe a cessé de fonctionner.

Au si non le problème ne viens pas de la deuxième boucle while car j'ai fait des tests et il y a bien sortie de la boucle while.

[transgohan]

Tu n'as rien d'autre après cette sortie de boucle qui pourrait occasionner ton souci.
Le message de "a cessé de fonctionner" est souvent représentatif d'une boucle ou d'un traitement infini.
Cela ne peut être occasionné par deux fclose().