local variables in GUI

[Genyuumaru]

I learned some new things in GUJass.

I use locals to make spells MUI. Also, to resume the coding in GUI (and not write a custom script every time I use the local) I came up with this:

custom script: local unit unit1
custom script: local integer number
custom script: set udg_temp_unit = unit1
custom script: set udg_temp_number = number1
Use temp_unit and temp_number in all the actions I want to...
wait 5 seconds
custom script: set udg_temp_unit = unit1
custom script: set udg_temp_number = number1
Use them further...
custom script: set unit1 = null

I HATE JASS! But I love locals

 

[Viikuna]

I just make spells in GUI, convert them to JASS and convert the globals with locals.

Then null the locals.

Works fine for me.

This is actually how I started learning Jass ,that GUI converted Jass just looks so ugly that soon you want to start cleaning it and using natives instead of BJs.
Naming your functions also makes code easier to read, and when you get NewGen and vJass, you can start to write globals inside your code.
Its great! , no more any use for useless udg_something variables and that stupid variable editor.

I HATE JASS! But I love locals

locals are great, but when you learn how to use structs you are gonna love them even more.

Dont hate Jass, without it we could edit wc3 at all

EDIT. No Jass, no GUI.

 

[Genyuumaru]

What does BJ in functions mean by the way?

 

[AceHart]

Blizzard JASS - the functions from Blizzard.j.

 

[Genyuumaru]

Ummm... What If I remove it -_- Nothing will change?

 

[Viikuna]

It is better to use native functions instead of BJs in most cases, because it saves some useless function calls. And some BJs contain some actions you dont really want to do.

Compare this:

native CreateUnit takes player id, integer unitid, real x, real y, real face returns unit

and this:

function CreateNUnitsAtLocFacingLocBJ takes integer count, integer unitId, player whichPlayer, location loc, location lookAt returns group
    return CreateNUnitsAtLoc(count, unitId, whichPlayer, loc, AngleBetweenPoints(loc, lookAt))
endfunction

function CreateNUnitsAtLoc takes integer count, integer unitId, player whichPlayer, location loc, real face returns group
    call GroupClear(bj_lastCreatedGroup)
    loop
        set count = count - 1
        exitwhen count < 0
        call CreateUnitAtLocSaveLast(whichPlayer, unitId, loc, face)
        call GroupAddUnit(bj_lastCreatedGroup, bj_lastCreatedUnit)
    endloop
    return bj_lastCreatedGroup
endfunction

function CreateUnitAtLocSaveLast takes player id, integer unitid, location loc, real face returns unit
    if (unitid == 'ugol') then
        set bj_lastCreatedUnit = CreateBlightedGoldmine(id, GetLocationX(loc), GetLocationY(loc), face)
    else
        set bj_lastCreatedUnit = CreateUnitAtLoc(id, unitid, loc, face)
    endif

    return bj_lastCreatedUnit
endfunction

native CreateUnitAtLoc takes player id, integer unitid, location whichLocation, real face returns unit

native CreateBlightedGoldmine takes player id, real x, real y, real face returns unit

First one is native, second one is GUI action: CreateUnit