[ES] No mas granadas para vos este round

nicolazo103 · **Author**

Bueno, estaba tranquilo en mi compu después de ver el partido y se me paso por la cabeza esto.
(que carajo tiene que ver con el partido? diran. A veces flasheaba que habia mas de una pelota en el campo, cuando en realidad eran globos)

No lo puedo probar porque no tengo el counter

pero no me dio errores
Es un plugin simple, por favor, lo que puedan mejorar posteenlo.

Code:

/* Script generated by Pawn Studio */

#include <amxmodx>
#include <hamsandwich>

new NadeCounter[33][3]
new gPlayers, Grenades[3], ShouldReset

public plugin_init()
{
    register_plugin("No more Nades for YOU", "1.0", "TheÜberRazorBack")
    
    RegisterHam(Ham_Spawn, "player", "fwSpawnPost", 1)
    
    gPlayers = get_maxplayers()
    
    Grenades[0] = register_cvar("amx_hegrenade_limit", "1")
    Grenades[1] = register_cvar("amx_flashbang_limit", "2")
    Grenades[2] = register_cvar("amx_smokegrenade_limit", "1")
    
    ShouldReset = register_cvar("amx_reset_counter", "0.0")
}
public fwSpawnPost(id)
{
    reset_nades()
}
reset_nades()
{
    for(new i; i < gPlayers; i++)
    {
        //me parece que esto esta al pedo pero bue :P
        if(!is_user_alive(i))
            continue;
        
        for(new a; a < 3; a++)
        {
            NadeCounter[i][a] = 0
        }
    }
}
public grenade_throw( id , grenadeindex , wId )
{
    if(wId == CSW_HEGRENADE) 
    {       
        if(NadeCounter[id][0] == get_pcvar_num(Grenades[0]))
            return PLUGIN_HANDLED
    
        NadeCounter[id][0]++
        client_print(id, print_chat, "Used %d of %d available he grenades", NadeCounter[id][0], get_pcvar_num(Grenades[0]))
        show_print(id, 0, "hegrenades")
    }
    else if(wId == CSW_FLASHBANG) {
        
        if(NadeCounter[id][1] == get_pcvar_num(Grenades[1]))
            return PLUGIN_HANDLED
        
        NadeCounter[id][1]++
        show_print(id, 1, "flashbangs")
    }
    else if(wId == CSW_SMOKEGRENADE) {
        
        if(NadeCounter[id][2] == get_pcvar_num(Grenades[2]))
            return PLUGIN_HANDLED
        
        NadeCounter[id][2]++
        show_print(id, 2, "smokegrenades")
    }
    
    if(get_pcvar_float(ShouldReset) > 0.0)
        set_task(get_pcvar_float(ShouldReset), "reset_nades", id)
    
    return PLUGIN_HANDLED
}
public show_print(id, number, const name[])
{
    client_print(id, print_chat, "Used %d of %d available %s ^n %s", NadeCounter[id][number], get_pcvar_num(Grenades[number]), name, 
    get_pcvar_float(ShouldReset) == 0.0? "The Nade Counter Doesn´t Reset until the end of the round": "The Nade Counter Resets")
    
    return PLUGIN_HANDLED
}

lucas_7_94 · 06-17-2010 , 11:05 Re: [ES] No mas granadas para vos este round

mandate un swith en los case de las granadas.

Alucard^ · 06-17-2010 , 11:27 Re: [ES] No mas granadas para vos este round

Code:

public fwSpawnPost(id)
{
    reset_nades()
}
reset_nades()
{
    for(new i; i < gPlayers; i++)
    {
        //me parece que esto esta al pedo pero bue :P
        if(!is_user_alive(i))
            continue;
        
        for(new a; a < 3; a++)
        {
            NadeCounter[i][a] = 0
        }
    }
}

Aca hay varias cosas mal... primero que tas haciendo reset_nades( ) en el player spawn de cada player...

O sea que cuando alguien spawnee se les resetea a TODOS, lo cual no tiene que ser asi...

Lo mas conveniente es que hookees el RoundStart y que ahi hagas esto...

Otra cosa es que el i < gPlayers esta mal, deberia ir i <= gPlayers.

