[Trigger] How to make this simply GUI MUI

[Anachron]

Event: A unit starts casting an ability
Condition: Abilty being cast equal to Plagued Brain
Action:
-----Do some action with target unit of ability being cast-----
wait 5 seconds
-----Do some action with target unit of ability being cast-----

Because of the target unit of ability being cast is null after 5 seconds, it does not work corretly. Can anyone help me please?

 

[Eleandor]

Make a global "unitvar" variable.

• Actions:
• Custom script: local unit udg_unitvar = GetSpellTargetUnit()
• -----Do some action with unitvar-----
• wait 5 seconds
• -----Do some action with unitvar-----
• Custom script: set udg_unitvar = null

 

[Anachron]

The actions I do also have to be in custom scripts or???
like Unit - Add Ability X to udg_unitvar. ?

 

[Eleandor]

nope.

*grumbles something about stupid 15 char limit*

 

[Anachron]

Edit: Do I need a variable for that? Without creating one before I can't use that variable in GUI, or?

Edit: Works fine now, thanks (&+rep)

 

[redscores]

Well, you cannot do that without a variable of any kind, best way is a unit variable, set the variable to the target unit of ability being cast and do the stuff, set the wait in, do the other stuff and set the unit var to no unit(to prevent leaks)


you need to create the variable in the variable editor. Yes.

 

[Anachron]

When I craete a loop and setted the udg_Tmp_Unit variable before, can I add waits in the loop and still make the trigger MUI?

 

[Eleandor]

If I understand you correctly, then yes.

 

[Anachron]

local integer udg_Tmp_Integer = ( 5 + GetUnitAbilityLevelSwapped('A00O', GetTriggerUnit()) )

IS RIGHT... wrong is..

EDIT:
set udg_Tmp_Integer = null

TYP CONFLICT!

EDIT2:
I 0-ed it and it works now.. wtf <.<

 

[redscores]

Maybe use jass and set a integer in jass? a local one? would make much more sense.

local integer udg_Tmp_Integer = ( 5 + GetUnitAbilityLevelSwapped('A00O', GetTriggerUnit())

thats a BAD idea

local integer Tmp_Integer = ( 5 + GetUnitAbilityLevelSwapped('A00O', GetTriggerUnit())

thats a good idea.

 

[Anachron]

• Custom script: local unit udg_Tmp_Unit = GetSpellTargetUnit()
• Custom script: local integer udg_Tmp_Integer = ( 5 + GetUnitAbilityLevelSwapped('A00O', GetTriggerUnit()) )
  • For each (Integer A) from 1 to Tmp_Integer, do (Actions)
    • Loop - Actions
      • Wait 1.00 game-time seconds
      • Unit - Cause (Triggering unit) to damage Tmp_Unit, dealing (12.50 + ((Real((Level of Weakening Epidemic for (Triggering unit)))) x 2.50)) damage of attack type spells and damage type magic
      • Set Tmp_Loc = (Position of Tmp_Unit)
      • Specialeffect - Create a special effect at Tmp_Loc using Abilities\Spells\Other\Volcano\VolcanoMissile.mdl
      • Custom script: call RemoveLocation(udg_Tmp_Loc)
      • Set Tmp_Effect = (Last created special effect)
      • Specialeffect - Destroy Tmp_Effect
  • Custom script: set udg_Tmp_Unit = null
  • Custom script: set udg_Tmp_Integer = 0

And whats wrong with this?

 

[Eleandor]

local integer udg_Tmp_Integer = ( 5 + GetUnitAbilityLevelSwapped('A00O', GetTriggerUnit()) )

IS RIGHT... wrong is..

EDIT:
set udg_Tmp_Integer = null

TYP CONFLICT!

EDIT2:
I 0-ed it and it works now.. wtf <.<

You can't, or at least shouldn't, null the following variables: integers, reals, strings. (it's possible I forgot one)

 

[Anachron]

And the new problem? Thanks for your graet help )

 

[redscores]

What the hell, why do you use udg as a local, it is used to sign globals -.- oh god.

 

[Anachron]

Umm because else I can not use them in GUI ???
( I am not that good in jass )

 

[redscores]

Simply do it in jass.^^

Convert your whole trigger in jass and use locals, it is so easy at such a trigger.

Simply add a local unit Target = TriggerUnit()

and remove udg_ from the vars and set them corretly in the trigger.


you could give me the map and i will make it in jass, so it is 100% MUI.

 

[Anachron]

With converting it into jass and remove the udg_ from units it also should be full MUI, doesn't it???

 

[redscores]

well, yeah, but the triggerunit leaks slightly.

 

[Eleandor]

No it doesn't. He was asking how to make GUI MUI, so I explained how you can *localise* a global variable. that's why you need the udg.

 

[Anachron]

My trigger does not work corrently, anyone know why?

 

[redscores]

Simply make it completely jass. So it is easier, no offense, but for MUI is JASS better.

 

[Anachron]

Omg I prefer GUI.. GUI Overview> Jass Overview....

please help with custom scripts and that like... >.<

 

[Eleandor]

show me your trigger you currently have and tell me the problem. Do you get a syntax error message when saving the map or does it just not work in game?

 

[GhostWolf]

Your trigger has no problems.

It could be coded better though.

You do not need to use the Special Effect variable. Simply create and destroy it in the following line (last created special effect).
You do not need to set the integer to 0 as it is a local.

A local variable loses its value when its function ends.

 

[Anachron]

It simply does not work with triggering it more then 1 time. (at the same time)