[Spell] Streak System

[Vinz]

These particular actions won't work.

• If (All Conditions are True) then do (Then Actions) else do (Else Actions)
  • If - Conditions
    • Streak_Level[TempInteger] Greater than 1
  • Then - Actions
    • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
      • If - Conditions
        • Streak_Level[TempInteger] Less than 23
      • Then - Actions
        • Set Streak_Level[TempInteger] = (Streak_Level[TempInteger] + 1)
      • Else - Actions
    • Game - Display to (All players) the text: (((PlayerColor[TempInteger] + ((Name of (Owner of (Killing unit))) + Endline)) + |r) + Streak_Names[Streak_Level[TempInteger]])
  • Else - Actions

Why?

 

[Chaosy]

idk, it's hard to say when we only got a small part of the trigger.

@off tpoic why did almia leave?

 

[Vinz]

It didn't work when I added that second If/Then/Else with the less than 23..

Edit: Or should I say it stopped working

 

[Nichilus]

Is there a reason why you don't use only 1 if/then/else?
Like this

• If (All Conditions are True) then do (Then Actions) else do (Else Actions)
  • If - Conditions
    • Streak_Level[TempInteger] Greater than 1
    • Streak_Level[TempInteger] Less than 23
  • Then - Actions
    • Set Streak_Level[TempInteger] = (Streak_Level[TempInteger] + 1)
    • Game - Display to (All players) the text: <text>
  • Else - Actions

 

[Orthane]

Is there a reason why you don't use only 1 if/then/else?
Like this

• If (All Conditions are True) then do (Then Actions) else do (Else Actions)
  • If - Conditions
    • Streak_Level[TempInteger] Greater than 1
    • Streak_Level[TempInteger] Less than 23
  • Then - Actions
    • Set Streak_Level[TempInteger] = (Streak_Level[TempInteger] + 1)
    • Game - Display to (All players) the text: <text>
  • Else - Actions

The world may never know.

However, You could do Greater then or equal to 1 and less than or equal to 23.

Where are you recycling the streak?

 

[Vinz]

I wanted to put a filter so the streak level doesn't go up anymore when reached 22(Godfury Slayer)

 

[Maker]

Is there a reason why you don't use only 1 if/then/else? Like this If (All Conditions are True) then do (Then Actions) else do (Else Actions) If - Conditions Streak_Level[TempInteger] Greater than 1 Streak_Level[TempInteger] Less than 23 Then - Actions Set Streak_Level[TempInteger] = (Streak_Level[TempInteger] + 1) Game - Display to (All players) the text: <text> Else - Actions

With that, when you get to lvl 23 streak, it will bug. With the next kill, you will not get the message. That is why you can't use the conditions like that.

 

[Almia]

idk, it's hard to say when we only got a small part of the trigger.

@off tpoic why did almia leave?

Off-Topic
Stop crying. Im still here.
I just cleaned up my account,after a "clash" happened.I will be back once every thing settles down.

On-Topic
I think removing the "Is Greater than 1" will fix the problem.

 

[Vinz]

The problem is, if I removed is Greater than 1, even if the player's killcount is equal to 1, it will display 'Killing Spree'..

 

[Almia]

Have you tried debugging every part of the system?

 

[Orthane]

I wanted to put a filter so the streak level doesn't go up anymore when reached 22(Godfury Slayer)

If you don't want it going above 22 then you need it to be less then or equal to 22. Not less then 23.


Instead of only posting that section, please post that entire trigger.

 

[Vinz]

Will try. And will post the triggers. Thanks!

TRIGGER

• Streak Kill
  • Events
    • Unit - A unit Dies
  • Conditions
    • ((Killing unit) is A Hero) Equal to True
    • ((Dying unit) is A Hero) Equal to True
    • (Owner of (Killing unit)) Not equal to Neutral Passive
    • (Owner of (Dying unit)) Not equal to Neutral Passive
    • ((Dying unit) belongs to an enemy of (Owner of (Killing unit))) Equal to True
  • Actions
    • Set TempInteger = ((Player number of (Owner of (Killing unit))) - 1)
    • Set X = ((Player number of (Owner of (Dying unit))) - 1)
    • Set Streak_HeadValue[TempInteger] = (Streak_HeadValue[TempInteger] + (Random real number between 15.00 and 40.00))
    • Set Streak_Reward[TempInteger] = (Streak_Reward[TempInteger] + (Random real number between 20.00 and 30.00))
    • Set Streak_HeadValue[X] = 20.00
    • Set Streak_Level[X] = 0
    • Set Streak_Reward[X] = 40.00
    • Game - Display to (All players) the text: (((PlayerColor[TempInteger] + PlayerName[TempInteger]) + |r has slain) + ( + ((PlayerColor[X] + PlayerName[X]) + ((|r + for ) + ( |cffffcc00 + ((String((Integer(Streak_Reward[TempInteger])))) + gold!))))))
    • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
      • If - Conditions
        • Streak_Level[TempInteger] Greater than 1
      • Then - Actions
        • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
          • If - Conditions
            • Streak_Level[TempInteger] Less than 23
          • Then - Actions
            • Set Streak_Level[TempInteger] = (Streak_Level[TempInteger] + 1)
          • Else - Actions
        • Game - Display to (All players) the text: (((PlayerColor[TempInteger] + ((Name of (Owner of (Killing unit))) + Endline)) + |r) + Streak_Names[Streak_Level[TempInteger]])
      • Else - Actions
    • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
      • If - Conditions
        • Streak_Level[X] Greater than 1
      • Then - Actions
        • Game - Display to (All players) the text: (((PlayerColor[TempInteger] + ((Name of (Owner of (Killing unit))) + Endline)) + ) + ((has ended + (PlayerColor[X] + PlayerName[X])) + Streak_Name[X]))
      • Else - Actions

 

[BronzeOregon]

And this still doesn't work?

 

[Vinz]

It works when player kills another, but the streak doesn't(the trigger shown in the first post) it never shows 'Killing Spree'.

 

[Maker]

It seems you are only incrementing streak level if the level is > 1. So it is not incremented when it is 0 or 1.

 

[Vinz]

Thanks Maker! I'll find a way to do it. I'm such unlogical.

Edit: So I removed the If/Then/Else with Less than 23 out of the other If/Then/Else.