Y el is_user_alive( ) cambiarlo por is_user_connected( )

Con respecto al grenade_throw( ) que es del csx.inc, se que funciona sin incluirlo, pero creo que para prevenir bugs o algo asi, conviene incluirlo. No toy seguro de esto, pero de todas formas no te da ningun inconveniente hacerlo ><.

Code:

    if(wId == CSW_HEGRENADE) 
    {      
        if(NadeCounter[id][0] == get_pcvar_num(Grenades[0]))
            return PLUGIN_HANDLED
    
        NadeCounter[id][0]++
        client_print(id, print_chat, "Used %d of %d available he grenades", NadeCounter[id][0], get_pcvar_num(Grenades[0]))
        show_print(id, 0, "hegrenades")
    }
    else if(wId == CSW_FLASHBANG) {
        
        if(NadeCounter[id][1] == get_pcvar_num(Grenades[1]))
            return PLUGIN_HANDLED
        
        NadeCounter[id][1]++
        show_print(id, 1, "flashbangs")
    }
    else if(wId == CSW_SMOKEGRENADE) {
        
        if(NadeCounter[id][2] == get_pcvar_num(Grenades[2]))
            return PLUGIN_HANDLED
        
        NadeCounter[id][2]++
        show_print(id, 2, "smokegrenades")
    }

Si no me equivoco esto se puede simplificar, no tengo ganas de fijarme mucho ahora pero algo como esto:

Code:

 if(NadeCounter[id][type] == get_pcvar_num(Grenades[type]) )

Pero deberias obtener el type... mira este plugin, te va a servir: http://forums.alliedmods.net/showthread.php?t=109973

Kiske · 06-17-2010 , 11:33 Re: [ES] No mas granadas para vos este round

PHP Code:


			
        for(new a; a < 3; a++)
        {
            NadeCounter[i][a] = 0
        }

PHP Code:


			
NadeCounter[i] = { 0, 0, 0 }

PD: Vazmar, te confundiste de Post.

nicolazo103 · *Last edited by nicolazo103; 06-17-2010 at 12:41.*

@Alucard^
Ok
A mi me parecia que esa parte se podia simplificar, pero ni idea como
Lo del type no te entendi

Fijate si esta mejor ahora

Code:

/* Script generated by Pawn Studio */

#include <amxmodx>

new NadeCounter[33][3]
new gPlayers, Grenades[3], ShouldReset

public plugin_init()
{
    register_plugin("No more Nades for YOU", "1.0", "TheÜberRazorBack")
    
    gPlayers = get_maxplayers()
    
    Grenades[0] = register_cvar("amx_hegrenade_limit", "1")
    Grenades[1] = register_cvar("amx_flashbang_limit", "2")
    Grenades[2] = register_cvar("amx_smokegrenade_limit", "1")
    
    ShouldReset = register_cvar("amx_reset_counter", "0.0")
    
    register_event("HLTV", "event_round_start", "a", "1=0", "2=0")
}
public event_round_start()
{
    reset_nades()
}
reset_nades()
{
    for(new i; i <= gPlayers; i++)
    {
        if(!is_user_connected(i))
            continue;
        
        /*for(new a; a < 3; a++)
        {
            NadeCounter[i][a] = 0
        }*/
        NadeCounter[i] = { 0, 0, 0 }
    }
}

public grenade_throw( id , grenadeindex , wId )
{
    switch(wId)
    {
        case CSW_HEGRENADE: {
                
            if(NadeCounter[id][0] == get_pcvar_num(Grenades[0]))
                return PLUGIN_HANDLED
    
            NadeCounter[id][0]++
            show_print(id, 0, "hegrenades")
        }
        case CSW_FLASHBANG: {
            if(NadeCounter[id][1] == get_pcvar_num(Grenades[1]))
                return PLUGIN_HANDLED
        
            NadeCounter[id][1]++
            show_print(id, 1, "flashbangs")
        }
        case CSW_SMOKEGRENADE: {
            if(NadeCounter[id][2] == get_pcvar_num(Grenades[2]))
                return PLUGIN_HANDLED
        
            NadeCounter[id][2]++
            show_print(id, 2, "smokegrenades")
        }
    }
    
    if(get_pcvar_float(ShouldReset) > 0.0)
        set_task(get_pcvar_float(ShouldReset), "reset_nades", id)
    
    return PLUGIN_HANDLED
}
public show_print(id, number, const name[])
{
    client_print(id, print_chat, "Used %d of %d available %s ^n %s", NadeCounter[id][number], get_pcvar_num(Grenades[number]), name, 
    get_pcvar_float(ShouldReset) == 0.0? "The Nade Counter Doesn´t Reset until the end of the round": "The Nade Counter Resets")
    
    return PLUGIN_HANDLED
}

