bilaloch
Messages postés556Date d'inscriptionlundi 6 octobre 2003StatutMembreDernière intervention31 octobre 20191 24 déc. 2006 à 20:50
nightlord666 => Merci pour toutes ces remarques. J'en tiendrai compte lors de mon optimisation de code, quand j'aurai tout le temps :)
Arnaud16022 => Je ne peux te repondre.
Joky => mdr pourquoi ta vision changera t'elle ? =D
BILALoch =)
cs_Joky
Messages postés1787Date d'inscriptionlundi 22 novembre 2004StatutMembreDernière intervention31 janvier 20092 24 déc. 2006 à 17:13
J'avoue que maintenant, je ne verrais plus Noël de la même façon lol
Arnaud16022
Messages postés1329Date d'inscriptionvendredi 15 août 2003StatutMembreDernière intervention16 juin 20102 24 déc. 2006 à 16:04
Je plussoie les commentaires de Nightlord, j'avoue ne pas avoir regardé réellement le code du main :angel:
Comment on fait déjà pour vider la console de manière portable ? j'avais déjà vu ça, mais ...
nightlord666
Messages postés746Date d'inscriptionvendredi 17 juin 2005StatutMembreDernière intervention23 mai 200710 24 déc. 2006 à 13:33
strcat(motSecret, "MARRON");
--> Niveau utilité, pourquoi ne pas remplacer ça par un strcpy ? Car dans ton code, on voit bien que normalement quand on arrive à cette ligne là, motSecret contient une chaine vide ("").
system("cls");
--> J'aime vraiment pas les commandes system... Moi j'ai compilé ton code sous Linux, résultat : un beau "cls: command not found"...
A remplacer par :
#ifdef _WIN32
system("cls");
#elif defined(_UNIX)
system("clear");
#else
#error "Système d'exploitation non pris en charge, l'écran ne sera pas effacé."
#endif
for (i = 1 ; i < 100 ; i++)
{
motSecret[i] = "";
motCache[i] = "";
}
C'est très mauvais : tu affecte à un char un const char*, ça m'etonne même que ton compilateur ne t'ai pas mis de warning... En plus, cette boucle est complétement inutile, car cela suffit :
motsecret[0] = 0; motCache[0] = 0;
Ensuite, dernière petite remarque : la longueur en anglais ne se dit pas lenght, mais length :-).
Je te met 7/10, il faut bien encourager les débutants, mais ça peut être amélioré :).
bilaloch
Messages postés556Date d'inscriptionlundi 6 octobre 2003StatutMembreDernière intervention31 octobre 20191 24 déc. 2006 à 12:47
Oui je suis tout a fait d'accord avec toi. Ce n'est pas optimisé. Mais bon :) Je reglerais ca et tu viendras noter plus tard ^^
PS : j'essayerai de dessiner un tit bonhomme qui se tue ;)
BILALoch
Arnaud16022
Messages postés1329Date d'inscriptionvendredi 15 août 2003StatutMembreDernière intervention16 juin 20102 24 déc. 2006 à 11:45
chtite déception là, j'espérais vaguement que tu avais dessiné un pendu dans la console ça aurait été rigolo :p
Pour la source bah pas grand chose à dire, si tu devrais séparer ton main en plusieurs fonctions au lieu d'en avoir une seule de 100 lignes .. c'est tout ^^
24 déc. 2006 à 20:50
Arnaud16022 => Je ne peux te repondre.
Joky => mdr pourquoi ta vision changera t'elle ? =D
BILALoch =)
24 déc. 2006 à 17:13
24 déc. 2006 à 16:04
Comment on fait déjà pour vider la console de manière portable ? j'avais déjà vu ça, mais ...
24 déc. 2006 à 13:33
--> Niveau utilité, pourquoi ne pas remplacer ça par un strcpy ? Car dans ton code, on voit bien que normalement quand on arrive à cette ligne là, motSecret contient une chaine vide ("").
system("cls");
--> J'aime vraiment pas les commandes system... Moi j'ai compilé ton code sous Linux, résultat : un beau "cls: command not found"...
A remplacer par :
#ifdef _WIN32
system("cls");
#elif defined(_UNIX)
system("clear");
#else
#error "Système d'exploitation non pris en charge, l'écran ne sera pas effacé."
#endif
for (i = 1 ; i < 100 ; i++)
{
motSecret[i] = "";
motCache[i] = "";
}
C'est très mauvais : tu affecte à un char un const char*, ça m'etonne même que ton compilateur ne t'ai pas mis de warning... En plus, cette boucle est complétement inutile, car cela suffit :
motsecret[0] = 0; motCache[0] = 0;
Ensuite, dernière petite remarque : la longueur en anglais ne se dit pas lenght, mais length :-).
Je te met 7/10, il faut bien encourager les débutants, mais ça peut être amélioré :).
24 déc. 2006 à 12:47
PS : j'essayerai de dessiner un tit bonhomme qui se tue ;)
BILALoch
24 déc. 2006 à 11:45
Pour la source bah pas grand chose à dire, si tu devrais séparer ton main en plusieurs fonctions au lieu d'en avoir une seule de 100 lignes .. c'est tout ^^