Lesterpig / openparty-mafia

An openparty extension for Mafia role game
GNU General Public License v3.0
4 stars 4 forks source link

Add Gambler #43

Closed mens-sana closed 8 years ago

mens-sana commented 8 years ago

Description du rôle :

Le Gambler : Il peut, chaque nuit, décider de mettre sa vie en jeu en tentant de deviner la victime des Mafioso. S'il a bon, il reçoit le nom d'un Mafioso au hasard et pourra réessayer une prochaine nuit. Cependant, s'il se trompe, il meurt...

Remarques :

J'ai ajouté la vérification des paris dans la phase mafia pour avoir les résultats avant la phase du Secouriste. J'ai aussi ajouté la fonction qui prive le gambler de son pouvoir dans index.js pour que le Docteur et le Secouriste puissent l'appeler tous les 2.

Lesterpig commented 8 years ago

Merci :)

Cependant, serait-il possible de placer la totalité du code du parieur dans son propre fichier ? L'éparpillement m'ennuie un chouïa :/

À priori, c'est faisable via la propriété beforeAll du rôle + les évènements. Si tu as besoin de conseils vis à vis des évènements n'hésite pas :stuck_out_tongue_winking_eye:

mens-sana commented 8 years ago

Quel événement je peux appeler pour intervenir à la fin du vote des mafieux et avant la phase du rescuer ?

J'aimerais bien déplacer gambleOutcome et lockGamblers dans gambler.js mais module.exports est bloqué par le return

Lesterpig commented 8 years ago

Après la phase de la mafia tu as l'évènement "mafiaVote" :

https://github.com/Lesterpig/openparty-mafia/blob/master/stages/mafia.j s#L66

Pour le rescuer, tu peux créer un évènement sans problème depuis le fichier https://github.com/Lesterpig/openparty-mafia/blob/master/stages /mafia_after.js#L11

S'il n'est pas possible d'isoler les fonctions du parieur je comprendrais, mais c'est mieux si on isole le plus possible ;)

Merci !

mens-sana commented 8 years ago

Voilà ^^ Tu avais raison, je n'utilisais pas assez les événements dans la 1ere version.

C'est plus propre :)

PS : au cas où, est ce que ca te gene de nommer le rôle Gambler sans traduction ? les traductions françaises sont pas géniales...

Lesterpig commented 8 years ago

Le code m'a l'air excellent et parfaitement isolé, super ! :smile:

Je fais quelques tests et je fusionne tout ça, merci !

Lesterpig commented 8 years ago

Hm concernant le nom, Parieur peut convenir pour l'instant. Ça ne coûtera pas beaucoup de temps de changer si jamais on trouve une meilleure traduction un jour. Je tiens à mon français :)