My own Script

[JeffreyQ]

I can't find any tutorial for this. I would like to know if i want to create a custom script. Can it call functions?

Example of what i did. But it gives errors

function ConeBool takes unit caster, unit victim, real radius returns boolean
	if (Cos(( GetUnitFacing(caster) - AngleBetweenPoints(GetUnitLoc(caster), GetUnitLoc(victim) )) > Cos( radius / 2.00 )) then
		return true
        endif

	return false
endfunction

I don't know what is wrong, anyone can help me? I am new to this and wanted to try out something new.
I want it so that i can use:

• custom script: set udg_inConeRadius = ConeBool(castingunit, pickedunit, 180)

Just an example.

Thanks for helping!

 

[ap0calypse]

castingunit and pickedunit aren't the correct codes...

Casting Unit = GetSpellAbilityUnit()
Picked Unit = GetEnumUnit()

Edit: okay, if the castingunit and pickedunit are variables (local variables), then it could be correct. but: in that case you didn't use the correct names in the function.

This is one of my test-functions:

function test takes integer I, integer INT returns string
    local string IntT = I2S(I) + I2S(INT)
    return IntT
endfunction

function testing takes nothing returns nothing
    local integer I = 66
    local integer INT = 6
    
    call BJDebugMsg(test(I, INT))
endfunction

It will show "666".
You can easily convert the lower function to GUI, but then it has to be "call BJDebugMsg(test(udg_I, udg_INT))".

 

[JeffreyQ]

Well, i wanted the units to be pass in as arguments. And i wanted to put this in the script of the map. Not in the trigger ]:

 

[ap0calypse]

Well, i wanted the units to be pass in as arguments. And i wanted to put this in the script of the map. Not in the trigger ]:

What're you talkig about?
I put the test-function in the map's script and the testing-function as a GUI-trigger when I tested it, just like in your example O_O

 

[Justify]

castingunit and pickedunit aren't the correct codes...

Casting Unit = GetSpellAbilityUnit()
Picked Unit = GetEnumUnit()

Edit: okay, if the castingunit and pickedunit are variables (local variables), then it could be correct. but: in that case you didn't use the correct names in the function.

Why not? Parameters and the values that're set in the function needn't have the same names, like you may see with your udg example.

The problem is the overuse of brackets.
To bad colors don't work with jass, so I'll show it without:

if (Cos(( GetUnitFacing(caster) - AngleBetweenPoints(GetUnitLoc(caster), GetUnitLoc(victim) )) > Cos( radius / 2.00 )) then

-> erase the first green bracket.

 

[JeffreyQ]

Urs works perfectly, but mine doesn't.
I tried writing the script but it didn't work. Lets assume i never put any function call now. The errors are from the script.

EDIT: AHH! Thanks alot Justify! +rep! Stupid brackets are confusing when without color.

 

[Justify]

Like I already said, you have a bracket to much...

EDIT: Didn't see your post^^

 

[JeffreyQ]

Hmm what is the different between CosBJ and Cos?

 

[Justify]

It's the same, but CosBJ calls Cos.
Its only a single call more.
Blizzard avoided to use natives directly for their
GUI, kind of stupid, but k.

EDIT: Forgot about the part from DSG :x ^^

 

[Dr Super Good]

Hmm what is the different between CosBJ and Cos?

CosBJ is stupid and works in degrees by converting the input into radians via multiplying with pi/180 and passing it to the Cos function.
Cos is accurate and works by complex maths converting the input value of radians into its coosponding trigonometric ratio.