'Qu'est-ce que tu ne comprends pas dans _map[i][j] = GROUND ?

Anonyme

21 août 2007 à 10:51:45

Bonjour !
J'ai dédié à mon problème un article dans mon blog, et le voici !, premier article. L'archive liée contient un makefile et les sources, ainsi que toutes les ressources. Un make suffira

La question est de savoir comment résoudre mon problème, et c'est précisément pourquoi je poste ici.
Merci d'avance.

rks`

21 août 2007 à 11:34:16

J'te dirais pas que j'ai la flemme de dl les sources hein mais bon

Si tu pouvais nous mettre juste la partie du code qui est interessante ce serait pas de refus :-°

pamaury

21 août 2007 à 12:31:51

Il m'a fallut du temps mais j'ai trouvé un bug(en fait deux):

int i,j;
[...]
try {
for (; i < 6; i++)
{
_map[i] = new int[6];
for (; j < 6; j++)
_map[i][j] = /*GROUND*/RED_ROBOT_RIGHT;
}
}

Ce code est faux car i commence à 0 puis j varie de 0 à 6 puis i passe à 1 et ... j varie de 6 à 6 !
Et oui, tu oublie de réinitialiser j à 0. Et j'ai constaté que tu faisait cette erreur potentielle à beaucoup d'endrois, il vaut mieux initialiser j directement dans le for:

try {
for (int i=0; i < 6; i++)
{
_map[i] = new int[6];
for (int j=0; j < 6; j++)
_map[i][j] = /*GROUND*/RED_ROBOT_RIGHT;
}
}

Enfin çà c'est le premier et vrai bug je crois.
Le deuxième c'est que ta fonction Ressource::replace_color(ou un truc comme çà) seg fault

En effet, tu manipule d'une part les pixels comme des entier 32-bit alors que ce n'est pas toujours vrai et d'autre part, ton pointeur sur pixel est Uint32 ce qui te fait sortir du tableau très vite à cause d'une malheureuse erreur d'arithmétique sur des pointeurs. Bref voilà un code à priori correct:

void
Resources::replace_color(SDL_Surface *surface, Uint32 pixel_to_replace, Uint32 new_pixel)
{
Uint8 *pixel;
Uint32 x, y, bpp = surface->format->BytesPerPixel;
SDL_BlitSurface(surface,NULL,surface,NULL);
// Locke la surface pour l'écrire.
if ( SDL_MUSTLOCK(surface) )
SDL_LockSurface(surface);
for (x = 0; x < (Uint32) surface->w; x++)
for (y = 0; y < (Uint32) surface->h; y++)
{
pixel = (Uint8*) surface->pixels + y * surface->pitch + x * bpp;
switch(bpp)
{
case 4:
if (*(Uint32 *)pixel == pixel_to_replace)
*(Uint32 *)pixel = new_pixel;
break;
case 2:
if (*(Uint16 *)pixel == pixel_to_replace)
*(Uint16 *)pixel = new_pixel;
break;
case 1:
if (*(Uint8 *)pixel == pixel_to_replace)
*(Uint8 *)pixel = new_pixel;
break;
default:
throw "$ERR";
}
}
// Unlocke la surface - opération achevée -
if ( SDL_MUSTLOCK(surface) )
SDL_UnlockSurface(surface);
}

Au passage, au dit "verouille" et pas "Locke"

Anonyme

21 août 2007 à 12:50:12

Merci, pour la première erreur, ça va, je pense avoir compris. C'est en partie à cause du warning : statement has no effect >.>
Pour ce qui est de la fonction replace_color, elle date d'un autre topic, j'ai pas pris le temps de tester et de vérifier que j'avais prise la bonne.
Pour le commentaire, je trouvais que taper vérouiller était trop long. (Excuse à 2 balles spotted)

EDIT : Je propose :

// Remplace la couleur.
void
Resources::replace_color(SDL_Surface *surface, Uint32 pixel_to_replace, Uint32 new_pixel)
{
Uint8 *pixel;
Uint32 x, y, bpp = surface->format->BytesPerPixel;
// Vérouille la surface pour l'écrire.
if (SDL_MUSTLOCK(surface))
SDL_LockSurface(surface);
// Parcours de la surface à la recherche de pixels à remplacer.
for (x = 0; x < (Uint32) surface->w; x++)
for (y = 0; y < (Uint32) surface->h; y++)
{
pixel = (Uint8*) surface->pixels + y * surface->pitch + x * bpp;
switch(bpp)
{
case 4:
if (*(Uint32 *)pixel == (Uint32) pixel_to_replace)
*(Uint32 *)pixel = (Uint32) new_pixel;
break;
case 2:
if (*(Uint16 *)pixel == (Uint16) pixel_to_replace)
*(Uint16 *)pixel = (Uint16) new_pixel;
break;
case 1:
if (*pixel == (Uint8) pixel_to_replace)
*pixel = (Uint8) new_pixel;
break;
default:
throw "$ERR";
}
}
// Déverrouille la surface - opération achevée -
if (SDL_MUSTLOCK(surface))
SDL_UnlockSurface(surface);
}

Qui ne marche pas, tout comme la fonction donnée précédemment. Pourriez vous m'aider en me disant pourquoi ?