Petites incohérence et améliorations à apporter dans le code

[voltou91]

Voici un début de liste de ce qu'il faudrait changer dans le futur :

• [ ] Mettre la fonction "killIfNeeded" dans la fonction "addHealth" et du coup la retirer partout où on enlève de la vie.

• [ ] Les fonctions comme "addMoney" ou "addExperience" utilise EditValueParameter alors que certaines fonctions comme "addHealth" ne l'utilise pas (pourtant ils ont les mêmes paramètres) (idem pour les fonctions sur les guildes).

• [ ] Certaines classes comme "Players" peuvent avoir le mot clé "static" pour fix les anti-patterns car elles ne contiennent aucune méthode non statique.

[voltou91]

En regardant dans la commande daily voici 2 changements à faire :

• [ ] Ici on save deux fois le player https://github.com/DraftBot-A-Discord-Adventure/DraftBot/blob/0c64a18295ea61b753de07710d27d0cce80d3b4a/src/commands/player/DailyCommand.ts#L125
• [ ] Ici on get l'inventaire dans "dailyNotReady" et "activeDailyItem", il faudrait le get avant "dailyNotReady" et le passer en paramètre dans ces deux fonctions afin de nous épargner une requête inutile https://github.com/DraftBot-A-Discord-Adventure/DraftBot/blob/0c64a18295ea61b753de07710d27d0cce80d3b4a/src/commands/player/DailyCommand.ts#L146-L151