ajout Rudder format factory #7

david540 commented 7 years ago

On pourra commencer à travailler sur Rudder1 et quand on en aura besoin, on pourra créer facilement des autres class: Rudder2, Rudder 3 , ... Avec des noms un peu plus parlant. Par contre je ne crois pas avoir réussi à faire correspondre ce pull Request à l'issue #7

matthieu637 commented 7 years ago

Merci c'est plus propre comme ça. Nommer la branche #7 suffit (et rappeller "#7" dans le pull request). Tu peux à nouveau utiliser une branche nommé david, c'était juste pour retirer les tests.

Je laisse faire la relecture complète du code à quelqu'un d'autre et repasserait en dernier.

matthieu637 commented 7 years ago

Pour l'utilisation de git et github c'est bon. Pour la compréhension de Factory aussi.

Par contre si je reprends les 4 points d'une relecture dans le wiki :

le code n'est pas documenté du tout : il faut massivement utiliser le raccourci ALT+ SHIFT + J sur chaque classe, chaque méthode et chaque attribut.
Les conventions sont respectées. Il reste des problèmes d’indentations : CTRL + SHIFT + F pour laisser eclipse indenter automatiquement le code.
Pas de static inutile ok.
Optimisation ok, sauf qu'avoir des méthodes "afficher" en Java n'est pas très propre. Il suffit de surcharger la méthode toString() et si on veut faire un print : System.out.println(objet).

Dernier détails : configurez bien vos projets (et même Eclipse entièrement) en UTF-8 ça évitera les problèmes.

david540 commented 7 years ago

En effet la méthode "afficher" était plus pour tester mon programme, je vais la passer en ToString() si on en a besoin ou la supprimer. Je vais faire un pull request avec mon code commenté.

matthieu637 commented 7 years ago

Tu peux continuer d'utiliser la branche #7 pour les corrections vu qu'elle n'a pas été détruite.

matthieu637 / cpp-2a-crawler

ajout Rudder format factory #7 #9