Aide pour un programme qui manipule une image BMP

Version imprimable

Salut à toutes et à tous.
Je suis entrin d'aprendre le C et la j'aborde la manipulation de differents fichiers et notament une image en BMP.
Alors voila ce que je veus faire:
-lire une image bmp en extrayant les elements descriptifs (tailles,largeur,etc..)
-récupérer pour chaque pixels de l'image les composant des couleurs ( les niveaux du vert du bleu et du jaune) je essayé de faire ça grace à un tableau dynamique ( allocation memoire et tout c'est la que ça cloche je pense :s)
-modifier les valeurs des composants de couleur (ça peut colcher ici aussi :s)

peut être que j'en demande beaucoup désolé si c'est le cas.

voila mon programe qui me fait la premiére étape c'est à dire la récuperation des élements déscriptifs mais il ne fait pas les autres étapes ( "erreur de segmentation ")

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
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
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
#include<stdio.h>
#include<stdlib.h>
#include<string.h>
 
typedef struct entete entete;
struct entete
{
  char format [3]; 
  int taille;
  int reserve;
  int BMP_Off;
  int BMP_Width;
  int BMP_Height;
};
 
 
typedef struct sCoul sCoul;
struct sCoul
{
  unsigned char R;
  unsigned char V;
  unsigned char B;
};
 
 
entete RecupEntete(FILE *fpin)
{
  entete E1;
  fread(&E1.format,2,1,fpin);
  fread(&E1.taille,4,1,fpin);
 
  fread(&E1.reserve,4,1,fpin);
 
  fread(&E1.BMP_Off,4,1,fpin);
 
  fread(&E1.reserve,4,1,fpin);
 
  fread(&E1.BMP_Width,4,1,fpin);
 
  fread(&E1.BMP_Height,4,1,fpin);
  return E1;  
}
 
entete RecupEntete(FILE *fpin);
 
 
 
int main(void)
{
  char nomFichier[]="";
  int i=0;
  int j=0;
  int k=0; 
  int x=0; 
  int y=0; 
  FILE*  f;
  FILE*  newFile;
  entete E;
  /*tabeau de couleurs ( R,V,B )*/
 
  sCoul *couleurs=NULL;
  sCoul *newColors=NULL;
  printf("Entrez le nom de l'image:  ");
  scanf("%s",nomFichier);
  strcat(nomFichier,".bmp");
 
        printf("%s\n",nomFichier);
 
	f = fopen(nomFichier,"rb");
	if (f==NULL)
	  {
	    printf("Erreur de chargement de l'image");
	    return(0);
	  }
  else
    {
      E=RecupEntete(f);
        printf("format: %d\n",E.reserve);
        printf("Taille du fichier: %d\n",E.taille); 
	printf("OFFSET: %d\n",E.BMP_Off);
	printf("Largeur du fichier: %d\n",E.BMP_Width);
        printf("Hauteur du fichier: %d\n",E.BMP_Height);     
 
 
        fseek(f, E.BMP_Off, SEEK_SET); /* positionnement du curseur
					sur le debut du corps*/
        i=0;
 
      /* allocation dynamique du tableau contenant les couleurs*/
 
      couleurs=malloc (E.taille*sizeof(sCoul));
      while (!feof(f))
	  {
            /* on remplis le tableau avec les valeurs de couleur des pixels*/
            fread(&couleurs[i], sizeof(sCoul),1,f);
 
            /*si le pixel est noir je le print*/
/* 	    if (couleurs[i].B+couleurs[i].V+couleurs[i].R==0) */
/* 	      { */
/* 		x=(i % E.BMP_Width); */
/* 		y=(i % E.BMP_Height); */
/* 		printf("PIXEL NOIR ! Coordonnée en (x:%d,y:%d)\n",x,y); */
/* 		j++; */
/* 	      } */
 	    i++; 
           }  
       j=0;
      while (j<i)
	{
	  newColors[j].R=255-couleurs[j].R ;
	  newColors[j].V=255-couleurs[j].V;
	  newColors[j].B-=255-couleurs[j].B;
                j++;
	}
 
      // je crée mon nouveau fichier
      newFile = fopen("Resu.bmp", "wb");
      if (newFile == NULL)
            printf ("Erreur ...");
      else
	{
	  fwrite(&E, sizeof(entete),1,newFile);
	  j=0;
             while(j<i)
	       {
		 fwrite(&newColors[j], (sizeof(sCoul)),1,newFile);
		 j++;
	       }  
           }
      fclose(newFile);   /* ajoute EOF au fichier et ferme le fichier */
 
 
    printf("appuyez sur une touche");
        getchar();
        getchar();
        return EXIT_SUCCESS;
    }
}

