Closed fuhrmanator closed 1 month ago
fuhrmanator
commented
1 month ago J'ai regardé le code - il n'y a pas de notion d'étudiant connecté (au niveau design). On trouve une liste d'étudiants et résultatsÉtudiants (un autre tableau). Lorsqu'un étudiant se déconnecte ou se connecte à un quiz, on devrait avoir un objet sur lequel les affichages dépendent (séparation modèle vue). C'est assez bâclé et a besoin d'un design plus propre.
fuhrmanator
commented
1 month ago Il devrait y avoir un design plus propre et documenté sur le plan de la synchronisation et les événements sur les socket (rythme étudiant/prof, étudiants qui se joignent avant le démarrage, après, etc.). C'est géré avec des booléennes, p.ex. https://github.com/ets-cfuhrman-pfe/EvalueTonSavoir/blob/2a2cba6415cad2bbfb875d76064f9a694e3deab5/client/src/pages/Student/JoinRoom/JoinRoom.tsx#L106
Il y a du code dupliqué (possiblement pour traiter un pb de synchronisation) mais ça corrompt le compte des connexions.
J'ai démarré un quiz et j'ai démarré deux connexions étudiants (joe et ron). Voici la capture d'écran de manage-room:
Il affiche 4 connexions (plutôt que 2) et les noms n'appariassent pas.
Si joe soumet une réponse on la voit et son nom (si on active l'affichage des noms), mais le compte des connexions est toujours faux:
Si ron soumet, c'est la même chose:
Bref, le nombre de connexions est faux, puis l'étudiant n'est pas dans le tableau avant de soumettre une réponse.