offlake
Messages postés190Date d'inscriptionmercredi 3 septembre 2008StatutMembreDernière intervention17 janvier 2009 7 oct. 2008 à 00:12
Good JOB
les prochaines sources seront Indenté!!
BY OFFLAKE
cs_GoldenEye
Messages postés527Date d'inscriptionvendredi 14 septembre 2001StatutMembreDernière intervention 6 octobre 20084 6 oct. 2008 à 23:59
- Le code est imbuvable (indentation, commentaires absents)
- Les bibliothèques utilisées sont totalement obsolètes et Brunews te l'a déjà fait remarquer.
- Je ne suis pas d'accord avec toi, le fait que le code "marche" ne suffit pas, il faut qu'il soit compréhensible par un tiers car dans la vraie vie, tu ne codes pas tout seul dans ton coin...
- Dernier élément par rapport à ton dernier commentaire ci-dessus, le niveau du bonhomme n'est pas une fonction croissante du nombre de sources déposées. A bon entendeur...
cs_petifa
Messages postés215Date d'inscriptiondimanche 20 février 2005StatutMembreDernière intervention10 mars 2014 6 oct. 2008 à 23:54
je suis pas d'accord avec toi, si ton code n'est pas très lisible il ne pourra pas être repris par quelqu'un d'autre, soigner l'indentation n'est pas a négliger, je dis ca pour ton main...
offlake
Messages postés190Date d'inscriptionmercredi 3 septembre 2008StatutMembreDernière intervention17 janvier 2009 6 oct. 2008 à 23:30
C chose là d'un expert comme toi
Moi je suis un apprenti informaticien
OFFLAKE
Spoolega
Messages postés65Date d'inscriptiondimanche 8 janvier 2006StatutMembreDernière intervention27 février 20111 6 oct. 2008 à 23:17
Bonjour,
Ton code n'est quasiment pas commenté -> En général quand on propose des classes, on décrit au moins les méthodes publiques... histoire de savoir ce que l'on manipule.
Une chose me dérange aussi : ta classe accnt comporte des bouts de code "graphique" (gotoxy / cprintf par ex...). Tout ceci va donc à l'encontre de la POO. Si tu développes une classe pour faire la gestion des comptes, celle ci ne doit en aucun cas se charger d'un quelconque affichage ; tu mélanges les choux et les carottes, c'est "grave" d'un point de vue conception logicielle (réutilisation quasi impossible / probablement problème en cas d'héritage).
++
offlake
Messages postés190Date d'inscriptionmercredi 3 septembre 2008StatutMembreDernière intervention17 janvier 2009 6 oct. 2008 à 10:46
avec une source deposé PETIFA c pas evident!!
offlake
Messages postés190Date d'inscriptionmercredi 3 septembre 2008StatutMembreDernière intervention17 janvier 2009 6 oct. 2008 à 10:45
ce qui compte c le principe du code
et il est executable
on dirai meme que ce nest pas un niveau debutant
BY:OFFLAKE
cs_petifa
Messages postés215Date d'inscriptiondimanche 20 février 2005StatutMembreDernière intervention10 mars 2014 6 oct. 2008 à 09:58
Slt quelques remarques :
- tu devrais soigner ton indentation,
- get et get2, tu devrais mettre un commentaire pour définit le but.. et ces fonctions ne sont pas des get mais des set car elles initialisent les variables de classe.
- évite de mettre tes texte en anglais et français mélanger (ligne 124-126)
- Ton VOID est un brouillon, c'est très mal indenté et trop pénible a lire ... jme suis arrêté la.
7 oct. 2008 à 00:12
les prochaines sources seront Indenté!!
BY OFFLAKE
6 oct. 2008 à 23:59
- Les bibliothèques utilisées sont totalement obsolètes et Brunews te l'a déjà fait remarquer.
- Je ne suis pas d'accord avec toi, le fait que le code "marche" ne suffit pas, il faut qu'il soit compréhensible par un tiers car dans la vraie vie, tu ne codes pas tout seul dans ton coin...
- Dernier élément par rapport à ton dernier commentaire ci-dessus, le niveau du bonhomme n'est pas une fonction croissante du nombre de sources déposées. A bon entendeur...
6 oct. 2008 à 23:54
6 oct. 2008 à 23:30
Moi je suis un apprenti informaticien
OFFLAKE
6 oct. 2008 à 23:17
Ton code n'est quasiment pas commenté -> En général quand on propose des classes, on décrit au moins les méthodes publiques... histoire de savoir ce que l'on manipule.
Une chose me dérange aussi : ta classe accnt comporte des bouts de code "graphique" (gotoxy / cprintf par ex...). Tout ceci va donc à l'encontre de la POO. Si tu développes une classe pour faire la gestion des comptes, celle ci ne doit en aucun cas se charger d'un quelconque affichage ; tu mélanges les choux et les carottes, c'est "grave" d'un point de vue conception logicielle (réutilisation quasi impossible / probablement problème en cas d'héritage).
++
6 oct. 2008 à 10:46
6 oct. 2008 à 10:45
et il est executable
on dirai meme que ce nest pas un niveau debutant
BY:OFFLAKE
6 oct. 2008 à 09:58
- tu devrais soigner ton indentation,
- get et get2, tu devrais mettre un commentaire pour définit le but.. et ces fonctions ne sont pas des get mais des set car elles initialisent les variables de classe.
- évite de mettre tes texte en anglais et français mélanger (ligne 124-126)
- Ton VOID est un brouillon, c'est très mal indenté et trop pénible a lire ... jme suis arrêté la.