Feedback #1
Feedback #1
Conversation
![github-classroom[bot]](https://avatars.githubusercontent.com/in/68187?s=60&v=4){kind=small}
| private Wheel backWheel ; | ||
| public TagAlongBike(Bike parentBike, Bike childBike) { | ||
| super(); | ||
| this.parentBike = parentBike; |
There was a problem hiding this comment.
C'est un vélo avec
- Un vélo parental,
- Un vélo enfant, et
- Deux roues.
Ca fait une vélo à 6 roues, et implique une simulation erronée.
| private Bike childBike; | ||
| private Wheel frontWheel; | ||
| private Wheel backWheel ; | ||
| public TagAlongBike(Bike parentBike, Bike childBike) { |
There was a problem hiding this comment.
Il a fallu un constructeur sans paramètre pour que l'injection de dépendance marche.
| - Oui, il existe déjà une abstraction de la classe 'Bike' qui est la classe 'Vehicle'(puisque 'Bike' etend 'Vehicle'). la méthode 'hetPush()' utilisée par 'Wheel' n'est pas astraite dans 'Vehicle', mais | ||
| pourrait être ajoutée en tant que méthode abstraite à 'Vehicle' ou en tant qu'interface séparée | ||
| - L'abstraction 'Vehicle' se trouve dans le paquetage transport | ||
| - Pour casser la dépendance cyclique, on pourrait introduire une interface, par exemple 'PushProvider', qui aurait une méthode 'getPush()'.'Bike' implémenterait cette interface ensuite, 'Wheel' ne dépendrait plus de 'Bike' directement, mais plutôt de l'interface 'PushProvider' |
There was a problem hiding this comment.
- Pour quoi introduire une nouvelle interface alors que
Vehicleexiste déjà ? - Il aura fallu passer par la classe
Vehiclepour éviter de charger le code avec des nouveaux types.
| @@ -0,0 +1,5 @@ | |||
| package fr.polytech.sim.transport; | |||
|
|
|||
| public interface PushProvider { | |||
There was a problem hiding this comment.
I'inteface pourrait être replacer par la classe abstraite Vehicle.
| @@ -16,4 +16,10 @@ protected NamedLogger(String name) { | |||
| this.name = name; | |||
| } | |||
|
|
|||
| protected String logMessage(String format, Object...args){ | |||
There was a problem hiding this comment.
Cette méthode ne respecte pas le patron "methode template" qui exige au moins une méthode abstraite. Voir support des cours pour plus d'info.
| ## Exercices 5 | ||
|
|
||
|
|
||
| ## Exercices 6 |
There was a problem hiding this comment.
Réalisation manquante dans le code.
| une horloge | ||
| - La méthode fabriqueb: permet de creer des objets sans spécifier la classe exacte de l'objet qui ser crée ce qui permet de centraliser la logique de création des instances, offrant ainsi plus de flexibilité pour changer le type d'objets | ||
| crées sans changer le code qui utilise ces objets. | ||
|
|
||
| ## Exercices 7 |
There was a problem hiding this comment.
Réalisation manquante de décorateur !
| ## Exercices 7 | ||
|
|
||
| ## Exercices 8 | ||
|
|
||
| -Le Patron de conception de la classe context avec ServiceLoader suit le patron de conception FACADE car en encapsulant la complexité de 'ServiceLoader' derrière une interface simplifiée, 'context' rend l'utilisation des services plus accessible et plus |
There was a problem hiding this comment.
Le code ne marche pas à cause de constructeur sans paramétre manquant.
| this.components.add(backWheel); | ||
| } | ||
| @Override | ||
| public double getPush(){ |
There was a problem hiding this comment.
À quoi ca sert cette rédifinission ?
👋! GitHub Classroom created this pull request as a place for your teacher to leave feedback on your work. It will update automatically. Don’t close or merge this pull request, unless you’re instructed to do so by your teacher.
In this pull request, your teacher can leave comments and feedback on your code. Click the Subscribe button to be notified if that happens.
Click the Files changed or Commits tab to see all of the changes pushed to
mainsince the assignment started. Your teacher can see this too.Notes for teachers
Use this PR to leave feedback. Here are some tips:
mainsince the assignment started. To leave comments on specific lines of code, put your cursor over a line of code and click the blue + (plus sign). To learn more about comments, read “Commenting on a pull request”.main. Click a commit to see specific changes.For more information about this pull request, read “Leaving assignment feedback in GitHub”.
Subscribed: @karelle19