lzear / LeekWars_Kikimeter

5 stars 3 forks source link

Supprimer les "say" du tableau d'Utilisation des PT #16

Closed abourdin closed 10 years ago

abourdin commented 10 years ago

Actuellement les data "blabla" viennent fausser les pourcentages dans le tableau d'utilisation des PT.

yLark commented 10 years ago

Pas à mon sens, les blabla consomment des PT. Certes, presque tout le monde ne les utilise que s'il en reste de disponible, mais tout de même. Dans la même logique, est-ce qu'on peut dire que les échecs faussent les pourcentages de PT ?

abourdin commented 10 years ago

Ahah, je me retrouve bien con, je n'avais jamais remarqué sur la commande say consommait des PT. Au temps pour moi !

Les échecs ne sont pas comptabilisés dans le compte d'armes/chips et PT utilisés actuellement ?

yLark commented 10 years ago

Ah oui d'accord, peut-être que tu vas reprendre un peu de ton code de poireaux ? :-P

Je n'ai pas bien compris le fond de ta pensée. Les PT dépensés pour une arme/chip qui finissent en échec sont comptabilisés comme échec.

abourdin commented 10 years ago

Non ça va, mes messages n'étaient envoyés qu'en fin de tour s'il y avait eu un kill de fait, donc ce n'était pas impactant, en revanche maintenant je comprends pourquoi certaines fois les messages n'étaient pas envoyés alors qu'ils auraient dû ^^ Si ça avait dû réduire mes PT et me faire sauter des actions, je pense que je m'en serais rendu compte :P

J'essayais simplement de comprendre ce que tu voulais dire par "les échecs faussent les pourcentages de PT".

yLark commented 10 years ago

Je disait juste que si les blabla faussent le calcul de PT, c'est pareil pour les échecs. Ils ont la même légitimité. Ça faisait partie de notre quiproquo, c'est pas important.

Foudge commented 10 years ago

Je rejoins un peu Corshclaw dans sa remarque initiale. Le but de la consommation moyenne des PT, c'est de voir si l'on exploite bien un maximum de PT pour le combat (cad armes et puces, que ça réussisse ou pas) alors que le "blabla" c'est 100% inutile et on est beaucoup à l'utiliser pour consommer les PT inutilisés.

Et c'est justement là le problème : comment évaluer le nombre de PT inutilisés si on les utilise en "blabla" ? Actuellement, il faut que le joueur fasse le calcul manuellement : PT moyen - (nb_blabla/nb_tours). Et c'est ce que je fais (à la louche). C'est un peu dommage de ne pas automatiser ce calcul, non ? J'en suis à me demander si je devrais pas abandonner l'utilisation du de la fonction say() juste pour avoir un résultat qui correspond réellement à ce qu'on cherche à avoir. Ou alors le remplacer par lama() qui lui ne consomme rien.

On pourrait peut-être rendre cela optionnel ? [x] prendre en compte le blabla dans les calculs de PT Option activée par défaut histoire de conserver le comportement actuel.

yLark commented 10 years ago

Oui, ou alors créer une autre donnée en plus de ptperturn : ptperturn2 qui ne tient pas compte des say(). La différence pourrait être documentée dans un tooltip au survol du texte, ou autre chose.

abourdin commented 10 years ago

Puisque le débat est relancé, on pourrait tout simplement rajouter une checkbox dans la config et un hook sur les calculs de somme de PT utilisés pour exclure ou non les commandes say, ça ne devrait pas avoir de grosses implications.

yLark commented 10 years ago

Justement, ce que j'ai proposé juste au-dessus me semble plus simple à gérer et me parait plus logique. C'est un autre type de données, donc autant créer une nouvelle colonne, voir même deux :

abourdin commented 10 years ago

Peut-être avec des noms un peu plus explicites, par exemple usedPTfiltered, ça évitera qu'on s'y perde par la suite.

yLark commented 10 years ago

Oui. Et je propose qu'on implémente ces tooltips à afficher au survol, ça coûte pas cher en dev et ça ajoutera pas mal de clarté pour l'utilisateur.

utiliser le tag title='' devrait suffire, non ? Pas besoin d'utiliser le class='tooltip' du site.

yLark commented 10 years ago

Les décomptes des PT sans say() sont stockés dans d'autres variables : https://github.com/Zear06/LeekWars_Kikimeter/commit/ebe174f60d50bed7bbc76803726ad4a720ebfe37