{"action":"opened","issue":{"url":"https://api.github.com/repos/kkakama/J-XMCDA/issues/19","repository_url":"https://api.github.com/repos/kkakama/J-XMCDA","labels_url":"https://api.github.com/repos/kkakama/J-XMCDA/issues/19/labels{/name}","comments_url":"https://api.github.com/repos/kkakama/J-XMCDA/issues/19/comments","events_url":"https://api.github.com/repos/kkakama/J-XMCDA/issues/19/events","html_url":"https://github.com/kkakama/J-XMCDA/issues/19","id":296571337,"number":19,"title":"Livraison 3","user":{"login":"oliviercailloux","id":1244880,"avatar_url":"https://avatars2.githubusercontent.com/u/1244880?v=4","gravatar_id":"","url":"https://api.github.com/users/oliviercailloux","html_url":"https://github.com/oliviercailloux","followers_url":"https://api.github.com/users/oliviercailloux/followers","following_url":"https://api.github.com/users/oliviercailloux/following{/other_user}","gists_url":"https://api.github.com/users/oliviercailloux/gists{/gist_id}","starred_url":"https://api.github.com/users/oliviercailloux/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/oliviercailloux/subscriptions","organizations_url":"https://api.github.com/users/oliviercailloux/orgs","repos_url":"https://api.github.com/users/oliviercailloux/repos","events_url":"https://api.github.com/users/oliviercailloux/events{/privacy}","received_events_url":"https://api.github.com/users/oliviercailloux/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2018-02-13T00:18:25Z","updated_at":"2018-02-13T00:18:25Z","closed_at":null,"author_association":"NONE","body":"Alts\r\nPas trouvé issue !\r\n\r\nCrits\r\n{Pom incorrect : url ; artifactId (voir BP Maven) ; à formatter ; jersey inutile. Noms de packages insuffisants.} (−1) \r\nCode à formater correctement (−1). \r\nProtéger contre nulls, documenter le contrat (voir BP nulls) (−1). \r\nLa méthode qui produit un critère devrait être statique, puisqu’elle n’a pas de sens au niveau d’une instance de critère. \r\nWarnings Eclipse (−1). \r\nIndiquer ordre des champs avec annotation. \r\nReprésentation d’un ensemble d’évaluations en CSV manquante (−0,5). \r\nÉvaluations vers Json : excellent usage de la table Guava (+1). \r\nTests attendus pour vérifier conversions. (−1)\r\n\r\nServiceDescr\r\n{Pom incorrect : url ; artifactId (voir BP Maven) ; à formatter ; jersey inutile. Noms de packages insuffisants.} (−1) \r\nNe pas masquer les exceptions (createJson entre autres) (−2). \r\nEncodage et décodage json doivent se faire entre String et objets typés pour plus de généralité (pas à partir de fichiers, sinon par exemple impossible à tester). \r\nDécodage en JSON devrait être rendu plus lisible et élégant (ne pas repasser à un JsonParser alors que vous avez déjà les données). \r\nLa méthode setMap n’encourage pas à une utilisation claire de cet objet : il devrait plutôt être modifié par l’utilisateur (ou utilisé en lecture) mais pas voir son contenu remplacé entièrement. \r\nProtéger contre null, documenter, voir BP null.\r\nWarnings Eclipse.\r\nDécodage Json ne doit pas être dans l’instance de l’objet.\r\nLes constantes doivent être séparées par des soulignés bas (CAPS_SEPARATED)\r\n\r\nBasicServlets (2)\r\n{Pom incorrect : url ; artifactId (voir BP Maven) ; à formatter ; jersey inutile. \r\nNoms de packages insuffisants.} (−1) \r\nRetirer fichier web.xml pour simplifier. (−1) \r\nLe conteneur (ni l’utilisateur d’ailleurs) ne peut pas faire la différence entre des servlets qui écoutent tous sur le même chemin. Le chemin devrait être, par exemple, /alternatives/get/id. \r\nC’est getEvaluations (au pluriel) qui est demandé, pas getEvaluation. \r\nBons choix de méthodes HTTP (+1). \r\nLes servlets de modification doivent lier des données à des problèmes (− 2). \r\nInutile de re-créer un objet Response qui contourne un standard existant : envoyer simplement une réponse HTTP classique, avec un code de statut et un contenu éventuels qui indiquent l’erreur éventuelle (−1). \r\nsetEvaluation ne contient pas assez de données. \r\nBonne utilisation de la conversion en XML (+1).\r\n\r\nOnline\r\n{Pom incorrect : url ; artifactId (voir BP Maven) ; à formatter ; jersey inutile. Noms de packages insuffisants.} (−1) Configuration correcte. Veuillez indiquer l’URL dans le fichier README du projet.\r\n\r\nSetDB1\r\nAttention, ce qui est implémenté pour le moment est du JDBC et non du JPA. Cela ne rapportera pas de points. Voir séances correspondantes.\r\n\r\nBasicClient\r\nObjectsXML\r\nGenSchema\r\nUseDB\r\nSOAP\r\nIntrouvables\r\n"}}
{"actor":{"display_login":"oliviercailloux"}}