Hero Selection Problem

MadCreator · May 18, 2013

Hi, I’m new to this but my main problem I am having is hero selection. I had made hero tavern to allow the player to pick their hero manually or random. The problem is, when a player picks a hero manually and then another player pick random, there still a chance to generate the same hero. Any help would be helpful :grin:

Hero Array
- Events
  - Map initialization
- Conditions
- Actions
  - -------- Set Array for Hero --------
  - Set Total_Heroes = 3
  - Set Random_Count = Total_Heroes
  - Set Hero_Array[1] = Paladin
  - Set Hero_Array[2] = Mountain King
  - Set Hero_Array[3] = Archmage
  - For each (Integer A) from 1 to 3, do (Actions)
    - Loop - Actions
      - Set Random_Data[(Integer A)] = (Integer A)
  - Trigger - Turn off (This trigger)

Manual Selection
- Events
  - Unit - A unit Sells a unit
- Conditions
  - ((Sold unit) is A Hero) Equal to True
- Actions
  - Player Group - Pick every player in (All players) and do (Actions)
    - Loop - Actions
      - Player - Make (Unit-type of (Sold unit)) Unavailable for training/construction by (Picked player)
  - Game - Display to (All players) the text: (A player had chosen + (Name of (Sold unit)))
  - Unit - Move (Sold unit) instantly to (Center of Hero Spawn <gen>)
  - Set Random_Data[Random_Hero] = Random_Count
  - Set Random_Count = (Random_Count - 1)

Random Selection
- Events
  - Unit - A unit Sells a unit
- Conditions
  - (Unit-type of (Sold unit)) Equal to Random Hero
- Actions
  - Set Random_Hero = (Random integer number between 1 and Random_Count)
  - Unit - Create 1 Hero_Array[Random_Data[Random_Hero]] for (Owner of (Sold unit)) at (Center of Hero Spawn <gen>) facing Default building facing degrees
  - Game - Display to (All players) the text: (A player had randomed + (Name of (Last created unit)))
  - Player Group - Pick every player in (All players) and do (Actions)
    - Loop - Actions
      - Player - Make (Unit-type of (Last created unit)) Unavailable for training/construction by (Picked player)
  - Set Random_Data[Random_Hero] = Random_Count
  - Set Random_Count = (Random_Count - 1)

deathismyfriend · May 18, 2013

U can store the heroes in an array then when the hero is picked de-index that picked hero. Also u leak locations which should be fixed look at the tutorial things that leak. Also u should take a look at my tutorial things a GUIer should know. It's in my sig. It will help u with a lot.

Tonex · May 18, 2013

Pretty sure this will do what you want. Based on principle of elminating the unwanted data in the array, then shifting everything down.

Hero Array
- Events
  - Map initialization
- Conditions
- Actions
  - -------- Set Array for Hero --------
  - Set Total_Heroes = 3
  - Set Hero_Array[1] = Paladin
  - Set Hero_Array[2] = Mountain King
  - Set Hero_Array[3] = Archmage
  - Trigger - Turn off (This trigger)

Manual Selection
- Events
  - Unit - A unit Sells a unit
- Conditions
  - ((Sold unit) is A Hero) Equal to True
- Actions
  - Player Group - Pick every player in (All players) and do (Actions)
    - Loop - Actions
      - Player - Make (Unit-type of (Sold unit)) Unavailable for training/construction by (Picked player)
  - Game - Display to (All players) the text: (A player has selected + (Name of (Sold unit)))
  - Unit - Move (Sold unit) instantly to (Center of Hero Spawn <gen>)
  - For each (Integer A) from 1 to Total_Heroes, do (Actions)
    - Loop - Actions
      - If (All Conditions are True) then do (Then Actions) else do (Else Actions)
        
        If - Conditions
        
        Or - Any (Conditions) are true
        
        Conditions
        
        (Unit-type of (Sold unit)) Equal to Hero_Array[(Integer A)]
        
        No unit-type Equal to Hero_Array[(Integer A)]
        
        Then - Actions
        
        Set Hero_Array[(Integer A)] = Hero_Array[((Integer A) + 1)]
        
        Set Hero_Array[((Integer A) + 1)] = No unit-type
        
        Else - Actions
  - Set Total_Heroes = (Total_Heroes - 1)

Random Selection
- Events
  - Unit - A unit Sells a unit
- Conditions
  - (Unit-type of (Sold unit)) Equal to Random Hero
