funcion obtener al mejor jugador

[Donald Duckling]

Hola tanto tiempo sin pasar por aca

Quisiera saber si esta funcion para obtener al mejor jugador del mapa está bien, creo que está mal hecha y por eso pido ayuda

Si lo está por favor diganme en donde está mal

Muchas gracias

Código PHP:

GetBestPlayer()
{
    static Frags[MAX_PLAYERS+1], id, best, is_admin;

    for(id = 1; id<=iMaxPlayers; id++)
    {
        if(is_user_bot(id) || !is_user_connected(id))
            continue;

        is_admin = is_user_admin(id)

        if(is_admin && !g_admins)
            continue;

        Frags[id] = get_user_frags(id)

        if (Frags[id] > Frags[best])
            best = id;
    }
    return best;
} 


[[N]drs]

Código PHP:

new bool:g_bIgnoreAdmins = true

GetBestPlayer()
{
    new iFrags[33], id, iBest

    for(id = 1; id<=iMaxPlayers; id++)
    {
        if(!is_user_connected(id) || is_user_bot(id))
            continue

        if(is_user_admin(id) && g_bIgnoreAdmins)
            continue

        iFrags[id] = get_user_frags(id)

        if (iFrags[id] > iFrags[iBest])
            iBest = id
    }
    
    return iBest
} 


[Donald Duckling]

Código PHP:

new bool:g_bIgnoreAdmins = true

GetBestPlayer()
{
    new iFrags[33], id, iBest

    for(id = 1; id<=iMaxPlayers; id++)
    {
        if(!is_user_connected(id) || is_user_bot(id))
            continue

        if(is_user_admin(id) && g_bIgnoreAdmins)
            continue

        iFrags[id] = get_user_frags(id)

        if (iFrags[id] > iFrags[iBest])
            iBest = id
    }
    
    return iBest
} 


Oh muchas gracias Nrds va finisimo. Disculpa habría una manera de optimizarla? o está bien así? voy a usar esa funcion para llamarla cada 1 segundo con un task y vi varios threads donde recomiendan optimizar ciertas funciones pero no se si entren en este caso

[[N]drs]

Oh muchas gracias Nrds va finisimo. Disculpa habría una manera de optimizarla? o está bien así? voy a usar esa funcion para llamarla cada 1 segundo con un task y vi varios threads donde recomiendan optimizar ciertas funciones pero no se si entren en este caso

Código PHP:

new bool:g_bIgnoreAdmins = true

GetBestPlayer()
{
    static iFrags[33], id, iBest
    arrayset(iFrags, 0, 33)
    id=0
    iBest=0

    for(id = 1; id<=iMaxPlayers; id++)
    {
        if(!is_user_connected(id) || is_user_bot(id))
            continue

        if(is_user_admin(id) && g_bIgnoreAdmins)
            continue

        iFrags[id] = get_user_frags(id)

        if (iFrags[id] > iFrags[iBest])
            iBest = id
    }
    
    return iBest
} 


[Donald Duckling]

Muchas gracias!

[[E]manuelitop15]

Muchas gracias!

Por qué llamarias a la funcion cada 1 segundo, si ya esta función te devuelve el id del mejor jugador, no creo q sea necesario obtenerlo varios veces en pocos segundos.
Qué es lo que tenes pensado hacer?

[Donald Duckling]

Por qué llamarias a la funcion cada 1 segundo, si ya esta función te devuelve el id del mejor jugador, no creo q sea necesario obtenerlo varios veces en pocos segundos.
Qué es lo que tenes pensado hacer?

Un hud donde muestre el jugador con mas kills del mapa al realizar algun evento en el servidor

Código PHP:

set_task(1.0, "best_hud", .flags="b")

public best_hud()
{
    static BestPlayer, UserName[32], FragsB;

    BestPlayer = GetBestPlayer()
    get_user_name(BestPlayer, UserName, charsmax(UserName))
    FragsB = get_user_frags(BestPlayer)


    set_dhudmessage(157,252,3, -1.0, 0.01, 0, 1.0, 1.0)

    if(!FragsB)
        formatex(formato, charsmax(formato), "AUN NO HAY MEJOR JUGADOR DEL MAPA")

    else
        formatex(formato, charsmax(formato), "MEJOR JUGADOR DEL MAPA: %s^nLLEVA ACTUALMENTE: %d KILLS", UserName, FragsB)

    ShowSyncHudMsg(0, hudhandler, formato)
}