merci d'avance pour toutes les réponses et désolé pour mon ortographe catastrophique.

Code:

1 2 3 4 5 6 int main(void) { char nomFichier[]=""; .... scanf("%s",nomFichier); strcat(nomFichier,".bmp");
nomFichier est inapte à stocker le nom du fichier, puisque ce tableau n'a qu'un caractère. L'écriture dans ce tableau va déborder et écraser ce qui est situé derrière. Le programme pourrait planter dès le scanf et c'est une malchance qu'il ne le fasse pas.
Déclarer plutôt
char nomFichier[256];
ou toute dimension suffisante pour stocker le résultat du scanf + ".bmp"
Code:

1 2 3 4 5 6 7 8 9 i=0; couleurs=malloc (E.taille*sizeof(sCoul)); while (!feof(f)) { /* on remplis le tableau avec les valeurs de couleur des pixels*/ fread(&couleurs[i], sizeof(sCoul),1,f); ... i++; }
L'arrêt de la boucle devrait être gouverné par la valeur de i, pas par la détection d'une fin de fichier:
while (i<E.taille)....

Tout d'abord merci pour votre réponse trés rapide, effectivement j'ai eu de la chance que ça ne plante pas au scanf.
J'ai effectué les changements dans le while mais enfaite ça change rien car je ne pense pas que ça plante ici vu que j'ai mis le test des pixel noir et les programme m'affiche les coordonnées de ces pixels mais quand il essai de faire le changement des couleurs c'est la que ca me fait "erreur de segmentation (core dumped)"
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 j=0; while (j<i) { newColors[j].R=255-couleurs[j].R ; newColors[j].V=255-couleurs[j].V; newColors[j].B-=255-couleurs[j].B; j++; } // je crée mon nouveau fichier newFile = fopen("Resu.bmp", "wb"); if (newFile == NULL) printf ("Erreur ..."); else { fwrite(&E, sizeof(entete),1,newFile); j=0; while(j<i) { fwrite(&newColors[j], (sizeof(sCoul)),1,newFile); j++; } } fclose(newFile); /* ajoute EOF au fichier et ferme le fichier */ printf("appuyez sur une touche"); getchar(); getchar(); return EXIT_SUCCESS; } }
voila c'est à partir de la que ça cloche je pense

25/02/2008, 20h33
souviron34

euh...

tu incrementes j pour fabriquer, et apres tu te sers de i comme limite et re-demarre avec j = 0... 8O
25/02/2008, 20h59
densha

comme je l'ai dit j'aprend donc je fait plein de faute si tu pouvais m'aider ça serai trés bien. merci
25/02/2008, 22h57
droggo

Fio,

Citation:

Envoyé par densha

Tout d'abord merci pour votre réponse trés rapide, effectivement j'ai eu de la chance que ça ne plante pas au scanf.

Non, tu as eu la malchance que ça ne plante pas, car tu as ainsi obtenu un programme au comportement indéterminé, alors que tu pouvais le penser bon. :)
25/02/2008, 23h29
souviron34
Code:

1 2 newColors[j].B-=255-couleurs[j].B;
ton moins, la, c'est bien ce que tu veux ??

