Detect Host System

[Moderator]

08:42, 8th Jul 2014
BPower:
Rejected.

 

[GywGod133]

maybe change the LONG ITE
into this

• If (All Conditions are True) then do (Then Actions) else do (Else Actions)
  • If - Conditions
    • ((Triggering player) slot status) Equal to Is playing
  • Then - Actions
    • Set Host = (Triggering player)
  • Else - Actions

 

[Pinzu]

What if the one playing red isn't host?

 

[Xonok]

There is a better way for detecting host.

Let all players load their number in gamecache, then synchronize the results. Over 90% of the time the remaining number will be the host.

 

[GywGod133]

There is a better way for detecting host.

Let all players load their number in gamecache, then synchronize the results. Over 90% of the time the remaining number will be the host.

oh.... can you give an example?

 

[Xonok]

Well, the JASS code is this:

call FlushGameCache(InitGameCache("gamehost.w3v"))
set udg_gamehost = InitGameCache("gamehost.w3v")
call StoreInteger ( udg_gamehost, "host", "host", GetPlayerId ( GetLocalPlayer () ) )
call TriggerSyncStart ()
call SyncStoredInteger ( udg_gamehost, "host", "host" )
call TriggerSyncReady ()
set udg_Host = GetStoredInteger ( udg_gamehost, "host", "host" )
call FlushStoredMission(udg_gamehost,"host")

I don't think it can be done in GUI though, due to the lack of sync natives.

 

[Wrda]

That won't work with bots.

 

[Xonok]

Bots don't really need to look for a player host either. They have their own ways.

 

[Wrda]

No, you don't get it. Suppose that a player named "jean" hosts a game with a bot named "anybot", the player joins the game, some other players join too, and he starts the game. Obviously the bot isn't a player, just a means to host a game, so when you use this method to detect the host ANY player ("jean" or any other players) will be host (the player who is more synchronized).

 

[Xonok]

My point still stands though. Bots still have one player that they consider the owner. Yes, using a bot means that detecting host should be done in some other way, but there are ways that bots can transfer information to the game.

 

[IcemanBo]

You only check player slot status in a linear way. How can you define the host this way?
All these waits, clear screens, printing "detecting" are not needed at all.
Just converting a GUI trigger into custom text is also not needed and useless.
Edit: And you don't check for all players.^^

 

[Vinz]

Just converting a GUI trigger into custom text is also not needed and useless.

This. The code looks bad. And you just check from player 8 down to 1, how does that even identify the host?

 

[gorillabull]

converted gui is more pro than just a plain gui script

 

[Tank-Commander]

I don't understand how this is doing anything useful, rather I don't even know how it's supposed to even begin to achieve the advertised task as the others above have already said, also yeah GUI to JASS conversion, just not worth it, makes no difference.

 

[EdgeOfChaos]

Don't convert GUI to JASS, just leave it.

Btw, a Player Group loop can eliminate the need for any If statement.

Why do you need the "Detecting" cinematic? It just wastes time.
-----

Here's how to improve it (something like this).

• AddEvents
  • Events
    • Map initialization
  • Conditions
  • Actions
    • Player Group - Pick every player in (All players) and do (Actions)
      • Loop - Actions
        • Trigger - Add to DetectHost <gen> the event (Player - (Picked player) types a chat message containing -host as An exact match)

• DetectHost
  • Events
  • Conditions
  • Actions
    • Player Group - Pick every player in (All players matching ((((Matching player) controller) Equal to User) and (((Matching player) slot status) Equal to Is playing))) and do (Actions)
      • Loop - Actions
        • Game - Display to (All players) the text: (Host detected: + (Name of (Picked player)))
        • Skip remaining actions

Takes way fewer lines, same functionality.

(Also remove player group leaks)

 

[Bannar]

Facts:
#1 this is bad.
I.e this does not detect host no matter if it's hosted via bot or not.

#2 Xonok is right about his example, and his solution is closest to approach we actually would want to use.

#3Wrda is right. There is no convenient way of detecting bot-host I can think of.

Considering facts: if we have to decide between cake and empty plate I'd go for cake, even if its pie that I'd like to eat.
Cake - sync solution.

 

[Vinz]

Cake

Are you like dissing me?

 

[Bannar]

Nope, tho I enjoy chocolate cake with strawberries on the top of that.
"If you know what I mean"

 

[GywGod133]

@Edge
Maybe you have player group leak :/
(Im not sure)

 

[EdgeOfChaos]

Yeah, I think it leaks, I just wrote it in 1 min as a demo of what can remove every If statement.

Not on a computer with WE right now, but:
Set TempForce = All Players Matching Condition...
For (TempForce)
...
DestroyForce(udg_TempForce)