search

matthieu637 / cpp-2a-info

CPP - Prépa des INP - Nancy | Projet Informatique 2ème année
https://matthieu637.github.io/cpp-2a-info/client.Reseau-class.html
MIT License
1 stars 4 forks source link

Resolution de l'issue avec les pseudos invalides #11

Closed david540 closed 7 years ago

david540 commented 7 years ago

Il fallait raise un RuntimeError avant le self.__connect = True (côté client) quand le serveur retourne une erreur (entier <0) Il y avait le même problème pour rejoindrePartie() quand on entrait le nom d'un joueur déjà pris par exemple

matthieu637 commented 7 years ago

Ok, mais ça serait mieux de ne pas déclencher d'exception, seulement retourner un nombre négatif et décrire l'erreur dans la doc. Dans notre cas, les exceptions sont réservés aux erreurs de programmation : j'appelle top avant creerPartie => stop le programme. Les retours négatifs ne représentent pas forcément des erreurs de programmation mais une mauvaise entrée utilisateur : rejoindrePartie avec un nom déjà pris, je redemande à l'utilisateur d'entrée un nom sachant que le retour a été négatif.

En théorie, on pourrait tout traiter par des exceptions, mais il faudrait introduire la gestion des exceptions à tout le monde (ce qui n'est pas au programme il me semble).

david540 commented 7 years ago

Voilà j'ai simplement mis le self.__connect= True dans un if si le serveur ne retourne pas d'erreur

matthieu637 commented 7 years ago

Je prends, merci.