Worldedit

[Crowolf]

Bonjour, voici mon premier sort en vJass. J'ai lu un tuto de FlamePhoenix sur Hive et j'ai essayé de faire un sort simple, un BlinkStrike. Mis à part le fait qu'il soit très simple (pas d'effet), je demande s'il y a moyen de l'améliorer au niveau du code, de supprimer les leaks etc...

Secret: Jass: scope BlinkStrike initializer Init //--------------------------------------------------------------------------- //---------------------------------------------------------------------------     globals         private constant integer Spell_ID = 'A000'         private constant real Damage_per_Lvl = 75.0     endglobals //--------------------------------------------------------------------------- //--------------------------------------------------------------------------- private function Conditions takes nothing returns boolean     return GetSpellAbilityId() == Spell_ID endfunction private function Actions takes nothing returns nothing     local unit Caster = GetSpellAbilityUnit()     local unit Target = GetSpellTargetUnit()     local real X = GetUnitX(Target)     local real Y = GetUnitY(Target)     call SetUnitX(Caster, X)     call SetUnitY(Caster, Y)     if IsUnitEnemy(Target, GetTriggerPlayer()) then         call UnitDamageTarget(Caster, Target, I2R(GetUnitAbilityLevel(Caster, Spell_ID))*Damage_per_Lvl, true, false, ATTACK_TYPE_NORMAL, DAMAGE_TYPE_UNKNOWN, WEAPON_TYPE_WHOKNOWS)         else     endif     set Caster = null     set Target = null endfunction //=========================================================================== private function Init takes nothing returns nothing     local trigger t = CreateTrigger(  )     call TriggerRegisterAnyUnitEventBJ( t, EVENT_PLAYER_UNIT_SPELL_EFFECT )     call TriggerAddCondition( t, Condition(function Conditions ))     call TriggerAddAction( t, function Actions ) endfunction endscope

Autre question, est-ce que vous avez des astuces pour connaître les noms des fonctions? Je galère énormément à ce niveau là.
Question bonus: Quel est l'équivalent en vJass d'un trigger periodique en GUI?
Merchi beaucouplus.
_________________

[Bantas]

[Crowolf]

Oui, je connais et utilise function list. Le problème c'est pour savoir ce que font ces functions, même si c'est souvent assez instinctif et pas trop dur. Il n'y a pas de petit texte explicatif comme en gui. Je suppose qu'avec la pratique, je devrais m'en sortir de mieux en mieux.
_________________

[Bantas]

[Brissou de Mourièssou]

Tu as un else qui ne sert à rien.
_________________

[Troll-Brain]

Pour la lisibilité / maintenabilité du code tu devrais appliquer des conventions différentes pour le choix de tes noms de variables/functions.

On en parle là bas :
http://worldedit.free.fr/forum/viewtopic.php?t=4466&start=0

Et "t" peut faire penser à un timer, "trig" serait plus judicieux.

PS :
Content que tu adhères à la secte
_________________

Le violet, c'est moche.

[Brissou de Mourièssou]

Et je te conseille des minuscules pour le début de tes noms de variables, ça respecte la sémantique du jass et surtout ça fait gagner du temps
_________________