Avis sur ce code

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,

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:

---

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:

---

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:

---

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.