si B vaut 0, 255-B vaut 255, -= donne -255 => crash :aie:
25/02/2008, 23h46
diogene
Citation:
Envoyé par souviron34

Code:

1 2 newColors[j].B-=255-couleurs[j].B;

ton moins, la, c'est bien ce que tu veux ??

si B vaut 0, 255-B vaut 255, -= donne -255 => crash :aie:
newColors[j].B est unsigned char et le comportement est bien défini (même si ce n'est peut-être pas celui attendu par le codeur compte tenu que newColors[j].B n'est pas initialisé) et ne peut entrainer de crash.
Par contre, newColors est déclaré comme un pointeur et je n'ai pas vu d'allocation le concernant (alors qu'elle existe pour couleurs) .
newColors[j] n'existe pas. Alors, là, c'est le crash!
26/02/2008, 11h53
densha

merci beaucoup, ça fonctionne apparemment aprés l'allocation mémoire pour les newColors que j'avais oublié :roll:.
Merci encore, une petite derniere question est-ce normale que le fichier que j'ai créé "resu.bmp" ne s'ouvre pas ?

je crois que ton probleme se situe plutot au niveau de :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
entete *RecupEntete(FILE *fpin)
{
  entete *E1;
  E1=(entete *)malloc(sizeof(entete));
  fread(E1->format,2,1,fpin);
  fread(E1->taille,4,1,fpin);
 
  fread(E1->reserve,4,1,fpin);
 
  fread(E1->BMP_Off,4,1,fpin);
 
  fread(E1->reserve,4,1,fpin);
 
  fread(E1->BMP_Width,4,1,fpin);
 
  fread(E1->BMP_Height,4,1,fpin);
  return E1;  
}

Pour le prototype (bien que ca n'a rien a voir avec le probleme) on le déclare plutot au début :

Code:

entete *RecupEntete(FILE *fpin);

faudra prendre ce changement en compte aprés :

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
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
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
int main(void)
{
  char nomFichier[60];
  int i=0;
  int j=0;
  int k=0; 
  int x=0; 
  int y=0; 
  FILE*  f;
  FILE*  newFile;
  entete *E;
  /*tabeau de couleurs ( R,V,B )*/
 
  sCoul *couleurs=NULL;
  sCoul *newColors=NULL;
  printf("Entrez le nom de l'image:  ");
  scanf("%s",nomFichier);
  strcat(nomFichier,".bmp");
 
        printf("%s\n",nomFichier);
 
	f = fopen(nomFichier,"rb");
	if (f==NULL)
	  {
	    printf("Erreur de chargement de l'image");
	    return(0);
	  }
  else
    {
      E=RecupEntete(f);
        printf("format: %d\n",E->reserve);
        printf("Taille du fichier: %d\n",E->taille); 
	printf("OFFSET: %d\n",E->BMP_Off);
	printf("Largeur du fichier: %d\n",E->BMP_Width);
        printf("Hauteur du fichier: %d\n",E->BMP_Height);     
 
 
        fseek(f, E->BMP_Off, SEEK_SET); /* positionnement du curseur
					sur le debut du corps*/
        i=0;
 
      /* allocation dynamique du tableau contenant les couleurs*/
 
      couleurs=malloc (E-->taille*sizeof(sCoul));
      while (!feof(f))
	  {
            /* on remplis le tableau avec les valeurs de couleur des pixels*/
            fread(&couleurs[i], sizeof(sCoul),1,f);
 
            /*si le pixel est noir je le print*/
/* 	    if (couleurs[i].B+couleurs[i].V+couleurs[i].R==0) */
/* 	      { */
/* 		x=(i % E->BMP_Width); */
/* 		y=(i % E->BMP_Height); */
/* 		printf("PIXEL NOIR ! Coordonnée en (x:%d,y:%d)\n",x,y); */
/* 		j++; */
/* 	      } */
 	    i++; 
           }  
       j=0;
       newColors=(sCoul *)malloc(i*sizeof(sCoul));
      while (j<i)
	{
	  newColors[j].R=255-couleurs[j].R ;
	  newColors[j].V=255-couleurs[j].V;
	  newColors[j].B-=255-couleurs[j].B;
                j++;
	}
 
      // je crée mon nouveau fichier
      newFile = fopen("Resu.bmp", "wb");
      if (newFile == NULL)
            printf ("Erreur ...");
      else
	{
	  fwrite(E, sizeof(entete),1,newFile);
	  j=0;
             while(j<i)
	       {
		 fwrite(&newColors[j], (sizeof(sCoul)),1,newFile);
		 j++;
	       }  
           }
      fclose(newFile);   /* ajoute EOF au fichier et ferme le fichier */
 
 
    printf("appuyez sur une touche");
        getchar();
        getchar();
        return EXIT_SUCCESS;
    }
free(E);
}

Ca fait longtemps que je cherche comment afficher une image et je pense que mes veux sont réaliser

J’ai compilé ce code

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
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
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
 
#include<stdio.h>
#include<stdlib.h>
#include<string.h>
 
typedef struct entete entete;
struct entete
{
  char format [3]; 
  int taille;
  int reserve;
  int BMP_Off;
  int BMP_Width;
  int BMP_Height;
};
 
 
typedef struct sCoul sCoul;
struct sCoul
{
  unsigned char R;
  unsigned char V;
  unsigned char B;
};
 
 
entete RecupEntete(FILE *fpin)
{
  entete E1;
  fread(&E1.format,2,1,fpin);
  fread(&E1.taille,4,1,fpin);
 
  fread(&E1.reserve,4,1,fpin);
 
  fread(&E1.BMP_Off,4,1,fpin);
 
  fread(&E1.reserve,4,1,fpin);
 
  fread(&E1.BMP_Width,4,1,fpin);
 
  fread(&E1.BMP_Height,4,1,fpin);
  return E1;  
}
 
entete RecupEntete(FILE *fpin);
 
 
 
 
int main(void)
{
  char nomFichier[60];
  int i=0;
  int j=0;
  int k=0; 
  int x=0; 
  int y=0; 
  FILE*  f;
  FILE*  newFile;
  entete *E;
  /*tabeau de couleurs ( R,V,B )*/
 
  sCoul *couleurs=NULL;
  sCoul *newColors=NULL;
  printf("Entrez le nom de l'image:  ");
  scanf("%s",nomFichier);
  strcat(nomFichier,".bmp");
 
        printf("%s\n",nomFichier);
 
	f = fopen(nomFichier,"rb");
	if (f==NULL)
	  {
	    printf("Erreur de chargement de l'image");
	    return(0);
	  }
  else
    {
      E=RecupEntete(f);
        printf("format: %d\n",E->reserve);
        printf("Taille du fichier: %d\n",E->taille); 
	printf("OFFSET: %d\n",E->BMP_Off);
	printf("Largeur du fichier: %d\n",E->BMP_Width);
        printf("Hauteur du fichier: %d\n",E->BMP_Height);     
 
 
        fseek(f, E->BMP_Off, SEEK_SET); /* positionnement du curseur
					sur le debut du corps*/
        i=0;
 
      /* allocation dynamique du tableau contenant les couleurs*/
 
      couleurs=malloc (E-->taille*sizeof(sCoul));
      while (!feof(f))
	  {
            /* on remplis le tableau avec les valeurs de couleur des pixels*/
            fread(&couleurs[i], sizeof(sCoul),1,f);
 
            /*si le pixel est noir je le print*/
/* 	    if (couleurs[i].B+couleurs[i].V+couleurs[i].R==0) */
/* 	      { */
/* 		x=(i % E->BMP_Width); */
/* 		y=(i % E->BMP_Height); */
/* 		printf("PIXEL NOIR ! Coordonnée en (x:%d,y:%d)\n",x,y); */
/* 		j++; */
/* 	      } */
 	    i++; 
           }  
       j=0;
       newColors=(sCoul *)malloc(i*sizeof(sCoul));
      while (j<i)
	{
	  newColors[j].R=255-couleurs[j].R ;
	  newColors[j].V=255-couleurs[j].V;
	  newColors[j].B-=255-couleurs[j].B;
                j++;
	}
 
      // je crée mon nouveau fichier
      newFile = fopen("Resu.bmp", "wb");
      if (newFile == NULL)
            printf ("Erreur ...");
      else
	{
	  fwrite(E, sizeof(entete),1,newFile);
	  j=0;
             while(j<i)
	       {
		 fwrite(&newColors[j], (sizeof(sCoul)),1,newFile);
		 j++;
	       }  
           }
      fclose(newFile);   /* ajoute EOF au fichier et ferme le fichier */
 
 
    printf("appuyez sur une touche");
        getchar();
        getchar();
        return EXIT_SUCCESS;
    }
free(E);
}

ça m’a donné erreur dans cette ligne

Code:

E=RecupEntete(f);

+ 8 warnings

29/02/2008, 10h33
diogene
Le type de E n'est pas le type de la valeur renvoyée par RecupEntete
Code:

1 2 3 4 5 6 7 8 9 10 11 entete RecupEntete(FILE *fpin) {...} int main(void) { ... entete* E; ... E=RecupEntete(f); ... }
29/02/2008, 15h32
dot-_-net
Citation:
Envoyé par diogene

Le type de E n'est pas le type de la valeur renvoyée par RecupEntete

Code:

1 2 3 4 5 6 7 8 9 10 11 entete RecupEntete(FILE *fpin) {...} int main(void) { ... entete* E; ... E=RecupEntete(f); ... }
Que dois je declarer E ??
29/02/2008, 15h43
Médinoc

C'est RecupEntete() qui doit retourner un pointeur, pointant vers la nouvelle structure allouée sur le tas.

Pour expliciter un peu ce que dit Medinoc :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
typedef struct entete entete;
....
entete * RecupEntete(FILE *fpin)
{
   entete * E1 = malloc(sizeof *E1);
   if(E1 != NULL)
   {
     fread(&E1->format,2,1,fpin);
     fread(&E1->taille,4,1,fpin);
....
   }
   return E1;
}
...
int main(void)
{
...
  entete *E = RecupEntete(f);
...
  free(E);
...
}

Ceci est aussi possible (mais pas forcément recommandé vu la taille de entete) :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
entete RecupEntete(FILE *fpin)
{
  entete E1;
  fread(&E1.format,2,1,fpin);
  fread(&E1.taille,4,1,fpin);
...
  return E1;
}
...
int main(void)
{
...
  entete E = RecupEntete(f);
...
}

03/03/2008, 14h47
densha

j'ai fait toutes les modif que vous m'avez indiqué mais mon fichier resu.BMP qui est créé ne veut toujours pas s'afficher. :?
Que faire ???? merci
03/03/2008, 14h54
dot-_-net

Citation:

Envoyé par densha

j'ai fait toutes les modif que vous m'avez indiqué mais mon fichier resu.BMP qui est créé ne veut toujours pas s'afficher. :?
Que faire ???? merci

Moi aussi ça n’a pas marcher pour moi
03/03/2008, 17h22
diogene

Mais, tu ne lis pas l'en-tête complet (qui doit faire 54 bytes sauf erreur), mais seulement le début.
Lorsque tu recopies, tu copies un en-tête tronqué et ton BMP est mal formé.
03/03/2008, 23h41
densha

ha oui ??? il me manque quoi dans mon entete alors ?? j'ai cherché partout il y a toujours le même model d'entete ??
04/03/2008, 10h36
diogene

Va voir ici, notamment le document de Wim Wouters