Discussion :

[cedd70]

Bonjour,

Je cherche une fonction pour connaitre la taille d'une image BMP.
j'ai deja écris ça qui marche parfois .. ( pour les petites image ça fonctionne pas)

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
 
 
int* x_y(char* lien)
{
	int ligne=0;
	int caractere = 0;
	int a = 0;
	int * x_y;
	x_y = (int *) malloc(sizeof(int)*(8));
	int * X;
	X = (int *) malloc(sizeof(int)*(2));
	FILE* source = NULL;
	source = fopen(lien, "rb");
	if(source != NULL)
	{
		while  (caractere != EOF)
		{
			caractere = fgetc(source);
			if (ligne >= 18 && ligne <= 21)
			{
				x_y [a] = caractere;
				a++;
			}
			else if (ligne >= 22 && ligne <= 25)
			{
				x_y [a] = caractere;
				a++;
			}
			ligne ++;
 
		}
	}
	fclose(source);	
	X[0]=x_y [0]+(x_y [1])*256+(x_y [2])*65536;//X
	X[1]=x_y [4]+(x_y [5])*256+(x_y [6])*65536;//Y
	return X;
}

Je sais pu se que j'ai écris ça fait lgt. Sans utiliser de libary

Merci

[dinobogan]

Il y a au moins 4 problèmes dans ton code :

1. la longueur et la largeur sont représentés par des entiers 32 bits. Pourquoi les lire octet par octet ? Même pour un soucis de endian, il me semble plus pratique de lire directement 4 octets à placer directement dans un tableau de char pour un soucis d'endian, ou alors directement dans un int si l'OS range les octets dans le même sens que le format BMP et que tu te moques de la compatibilité,
2. tu lis les données à une place fixe dans le fichier, sans tenir compte du format de l'entête de fichier et de l'entête des données,
3. tu ne regardes que 3 octets pour la taille au lieu de 4. Ok, dans l'écrasante majorité des cas, une taille dépassant 16 millions de pixels n'arrive pas, mais autant être propre et gérer correctement le format,
4. faire un malloc de 8 octets pour stocker consécutivement deux int, c'est un peu abusé.... Il vaudrait mieux déclarer un tableau de char de 4 éléments sur la stack : (codeinline]char tab_taille[ 4 ];[\codeinline] et deux int : un pour la largeur et l'autre pour la hauteur.

Le format de fichier BMP se trouve en quelques clics, un peu de persévérance

[cedd70]

Merci
je prend ca en note pour améliorer mon programme merci