JASS Issue

Ahem, this is a horrible piece of script, but oh well. All I did basically was copy and paste some converted text. What I'm attempting to do is create a variable that I can use for a few more functions, then remove it. Normally I would just use a Global but I have to be able to have 9 of these 'tempWeapon' variables for each player.

Thanks.

[quote]
// Add this function in front of the other one
function ConditionInAFunction takes nothing returns boolean
return GetUnitTypeId(GetFilterUnit()) == 'e001'
endfunction

// At top of function
local unit tempWeapon

set tempWeapon = GroupPickRandomUnit(GetUnitsInRangeOfLocMatching(70.00,GetUnitLoc(udg_Hero[GetForLoopIndexA()]), Condition(ConditionInAFunction)))

Somthing must be wrong with that, I get warning for 'endif's for the first 4 lines, and a 'Expected name' for the final line.

What the heck are you doing? cause it seems you are inserting a function inside a gui trigger, please learn JASS the real way.

I dont know JASS, I was just following what Zoxc suggested.

Well you said you converted the trigger into JASS.........

Not really,
I guess that wasn't really clear. Anyway I try to stick with GUI, so if you would be so kind as to help in that format (With custom scripts of course).

If youre using GUI you need to do this:

// Add this global script ( The trigger with the map name )

// At top of the trigger

// Then this where you want it

// At end of trigger

Locals can only be used in the function its created in. It can't be used in conditions except custom script coditions. And Pick All Actions, cuz WE or creating a new function for that.

Lovely little Error list.

Here is what my trigger looks like;


Once it works I'll replace all of those 'Random unit from..' with tempWeapon. Also, this If/Then/Else is one of 9 in this trigger, checking for 9 different unit types.

Please do what I say:

Yes, however this runs in a loop every .05 seconds and the local might be needed for all 9 If/Then/Else functions at the same time. If I place the scripts in those locations will it still work properly?