profcfuhrmanets / log210-systeme-gestion-bordereau-node-express-ts

Système de gestion des bordereaux (SGB) utilisé dans le cadre du cours de LOG210.
https://profcfuhrmanets.github.io/log210-systeme-gestion-bordereau-node-express-ts/
0 stars 17 forks source link

Le SGB ne suit pas la méthode de conception proposée dans le cours #9

Open Olon11 opened 2 months ago

Olon11 commented 2 months ago

Les modèles ne sont pas des classes qui représentent les classes conceptuelles du MDD de SGB. Ils ressemblent plus à des contrôleurs.

Par exemple Course ne contient pas de propriété id, prealable ou titre. Il contient une méthode statique qui retourne tous les cours. C'est plutôt le rôle du contrôleur de faire ça. À cause de ça, les contrôleurs ne font que transmettre l'appel de méthode vers le « modèle », ce qui n'apporte rien.

fuhrmanator commented 2 months ago

C'est pas important pour l'instant, surtout que tu spécifie que regarder la conception n'est pas le but #10

Olon11 commented 2 months ago

Oui, je suis d'accord, mais il faut donner l'exemple aux étudiants. Un étudiant pourrait vouloir s'inspirer de la structure du SGB et alors, son projet serait incorrect.

Olon11 commented 2 months ago

C'est pire que ce que je pensais. Par exemple, dans Insert Grade, le routeur donne l'objet query au contrôleur, qui le donne à la classse du domaine. Exactement ce qu'on dit aux étudiants de ne pas faire.

Pourquoi help wanted? Je peux m'en charger.

Olon11 commented 2 months ago

J'ai fait une restructuration pour insert grade dans e9a570a