victorsmits/Ludotheque

Current Issues

Opened this issue · 0 comments

victorsmits commented

  1. Structure du projet pas claire (.json dans les fichers src, un autre dans GameLibrary/json).

  2. Authentification n'a pas de constructeur mais est appelé comme si => cascade de problème (getUserName fonctionne pas dans authentificationTest)

  3. Pourquoi utiliser des randoms pour les identifiants ???? Ca ne garanti pas que les identifiants soient uniques => mauvaise méthode ! (ou méthode à modifier pour le garantir)

  4. Test des méthodes de la mauvaise classe (authentification test => setPassword vient de Person ! ) => références inconnues

  5. Duplications de code !!! :( (userFacadeImp)

  6. Diagramme de classe pas clair du tout sur github (rectangle étrange autour des classes "game", flèches mal placées => utilise quel classe ?)

  7. Adapter config de maven pour que ça fonctionne sur jenkins

  8. Manque de commentaires dans certains fichiers / commentaire inutile (//if not found, return "Not Found", //enter username alors que string avant, etc)

  9. Adherent facade complètement vide ??? => programme non terminé

  10. Pourquoi final ?? bloquer l'héritage sans raison ? manager et game library => bloque l'extensibilité

  11. Tout en statique => provoque des problèmes au niveau des tests unitaires (ajoute membres de liste dans une classe de test, puis ajoute de nouveau dans une autre classe de tests, mais comme liste = variable statiques, les membres des 2 tests s'ajoutent à la même liste => un des deux tests fails)

  12. Borrow:

    • De nouveau random

  13. GameFactory.java & GameFactoryIF, est-ce un abandon ??

  14. Person:

    • this.id = this.id = (long) ((Math.random()*((9999-1000)+1))+1000); => erreur de parenthèses dans la formule (si le but est bien d'avoir un ID à 4 chiffres, compris entre 1001 et 9001)
    • Partie recherche dans la classe personne semble illogique

  15. userFacadeImp

    • Duplication de codes !! (49 lignes de code dupliquée)

  16. CreateGameFactory: return type game(Toy, videogame ou boardgame) mais expect return un interface "FactoryGame"

  17. Le D.P Factory réduit l'extensibilité (si on ajoute un nouveau sous-type de Game, on doit également modifier la classe GameFactory et GameFactoryIF)

  18. Faire une interface pour une factory ???