- Actions
  - Set Random_Hero = (Random integer number between 1 and Random_Count)
  - Unit - Create 1 Hero_Array[(Random integer number between 1 and Total_Heroes)] for (Owner of (Sold unit)) at (Center of Hero Spawn <gen>) facing Default building facing degrees
  - Game - Display to (All players) the text: (A player has randomly selected + (Name of (Last created unit)))
  - Player Group - Pick every player in (All players) and do (Actions)
    - Loop - Actions
      - Player - Make (Unit-type of (Last created unit)) Unavailable for training/construction by (Picked player)
  - For each (Integer A) from 1 to Total_Heroes, do (Actions)
    - Loop - Actions
      - If (All Conditions are True) then do (Then Actions) else do (Else Actions)
        
        If - Conditions
        
        Or - Any (Conditions) are true
        
        Conditions
        
        (Unit-type of (Last Created Unit)) Equal to Hero_Array[(Integer A)]
        
        No unit-type Equal to Hero_Array[(Integer A)]
        
        Then - Actions
        
        Set Hero_Array[(Integer A)] = Hero_Array[((Integer A) + 1)]
        
        Set Hero_Array[((Integer A) + 1)] = No unit-type
        
        Else - Actions
  - Set Total_Heroes = (Total_Heroes - 1)

ap0calypse · May 18, 2013

@Tonex: no reason to loop in the random selection-trigger, since you already know which hero is created (Hero_Array[Random_Hero]).

Tonex · May 18, 2013

You mean the loop that shifts everything down? Because that is still needed to remove the randomly selected hero from the available random heroes?

deathismyfriend · May 18, 2013

thats not the most efficient indexing method. look at how i do indexing in my tutorial.
http://www.hiveworkshop.com/forums/tutorial-submission-283/things-gui-user-should-know-233242/

Also dont use integer A instead make ur own variable and use that. You also leak locations.

Tonex · May 18, 2013

It's easy enough to use and I was just too lazy to remove leaks. It's not really required if it's only 7 or 8 points, as long as other leaks are removed.

deathismyfriend · May 18, 2013

leaks should always be removed and all triggers should be as efficient as possible lol.
Ignoring leaks and ignoring efficiency leads to a lagging game.

ap0calypse · May 18, 2013

Ignoring efficiency does not have to result in a lagging game.
Edit: that doesn't mean you should ignore it all the time, though

.
Efficiency is what we strive for (too much inefficiency will make the game noticeably heavier) endedit

But that aside: the reason why you do not have to loop, is because the list does not need to stay in order.
If that were the case, then you'd be right, but as it is: you can just replace the last index with the current index and it's done (no loop involved).

Here, this means:

Set Hero_Array[Random_Hero] = Hero_Array[Total_Heroes]
Set Total_Heroes = Total_Heroes - 1

The randomed hero will be removed from the list with this.

Edit: just checked, this is basically what deathismyfriend's tutorial also says (my version is more succinct because it's a very specific example instead of a general one).

deathismyfriend · May 18, 2013

u should also null the last hero variable b4 reducing index since it wont be used again.

Tonex · May 18, 2013

@ap0calypse My bad, I'm just used to retaining order from when I wrote some programs to simulate those mathematical card tricks.

ap0calypse · May 18, 2013

Tonex said:
@ap0calypse My bad, I'm just used to retaining order from when I wrote some programs to simulate those mathematical card tricks.

No problem at all

. I've created some card-games in the past, and those sorting loops pop up everywhere.

MadCreator · May 19, 2013

Thanks for the help and feedback, they were super helpful but I do have a question I would like to ask. Since Apocalypse mention you don’t need a loop on the random selection and Deathismyfriend also mention that the last hero variable need to be null before reducing the index. I try to figure out how to set to that but I got some undesirable result. Please let me know what I did wrong here.

Random Selection Copy 2
- Events
  - Unit - A unit Sells a unit
- Conditions
  - (Unit-type of (Sold unit)) Equal to Random Hero
- Actions
  - Set Random_Hero = (Random integer number between 1 and Random_Count)
  - Unit - Create 1 Hero_Array[(Random integer number between 1 and Total_Heroes)] for (Owner of (Sold unit)) at (Center of Hero Spawn <gen>) facing Default building facing degrees
  - Game - Display to (All players) the text: (A player had randomed + (Name of (Last created unit)))
  - Player Group - Pick every player in (All players) and do (Actions)
    - Loop - Actions
      - Player - Make (Unit-type of (Last created unit)) Unavailable for training/construction by (Picked player)
  - Set Hero_Array[Random_Hero] = Hero_Array[Total_Heroes]
  - Set Hero_Array[Random_Hero] = No unit-type
  - Set Total_Heroes = (Total_Heroes - 1)

deathismyfriend · May 19, 2013

u leak a location.
Also y do this ?

