GESTION DES COMPTES BANCAIRES

cs_petifa Messages postés 215 Date d'inscription dimanche 20 février 2005 Statut Membre Dernière intervention 10 mars 2014 - 6 oct. 2008 à 09:58
offlake Messages postés 190 Date d'inscription mercredi 3 septembre 2008 Statut Membre Dernière intervention 17 janvier 2009 - 7 oct. 2008 à 00:12
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/48127-gestion-des-comptes-bancaires

offlake Messages postés 190 Date d'inscription mercredi 3 septembre 2008 Statut Membre Dernière intervention 17 janvier 2009
7 oct. 2008 à 00:12
Good JOB
les prochaines sources seront Indenté!!
BY OFFLAKE
cs_GoldenEye Messages postés 527 Date d'inscription vendredi 14 septembre 2001 Statut Membre Dernière intervention 6 octobre 2008 4
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és 215 Date d'inscription dimanche 20 février 2005 Statut Membre Dernière intervention 10 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és 190 Date d'inscription mercredi 3 septembre 2008 Statut Membre Dernière intervention 17 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és 65 Date d'inscription dimanche 8 janvier 2006 Statut Membre Dernière intervention 27 février 2011 1
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és 190 Date d'inscription mercredi 3 septembre 2008 Statut Membre Dernière intervention 17 janvier 2009
6 oct. 2008 à 10:46
avec une source deposé PETIFA c pas evident!!
offlake Messages postés 190 Date d'inscription mercredi 3 septembre 2008 Statut Membre Dernière intervention 17 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és 215 Date d'inscription dimanche 20 février 2005 Statut Membre Dernière intervention 10 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.