[JASS] Condensed if/then/else

[Bribe]

When converting to JASS directly from GUI, I get two basic types of if/then/else functions above many triggers' actions...

One is really long format:

function SLust2 takes nothing returns boolean
if ( not ( GetUnitAbilityLevelSwapped(GetSpellAbilityId(), GetTriggerUnit()) == 1 ) ) then
return false
endif
return true
endfunction

The other is much shorter format:

function SLustX takes nothing returns boolean
return ( UnitHasBuffBJ(GetTriggerUnit(), 'B004') == false )
endfunction

Is there a way to make the longer condition (above) into the same format as the shorter condition (below)? +Rep even if someone answered this question before you, I want to understand this function because I use it heavily in the map.

 

[YourNameHere]

function SLust2 takes nothing returns boolean
    return GetUnitAbilityLevel(GetTriggerUnit(), GetSpellAbilityId()) == 1
endfunction

Would return true if the Unit's level of the casted ability is 1.

 

[Berb]

The second one isn't an if-statement, its a boolean expression. A function that returns a true/false value. The first one includes an if-statement.

An if-statement is:

if (someBoolean) then
    // ...
else
    // ....
endif

If you were to combine a boolean expression with an if-statement, you would get:

if (SLust2()) then
    // ....
else
    // ....
endif

Basically, there is only one type of if-statement and that's as I have written it. What you could do is:

function SLust2 takes nothing returns boolean
    return not (GetUnitAbilityLevelSwapped(GetSpellAbilityId(), GetTriggerUnit()) == 1) then
endfunction

 

[Bribe]

function...
return not (GetUnityAbi... == 1) then
endfunction

Is that a false comparison, since you're using not?

 

[TriggerHappy]

It's basically just making the expression the opposite;
if not (true) then is the same as if (false) then.

 

[Berb]

Or in your situation, having:

return not (someInteger == 1)

Is the same thing as:

return (someInteger != 1)

There are lots of variations.