DraftBot
DraftBot copied to clipboard
Refactor Grouping parameters for commands
suggestion de refactor (dans une prochaine maj) : pouvoir présenter à l'entrée de la commande un objet contenant embed + player + language + translationmodule de la commande en cours.
Je m'explique : Beaucoup trop de commandes dans le bot demande généralement ces 4 paramètres, et ca gène la lecture des signatures des fonctions. La plupart du temps les 4 sont passés sans réflexion, et aucune autre alternative est nécessaire pour la commande. En réunissant les 4, on réduit la charge de pensée, et ainsi on simplifie la lecture du code.
Exemple sur cette entête : on passerait de 6 à 3 arguments : CommandState (les 4 paramètres surutilisés), League et scoreToAward. En ayant juste ca, on sait ce que la commande fait sans avoir à ignorer les paramètres redondants
Originally posted by @romain22222 in https://github.com/DraftBot-A-Discord-Adventure/DraftBot/pull/1776#discussion_r1140766953