If too many if then else is used in one trigger does that cause it to be disabled?

[Blackrockorc]

The reason i ask if too many if then else's are used in one trigger can is cause a trigger to be disabled is because im getting an error after i decided to combine many of the simple triggers i had in my map. it says script error. all all the triggers work separately.

ex... all triggers which i combined follow this format.

• Taurania Spawns
  • Events
    • Unit - A unit Finishes training a unit
  • Conditions
  • Actions
    • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
      • If - Conditions
        • (Unit-type of (Trained unit)) Equal to Kodo Beast (Tauren)
      • Then - Actions
        • Set KodoBeast = (KodoBeast + 1)
        • Unit - Remove (Trained unit) from the game
      • Else - Actions
        • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
          • If - Conditions
            • (Unit-type of (Trained unit)) Equal to Tauren
          • Then - Actions
            • Set Tauren = (Tauren + 1)
            • Unit - Remove (Trained unit) from the game
          • Else - Actions
            • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
              • If - Conditions
                • (Unit-type of (Trained unit)) Equal to Sprit Walker
              • Then - Actions
                • Set Spritwalker = (Spritwalker + 1)
                • Unit - Remove (Trained unit) from the game
              • Else - Actions

• Centrax Spawns
  • Events
    • Unit - A unit Finishes training a unit
  • Conditions
  • Actions
    • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
      • If - Conditions
        • (Unit-type of (Trained unit)) Equal to Centaur Khan
      • Then - Actions
        • Set CentaurKahn = (CentaurKahn + 1)
        • Unit - Remove (Trained unit) from the game
      • Else - Actions
        • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
          • If - Conditions
            • (Unit-type of (Trained unit)) Equal to Centaur Archer
          • Then - Actions
            • Set CentaurArcher = (CentaurArcher + 1)
            • Unit - Remove (Trained unit) from the game
          • Else - Actions
            • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
              • If - Conditions
                • (Unit-type of (Trained unit)) Equal to Centaur Sorcerer
              • Then - Actions
                • Set CentaurSorcerer = (CentaurSorcerer + 1)
                • Unit - Remove (Trained unit) from the game
              • Else - Actions
                • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
                  • If - Conditions
                    • (Unit-type of (Trained unit)) Equal to Centaur Drudge
                  • Then - Actions
                    • Set CentaurDrudge = (CentaurDrudge + 1)
                    • Unit - Remove (Trained unit) from the game
                  • Else - Actions

there are 36 bases so i did triggers like this for each base. and combined them since they are all pretty similar, so why do they work separate but when i put em all in one they don't work?

 

[Raven0]

no on the if/then/else limit

and I didn't see anything wrong with the code

mind attaching your map?

 

[Blackrockorc]

The trigger spawns/baseburn are the ones that stopped working after i combined the triggers similar into one. there use to b 36 burns and 36 spawn triggers but now i did them into 1 of each

 

[Raven0]

sorry I was wrong

The error you'r getting is "too many nested ifs"

The limit is 50 apparently

 

[Blackrockorc]

sorry I was wrong

The error you'r getting is "too many nested ifs"

The limit is 50 apparently

lol well thank you for tip.

 

[SanKakU]

is that a gui limit or does it apply to jass as well?

 

[Raven0]

I would assume it applies to jass b/c gui is just converted to jass.

In an actual programming language you would just use a case statement for htis

 

[Bribe]

It almost certainly does not apply to JASS; it sounds unsurprisingly like a GUI limitation. If there are too many branches, the processing time to fold all of them could potentially crash World Editor.

 

[Maker]

You can get around that by using arrays and loops. Create a unit type array, put your units there. Create an integer array. Then loop through the unit type array and when match is found, increase the corresponding integer.

MaxIndex is the maximum index of the unit type array that's used for a unit type.

For example:
UnitTypes[0] = Footman
UnitTypes[1] = Knight
UnitTypes[2] = Rifleman

MaxIndex would be 2 there.

• Untitled Trigger 002
  • Events
  • Conditions
  • Actions
    • Set UnitType = (Unit-type of (Trained unit))
    • For each (Integer loopA) from 0 to MaxIndex, do (Actions)
      • Loop - Actions
        • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
          • If - Conditions
            • UnitType Equal to UnitTypes[loopA]
          • Then - Actions
            • Set UnitCount[loopA] = (UnitCount[loopA] + 1)
            • Unit - Remove (Trained unit) from the game
            • Skip remaining actions
          • Else - Actions

 

[Titanhex]

Yes, there are 100% without a doubt a limit to the number of If statements allowed in both JASS and GUI. While I'm unfamiliar with the GUI limit (Probably 36), the JASS limit exists and I've encountered it. I think I encountered it mainly through an elseif limit, but I could be wrong. It doesn't compile with an error, it just fails to run anything past a certain if limit and the thread stops.

 

[Blackrockorc]

It almost certainly does not apply to JASS; it sounds unsurprisingly like a GUI limitation. If there are too many branches, the processing time to fold all of them could potentially crash World Editor.

idk about world editor but i use to get cylnic redundency check error in game.