pouja
Messages postés3Date d'inscriptionmardi 20 avril 2010StatutMembreDernière intervention20 avril 2010
-
20 avril 2010 à 17:36
cs_Julien39
Messages postés6414Date d'inscriptionmardi 8 mars 2005StatutModérateurDernière intervention29 juillet 2020
-
22 mai 2010 à 19:18
Cette discussion concerne un article du site. Pour la consulter dans son contexte d'origine, cliquez sur le lien ci-dessous.
cs_Julien39
Messages postés6414Date d'inscriptionmardi 8 mars 2005StatutModérateurDernière intervention29 juillet 2020371 22 mai 2010 à 19:18
Il ne faut pas confondre bien commenté et beaucoup de commentaires. Je pense que tu devrais utiliser un peu plus de JavaDoc et moins de //.
Normalement, les // c'est utilisé surtout pour commenter les lignes de code à ne pas exécuter. On utilise /** */ Pour la javadoc et il y a des manières normalisées pour commenter.
Mais bon, pour un premier projet, on ne peut pas tout savoir et 'est déjà pas mal d'avoir fait l'effort d'écrire des commentaires trop de sources en manquent.
Sinon, on a tendance à se perdre un peu dans les classes. tu pourrais utiliser un peu mieux les objets et les Enum en particulier que tu n'utilises pas du tout et qui pourrait bien t'aider.
Et tu devrais faire attention quand tu redéfinis des méthodes comme equals a ne pas modifier l'objet courant, ce n'est pas grave au fond, mais on s'attend à ce que cette méthode ne modifie rien, si on était en c++ on ajouterais un const. Tu pourrais avoir quelques problèmes si tu travaillais avec d'autres personnes.
Qui plus est, je pense que tu utilises mal les cast dans tes méthodes equals, je pense que tu ne connais pas le polymorphisme en JAVA. En gros, ca veut dire que le cast est inutile.
dje_jay
Messages postés58Date d'inscriptionmercredi 17 décembre 2003StatutMembreDernière intervention16 février 20112 21 avril 2010 à 12:09
22 mai 2010 à 19:18
Normalement, les // c'est utilisé surtout pour commenter les lignes de code à ne pas exécuter. On utilise /** */ Pour la javadoc et il y a des manières normalisées pour commenter.
Mais bon, pour un premier projet, on ne peut pas tout savoir et 'est déjà pas mal d'avoir fait l'effort d'écrire des commentaires trop de sources en manquent.
Sinon, on a tendance à se perdre un peu dans les classes. tu pourrais utiliser un peu mieux les objets et les Enum en particulier que tu n'utilises pas du tout et qui pourrait bien t'aider.
Et tu devrais faire attention quand tu redéfinis des méthodes comme equals a ne pas modifier l'objet courant, ce n'est pas grave au fond, mais on s'attend à ce que cette méthode ne modifie rien, si on était en c++ on ajouterais un const. Tu pourrais avoir quelques problèmes si tu travaillais avec d'autres personnes.
Qui plus est, je pense que tu utilises mal les cast dans tes méthodes equals, je pense que tu ne connais pas le polymorphisme en JAVA. En gros, ca veut dire que le cast est inutile.
21 avril 2010 à 12:09
http://hackaday.com/2010/03/10/game-glove-learns-your-weakness/
(Sans rancunes! ;-) )