cptpingu
Messages postés3837Date d'inscriptiondimanche 12 décembre 2004StatutModérateurDernière intervention28 mars 2023
-
18 mars 2011 à 10:35
MISBEB -
11 août 2012 à 04:17
Cette discussion concerne un article du site. Pour la consulter dans son contexte d'origine, cliquez sur le lien ci-dessous.
tes un plaisantin si tu ne veux pas aider les gens alors pourkw tu mets tn code ici?
cs_azerty25
Messages postés1114Date d'inscriptionjeudi 19 décembre 2002StatutMembreDernière intervention 6 mai 2007 22 mars 2011 à 12:09
Ca utilise quelle stratégie pour faire l'évaluation ?
ctx_man
Messages postés285Date d'inscriptionmardi 28 décembre 2004StatutMembreDernière intervention20 janvier 20133 20 mars 2011 à 23:36
J'ai jeté un oeil aux autres codes sources de l'auteur, c'est toujours les mêmes problèmes : illisible, tout le code dans les headers, quasiment aucun commentaire utile.
Il faudrait effectivement ouvrir un livre de C. Ou même d'à peu près n'importe quel langage, car tous disent la même chose : il faut commenter et travailler la lisibilité du code. La seule chose que précisera en plus un livre de C ou C++ c'est qu'on ne met pas le code dans les headers (et deux/trois autres détails tels que ne pas utiliser de goto a moins que ca ne soit véritablement nécessaire, vérifier le retour des fonctions car elles peuvent échouer, éviter d'imbriquer 15 scopes, ...)
cptpingu
Messages postés3837Date d'inscriptiondimanche 12 décembre 2004StatutModérateurDernière intervention28 mars 2023123 20 mars 2011 à 12:09
@Eomer212: C'est du C, pas du C++. Pour le reste, c'est vrai que la qualité du code est assez faible.
eomer212
Messages postés13Date d'inscriptionlundi 7 février 2011StatutMembreDernière intervention 8 février 2011 20 mars 2011 à 05:15
un code à ne surtout pas regarder, car il est l'antithése de ce qu'il faut faire.
completement rigide, les fonctions sont dans les headers, illisibles, incomprehensible, une somme de bricolages qui fonctionne par l'operation du saint esprit,
bref, n'aurait jamais du etre publié...
ouvre un vrai bouquin de C++ une fois dans ta vie.
ctx_man
Messages postés285Date d'inscriptionmardi 28 décembre 2004StatutMembreDernière intervention20 janvier 20133 18 mars 2011 à 18:10
- Le code ne compile pas
- Tout le code est dans les headers
- Aucun commentaire ou presque
- Lisibilité quasi nulle, notamment a cause de l'absence de commentaire, il est très complexe de comprendre une expression telle que : "while((A[i]!=43)&&(A[i]!=45)&&(A[i]!=47)&&(A[i]!=42)&&(A[i]!=40)&&(A[i]!=41)&&(A[i]!='^')&&(A[i]!='E')&&(A[i]!='\0'))"
Bacterius
Messages postés3792Date d'inscriptionsamedi 22 décembre 2007StatutMembreDernière intervention 3 juin 201610 18 mars 2011 à 13:13
Readme : "[...] car j'ai volontairement ajouté des erreurs, non pas dans le but de vous faire du mal mais dans le but de vous permettre d'apprendre à coder des interfaces graphique."
T'es gentil mais on est assez grands pour lire un code quand il nous intéresse. Alors merci de poster un code qui compile et de laisser aux autres leur libre arbitre.
cptpingu
Messages postés3837Date d'inscriptiondimanche 12 décembre 2004StatutModérateurDernière intervention28 mars 2023123 18 mars 2011 à 10:35
Vu dans le README:
> "il n'est pas nécessaire de prendre ce code, le compiler et l'executer au premier contact, car j'ai volontairement ajouté des erreurs,"
Hein ? Ça n'a aucun sens. Merci de poster un code qui compile.
11 août 2012 à 04:17
22 mars 2011 à 12:09
20 mars 2011 à 23:36
Il faudrait effectivement ouvrir un livre de C. Ou même d'à peu près n'importe quel langage, car tous disent la même chose : il faut commenter et travailler la lisibilité du code. La seule chose que précisera en plus un livre de C ou C++ c'est qu'on ne met pas le code dans les headers (et deux/trois autres détails tels que ne pas utiliser de goto a moins que ca ne soit véritablement nécessaire, vérifier le retour des fonctions car elles peuvent échouer, éviter d'imbriquer 15 scopes, ...)
20 mars 2011 à 12:09
20 mars 2011 à 05:15
completement rigide, les fonctions sont dans les headers, illisibles, incomprehensible, une somme de bricolages qui fonctionne par l'operation du saint esprit,
bref, n'aurait jamais du etre publié...
ouvre un vrai bouquin de C++ une fois dans ta vie.
18 mars 2011 à 18:10
- Tout le code est dans les headers
- Aucun commentaire ou presque
- Lisibilité quasi nulle, notamment a cause de l'absence de commentaire, il est très complexe de comprendre une expression telle que : "while((A[i]!=43)&&(A[i]!=45)&&(A[i]!=47)&&(A[i]!=42)&&(A[i]!=40)&&(A[i]!=41)&&(A[i]!='^')&&(A[i]!='E')&&(A[i]!='\0'))"
18 mars 2011 à 13:13
T'es gentil mais on est assez grands pour lire un code quand il nous intéresse. Alors merci de poster un code qui compile et de laisser aux autres leur libre arbitre.
18 mars 2011 à 10:35
> "il n'est pas nécessaire de prendre ce code, le compiler et l'executer au premier contact, car j'ai volontairement ajouté des erreurs,"
Hein ? Ça n'a aucun sens. Merci de poster un code qui compile.