Alucard^ · 06-17-2010 , 12:33 Re: [ES] No mas granadas para vos este round

Antes que nada plz, acordate de acomodar el codigo con CTRL+I porque sino molesta leerlo :p

Code:

reset_nades()
{
    for(new i; i <= gPlayers; i++)
    {
        if(!is_user_connected(i))
            continue;
    }
            for(new i; i <= gPlayers; i++)
            {
                if(!is_user_connected(i))
                    continue;
                
                /*for(new a; a < 3; a++)
                {
                NadeCounter[i][a] = 0
                }*/
                NadeCounter[i] = { 0, 0, 0 }
            }
        }
    }
}

Mas alla del error de llave que hay aca, que quisiste hacer usando 2 loops? xD

Code:

reset_nades()
{
    for(new i; i <= gPlayers; i++)
    {
        if(!is_user_connected(i) )
            continue;
        
        NadeCounter[i] = { 0, 0, 0 }
    }
}

Fixed.

Con respecto a lo de type, ahora me tengo que ir pero... te fijaste en mi plugin? Fijate que yo bloqueo todas las armas y las granadas pero dependiendo de la cvar que tiene, y no hago case o condiciones por cada una de las armas o granadas. Simplificalo de esa forma y listo =p

nicolazo103 · *Last edited by nicolazo103; 06-17-2010 at 12:46.*

Lo que pasa es que puse lo de kiske a lo ultimo porque me olvide y me quedo para el tuje.
Si, me fije en tu plugin, lo que no entiendo es lo de GetWeaponboxType(iWeapon) que lo pones como get_user_weapon(id)

Alucard^ · 06-17-2010 , 14:21 Re: [ES] No mas granadas para vos este round

Quote:

Originally Posted by nicolazo103

Lo que pasa es que puse lo de kiske a lo ultimo porque me olvide y me quedo para el tuje.
Si, me fije en tu plugin, lo que no entiendo es lo de GetWeaponboxType(iWeapon) que lo pones como get_user_weapon(id)

Ah jajaja na a eso no le des bola... lo que pasa es que el GetWeaponboxType( ) tenia otras cosas mas, otros metodos que habia testeado antes pero, al final funciono bien el get_user_weapon( ) y de paso lo deje asi porque queda mas powa ese name :E aparte, el otro es GetGrenadeType o algo asi, entonces asi queda parecido =p. Pero en resumen, es lo mismo.

nicolazo103 · *Last edited by nicolazo103; 06-17-2010 at 16:19.*

Me confundi GetWeaponboxType con GetWeaponType

Pero aun asi sigo sin entender como queres que lo haga

algo asi?

Code:

new Granada [][] {
    CSW_HEGRENADE,
    CSW_FLASHBANG,
    CSW_SMOKEGRENADE
}
new NadeCounter[33][Granada]
new Cvars[Granada]
/////////
Cvars[0] = register_cvar("amx_he_limit", "2")

////////
new type = get_weaponid(get_user_weapon(id))

if(type && NadeCounter[id][type] == get_pcvar_num(Grenade[type])

Todavia tengo que pensar bien la idea porque me parece que podria usar wId

EDIT: Me parece que entendi, voy a tratar de hacerlo
EDIT2: Como puedo usar un operador condicional con 3 opciones?

Alucard^ · 06-17-2010 , 17:02 Re: [ES] No mas granadas para vos este round

Emmm con 3 opciones no se puede, se pueden ir encadenando varios, pero tienen que estar relacionados por asi decirlo... mostrame que es lo que queres y te digo si se puede.