[JASS] "must not take any arguments when used as code"

[Darkholme]

How do I get around this? I want to call a function from a ForGroupBJ(). There are 2 different scenarios, and one of them will need a variable integer passed to it. What do I do?

Code Snippet

      if (remainder < 0) then
        set remainder = remainder/allCount
        if (remainder<1) then
        call ForGroupBJ(all, function DeincrementLife(1))
        else
        call ForGroupBJ(all, function DeincrementLife(remainder))
        endif
      endif

 

[Earth-Fury]

store it in a global, call ForGroup, and use the global in the callback. Also, don't use ForGroupBJ... use:

native ForGroup takes group whichGroup, code callback returns nothing

 

[Darkholme]

store it in a global, call ForGroup, and use the global in the callback. Also, don't use ForGroupBJ... use:

native ForGroup takes group whichGroup, code callback returns nothing

I'm actually not sure how to make a global in Jass. I've done it with the menu in GUI before though. How DO I declare a global in jass?

 

[Herman]

Same way

Just in Jass, you must refer to them with "udg_" in front of them, which stands for "User Defined Global", because wc already generates it's own globals, but it needs to differentiate between them in the code (like when you create a trigger then convert it, there is a generated trigger global)

 

[Earth-Fury]

you really should just use the JASS NewGen editor so you can define your own globals. Get it here: http://wc3campaigns.net/showthread.php?t=90999

Then you define globals in globals blocks:

globals
    [constant] <type> <name> [= <default value>] // The basic syntax
    unit MyUnit // a valid example
    integer MyInt = 34
    constant integer THIS_VAR_CAN_NOT_BE_SET = 34523
endglobals

You can define as many or as few as you want in as many global blocks as you want. Where a block of globals is in the maps script doesn't matter, you can use them anywhere. constant globals can not have their values changed. Do not initialize globals using functions. Eg, this will crash WC3:

globals
    unit u = CreateUnit(...)
endglobals

 

[DiscipleOfLife]

In this case you could use bj_forLoopAIndex for example (so you don't need to declare your own globals)
Example:

function DeincrementLife takes nothing returns nothing
    call SetUnitState(GetEnumUnit(), UNIT_STATE_LIFE, GetWidgetLife(GetEnumUnit())-bj_forLoopAIndex)
endfunction
//......
      if (remainder < 0) then
        set remainder = remainder/allCount
        if (remainder<1) then
        set bj_forLoopAIndex = 1
        call ForGroup(all, function DeincrementLife)
        else
        set bj_forLoopAIndex = remainder
        call ForGroup(all, function DeincrementLife)
        endif
      endif