search

fuhrmanator / log210-ndc-quarto

Source du manuel Analyse et conception de logiciels (Quarto Markdown)
https://fuhrmanator.github.io/log210-ndc-quarto/
8 stars 2 forks source link

Ajout d'exercices pour les GRASP #111

Closed Olon11 closed 1 year ago

Olon11 commented 1 year ago

105

fuhrmanator commented 1 year ago

J'ai tenté faire une solution cohérente avec la solution au problème de polymorphisme, dans la fabrique on y ajoute un type ayant les 3 possibilités dans addBird(). Ça élimine le cas d'erreur. Mais, ça risque d'être compliqué à comprendre pour les débutants en TypeScript. On pourrait faire un lien vers le Playground.

Olon11 commented 1 year ago

Je suis d'accord avec les changements.

J'ai tenté faire une solution cohérente avec la solution au problème de polymorphisme, dans la fabrique on y ajoute un type ayant les 3 possibilités dans addBird(). Ça élimine le cas d'erreur. Mais, ça risque d'être compliqué à comprendre pour les débutants en TypeScript. On pourrait faire un lien vers le Playground

Je ne suis pas certain de comprendre. Ces changements n'apparaissent pas dans ton commit. À moins que ce soit juste une réflexion...

Olon11 commented 1 year ago

59

fuhrmanator commented 1 year ago

Ces changements n'apparaissent pas dans ton commit. À moins que ce soit juste une réflexion...

C'est dans le lien du Playground (vois-tu le code)? Je disais que ce lien peut être éventuellement une solution alternative "avancée" en TypeScript, mais l'expliquer est p-e trop long.

Olon11 commented 1 year ago

Oui, j'ai vu le code dans le lien. Le commentaire était ambigu.

Effectivement, ça pourrait être confus pour les lecteurs qui ne sont pas familiers avec TypeScript. Je crois que c'est mieux de conserver le code plus simple.

fuhrmanator commented 1 year ago

Merci @Olon11 !