Discussion :

[Soin...]

Yop tous le monde

Alors voilà, j'ai eu l'idée de faire quelque chose d'un peut spécial et un peut idiot, c'est d'encapsuler toutes les fonctions / variables de base de la SDL. J'ai finis jusqu'au évènements (gestion clavier / clic). Et maintenant j'aimerai savoir si ce que j'ai fais est bien avant de me lancer dans un Tétris ou une connerie du genre (faut bien s'en servir quand même ).

Je sais que je suis en train d'essayer de réinventer la poudre, mais c'était un projet à but totalement pédagogique, il n'y avait rien de bien sérieux là dedans

voici mon GITHUB qui contient toutes les classes et méthodes que j'ai implémenté

J'aimerais avoir vos avis sur ce qui est bien, pas bien ou même ce qui pourrait être plus optimisé par un autre moyen

Cordialement,

[Iradrille]

Hello,

Ta classe coor remplie 2 rôles et devrait être découpée en 2 classes :
- une classe coor (coordonnées) qui ne contient que des membres x / y.
- une 2eme classe pour la création de fenêtre : width / height / bpp.
-> bpp n'à a-priori rien à faire dans une classe coor.

Tes exceptions sont aussi inutiles vu que tu les catch juste derrière :

Code :

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

1
2
3
4
5
6
7
8
try {
   if (!(newS = SDL_CreateRGBSurface(SDL_HWSURFACE, coord.getCoorX(), coord.getCoorY(), coord.getBpp()\
				      , 0, 0, 0, 0)))
      throw std::string("Error cannot create Surface");
}
catch (std::string const& str){
   std::cerr << str << std::endl;
}

Pourrait très bien être

Code :

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

1
2
3
4
if (!(newS = SDL_CreateRGBSurface(SDL_HWSURFACE, coord.getCoorX(), coord.getCoorY(), coord.getBpp()\
				   , 0, 0, 0, 0)))
    std::cerr << "Error cannot create Surface" << std::endl;
}

Ou, avec des exceptions

Code :

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

1
2
3
4
if (!(newS = SDL_CreateRGBSurface(SDL_HWSURFACE, coord.getCoorX(), coord.getCoorY(), coord.getBpp()\
				   , 0, 0, 0, 0)))
    throw std::string("Error cannot create Surface");
}

-> Si tu utilises des exceptions, les rattrapes pas juste derrière, ça les rend inutiles.