search

lzear / LeekWars_Kikimeter

5 stars 3 forks source link

Restructuration des objets #8

Closed yLark closed 9 years ago

yLark commented 10 years ago

Je n'ai pas la maîtrise suffisante de la POO et de JavaScript, mais je pense qu'il serait bon de reprendre la structure des objets. J'ai essayé, mais je n'ai pas vraiment réussi à faire ce que je voulais, donc j'ai abandonné.

ECMAScript5 permet des choses pas mal. On pourrait avoir des beaux objets et leurs attributs/méthodes du genre : currentFight.leek[name].dmg_out, currentFight.FightId, currentFight.leekCount, currentFight.team[1].name, currentFight.team[1].leekCount, currentFight.leek[name].dmg_out.turn(4) : pour avoir les dommages subis jusqu'au tour 4, et plein de fonctions pour calculer (et pas stocker) les données du genre TPperTurn, etc.

Cela permettra un code beaucoup plus propre et efficace. Notamment, pour les deux enhancement que je me suis assigné récemment. Sans cette amélioration des objets, j'arriverai à les faire, mais ça sera avec plein de fonctions toutes moches de partout.

lzear commented 10 years ago

Je vais essayer de faire des modifs sur les classes. Je vais faire ça dans une nouvelle branche pour qu'on puisse collaborer sans que ça nuise au script actuel.

lzear commented 10 years ago

Je pense avoir terminé les modifications. J'imagine qu'il faudrait que je documente un peu le code, je m'en occuperai prochainement. Il y a toutes les idées que tu avais données, avec une syntaxe un peu différente parfois : currentFight.leeks[name].dmg_out, currentFight.FightId, currentFight.nbLeeks, currentFight.team[1].name, currentFight.team[1].nbLeeks, currentFight.leeks[name].round[4].dmg_out : dommages émis pendant le tour 4

J'ai laissé ça dans une branche à part, le temps que l'on s'assure qu'il n'y a pas de problèmes à régler. N'hésite pas si tu as des remarques.

yLark commented 10 years ago

Excellent, je vais jeter un oeil dès que j'ai le temps.

yLark commented 9 years ago

Ça me semble fonctionner à merveille. Je close.