Set Random_Hero = (Random integer number between 1 and Random_Count)
Unit - Create 1 Hero_Array[(Random integer number between 1 and Total_Heroes)] for (Owner of (Sold unit)) at (Center of Hero Spawn <gen>) facing Default building facing degrees

u set a random integer twice. They are different values but the upper one isnt used.

MadCreator · May 20, 2013

Thanks for pinpointing that out. I fix that and I am very aware about leak a location, but my biggest concern I am having is -is understanding how to run my random selection trigger without the loops as Apocalypse had mention, because I am still getting hero that already been selected without loops. Other than that, I will probably stick with the example Tonex had written out, since I am not advance with GUI.

deathismyfriend · May 20, 2013

basically u set a boolean for the unit. lets say i make 10 heroes. Well store their unit type into an array. Then if player selects that unit set the boolean to false. for the condition just check if boolean is true. If it is true then give the player the hero

Tonex · May 20, 2013

Instead of

For each (Integer A) from 1 to Total_Heroes, do (Actions)
- Loop - Actions
  - If (All Conditions are True) then do (Then Actions) else do (Else Actions)
    - If - Conditions
      - Or - Any (Conditions) are true
        
        Conditions
        
        (Unit-type of (Sold unit)) Equal to Hero_Array[(Integer A)]
        
        No unit-type Equal to Hero_Array[(Integer A)]
    - Then - Actions
      - Set Hero_Array[(Integer A)] = Hero_Array[((Integer A) + 1)]
      - Set Hero_Array[((Integer A) + 1)] = No unit-type
    - Else - Actions

You could put

For each (Integer A) from 1 to Total_Heroes, do (Actions)
- Loop - Actions
  - If (All Conditions are True) then do (Then Actions) else do (Else Actions)
    - If - Conditions
      - (Unit-type of (Sold unit)) Equal to Hero_Array[(Integer A)]
    - Then - Actions
      - Set Hero_Array[(Integer A)] = Hero_Array[Total_Heroes]
      - Set Hero_Array[Total_Heroes] = No unit-type
    - Else - Actions

And in the other one, replace

Set Random_Hero = (Random integer number between 1 and Random_Count)
Unit - Create 1 Hero_Array[(Random integer number between 1 and Total_Heroes)] for (Owner of (Sold unit)) at (Center of Hero Spawn <gen>)

With

Set Random_Hero = (Random integer number between 1 and Total_Heroes)
Unit - Create 1 Hero_Array[Random_Hero] for (Owner of (Sold unit)) at (Center of Hero Spawn <gen>)

And

For each (Integer A) from 1 to Total_Heroes, do (Actions)
- Loop - Actions
  - If (All Conditions are True) then do (Then Actions) else do (Else Actions)
    - If - Conditions
      - Or - Any (Conditions) are true
        
        Conditions
        
        (Unit-type of (Last Created Unit)) Equal to Hero_Array[(Integer A)]
        
        No unit-type Equal to Hero_Array[(Integer A)]
    - Then - Actions
      - Set Hero_Array[(Integer A)] = Hero_Array[((Integer A) + 1)]
      - Set Hero_Array[((Integer A) + 1)] = No unit-type
    - Else - Actions

With

Set Hero_Array[Random_Hero] = Hero_Array[Total_Heroes]
Set Hero_Array[Total_Heroes] = No unit-type

deathismyfriend · May 20, 2013

the problem is thats what he shouldnt do. integer A is less efficient and shouldnt be used. u still leak. Setting a boolean is a lot better option than changing the indexes.

Tonex · May 20, 2013

Yes yes, I know that I'm still leaking a point. In this situation, it's fine using Integer A because there are no waits within the loop to mess up Integer A. The Boolean system is highly discouraged because there's the off chance that the game will choose the same Boolean that is set to false 100 or even 1000 times before choosing a hero. There's even a chance that every time you try to randomly select a hero it will keep looping to the same hero enough times so that the trigger quits without doing anything.

deathismyfriend · May 21, 2013

integer A is slower. Whenever u use it as an index it uses the BJ instead of just getting the integer.
booleans will not do that unless u miss handle them.

Tonex · May 21, 2013

I'm saying that atm I for the trigger I'm not caring about efficiency, just making sure that the poor guy knows how everything works. Can you show me an example of the Boolean system? I'm interested in how it would work.

deathismyfriend · May 21, 2013

here u go this will just say u dont get ur unit when u try to buy the same unit twice.
Note: u need to use a multi session emulator to use this.

Tonex · May 21, 2013

Yeah so the problem with your Boolean method is that the player may, even though it's a very very very slight chance, come up with 1000 "u dont get ur unit" messages before finally getting their unit or giving up. With my solution there is always a 100% chance that the player will get a unit.

deathismyfriend · May 21, 2013

