[Trigger] How to see what three integers are highest?

[Airandius]

I have a map where 8 players each have their own integer value.
How can I see who has the highest?

 

[Kercyn]

The only thing I can think of is multiple If/Then statements, but that's really not effective. There is no multiple variable comparison in GUI (that I know of), but that may be possible in JASS.

 

[Super-Sheep]

I dont even understand the question. Player 8 has the highest number?

 

[Zombie007]

no randomly player with highest integer number is viewed.

like
Player1 integer=1
Player2 integer=6
Player3 integer=4
etc...
Player 2 have highest integer vaule.



Wait please i try do this

 

[Teelo]

Well since a binary search is clearly not possible here...

//** Get highest player value
// Requires the values of the players to search through to be in a variable.
// Substitute some function call or whatever you use to get this value as needed.
// 
// @param: none
// @returns: the native player number of the player with the highest player value
//**
function getHighestPlayerValue takes nothing returns integer
  local integer highestPlayerNum = 0
  local integer highestValue = 0
  local integer index = 0

  loop
    exitwhen index == 12
    if (udg_theVariableWhereYouStoredTheValues[index] > highestValue) then
      set highestPlayerNum = index
      set highestValue = udg_variable[index]
    endif
    set index = index + 1
  endloop
  return highestPlayerNum
endfunction

 

[Kercyn]

If the number of players is lower than 12, wouldn't

exitwhen index == 12

or

set index = index + 1

cause problems?

Maybe you could count every human player in the game and then use the exitwhen command.

 

[Avator]

The only way to do this in GUI is

• Actions
  • Set Integer[1] = HighestInteger
  • If all conditions are True, Then do Then Actions, Else do Else Actions
    • If - Conditions
      • Integer[1] Less then or equal to Integer[2]
    • Then - Actions
      • Set Integer[2] = HighestInteger
    • Else - Actions
  • If all conditions are True, Then do Then Actions, Else do Else Actions
    • If - Conditions
      • HighestInteger Less then or equal to Integer[3]
    • Then - Actions
      • Set Integer[3] = HighestInteger
    • Else - Actions
  • If all conditions are True, Then do Then Actions, Else do Else Actions
    • If - Conditions
      • HighestInteger Less then or equal to Integer[4]
    • Then - Actions
      • Set Integer[4] = HighestInteger
    • Else - Actions
  • If all conditions are True, Then do Then Actions, Else do Else Actions
    • If - Conditions
      • HighestInteger Less then or equal to Integer[5]
    • Then - Actions
      • Set Integer[5] = HighestInteger
    • Else - Actions
  • If all conditions are True, Then do Then Actions, Else do Else Actions
    • If - Conditions
      • HighestInteger Less then or equal to Integer[6]
    • Then - Actions
      • Set Integer[6] = HighestInteger
    • Else - Actions
  • If all conditions are True, Then do Then Actions, Else do Else Actions
    • If - Conditions
      • HighestInteger Less then or equal to Integer[7]
    • Then - Actions
      • Set Integer[7] = HighestInteger
    • Else - Actions
  • If all conditions are True, Then do Then Actions, Else do Else Actions
    • If - Conditions
      • HighestInteger Less then or equal to Integer[8]
    • Then - Actions
      • Set Integer[8] = HighestInteger
    • Else - Actions

 

[Teelo]

If the number of players is lower than 12, wouldn't

exitwhen index == 12

or

set index = index + 1

cause problems?

Maybe you could count every human player in the game and then use the exitwhen command.

Never caused problems on my maps.

To do that in gui, just write all those lines with custom script commands

 

[raft_pl]

I heard that you can't have more than one local in GUI... I mean, only the first custom scripted local gets registered, the others dont.
So that a trigger like this wouldn't work;

• Actions
  • Custom script: local integer a = 1
  • Custom script: local integer b = 2
  • Custom script: local unit u = GetTriggerUnit()
  • Custom script: call SetUnitState(u, UNIT_STATE_LIFE, (a + b))

Not sure tho.

 

[Caleb]

i think the hint is that it has something to do with the math - max function. this function compares two values that determines which value is higher.

this trigger is for example:

Event:
Player - Player 1 (Red) types a chat message containing compare as An exact match

Condition:none

Action:
Game - Display to (All players) the text: (String((Max((Life of Peasant 0056 <gen>), (Life of Footman 0057 <gen>)))))

this triggers simply display the hit point of footman because it has more hit point than that of the peasant

 

[GhostWolf]

It's a simple freaking loop, exactly like Teelo did. You don't have to copy it a letter by letter, understand how it works and translate it to your code -.-

 

[Teelo]

Edit: Wasnt paying attention.

 

[Airandius]

Lol, I never insisted on anything. You got pist at the wrong person.
I'll use avators way, it might not be the best, but at least I understand it. xD

Still thanks to all + rep

 

[GhostWolf]

