Acquis. Bon travail, attention à produire du code clair (organisation, nommage, indentation, commentaires, respect des conventions, optimisations). Voir remarques détaillées ci-dessous :
Classes à créer avec une majuscule en première lettre.
Controller à placer dans le package controller
Entités à réorganiser pour plus de clarté :
- Constantes
- Attributs
- Constructeurs
- Getters/Setters
- Equals/ToString
Variables à déclarer en camel case par exemple listContact au lieu de ListContact
Préférer ici une méthode du repository permettant de rechercher par prénom et par nom plutôt que de tout récupérer depuis la BDD et faire la recherche en Java.
|
List<contact> ListContact = (List<contact>) contactRepository.findAll(); |
|
ArrayList<contact> selection = new ArrayList<>(); |
|
|
|
|
|
for (contact l : ListContact) { |
|
if ((prenom.equals(l.getPrenom())) && (nom.equals(l.getNom()))) { |
|
|
|
selection.add(l); |
|
System.out.println(l); |
|
} |
|
|
|
} |
Attention aussi au defaultValue dans ton controller, par défaut la recherche se fera avec "aucun" ce qui n'a pas de sens.
Acquis. Bon travail, attention à produire du code clair (organisation, nommage, indentation, commentaires, respect des conventions, optimisations). Voir remarques détaillées ci-dessous :
Classes à créer avec une majuscule en première lettre.
Controller à placer dans le package
controllerEntités à réorganiser pour plus de clarté :
Variables à déclarer en camel case par exemple
listContactau lieu deListContactPréférer ici une méthode du repository permettant de rechercher par prénom et par nom plutôt que de tout récupérer depuis la BDD et faire la recherche en Java.
demojava/src/main/java/com/ipiecoles/javaexo/mesi/model/GreetingController.java
Lines 27 to 38 in 05aab9b
Attention aussi au
defaultValuedans ton controller, par défaut la recherche se fera avec "aucun" ce qui n'a pas de sens.