Merci pour la coloration du code.
| public void addOption(Option opt) {
options = new ArrayList<>();
options.add(opt);
}
|
J'aurais mis le options = new ArrayList
dans le constructeur de ta classe. En effet, avec ton code, chaque fois qu'on ajoute une option à une voiture, tu :
- remplaces la liste des options par une nouvelle liste (vite) ;
- ajoutes à cette nouvelle liste l'option qu'on t'a demandé t'ajouter.
Résultat des courses : toutes les options qui avaient été ajoutées avant sont perdues ! Il semble donc plus logique de d'assigner à option une liste vide au moment où tu crées ton véhicule, puis à te contenter de rajouter des options par la suite. C'est pour cela que je te conseille de mettre la première ligne dans le constructeur plutôt que dans addOption.
Après, c'est difficile de dire ce qui est « bien écrit » ou non sans avoir le reste du code et sans trop avoir le contexte du cours (à quoi les profs font attention, etc.).
Comme je l'ai déjà dit, je ne suis pas expert en Java… Et notre expert local est parti en vacances. Si quelqu'un d'autre a des idées, qu'ils n'hésite pas à répondre !