Debug this simple JASS code so it works OR Suggest a solution to my problem

[Nogusta]

My third and final question in the set:
How Do I Attach Item Tables to Newly Created Units?
and
When This Unit Dies Game Crashes. Why?

Well here goes:

Hey guys, let me tell you this one is pretty complicated and annoying, I suspect no one but Darg himself could figure it out:

I'm attempting to make a code that destroys the entrance and exit you could say, of any pair of waygates in which one of the waygates dies. (The gates are NOT pre-placed and are vulnerable to regular attacks.)

Here is a slightly bugged code I am trying to use:

Code

function your_trigger_actions takes nothing returns nothing
local unit u = GetDyingUnit()
local integer i = 0
loop
exitwhen ( GetBooleanOr( udg_WaygateA = u, udg_WaygateB = u )) %<--Problem%
set i = ( i + 1 )
endloop
set u = null
call ExplodeUnitBJ( udg_WaygateA )
call ExplodeUnitBJ( udg_WaygateB )
endfunction

For some reason WE says that it has an error and says it's expecting a "==" in the line indicated above.

But skipping over that let me tell you something about this code. It uses two global arrays of size 100. Another trigger not stated here stores a number of waygates within both arrays.

%%%%%The Problem I'm Trying to Solve%%%%%

You see the players can build an undetermined amount of waygates in this map. When a waygate finishes construction then a sister waygate will be created via triggers and automatically link up with the built one.

When this occurs the built waygate will also be stored within the index of the array 'WaygateA' and the sister waygate will be stored within the index of the array 'WaygateB'.

Making this even more complicated is the fact that a counter is attached to the indexes of the arrays. Every time a pair of waygates are stored within the pair of arrays the counter will go up by 1 so the next pair of waygates will be stored in the next entry of their corresponding arrays all the way up to 100.

This system makes sure that built and sister waygates all have the same index number in the two arrays so they can be used in the code above.

(Takes a breath) Finally, the code above simply makes it so when a unit dies the trigger will check for whether the dying unit was a waygate stored within either array and if it was attempt to destroy any waygate with the corresponding index number of the dying waygate therefore destroying both the built and sister waygates.

However of course... it doesn't do so for a reason that is beyond me. In fact I believe the loop in the code above may never even exit. But I am unfortunately fairly new at JASS and pretty nasty with its syntax.

Well the bottom line is: A little help guys?

 

[SD_Ryoko]

First I'll tell you, WE is right.

When doing an equal comparison, use ==
When setting a variable, use =
When doing an or, use ||
When doing an and, use &&

[edit]

And if you've got it wrong, WE will disabled that trigger.

 

[Nogusta]

I'd say it's a little more complicated then that. I previously used double equal signs and WE said it was alright. However it did say I needed something like "init(Trigger Name) function" or something. I think this is because I cleared all the custom script when I first converted the trigger in the first place.

So I remade the trigger and didn't clear all the JASS script already there. I just simply substituted my revised code:

Code

function your_trigger_actions takes nothing returns nothing
local unit u = GetDyingUnit()
local integer i = 0
loop
exitwhen ( GetBooleanOr( udg_WaygateA == u, udg_WaygateB == u ))
set i = ( i + 1 )
endloop
set u = null
call ExplodeUnitBJ( udg_WaygateA )
call ExplodeUnitBJ( udg_WaygateB )
endfunction

If my programming knowledge is correct this should accomplish what's described in my first post but it just doesn't, unless I still did something wrong with that initialize function I mentioned earlier even though WE didn't disable the trigger. I'm sure every trigger in accordance with this script is correct. The only thing that isn't correct is this script itself. Maybe I just need a whole new script, but I can't think of one.

If you read my problem it says I want it so when a waygate gets killed the waygate its connected to gets killed. I tried all the stuff I said above even before submitting this thread.

Basically the result of this script that I get is: When I try killing a waygate in game, it just dies and its sister waygate doesn't.

I'm really missing something here.

 

[Shush]

Yeah, in every trigger theres a function that actually creates it and adds the events, conditions and actions to it.

You can do it yourself too, like this:

function initTrig_mytrigname takes nothing returns nothing
    set gg_trg_mytrig = CreateTrigger( )
    call TriggerRegisterAnyUnitEven (UNIT_SPELL_EFFECT) //This registers a "unit starts the effect of an ability" event to the trigger.    
    call TriggerAddCondition(gg_trg_mytrig, your_trigger_condition)
    call TriggerAddAction(gg_trg_mytrig, your_trigger_actions )

Just put this at the end of your trigger.

P.S your_trigger_condition is the function used for the conditions.

 

[Shush]

Ugh, sorry for double post, Can't edit.

Anyways, forgot to put endfunction in it

 

[Nogusta]

Sorry for being a little thick here but could you re-post that end code so it actually applies to the code I posted?

My script was never made from an existing trigger, I did it from a completely blank state.

 

[Shush]

Oh. no problem

function your_trigger_actions takes nothing returns nothing
   local unit u = GetDyingUnit()
   local integer i = 0
   loop
   exitwhen ( GetBooleanOr(    udg_WaygateA[i] == u, udg_WaygateB[i] == u )) %<--Problem%
   set i = ( i + 1 )
   endloop
   set u = null
   call ExplodeUnitBJ( udg_WaygateA[i] )
   call ExplodeUnitBJ( udg_WaygateB[i] )
endfunction

//======================================
function initTrig takes nothing returns nothing
   set gg_trg_Name_Of_Trigger =        CreateTrigger()
   call TriggerRegisterEvent(   ) //Event  you desire
   call TriggerAddCondition(gg_trg_Name_Of_Trigger, your_trigger_condition)
   call TriggerAddAction(gg_trg_Name_Of_Trigger, your_trigger_actions)
endfunction