[JASS] Scope

[Destroyer95]

I started with scopes today and i wanted to practice so i made something. But it doesn't work. Why?

scope vars
function init takes nothing returns nothing
    set target = GetSpellTargetUnit()
    set caster = GetSpellAbilityUnit()
    set x = GetUnitX(target)
    set y = GetUnitY(target)
    set c = GetUnitLoc(caster)
    set angle = GetUnitFacing(caster)
endfunction
endscope

Thx!

 

[Squiggy]

scope vars initializer *InitTrig*

Would work
Be sure to name *InitTrig* to the same as you named your initialization trigger

 

[Destroyer95]

could u explain a bit? im kinda noob at this xD

EDIT: I did that and it still doesn't work. Here's the scrip now

scope vars initializer InitTrig_Spell

function init takes nothing returns nothing
    set target = GetSpellTargetUnit()
    set caster = GetSpellAbilityUnit()
    set x = GetUnitX(target)
    set y = GetUnitY(target)
    set c = GetUnitLoc(caster)
    set angle = GetUnitFacing(caster)
endfunction
endscope

 

[Coolty44]

All triggers in JASS need an init function, or they wont work properly (I think libarys are exempt from this rule sometimes though). This is a basic init function that you'll find in alot of JASS spells (Atleast the one I use anyway)

    private function Init takes nothing returns nothing
        local trigger I = CreateTrigger()
        call TriggerRegisterAnyUnitEventBJ(I,EVENT_PLAYER_UNIT_SPELL_CAST)
        call TriggerAddCondition(I,Condition(function Conditions))
        call TriggerAddAction(I,function Actions)
        set I = null
    endfunction

Copy that, change the scope part to look like this

scope Vars initializer Init

Its best you reserve using "Init" as your initializer function, and call everything else something diffrent like Actions and Conditions

Also, whatever your main part is named (the actions, what it does), be sure to change the TriggerAddAction part to your action functions name, same goes for the Condition. If you dont have a condition, then remove the TriggerAddCondition line. If it isnt made for a spell, then you'll need a diffrent event line too.

It should work now.

 

[Squiggy]

A scope is like a wrapper, inside a scope you can name handles as the same as in any other scope as long as the scope has another name.

Your initTrig would look like this

   private function InitTrig takes nothing returns nothing
        local trigger t = CreateTrigger()
        call TriggerRegisterAnyUnitEventBJ(t, blah)
        call TriggerAddCondition(t, Condition(function Conditions))
        call TriggerAddAction(t, function Actions)
        set t = null
    endfunction

The name of the initializer has to be the same as the init trigger

For I am also just learning jass, I am not that expierienced ^.^, so that's almost all I can tell about scopes.. maybe a bit more


//edit: aww beat me to it

 

[aznricepuff]

Um, you dont need inittrigs with vjass. You could do this just fine:

scope A initializer init
private function init takes nothing returns nothing
endfunction
endscope

Scopes don't even need initializers.

And by "it doesnt work" what do you mean? If it doesn't compile, then are you sure you declared your variables (target, etc.) as globals?

And GetSpellTargetUnit() and GetSpellAbilityUnit(), and every other event response function don't work inside initializers. They'll only work inside trigger actions or conditions. Maybe you want something like this:

scope vars initializer init
globals
    private unit target
    private unit caster
    private real x
    private real y
    private location c
    private real angle
endglobals
private function actions takes nothing returns nothing
    set target = GetSpellTargetUnit()
    set caster = GetSpellAbilityUnit()
    set x = GetUnitX(target)
    set y = GetUnitY(target)
    set c = GetUnitLoc(caster)
    set angle = GetUnitFacing(caster)
endfunction
private function init takes nothing returns nothing
    local trigger t = CreateTrigger()
    call TriggerRegisterAnyUnitEventBJ(t, EVENT_PLAYER_UNIT_SPELL_EFFECT)
    call TriggerAddAction(t, function actions)
endfunction
endscope

 

[Destroyer95]

scope vars initializer Init
globals
     constant integer id = 'spel'
     private unit target
     private real x
     private real y
     private unit caster
     private location c
     private real angle
endglobals
private function actions takes nothing returns nothing
    set target = GetSpellTargetUnit()
    set caster = GetSpellAbilityUnit()
    set x = GetUnitX(target)
    set y = GetUnitY(target)
    set c = GetUnitLoc(caster)
    set angle = GetUnitFacing(caster)
endfunction
private function Init takes nothing returns nothing
    local trigger t = CreateTrigger()
    call TriggerRegisterAnyUnitEventBJ(t, EVENT_PLAYER_UNIT_SPELL_EFFECT)
    call TriggerAddAction(t, function actions)
endfunction
endscope

Still doesn't work. It says syntax error (for scope vars) and statement outside of function for all others. And it also says that those varialbes are undeclared (in actions).

 

[Peetoon]

Are you sure you're using JNGP(JassNewGenPack)? Scopes are a part of vJASS, not of normal JASS, and you need JNGP for vJASS.

 

[Destroyer95]

Yea im sure

 

[Peetoon]

What aznricepuff said works perfectly fine for me.

 

[Yixx]

Do not use the 'Syntax Check' button, it does not work and just sits there for nothing =\
Save the map instead (yes, save), that compiles the code and checks for REAL mistakes,,

 

[Cokemonkey11]

Yixx is right, in the current JNGP the syntax checker button does not recognize vJass,