Problématique Adapter Pattern

[Guiloux]

Description

Nous pouvons désormais filtrer nos films par acteur grâce à une API de filtrage. Cette API se trouve dans le dossier lib à la racine de notre projet. Le problème, c'est que la première version n'est pas très rapide et met une seconde à afficher les films filtrés : c'est énorme !

Nous avons décidé de développer une nouvelle version de cette API : lib/filter-v2. Seulement voilà, cette nouvelle version ne s'utilise pas tout à fait pareil.

_// Voici comment on utilise la V1_
const FilterLib = new FilterV1(this.Movies, actor)
const FilteredMoviesV1 = await FilterLib.filterByActor()

_// Voici comment on utilise la V2_
const FilteredMoviesV2 = await FilterV2.filterByActor(this.actor, this.Movies)

• Dans la première version, on doit instancier la librairie puis appeler la méthode filterByActor.

• Dans la deuxième version, nous n'avons plus besoin d'instancier : nous passons par une méthode statique. De plus, l'ordre des paramètres a été modifié.

User Stories

Première User Story

• (GIVEN) Je suis sur la page d'accueil de Filmo Patterns
• (WHEN) Je filtre les films d'Arnold Schwarzenegger
• (THEN) Les films d'Arnold Schwarzenegger s'affichent en utilisant la nouvelle API

[JohanPeraldi]

Dans le screencast présentant la solution à l'exercice dans le cours d’OpenClassrooms, Thomas instancie son adapteur de filtre en inversant les arguments attendus (ligne 17 de FilterForm.js):

const FilterLib = new FilterMoviesAdapter(actor, this.Movies)

Or c’est bien les films qu’il faut passer en premier argument et l’acteur en deuxième, comme indiqué dans la solution présentée ici:

const FilterLib = new FilterMoviesAdapter(this.Movies, actor)

[tdimnet]

Good call @JohanPeraldi ! Il n'y a plus qu'à espérer qu'OC fasse quelque chose ^^

[JohanPeraldi]

Excellent cours sur les design patterns cela dit! Merci.

[Barberousse93]

C'EST UNE ESCROQUERIE !! :D Il suffit de commenter la ligne : await new Promise(resolve => setTimeout(resolve, 1000)) pour que soudainement, l'exécution soit instantanée !! Bon... je sors...

[tdimnet]

Oops ^^

[Elishaya13]

L'acteur à été ajouté dans le constructeur des models , soit dans ExternalMovie.js pour "sylvester", soit dans Movie.js pour "arnold." (this._actor = "arnold") Voila si comme moi vous buguiez à vous demander comment l'app pouvait savoir si c'était arnold ou sylvester, ne trouvant pas l'information dans les data json ...