Aide pour un programme qui manipule une image BMP

**densha** · 25/02/2008, 17h04

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 :

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

**diogene** · 25/02/2008, 17h34

Code :

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

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 :

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

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

**densha** · 25/02/2008, 21h07

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 :

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

**souviron34** · 25/02/2008, 21h33

euh...

tu incrementes j pour fabriquer, et apres tu te sers de i comme limite et re-demarre avec j = 0...

**densha** · 25/02/2008, 21h59

comme je l'ai dit j'aprend donc je fait plein de faute si tu pouvais m'aider ça serai trés bien. merci

**droggo** · 25/02/2008, 23h57

Fio,

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.

**souviron34** · 26/02/2008, 00h29

Code :

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

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

**diogene** · 26/02/2008, 00h46

Envoyé par souviron34

Code :

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

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

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!

**densha** · 26/02/2008, 12h53

merci beaucoup, ça fonctionne apparemment aprés l'allocation mémoire pour les newColors que j'avais oublié

.
Merci encore, une petite derniere question est-ce normale que le fichier que j'ai créé "resu.bmp" ne s'ouvre pas ?

**siegfried64** · 28/02/2008, 17h37

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

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

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

entete *RecupEntete(FILE *fpin);

faudra prendre ce changement en compte aprés :

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

**dot-_-net** · 28/02/2008, 23h34

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 :

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

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

     E=RecupEntete(f);

+ 8 warnings

**diogene** · 29/02/2008, 11h33

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

Code :

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

1
2
3
4
5
6
7
8
9
10
11
entete RecupEntete(FILE *fpin)
{...}

int main(void)
{
...
  entete* E;
...
  E=RecupEntete(f);
...
}

**dot-_-net** · 29/02/2008, 16h32

Envoyé par diogene

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

Code :

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

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 ??

**Médinoc** · 29/02/2008, 16h43

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

**diogene** · 29/02/2008, 17h21

Pour expliciter un peu ce que dit Medinoc :

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

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

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

**densha** · 03/03/2008, 15h47

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

**dot-_-net** · 03/03/2008, 15h54

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

**diogene** · 03/03/2008, 18h22

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

**densha** · 04/03/2008, 00h41

ha oui ??? il me manque quoi dans mon entete alors ?? j'ai cherché partout il y a toujours le même model d'entete ??

**diogene** · 04/03/2008, 11h36

Va voir ici, notamment le document de Wim Wouters

Aide pour un programme qui manipule une image BMP

C

Discussions similaires

Partager

Partager