Discussion :

[-Mod-]

Bonjour.

Il semblerait que j'ai un problème d'arrondi dans l'un de mes codes.

Voici les lignes en cause (toutes les variables sont de type long) :

Code :

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

1
2
3
 
long tileX = spriteTile % (sprite->info.Width / sprite->tile.x);
long tileY = spriteTile / (sprite->info.Width / sprite->tile.y);

Ce code sert à calculer à partir d'un numéro d'index des coordonnées dans un sprite. L'image est découpée en plusieurs cases en hauteur et largeur.

A priori, il me semble que tileX et tileY contiennent strictement une valeur entière.

spriteTile = numéro d'index.
sprite->info.Width = largeur du sprite en pixel, dans le cas présenté plus bas, 216 pixels.
sprite->tile.x = largeur d'une case.
sprite->tile.y = hauteur d'une case.

L'indexation est de ce type (exemple) :

00 - 01 - 02 - 03
04 - 05 - 06 - 07
08 - 09 - 10 - 11
12 - 13 - 14 - 15

Le 00 renvoie les coordonnées (0,0), le 09 (1,2), le 15 (3,3), etc...

Ces coordonnées sont ensuite traitées pour un affichage de sprite qui ne pose pas de problème.

Lorsque je laisse les calculs de tileX et tileY tels que présentés plus haut, ça bugue :/ .Visuellement, ce problème cause ceci :

Voici le sprite original, de taille 216x48 :




Voici les images affichées pour sprite->tile.x = 24 et sprite->tile.y = 48, et spriteTile de 0 à 8 (le noir n'est pas le fond d'écran, mais ce qui m'est affiché par le sprite) :




Image affichée pour sprite->tile.x = 24 et sprite->tile.y = 30 et spriteTile de 0 à 8




Lorsque je remplace ceci :

Code :

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

1
2
long tileX = spriteTile % (sprite->info.Width / sprite->tile.x);
long tileY = spriteTile / (sprite->info.Width / sprite->tile.y);

par cela, ou autres valeurs, tout fonctionne correctement :

Code :

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

1
2
long tileX = 5;
long tileY = 0;

Et avec tileX de 0 à 8, j'obtiens :



Soit le sprite correctement affiché. D'où vient le problème ?

La fonction complète :

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
// On affiche une tile du sprite.
// sprite = Pointeur vers les données du sprite.
// spriteTile = Numéro de la tile à afficher.
// positionX = Abscisse pour l'affichage de la tile.
// positionY = Ordonnée pour l'affichage de la tile.
 
	void DXS_DrawSpriteTile(M2ESPRITE* sprite, long spriteTile, float positionX, float positionY)
	{
 
		RECT tile = {0};
 
		spriteTile -= 1;
 
		long tileX = spriteTile % (sprite->info.Width / sprite->tile.x);
		long tileY = spriteTile / (sprite->info.Width / sprite->tile.y);
 
		tile.left = tileX * sprite->tile.x;
		tile.top = tileY * sprite->tile.y;
		tile.right = (tileX + 1) * sprite->tile.x;
		tile.bottom = (tileY + 1) * sprite->tile.y;
 
		// Mise à jour de la position du sprite pour l'affichage.
			sprite->position.x = positionX / sprite->scale.x;
			sprite->position.y = positionY / sprite->scale.y;
 
		// Affichage du sprite.
			sprite->sprite->Draw(	sprite->texture,								// Données sur la texture du sprite.
									&tile,							// Rectangle source sur la texture.
									&sprite->center,						// Position du centre du sprite, par rapport à 
																	// son bord haut gauche.
									&sprite->position,					// Position du sprite.
									D3DCOLOR_ARGB(	sprite->alpha,			// Génère la couleur de diffusion et d'alpha.
													sprite->rouge,
													sprite->vert,
													sprite->bleu));	
	}

[souviron34]

salut

Primo le type flottant par défaut en C est double, donc ce serait mieux de passer des doubles que des floats.

Secondo, est-ce que sprite->scale.x ou .y est flottant ? sinon :

Code :

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

1
2
3
 
	sprite->position.x = (int)(positionX / (double)sprite->scale.x);
	sprite->position.y = (int)(positionY / (double)sprite->scale.y);

Tertio, tu peux simplifier du code et accélerer :

Code :

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

1
2
3
 
		tile.right = tile.left + sprite->tile.x;
		tile.bottom = tile.top + sprite->tile.y;

Enfin :

Code :

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

1
2
3
 
long tileX = spriteTile % (long)((double)sprite->info.Width / (double)sprite->tile.x));
long tileY = (long)((double)spriteTile / ((double)sprite->info.Width / (double)sprite->tile.y));

Car en C par défaut la division est entière..

[mchk0123]

En C, grâce à la promotion numérique, il suffit que seulement l'une des opérande soit flottante pour que la division se fasse en flottant.

Donc ce qui suit :

Code :

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

1
2
3
 
	sprite->position.x = (int)(positionX / (double)sprite->scale.x);
	sprite->position.y = (int)(positionY / (double)sprite->scale.y);

Peut être simplifié en :

Code :

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

1
2
3
 
	sprite->position.x = (int)(positionX / sprite->scale.x);
	sprite->position.y = (int)(positionY / sprite->scale.y);

Même chose pour :

Code :

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

1
2
3
 
long tileX = spriteTile % (long)((double)sprite->info.Width / (double)sprite->tile.x));
long tileY = (long)((double)spriteTile / ((double)sprite->info.Width / (double)sprite->tile.y));

A remplaçer par :

Code :

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

1
2
3
 
long tileX = spriteTile % (long)((double)sprite->info.Width / sprite->tile.x));
long tileY = (long)(spriteTile / ((double)sprite->info.Width / sprite->tile.y));

[-Mod-]

Merci pour ces précisions, ça me sera sans doute utile pour la suite

En fait je n'avais qu'une lettre à changer à cette ligne pour tout corriger :

Code :

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

long tileY = spriteTile / (sprite->info.Width / sprite->tile.y);

Qui devient :

Code :

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

long tileY = spriteTile / (sprite->info.Width / sprite->tile.x);

Une conne erreur ^^"

[souviron34]

En C, grâce à la promotion numérique, il suffit que seulement l'une des opérande soit flottante pour que la division se fasse en flottant.

Donc ce qui suit :

...

Peut être simplifié en :

....

Bien sûr que tu as raison, mais je préfère la première écriture pour raison d'homogénéité...

Au moins, si tu veux comprendre, ou modifier, ou passer dans un autre langage, tu sais que là pour les 2 il te faut des flottantS.

Mais bon, chacun fait ce qu'il veut, pusique c'est autorisé

[mchk0123]

Bah ! Je dois être trop feinéant !