Tetris Classique

Utilisateur anonyme - 2 oct. 2013 à 17:49
Utilisateur anonyme - 28 oct. 2015 à 16:56

Cette discussion concerne un article du site. Pour la consulter dans son contexte d'origine, cliquez sur le lien ci-dessous.

https://codes-sources.commentcamarche.net/source/100152-tetris-classique

Utilisateur anonyme
28 oct. 2015 à 16:56

Modification du 28/10/2015 : Code Source Modifié en rapport avec les Commentaires de la communauté. Je n'ai pas pu intervenir avant par manque de Temps.

mogwai93

Messages postés: 361
Date d'inscription: mardi 31 décembre 2002
Statut: Membre
Dernière intervention: 19 mars 2015

7 oct. 2013 à 10:22

Dans Pieces.cpp, beaucoup de :
if (PieceCourante == N) {}
if (PieceCourante == N+1) {}
...
==> soit rajouter un else, soit mettre un switch

le contenu de chaque switch (sens par exemple) se ressemblent
==> créer une fonction avec des paramètres différents pour chaque switch
tu gagnes en taille (petites méthodes),
tu peux perdre ou gagner en lisibilité (dépend de tes commentaires)

if (Depart == true && ValidPieces == true)
==> if (Depart && ValidPieces) suffit !

if (Pause == false)
==> tu peux remplacer par if (!Pause)

Utilisateur anonyme > mogwai93

Messages postés: 361
Date d'inscription: mardi 31 décembre 2002
Statut: Membre
Dernière intervention: 19 mars 2015

7 oct. 2013 à 12:52

@ mogwai93 : Merci pour tes Suggestions. Effectivement avec cette méthode on peut compacter le code. Pour les variables d'état, c'est vrai que c'est plus simple de les écrire comme cela.

Merci. @+.