Discussion :

[hpfx]

Bonjour, j'ai fais une petite fonction qui copie un bout d'une image dans une nouvelle. et j'aimerais avoir votre avis sur la réalisation (est-ce optimisé, peut-on faire mieux ?)

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
// copie une partie de l'image <imgsrc> dans une nouvelle image de taille (newwidth;newheight).
SDL_Surface* copie_img(SDL_Surface * imgsrc, const Uint16 newwidth, const Uint16 newheight, const Uint16 posx, const Uint16 posy)
{
    SDL_Surface *tmp_tile=SDL_CreateRGBSurface(SDL_HWSURFACE, newwidth, newheight, 32, 0, 0, 0, 0);
    SDL_Surface *img= NULL;
    SDL_SetColorKey(tmp_tile, SDL_RLEACCEL | SDL_SRCCOLORKEY, SDL_MapRGB( screen->format, 0, 0, 0) );
    img = SDL_DisplayFormat( tmp_tile );
    if( img != NULL )
    {
        SDL_FreeSurface( tmp_tile );
    }
    else
    {
        img=tmp_tile; 
    }
    SDL_Rect clip;
    clip.x=posx; clip.y=posy; clip.w=newwidth; clip.h=newheight;
    apply_surface(0,0,imgsrc,img,&clip);
    return img;
}

voilà, vos remaques, vos avis.

PS : au fait, la classique fonction apply_surface() est comme ceci :

Code :

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

1
2
3
4
5
6
7
8
9
void apply_surface( int x, int y, SDL_Surface* source, SDL_Surface* destination, SDL_Rect* clip = NULL )
{
    SDL_Rect offset;
 
    offset.x = x;
    offset.y = y;
 
    SDL_BlitSurface( source, clip, destination, &offset );
}

[coyotte507]

Salut,

La fonction semble correcte, juste quelques broutilles



Sur la dernière ligne tu n'as pas besoin d'appeler apply_surface, tu peux directement faire un SDL_BlitSurface:

Code :

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

SDL_BlitSurface(imgsource, &clip, img, NULL);

Code :

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

    SDL_Surface *tmp_tile=SDL_CreateRGBSurface(SDL_HWSURFACE, newwidth, newheight, 32, 0, 0, 0, 0);

Après ça, tu as besoin de vérifier que tmp_tile != NULL, sinon pour les prochaines fonctions tu auras très probablement une segfault (si tmp_tile == NULL).

Code :

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

1
2
    SDL_Rect clip;
    clip.x=posx; clip.y=posy; clip.w=newwidth; clip.h=newheight;

Ca peut être simplifé en:

Code :

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

    SDL_Rect clip = {posx, posy, newwidth, newheight};

Concernant l'optimisation, peut-être qu'en connaissant le bpp de l'écran (en faisant par exemple SDL_GetVideoSurface()->format->BitsPerPixel), on peut éviter d'appeler SDL_DisplayFormat.

Enfin, il 'faudrait' gérer les erreurs comme celle de SDL_BlitSurface, mais ce n'est pas obligatoire en premier lieu.

[hpfx]

Concernant l'optimisation, peut-être qu'en connaissant le bpp de l'écran (en faisant par exemple SDL_GetVideoSurface()->format->BitsPerPixel), on peut éviter d'appeler SDL_DisplayFormat.

Ha c'est interessant, tu peux expliquer ?

J'avoue que j'utilise a tout les coups SDL_DisplayFormat sur chaques nouvelles image (que ca soit chargé depuis un fichier, copiée ou créée) sans trop savoir ce quelle fait finalement, j'ai lu que ca l'optimise, donc je fais bêtement
Dans quel cas le bpp peut influer sur l'utilisation de cette fonction ?

Merci.

[coyotte507]

Ha c'est interessant, tu peux expliquer ?

J'avoue que j'utilise a tout les coups SDL_DisplayFormat sur chaques nouvelles image (que ca soit chargé depuis un fichier, copiée ou créée) sans trop savoir ce quelle fait finalement, j'ai lu que ca l'optimise, donc je fais bêtement
Dans quel cas le bpp peut influer sur l'utilisation de cette fonction ?

Merci.

Je pensais que ça n'affectait que le bpp et les flags de la surface, pour les mettre pareil que la surface d'affichage, mais en fait ça change aussi d'autres paramètres ...

Donc pas la peine d'y toucher