Discussion :

[girv97]

Bonjour,

Je débute en programmation(c++) et j'ai créé un petit dans la console dans lequel le joueur 1 entre un mot, le programme mélange les lettres et le joueur 2 doit deviner le mot qui est maintenant mélangé. Je voulais savoir ce que vous pensez du code, ce qui serait à améliorer, à changer, à optimiser, etc.

Merci à l'avance de votre aide !

Voici mon programme : main.cpp

[Trademark]

Salut,

Bonne initiative, il y a plein de choses à corriger plus ou moins grave, d'abord les graves :

1. Supprime toutes les variables globales et déclarent les au plus proches de leur lieu d'utilisation.
2. endl<< endl << endl<< endl << endl << endl<< endl << endl: Ça se voit que c'est pas normal de faire ça... En plus qu'est-ce qui empêche l'utilisateur de remonter dans la console, et si la console est plus grande ? Vu que c'est une application console, il n'existe malheureusement pas de solutions portables mais tu peux t'inspirer de ça. Au pire il faut que tu fasses une boucle.
3. Il y a plein de redondance dans ton code, utilise des fonctions pour chaque petit bout de code identique !
4. Utilise des fonctions pour chaque fonctionnalités différentes.
5. Les instructions goto sont à proscrire et d'autant plus par un débutant.

Les choses moins graves maintenant (mais à corriger quand même !) :

1. Tu fais du C++ donc srand(time(0)) peut être remplacé par la version C++, n'oublie pas d'activer C++11 dans ton compilateur néanmoins (-std=c++11 dans gcc, se renseigner pour les autres).
2. Certains noms de variables ne sont pas clair comme motamel.
3. Utilise des structures pour éviter de dupliquer ton code, par exemple représente un utilisateur et un jeu dans une structure, ça t'évitera aussi de passer beaucoup de variables en paramètres.
4. Nomme tes variables soit toutes en français, soit toutes en anglais, il vaut mieux éviter de mixer les deux...

Bon courage !

[Iradrille]

Hello,

@Trademark à fait pas mal de bonnes remarques, deux ou trois trucs supplémentaires :

• évite using namespace std;
• utilise les fonction existantes quand c'est possible : ta fonction melangerMot peut utiliser std::random_shuffle
• attention aux lignes trop longues qui nuisent à la lisibilité, essaie de te limiter à 80 ou 100 caractères : 80 est la "norme" depuis 30 ou 40 ans, 100 permet d'avoir 2 documents ouverts cote à cote sur Visual Studio (et probablement tous les IDE) en 1920x1080 (ou plus grande résolution)

[girv97]

merci beaucoup pour votre aide !

@trademark
-pour les endl << endl << endl... je savais aussi que ce n'était pas correct mais je navais aucune idée comment régler ça...merci pour l'idée de la boucle... Y a t il un moyen d'effacer ce qui est dans la console à un point du programme ?
-pour les bouts de codes identiques et les fonctions il faudrait que je sois un peu plus à l'aise en c++
-pour les goto ç'est à cause que j'ai programmé et ti-basic ;p quel moyen est a utilisé en C++, les boucles ?
-ensuite pour les variables(fr/an, nom bizarre, etc.) je sais que j'ai fait ça un peu n'importe comment !
-les structures je vais aller voir un tuto

@iradrille
-il me semble qu'utilisé "using namespace std" nous simplifie la vie ?
-pour les fonctions pourquoi ne pas les créer, je suis débutant et ça m'aide a progresser...

merci encore de votre aide! pouvez vous encore m'éclairer sur mes petites questions ? ça serait apprécié

[Iradrille]

Y a t il un moyen d'effacer ce qui est dans la console à un point du programme ?

Une solution simple mais non portable

Code :

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

1
2
system("clear"); // sous Linux (ça ne nettoie pas vraiment la console, mais écrit seulement assez de retour à la ligne pour cacher ce qu'il y avait avant)
system("cls"); // sous Windows

pour les goto ç'est à cause que j'ai programmé et ti-basic ;p quel moyen est a utilisé en C++, les boucles ?

De manière générale boucle (for / while / do while) ou condition (if), ici plutôt une boucle.

il me semble qu'utilisé "using namespace std" nous simplifie la vie ?

Cela simplifie la vie, mais ouvre la porte à des conflits de noms possible : tu ne peux par exemple pas créer de fonction nommée "cout", ou tout autre nom utilisé par la STL.
En séparant les déclarations dans un .h et implémentations dans un .cpp, tu peux éventuellement utiliser "using namespace std" dans un .cpp, jamais dans un .h en tout cas.

pour les fonctions pourquoi ne pas les créer, je suis débutant et ça m'aide a progresser...

Dans une optique d'apprentissage de l'algo, oui, tu peux recréer des choses existantes pour voir comment ça marche.
Mais en dehors de ça, il vaut mieux utiliser des fonctions existantes (qui ont été prouvées fonctionnelles ).

[girv97]

Merci beaucoup encore une fois !

Je me demandais comment faire pour changer seulement une variable dans une fonction. Par exemple, quand c'est le joueur 1 qui a écrit le mot, le joueur 2 doit le deviner, mais comment changer le rôle des 2 joueur dans la partie suivante sans avoir a utilisé if(game == 1) et du coup d'écrire la ligne de code 2 fois ?