Here, if you're too stupid to actually read what the Jass code does, this is what it does

For [B]all_numbers[/B] do:
    if [B]current_number[/B] is bigger then [B]biggest_number[/B] then set [B]biggest_number[/B] to [B]current_number[/B]

Is that so freaking hard to understand?

 

[Kercyn]

Here, if you're too stupid to actually read what the Jass code does, this is what it does

For [B]all_numbers[/B] do
    if [B]current_number[/B] is bigger then [B]biggest_number[/B] then set [B]biggest_number[/B] to [B]current_number[/B]

Is that so freaking hard to understand?

Erm, did you even think he might not know JASS? Nobody was born knowing everything. Would you understand an advanced C++ app if the only thing you knew was "Hello World!"? If you're not used to programming, it's not that easy, you know...

 

[Deaod]

You know how to read texts, right? Reading (Jass) code is the same, its like talking to the CPU (very indirectly on a technical level though).
You can even recreate that function in GUI (much worse but its possible).

 

[onix_noob]

The only way to do this in GUI is (onix's edit)

• Actions
• For each intiger from 1 to 12 do actions
  • Actions
    • If all conditions are True, Then do Then Actions, Else do Else Actions
      • If - Conditions
        • Integer[intigerA] Greater then or equal to HighestInteger
      • Then - Actions
        • Set HighestInteger = Integer[IntigerA]
      • Else - Actions

i write it by my hands i am 2 lazy to open the world editor but i am sure it will work 100% and is leak free so u don't need nothing more

 

[Kercyn]

You know how to read texts, right? Reading (Jass) code is the same, its like talking to the CPU (very indirectly on a technical level though).
You can even recreate that function in GUI (much worse but its possible).

Yes, I know all that. What I say is that there are people that DO NOT know how to program. Which means that they actually can't read code. Is it that terrible? Is he a retard just because he can't read JASS?

 

[onix_noob]

Yes, I know all that. What I say is that there are people that DO NOT know how to program. Which means that they actually can't read code. Is it that terrible? Is he a retard just because he can't read JASS?

Jass is good for separate multiboards, long spells, and easy to make spells MPI and MPU and ofc to evade leaks if u are careful. Almost all other systems can be done in GUI but u need alot of expirience in it

 

[Avator]

Alright folks, that's enough. The thread starter has made clear that the solution of his problem has been found and knows how he's going to deal with it.

No more posts.

 

[GhostWolf]

Yes, I know all that. What I say is that there are people that DO NOT know how to program. Which means that they actually can't read code. Is it that terrible? Is he a retard just because he can't read JASS?

It's not that they can't, it is that they don't even bother trying. I could figure what a small code like that does before I knew anything about programming or Jass.

Jass is good for separate multiboards, long spells, and easy to make spells MPI and MPU and ofc to evade leaks if u are careful. Almost all other systems can be done in GUI but u need alot of expirience in it

I find it hard to see how this relates to the last previous posts, or to the thread at all.

 

[onix_noob]

I find it hard to see how this relates to the last previous posts, or to the thread at all.

I find it hard to see how this relates to the last previous posts, or to the thread at all. I couldn't answer better than u already do, thanks for you help read my signature

 

[GhostWolf]

I find it hard to see how this relates to the last previous posts, or to the thread at all. I couldn't answer better than u already do, thanks for you help read my signature

The first part made no sense.
And does the second part mean I have a big penis? (should I be happy or sad?)

 

[Schuhu]

Wll, we have two "only" ways now, but i will throw in a third:

• Leaderboard - Create a leaderboard for (All players) titled Hello i count.
• Set board = (Last created leaderboard)
• Leaderboard - Hide board
• -------- IntegerVar[A] is the integer variable you have connected to each player --------
• For each (Integer A) from 1 to 8, do (Actions)
  • Loop-Actions
    • Leaderboard - Change the value for (Player((Integer A))) in (Last created leaderboard) to Integer[(Integer A)]
• Leaderboard - Sort board by Value in decending order
• Set TempPlayer = (Player in position 1 of board)
• Set High = Integer[(Player number of TempPlayer)]
• Bestenliste - Destroy board

I don't know if the hide board is needed
Now, in TempPlayer you have the leading player and High is his value.

 

[Teelo]

Lol. Leaderboard. Good one ^^

 

[Lord_BoNes]

Did we all forget that the leaderboard can sort itself? LOL

 

[Schuhu]

Hell, whats the english version of this one:

• Leaderboard - Sort board by Value in Absteigend order

I have no idea, lol.

 

[Lord_BoNes]

I take it that it means decending order (from highest to lowest).

 

[Schuhu]

Sounds good, i edited it.

 

[Teelo]

It works, but doing a mergesort O(nlogn) and then a binary search O(logn) is definitely less efficient than my method, doing a sequential search O(n).

 

[Airandius]

People, the problem is already solved, stop spamming.
And leader boards suck.

/no replies