oliviercailloux
commented
6 years ago Je ne trouve pas bsarkarim ni AymaneBarkaoui dans les inscrits au cours. Êtes-vous bien inscrits sous un nom qui ressemble à votre login ?
Open oliviercailloux opened 6 years ago
oliviercailloux
commented
6 years ago Je ne trouve pas bsarkarim ni AymaneBarkaoui dans les inscrits au cours. Êtes-vous bien inscrits sous un nom qui ressemble à votre login ?
Travis non configuré
POM dépendance json non standard suspecte dépendance vers norme servlet inutile puisque dépendance vers tout java ee est présente dépendances vers SLF4J et logback inutiles, JUL est à utiliser pour le log guava à mettre à jour (artifactId en minuscule, j’avais omis de préciser cette consigne, mais laissez le nom du dépôt GitHub en majuscules)
Qualité du code formatter le code !
Noms packages incorrects : vous ne controllez probablement pas le domaine correspondant (BP Maven)
Ne pas importer le code de JLP, l’utiliser comme bibliothèque !
27 erreurs et warnings détectés par Eclipse, à corriger.
BP Exceptions
Éviter toutes les méthodes en static, programmer en orienté-objet ; fournir des classes et des méthodes réutilisables et une découpe claire des responsabilités.
Servlets doivent répondre aux bonnes méthodes HTTP (réfléchir si Get ou Post ou autre).
Implémenter échec rapide, par exemple, renvoyer erreur lors appel servlet si problème détecté.
Pourquoi certaines variables sont au niveau de la classe et d’autres de la méthode ? Logique obscure.
Logger (en utilisant JUL).