profcfuhrmanets / log210-enonce-lab0

Laboratoire d'introduction aux technologies utilisées en LOG210
MIT License
0 stars 4 forks source link

Confusion avec le changement de page (index2.pug) #4

Closed fuhrmanator closed 2 years ago

fuhrmanator commented 3 years ago

Plusieurs étudiants ont remarqué la confusion avec cette partie du laboratoire. Voici un commentaire détaillé laissé sur Discord:

Y'a beaucoup de gens qui ont été confus pas cette instruction. Il suffit de lire l'historique du salon pour s'en rendre compte. (moi compris)

Autres exemples:

  • Ce bouton devra redémarrer le jeux et afficher le contenu de la page de jeux en utilisant le fichier index2.pug
  • Est-ce que quelqu'un peut me dire avec la plus grande certitude ce qu'est la "page de jeux" ?
  • Est-ce index.puml au complet ? (l'énoncé désigne index.pug et index.puml comment étant le même fichier)
  • Est-ce le score des joueurs ?
  • Est-ce seulement le nom des joueurs ?
  • Est-ce que la "page de jeux" permet de jouer ?

Les views doivent avoir un meilleur nom.

L'utilisation de l'API est mauvaise. La route /api/v1/redemarrerNouveauJeu est la même que /api/v1/redemarrerJeu. Une api devrait seulement permettre les interactions direct avec l'état du jeu comme démarrer, jouer, terminer et redémarrer.

C'est l'application qui devrait se chargé de renvoyer les views comme "index2.pug". Sinon le javascript coté client devrait réutilisé les donnée reçus par l'API pour mettre à jour de document. (sans le document.write)

L'utilisation de document.write() m'a vraiment surpris. C'est selon moi la pire méthode possible pour modifier une page web. Pire que window.location.reload().

Une solution peut être de

Olon11 commented 3 years ago

Je propose une page qui permet d'afficher les joueurs en fonction de leur rang (ex.: le plus de parties gagnées ou le meilleur ratio).