RandomHero2EventsLocalVariablesRandomNumber=0<Integer>x=0<Integer>ConditionsActionsVariable-SetRandomNumber=(Randomintegerbetween1and18)General-Foreachintegerxfrom1to12withincrement1,do(Actions)ActionsGeneral-If(Conditions)thendo(Actions)elsedo(Actions)IfHeroSelection-RandomPlayer[x]==TrueThenGeneral-If(Conditions)thendo(Actions)elsedo(Actions)IfHeroSelectionPlayerPicked?[x]==FalseHeroSelection-TerranHeroes<3OrConditionsRandomNumber==1RandomNumber==2RandomNumber==3RandomNumber==10RandomNumber==11RandomNumber==12ThenGeneral-If(Conditions)thendo(Actions)elsedo(Actions)IfHeroSelection-HeroPicked[RandomNumber]==FalseThenDialog-DestroyHeroSelectionDialogItems[RandomNumber]Variable-SetHeroSelection-RandomPlayer[x]=FalseVariable-SetHeroSelection-HeroPicked[x]=TrueVariable-SetHeroSelectionPlayerPicked?[x]=TrueVariable-SetHeroSelection-TerranHeroes=(HeroSelection-TerranHeroes+1)Unit-Create1HeroIndex[RandomNumber]forplayerxatTerranHeroRevivefacing270.0degrees(NoOptions)Unit-Make(Lastcreatedunit)InvulnerableUnit-Add1Hero-NoVisionto(Lastcreatedunit)from(Lastcreatedunit)Variable-SetHero(Unit)[x]=(Lastcreatedunit)Variable-SetHeroSelectionNumberofPlayers=(HeroSelectionNumberofPlayers-1)UI-Display(((Text(Nameofplayerx)withcolor(Color((Currentplayerxcolor))))+" has picked ")+((Nameof(Unittypeof(Lastcreatedunit)))+"."))for(Allplayers)toSubtitleareaCamera-Panthecameraforplayerxto(TerranHeroRevivefacing270.0degrees)over0.0secondswithExistingVelocity%initialvelocity,10%deceleration,andDoNotusesmartpanningGeneral-If(Conditions)thendo(Actions)elsedo(Actions)IfGameMode==FalseThenDialog-HideHeroSelectionDialogBox[0]for(Playergroup(x))ElseDialog-HideHeroSelectionDialogBox[1]for(Playergroup(x))ElseTrigger-RunRandomHero2(CheckConditions,Don'tWaituntilitfinishes)ElseTrigger-RunRandomHero2(CheckConditions,Don'tWaituntilitfinishes)General-If(Conditions)thendo(Actions)elsedo(Actions)IfHeroSelectionPlayerPicked?[x]==FalseHeroSelection-ProtossHeroes<3OrConditionsRandomNumber==4RandomNumber==5RandomNumber==6RandomNumber==13RandomNumber==14RandomNumber==15ThenGeneral-If(Conditions)thendo(Actions)elsedo(Actions)IfHeroSelection-HeroPicked[RandomNumber]==FalseThenDialog-DestroyHeroSelectionDialogItems[RandomNumber]Variable-SetHeroSelection-RandomPlayer[x]=FalseVariable-SetHeroSelection-HeroPicked[x]=TrueVariable-SetHeroSelectionPlayerPicked?[x]=TrueVariable-SetHeroSelection-ProtossHeroes=(HeroSelection-ProtossHeroes+1)Unit-Create1HeroIndex[RandomNumber]forplayerxatProtossHeroRevivefacing270.0degrees(NoOptions)Unit-Make(Lastcreatedunit)InvulnerableUnit-Add1Hero-NoVisionto(Lastcreatedunit)from(Lastcreatedunit)Variable-SetHero(Unit)[x]=(Lastcreatedunit)Variable-SetHeroSelectionNumberofPlayers=(HeroSelectionNumberofPlayers-1)UI-Display(((Text(Nameofplayerx)withcolor(Color((Currentplayerxcolor))))+" has picked ")+((Nameof(Unittypeof(Lastcreatedunit)))+"."))for(Allplayers)toSubtitleareaCamera-Panthecameraforplayerxto(ProtossHeroRevivefacing270.0degrees)over0.0secondswithExistingVelocity%initialvelocity,10%deceleration,andDoNotusesmartpanningGeneral-If(Conditions)thendo(Actions)elsedo(Actions)IfGameMode==FalseThenDialog-HideHeroSelectionDialogBox[0]for(Playergroup(x))ElseDialog-HideHeroSelectionDialogBox[1]for(Playergroup(x))ElseTrigger-RunRandomHero2(CheckConditions,Don'tWaituntilitfinishes)ElseTrigger-RunRandomHero2(CheckConditions,Don'tWaituntilitfinishes)General-If(Conditions)thendo(Actions)elsedo(Actions)IfHeroSelectionPlayerPicked?[x]==FalseHeroSelection-ZergHeroes<3OrConditionsRandomNumber==7RandomNumber==8RandomNumber==9RandomNumber==16RandomNumber==17RandomNumber==18ThenGeneral-If(Conditions)thendo(Actions)elsedo(Actions)IfHeroSelection-HeroPicked[RandomNumber]==FalseThenDialog-DestroyHeroSelectionDialogItems[RandomNumber]Variable-SetHeroSelection-RandomPlayer[x]=FalseVariable-SetHeroSelection-HeroPicked[x]=TrueVariable-SetHeroSelectionPlayerPicked?[x]=TrueVariable-SetHeroSelection-ZergHeroes=(HeroSelection-ZergHeroes+1)Unit-Create1HeroIndex[RandomNumber]forplayerxatZergHeroRevivefacing270.0degrees(NoOptions)Unit-Make(Lastcreatedunit)InvulnerableUnit-Add1Hero-NoVisionto(Lastcreatedunit)from(Lastcreatedunit)Variable-SetHero(Unit)[x]=(Lastcreatedunit)Variable-SetHeroSelectionNumberofPlayers=(HeroSelectionNumberofPlayers-1)UI-Display(((Text(Nameofplayerx)withcolor(Color((Currentplayerxcolor))))+" has picked ")+((Nameof(Unittypeof(Lastcreatedunit)))+"."))for(Allplayers)toSubtitleareaCamera-Panthecameraforplayerxto(ZergHeroRevivefacing270.0degrees)over0.0secondswithExistingVelocity%initialvelocity,10%deceleration,andDoNotusesmartpanningGeneral-If(Conditions)thendo(Actions)elsedo(Actions)IfGameMode==FalseThenDialog-HideHeroSelectionDialogBox[0]for(Playergroup(x))ElseDialog-HideHeroSelectionDialogBox[1]for(Playergroup(x))ElseTrigger-RunRandomHero2(CheckConditions,Don'tWaituntilitfinishes)ElseTrigger-RunRandomHero2(CheckConditions,Don'tWaituntilitfinishes)Else
I know this is a long trigger and please I don't need help optimizing it. It works fine the way it is, save one part.
HeroSelection-HeroPicked[RandomNumber]==False
This is added when a player picks a hero in a different trigger. This is not working the way it's intended. This is here to state IS the hero that the random number picked already picked?
Right, ok that would specify after the hero has been picked to knock that hero off the selection, which needs to be changed, but that doesn't fix the issue at hand.
I need this to check if the hero has been picked already before creating the hero.
I know it's:
HeroSelection-HeroPicked[RandomNumber]==False
I need to change the random number section somehow to ask has the hero been picked. I just can't wrap my head around it.
EDIT:
After looking at this more.
X = player
Random # = number 1-18 which represents the heroes
Hmm this should work, and changing what you said would lock out that hero from the pool again.
BUT from the reports I've heard, a few players have got duplicate heroes. I asked for more details and he said the one player picked Renzo, then the 2nd player randomed and got Renzo again.
So I don't know
EDIT 2:
Just looked at the first trigger that picks the hero by selection and I had the same set variable wrong.
Changed it to:
Variable-SetHeroSelection-HeroPicked[Index]=True
I think that will fix it. Thanks :)
EDIT 3:
I've noticed that every time I've gone random. It has given me a terran hero. #'s 1-3 & 10-12. Is this just coincidence or is something I have setup not working right?
RandomHero2EventsLocalVariablesRandomNumber=0<Integer[12]>
x = 0 <Integer>
Conditions
Actions
General - For each integer x from 1 to 12 with increment 1, do (Actions)
Actions
General - If (Conditions) then do (Actions) else do (Actions)
If
Hero Selection - Random Player[x] == True
Then
Variable - Set Random Number[x] = (Random integer between 1 and 18)
UI - Display (Text(Random Number[x])) for (Player group(x)) to Subtitle area
General - If (Conditions) then do (Actions) else do (Actions)
If
Or
Conditions
Random Number[x] == 1
Random Number[x] == 2
Random Number[x] == 3
Random Number[x] == 10
Random Number[x] == 11
Random Number[x] == 12
Then
General - If (Conditions) then do (Actions) else do (Actions)
If
Hero Selection Player Picked?[x] == False
Hero Selection - Terran Heroes < 3HeroSelection-HeroPicked[RandomNumber[x]]==FalseThenDialog-DestroyHeroSelectionDialogItems[RandomNumber[x]]Variable-SetHeroSelection-RandomPlayer[x] =FalseVariable-SetHeroSelection-HeroPicked[RandomNumber[x]]=TrueVariable-SetHeroSelectionPlayerPicked?[x]=TrueVariable-SetHeroSelection-TerranHeroes =(HeroSelection-TerranHeroes+1)Unit-Create1HeroIndex[RandomNumber[x]]forplayerxatTerranHeroRevivefacing270.0degrees(NoOptions)Unit-Make(Lastcreatedunit)InvulnerableUnit-Add1Hero-NoVisionto(Lastcreatedunit)from(Lastcreatedunit)Variable-SetHero(Unit)[x]=(Lastcreatedunit)Variable-SetHeroSelectionNumberofPlayers =(HeroSelectionNumberofPlayers-1)UI-Display(((Text(Nameofplayerx)withcolor(Color((Currentplayerxcolor))))+"haspicked")+((Nameof(Unittypeof(Lastcreatedunit)))+"."))for(Allplayers)toSubtitleareaCamera-Panthecameraforplayerxto(TerranHeroRevivefacing270.0degrees)over0.0secondswithExistingVelocity%initialvelocity,10%deceleration,andDoNotusesmartpanningGeneral-If(Conditions)thendo(Actions)elsedo(Actions)IfGameMode ==FalseThenDialog-HideHeroSelectionDialogBox[0]for(Playergroup(x))ElseDialog-HideHeroSelectionDialogBox[1]for(Playergroup(x))ElseTrigger-RunRandomHero2(CheckConditions,Don'tWaituntilitfinishes)ElseTrigger-RunRandomHero2(CheckConditions,Don'tWaituntilitfinishes)General-If(Conditions)thendo(Actions)elsedo(Actions)IfOrConditionsRandomNumber[x] ==4RandomNumber[x] ==5RandomNumber[x] ==6RandomNumber[x] ==13RandomNumber[x] ==14RandomNumber[x] ==15ThenGeneral-If(Conditions)thendo(Actions)elsedo(Actions)IfHeroSelectionPlayerPicked?[x]==FalseHeroSelection-ProtossHeroes<3HeroSelection-HeroPicked[RandomNumber[x]]==FalseThenDialog-DestroyHeroSelectionDialogItems[RandomNumber[x]]Variable-SetHeroSelection-RandomPlayer[x] =FalseVariable-SetHeroSelection-HeroPicked[RandomNumber[x]]=TrueVariable-SetHeroSelectionPlayerPicked?[x]=TrueVariable-SetHeroSelection-ProtossHeroes =(HeroSelection-ProtossHeroes+1)Unit-Create1HeroIndex[RandomNumber[x]]forplayerxatProtossHeroRevivefacing270.0degrees(NoOptions)Unit-Make(Lastcreatedunit)InvulnerableUnit-Add1Hero-NoVisionto(Lastcreatedunit)from(Lastcreatedunit)Variable-SetHero(Unit)[x]=(Lastcreatedunit)Variable-SetHeroSelectionNumberofPlayers =(HeroSelectionNumberofPlayers-1)UI-Display(((Text(Nameofplayerx)withcolor(Color((Currentplayerxcolor))))+"haspicked")+((Nameof(Unittypeof(Lastcreatedunit)))+"."))for(Allplayers)toSubtitleareaCamera-Panthecameraforplayerxto(ProtossHeroRevivefacing270.0degrees)over0.0secondswithExistingVelocity%initialvelocity,10%deceleration,andDoNotusesmartpanningGeneral-If(Conditions)thendo(Actions)elsedo(Actions)IfGameMode ==FalseThenDialog-HideHeroSelectionDialogBox[0]for(Playergroup(x))ElseDialog-HideHeroSelectionDialogBox[1]for(Playergroup(x))ElseTrigger-RunRandomHero2(CheckConditions,Don'tWaituntilitfinishes)ElseTrigger-RunRandomHero2(CheckConditions,Don'tWaituntilitfinishes)General-If(Conditions)thendo(Actions)elsedo(Actions)IfOrConditionsRandomNumber[x] ==7RandomNumber[x] ==8RandomNumber[x] ==9RandomNumber[x] ==16RandomNumber[x] ==17RandomNumber[x] ==18ThenGeneral-If(Conditions)thendo(Actions)elsedo(Actions)IfHeroSelectionPlayerPicked?[x]==FalseHeroSelection-ZergHeroes<3HeroSelection-HeroPicked[RandomNumber[x]]==FalseThenDialog-DestroyHeroSelectionDialogItems[RandomNumber[x]]Variable-SetHeroSelection-RandomPlayer[x] =FalseVariable-SetHeroSelection-HeroPicked[RandomNumber[x]]=TrueVariable-SetHeroSelectionPlayerPicked?[x]=TrueVariable-SetHeroSelection-ZergHeroes =(HeroSelection-ZergHeroes+1)Unit-Create1HeroIndex[RandomNumber[x]]forplayerxatZergHeroRevivefacing270.0degrees(NoOptions)Unit-Make(Lastcreatedunit)InvulnerableUnit-Add1Hero-NoVisionto(Lastcreatedunit)from(Lastcreatedunit)Variable-SetHero(Unit)[x]=(Lastcreatedunit)Variable-SetHeroSelectionNumberofPlayers =(HeroSelectionNumberofPlayers-1)UI-Display(((Text(Nameofplayerx)withcolor(Color((Currentplayerxcolor))))+"haspicked")+((Nameof(Unittypeof(Lastcreatedunit)))+"."))for(Allplayers)toSubtitleareaCamera-Panthecameraforplayerxto(ZergHeroRevivefacing270.0degrees)over0.0secondswithExistingVelocity%initialvelocity,10%deceleration,andDoNotusesmartpanningGeneral-If(Conditions)thendo(Actions)elsedo(Actions)IfGameMode ==FalseThenDialog-HideHeroSelectionDialogBox[0]for(Playergroup(x))ElseDialog-HideHeroSelectionDialogBox[1]for(Playergroup(x))ElseTrigger-RunRandomHero2(CheckConditions,Don'tWaituntilitfinishes)ElseTrigger-RunRandomHero2(CheckConditions,Don'tWaituntilitfinishes)Else
So I changed up the order of things and put a display text message in there.
From what I understand shouldn't this ONLY be displaying the text of the random # ONE time for whatever hero was picked?
Because in game it's showing multiple times. Most likely 12 times, which doesn't make sense to me.
RandomHero2EventsLocalVariablesRandomNumber=0<Integer[12]>
x = 0 <Integer>
Conditions
Actions
General - For each integer x from 1 to 12 with increment 1, do (Actions)
Actions
General - If (Conditions) then do (Actions) else do (Actions)
If
Hero Selection - Random Player[x] == True
Then
Variable - Set Random Number[x] = (Random integer between 1 and 18)
General - If (Conditions) then do (Actions) else do (Actions)
If
Or
Conditions
Random Number[x] == 1
Random Number[x] == 2
Random Number[x] == 3
Random Number[x] == 10
Random Number[x] == 11
Random Number[x] == 12
Then
General - If (Conditions) then do (Actions) else do (Actions)
If
Hero Selection Player Picked?[x] == False
Hero Selection - Terran Heroes < 3HeroSelection-HeroPicked[RandomNumber[x]]==FalseThenDialog-DestroyHeroSelectionDialogItems[RandomNumber[x]]Variable-SetHeroSelection-RandomPlayer[x] =FalseVariable-SetHeroSelection-HeroPicked[RandomNumber[x]]=TrueVariable-SetHeroSelectionPlayerPicked?[x]=TrueVariable-SetHeroSelection-TerranHeroes =(HeroSelection-TerranHeroes+1)Unit-Create1HeroIndex[RandomNumber[x]]forplayerxatTerranHeroRevivefacing270.0degrees(NoOptions)Unit-Make(Lastcreatedunit)InvulnerableUnit-Add1Hero-NoVisionto(Lastcreatedunit)from(Lastcreatedunit)Variable-SetHero(Unit)[x]=(Lastcreatedunit)Variable-SetHeroSelectionNumberofPlayers =(HeroSelectionNumberofPlayers-1)UI-Display(((Text(Nameofplayerx)withcolor(Color((Currentplayerxcolor))))+"haspicked")+((Nameof(Unittypeof(Lastcreatedunit)))+"."))for(Allplayers)toSubtitleareaCamera-Panthecameraforplayerxto(TerranHeroRevivefacing270.0degrees)over0.0secondswithExistingVelocity%initialvelocity,10%deceleration,andDoNotusesmartpanningGeneral-If(Conditions)thendo(Actions)elsedo(Actions)IfGameMode ==FalseThenDialog-HideHeroSelectionDialogBox[0]for(Playergroup(x))ElseDialog-HideHeroSelectionDialogBox[1]for(Playergroup(x))ElseTrigger-RunRandomHero2(CheckConditions,Don'tWaituntilitfinishes)ElseGeneral-If(Conditions)thendo(Actions)elsedo(Actions)IfOrConditionsRandomNumber[x] ==4RandomNumber[x] ==5RandomNumber[x] ==6RandomNumber[x] ==13RandomNumber[x] ==14RandomNumber[x] ==15ThenGeneral-If(Conditions)thendo(Actions)elsedo(Actions)IfHeroSelectionPlayerPicked?[x]==FalseHeroSelection-ProtossHeroes<3HeroSelection-HeroPicked[RandomNumber[x]]==FalseThenDialog-DestroyHeroSelectionDialogItems[RandomNumber[x]]Variable-SetHeroSelection-RandomPlayer[x] =FalseVariable-SetHeroSelection-HeroPicked[RandomNumber[x]]=TrueVariable-SetHeroSelectionPlayerPicked?[x]=TrueVariable-SetHeroSelection-ProtossHeroes =(HeroSelection-ProtossHeroes+1)Unit-Create1HeroIndex[RandomNumber[x]]forplayerxatProtossHeroRevivefacing270.0degrees(NoOptions)Unit-Make(Lastcreatedunit)InvulnerableUnit-Add1Hero-NoVisionto(Lastcreatedunit)from(Lastcreatedunit)Variable-SetHero(Unit)[x]=(Lastcreatedunit)Variable-SetHeroSelectionNumberofPlayers =(HeroSelectionNumberofPlayers-1)UI-Display(((Text(Nameofplayerx)withcolor(Color((Currentplayerxcolor))))+"haspicked")+((Nameof(Unittypeof(Lastcreatedunit)))+"."))for(Allplayers)toSubtitleareaCamera-Panthecameraforplayerxto(ProtossHeroRevivefacing270.0degrees)over0.0secondswithExistingVelocity%initialvelocity,10%deceleration,andDoNotusesmartpanningGeneral-If(Conditions)thendo(Actions)elsedo(Actions)IfGameMode ==FalseThenDialog-HideHeroSelectionDialogBox[0]for(Playergroup(x))ElseDialog-HideHeroSelectionDialogBox[1]for(Playergroup(x))ElseTrigger-RunRandomHero2(CheckConditions,Don'tWaituntilitfinishes)ElseGeneral-If(Conditions)thendo(Actions)elsedo(Actions)IfOrConditionsRandomNumber[x] ==7RandomNumber[x] ==8RandomNumber[x] ==9RandomNumber[x] ==16RandomNumber[x] ==17RandomNumber[x] ==18ThenGeneral-If(Conditions)thendo(Actions)elsedo(Actions)IfHeroSelectionPlayerPicked?[x]==FalseHeroSelection-ZergHeroes<3HeroSelection-HeroPicked[RandomNumber[x]]==FalseThenDialog-DestroyHeroSelectionDialogItems[RandomNumber[x]]Variable-SetHeroSelection-RandomPlayer[x] =FalseVariable-SetHeroSelection-HeroPicked[RandomNumber[x]]=TrueVariable-SetHeroSelectionPlayerPicked?[x]=TrueVariable-SetHeroSelection-ZergHeroes =(HeroSelection-ZergHeroes+1)Unit-Create1HeroIndex[RandomNumber[x]]forplayerxatZergHeroRevivefacing270.0degrees(NoOptions)Unit-Make(Lastcreatedunit)InvulnerableUnit-Add1Hero-NoVisionto(Lastcreatedunit)from(Lastcreatedunit)Variable-SetHero(Unit)[x]=(Lastcreatedunit)Variable-SetHeroSelectionNumberofPlayers =(HeroSelectionNumberofPlayers-1)UI-Display(((Text(Nameofplayerx)withcolor(Color((Currentplayerxcolor))))+"haspicked")+((Nameof(Unittypeof(Lastcreatedunit)))+"."))for(Allplayers)toSubtitleareaCamera-Panthecameraforplayerxto(ZergHeroRevivefacing270.0degrees)over0.0secondswithExistingVelocity%initialvelocity,10%deceleration,andDoNotusesmartpanningGeneral-If(Conditions)thendo(Actions)elsedo(Actions)IfGameMode ==FalseThenDialog-HideHeroSelectionDialogBox[0]for(Playergroup(x))ElseDialog-HideHeroSelectionDialogBox[1]for(Playergroup(x))ElseTrigger-RunRandomHero2(CheckConditions,Don'tWaituntilitfinishes)ElseElse
Fixed it. I had to remove the 3 looping things if the number was picked. It was always fixing UNTIL one of those six numbers was picked.
I know this is a long trigger and please I don't need help optimizing it. It works fine the way it is, save one part.
This is added when a player picks a hero in a different trigger. This is not working the way it's intended. This is here to state IS the hero that the random number picked already picked?
You have
Looks like that should be
instead.
@SBeier: Go
Right, ok that would specify after the hero has been picked to knock that hero off the selection, which needs to be changed, but that doesn't fix the issue at hand.
I need this to check if the hero has been picked already before creating the hero.
I know it's:
I need to change the random number section somehow to ask has the hero been picked. I just can't wrap my head around it.
EDIT:
After looking at this more.
Hmm this should work, and changing what you said would lock out that hero from the pool again.
BUT from the reports I've heard, a few players have got duplicate heroes. I asked for more details and he said the one player picked Renzo, then the 2nd player randomed and got Renzo again.
So I don't know
EDIT 2:
Just looked at the first trigger that picks the hero by selection and I had the same set variable wrong.
Changed it to:
I think that will fix it. Thanks :)
EDIT 3:
I've noticed that every time I've gone random. It has given me a terran hero. #'s 1-3 & 10-12. Is this just coincidence or is something I have setup not working right?
@Zero0018: Go
How about you try using a variable of the type 'unit type', modify the variable to be the hero that was chosen and use that as a reference point?
So I changed up the order of things and put a display text message in there.
From what I understand shouldn't this ONLY be displaying the text of the random # ONE time for whatever hero was picked?
Because in game it's showing multiple times. Most likely 12 times, which doesn't make sense to me.
This is part one of the trigger:
Fixed it. I had to remove the 3 looping things if the number was picked. It was always fixing UNTIL one of those six numbers was picked.
@Zero0018: Go
How do I use this? just copy and paste?