Discussion :

[AceSir]

Salut !

Je m'explique. J'ai une image avec fond blanc que je tourne de 90° et que je colle sur une fenêtre de couleur blanche. Le problème c'est qu'après la rotation apparait quelques petits traits noirs. Je me demandais donc s'il était possible de corriger ceci ?

[Obsidian]

Bonjour,

On peut voir ton code ?

[AceSir]

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
 
void Window::determinatePos(Window::Position p, SDL_Surface** s, SDL_Rect* r) {
  SDL_Surface* tmp;
  switch (p) {
    case NORTH:
      r->x = WIDTH / 2 - (*s)->w / 2;
      r->y = 0;
      break;
    case EAST:
      r->x = WIDTH - Card::HEIGHT - SHIFT_SEL_CARD;
      r->y = HEIGHT / 2 - (*s)->w  / 2;
      tmp = rotozoomSurface(*s, -90.0, 1.0, 0);
      SDL_FreeSurface(*s);
      *s = tmp;
      break;
    case SOUTH:
      r->x = WIDTH / 2 - (*s)->w / 2;
      r->y = HEIGHT - Card::HEIGHT - 2 * SHIFT_SEL_CARD;
      break;
    case WEST:
      r->x = 0;
      r->y = HEIGHT / 2 - (*s)->w  / 2;
      tmp = rotozoomSurface(*s, 90.0, 1.0, 0);
      SDL_FreeSurface(*s);
      *s = tmp;
      break;
  }
}

s étant déclaré de la sorte :

Code :

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

1
2
3
 
SDL_Surface* s = SDL_CreateRGBSurface(SDL_HWSURFACE, w, h, BPP, 0, 0, 0, 0);
SDL_FillRect(s, NULL, SDL_MapRGB(s->format, 255, 255, 255));

[edgarjacobs]

Hello,

Essaie en mettant le dernier paramètre de la fonction à 1:

Code :

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

mp = rotozoomSurface(*s, 90.0, 1.0, 1);

Code :

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

1
2
3
4
5
6
7
8
9
10
11
/* SDL_rotozoom.h:

Rotates and zoomes a 32bit or 8bit 'src' surface to newly created 'dst' surface.
 'angle' is the rotation in degrees. 'zoom' a scaling factor. If 'smooth' is 1
 then the destination 32bit surface is anti-aliased. If the surface is not 8bit
 or 32bit RGBA/ABGR it will be converted into a 32bit RGBA format on the fly.

*/

    DLLINTERFACE SDL_Surface *rotozoomSurface(SDL_Surface * src, double angle, double zoom, int smooth);

[AceSir]

Si je met le paramètre à 1, l'image devient floue et les traits noirs sont toujours présents...

[Kannagi]

moi il y a quelque chose qui m'intrigue tu fais

Code :

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

1
2
 
SDL_Surface** s

s c'est un tableau image ?

Après tu fais un :

Code :

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

1
2
3
4
 
(*s)->w
//...
rotozoomSurface(*s, 90.0, 1.0, 0);

plus simple de faire un s[0]->w et s[0] non ?

Apres je vois un danger la :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
void Window::determinatePos(Window::Position p, SDL_Surface** s, SDL_Rect* r) 
{
  SDL_Surface* tmp;
  switch (p) {
 
    //...
      tmp = rotozoomSurface(*s, 90.0, 1.0, 0);
      SDL_FreeSurface(*s);
      *s = tmp;
      break;
  }
}

alors tu libère *s ????
tu recharge un tableau image a chaque fois ou une image ou quoi ?
Apres tu fais *s = tmp qui sert a rien.
C'est plus logique de libérer tmp , d’ailleurs oui il faut le libérer vu que tu ne libère pas.

bref tu joue avec les pointeurs et j'ai pas impression que tu sais ce que tu fais avec .
Ah oui je viens de voir que s est un truk tous simple pour quoi alors utiliser un double pointeur et après libère les bonnes SDL_Surface.

[AceSir]

s est un pointeur sur un pointeur de SDL_Surface. C'est pour modifier la valeur du pointeur, sans avoir à le retourner par la fonction, tout simplement.

s[0] fonctionnerait très bien, certes, mais comme ce n'est pas un tableau, je préfère rester avec ma notation

Oui *s est libéré (puisque la surface initiale ne me sert plus à rien) et prend la valeur de tmp (la surface modifié). Ce dernier est libéré par la suite, mais c'est plus loin dans le code et je ne l'ai pas posté

[Kannagi]

Oh ok peut t'on voir un peu le résultat ?
Parce que je ne suis jamais tombé sur ce problème.

[AceSir]

Voilà

[LittleWhite]

Bonjour,

Essayez avec une taille de sprite en puissance de 2 (par exemple 32x64, ou autre, mais, donc les deux membres sont des puissances de deux)

La transformation que fait SDL_gfx_rotozoom fonctionnera mieux, car il pourra matcher tous les pixels.

[AceSir]

J'ai redimensionné les sprites en 84*122 mais pas de changement

[LittleWhite]

Lorsque je dis des puissances de 2, c'est 2² / 2³ / 2⁴ / 2⁵ ... , soit 16,32,64,128,256.
Soit, une taille du genre 64x128

[AceSir]

Ah oui, excuse moi j'avais lu multiple

Mais ça va être particulièrement moche si je fais ça... Je pense que je vais plutôt tourner l'image manuellement, l'enregistrer sous un nouveau nom et bliter cette image au lieu d'effectuer des rotozoom

[LittleWhite]

Oui, avoir les sprites tout prêt dans les quatre rotations, c'est mieux, sachant que les algos de Gimp ou Photoshop sont plus avancés (donc, meilleur ).

[AceSir]

Et bien finalement, j'ai trouvé une solution : retourner le sprite, puis le coller sur la surface et non l'inverse. Ainsi, je n'ai plus de bandes noirs !

Merci pour tout en tout cas !