if ur doing a random hero selection then use an indexer. But this should still be used for when ur selecting a hero.

MadCreator · May 21, 2013

I am open minded to all suggestion, I even went poking around for tutorial for Boolean.

Set Random_Hero = (Random integer number between 1 and Total_Heroes)
Unit - Create 1 Hero_Array[Random_Hero] for (Owner of (Sold unit)) at (Center of Hero Spawn <gen>

Had produce some undesirable result, I am no expert like the two of you but my guess it have something to do with Random_Hero because when I remove it, it work fine.

As for the Boolean, I manage to use the tutorial from http://www.hiveworkshop.com/forums/trigger-gui-editor-tutorials-279/hero-selection-systems-38699/, but the result I was looking for random wasn't what I wanted. I wanted it to be set, if a hero is pick it can’t be pick again. The tutorial have the triggers I was using, but if you want me to post it on here, let me know.

deathismyfriend · May 21, 2013

if u want it to be random then u should use an indexing type of system. If u use the hashtables like i show u in the map then just index each hero into an integer array and use the hashtable to index these. basically since boolean takes up 0 in the hashtable i use then use 1 for the integer index. You will need to save index number as 1 of tempInt in the hashtable. ( tempInt being the unit type id, the custom script i used)
When someone picks random hero u will need to change the index and the boolean for the hashtable. This will allow it to work when 1 player chooses random and one doesnt. This way no1 willl be able to get the same hero.

Tonex · May 21, 2013

He disables the training of a selected hero and the unit is removed from the array both ways, so players won't be ale to select the same hero anyways.

deathismyfriend · May 21, 2013

that works to lol.

He should de-index the heroes a better way tho instead of just reducing all the indexes.

Tonex · May 21, 2013

Yeah, ap0calypse pointed that out

deathismyfriend · May 21, 2013

if he doesnt know how to de-index its easy. something like this should do.

The triggering unit will need to be changed to whatever it is. If needed.
This will need to be run whenever a hero is bought / trained

Actions
- Set unitTypeId = (Unit-type of (Triggering unit))
- For each (Integer tempInt) from 1 to maxIndex, do (Actions)
  - Loop - Actions
    - If (All Conditions are True) then do (Then Actions) else do (Else Actions)
      - If - Conditions
        
        unitTypeId Equal to heroIndex[tempInt]
      - Then - Actions
        
        Set heroIndex[tempInt] = heroIndex[maxIndex]
        
        Set heroIndex[maxIndex] = No unit-type
        
        Set maxIndex = (maxIndex - 1)
      - Else - Actions

Tonex · May 21, 2013

Yup yup

Tonex said:
For each (Integer A) from 1 to Total_Heroes, do (Actions)

Loop - Actions

If (All Conditions are True) then do (Then Actions) else do (Else Actions)

If - Conditions

(Unit-type of (Sold unit)) Equal to Hero_Array[(Integer A)]

Then - Actions

Set Hero_Array[(Integer A)] = Hero_Array[Total_Heroes]

Set Hero_Array[Total_Heroes] = No unit-type

Else - Actions

Though I left the

Set Total_Heroes = Total_Heroes - 1

outside of the loop.

deathismyfriend · May 21, 2013

ooo sry about that i didnt see urs. umm u should use a tempInt rather than integer A. integer A calls a BJ when its used so its slower and less efficient than using ur own integer like i did.

MadCreator · May 22, 2013

I haven’t toy around with the editor today. Yeah, I think my weakness is de-index; maybe I shouldn't had gotten ahead of myself with my silly idea. I shall play around what you two had posted and if I am having any trouble, I shall let you guys know. Also, want to thank you guys for taken the time to help me.

deathismyfriend · May 22, 2013

Np lol and good luck umm my tutorial has a chapter for indexing called how to index. It shows u how to make a working spell using indexing / de-indexing. It should be able to help u. The tutorial is called things a GUIer should know
It's in my sig. Just click the link

MadCreator · May 22, 2013

Yeah, I think I should stick with simple stuff for now on until I understand what I am doing and how everything work. It would be best if I go through your entire lesson and just make baby map with triggers doing silly stuff haha. But this won't be the last of me posting, I am sure you will see more in the future haha

Ralle · May 22, 2013

A hero selection system is not being run for very long, so the leaks are minor. Let's worry more about triggers that run 1000s of times throughout the game. I don't think a real indexer is needed for this simple purpose.

deathismyfriend · May 22, 2013

np if u need any help feel free to ask lol.

@Ralle
i think that every trigger should be leak free and as efficient as possible.
Yes periodic triggers are the most important but if u get enough inefficient triggers running back to back in a short time it can still cause lag.

Hero Selection Problem

Attachments

Similar threads