[ES] Esta bien como "optimise" el code?

Raddish · **Author**

NO OPTIMIZADO:

PHP Code:


			
case 1:
        {
            if (g_bancoz[id] == 5)
            {
                zp_colored_print(id, "^x04[ZA]^x01 Has llegado al Nivel Maximo!")
            }
            if (g_bancoz[id] == 4)
            {
                if (g_puntosz[id] >= 24)
                {
                    g_bancoz[id]++
                    g_puntosz[id] -= 24
                    g_gastadosz[id] += 24
                }
            }
            if (g_bancoz[id] == 3)
            {
                if (g_puntosz[id] >= 12)
                {
                    g_bancoz[id]++
                    g_puntosz[id] -= 12
                    g_gastadosz[id] += 12
                }
            }
            if (g_bancoz[id] == 2)
            {
                if (g_puntosz[id] >= 6)
                {
                    g_bancoz[id]++
                    g_puntosz[id] -= 6
                    g_gastadosz[id] += 6
                }
            }
            if (g_bancoz[id] == 1)
            {
                if (g_puntosz[id] >= 3)
                {
                    g_bancoz[id]++
                    g_puntosz[id] -= 3
                    g_gastadosz[id] += 3
                }
            }
            if (g_bancoz[id] == 0)
            {
                if (g_puntosz[id] >= 1)
                {
                    g_bancoz[id]++
                    g_puntosz[id] -= 1
                    g_gastadosz[id] += 1
                }
            }
            menu_puntosz(id)
        }

"OPTIMIZADO":

PHP Code:


			
case 1:
        {
            static bancoz
            bancoz = g_bancoz[id]
            switch(bancoz)
            {
                case 5:menu_puntosz(id)
                case 4:
                {
                    if (g_puntosz[id] >= 24)
                    {
                        g_bancoz[id]++
                        g_puntosz[id] -= 24
                        g_gastadosz[id] += 24
                    }
                }
                case 3:
                {
                    if (g_puntosz[id] >= 12)
                    {
                        g_bancoz[id]++
                        g_puntosz[id] -= 12
                        g_gastadosz[id] += 12
                    }
                }
                case 2:
                {
                    if (g_puntosz[id] >= 6)
                    {
                        g_bancoz[id]++
                        g_puntosz[id] -= 6
                        g_gastadosz[id] += 6
                    }
                }
                case 1:
                {
                    if (g_puntosz[id] >= 3)
                    {
                        g_bancoz[id]++
                        g_puntosz[id] -= 3
                        g_gastadosz[id] += 3
                    }
                }
                case 0:
                {
                    if (g_puntosz[id] >= 1)
                    {
                        g_bancoz[id]++
                        g_puntosz[id] -= 1
                        g_gastadosz[id] += 1
                    }
                }
            }
            menu_puntosz(id)
        }

01101101 · 01-22-2010 , 03:16 Re: [ES] Esta bien como "optimise" el code?

optimice

Eso que pusiste es la misma mierda distinto olor, ves porque te digo que te falta aprender MUUUCHO?

PHP Code:


			
new Float:mejoria[] = { 220.0, 250.0, 280.0, 310.0, 350.0 }

//////////////
{
    case 1:
    {
        g_speed[id] > 4 ? client_print(id, print_chat, "Mejoria maxima") : g_speed[id]++
    }
    show_menu_pija(id)
}

/////////////

set_pev(id, pev_maxspeed, mejoria[g_speed[id]])

Habia que pensar mucho?

Raddish · 01-22-2010 , 08:13 Re: [ES] Esta bien como "optimise" el code?

Creo que entendi, aver si decime si esta bien esto. no me compila pero we

es la primera ves q uso eso de

variable > numero ? mayor : menor

u.u

PHP Code:


			
/* Script generated by Pawn Studio */

#include <amxmodx>
#include <amxmisc>

#define PLUGIN    "New Plugin"
#define AUTHOR    "Unknown"
#define VERSION    "1.0"

new g_puntos[33]
new g_speed[33]

public plugin_init()
{
    register_plugin(PLUGIN, VERSION, AUTHOR)
    // Add your own code here
}

public asd(id)
{
    new Float:mejoria[] = { 220.0, 250.0, 280.0, 310.0, 350.0 }
    new const puntos1[] = { 1, 3, 6, 12, 24 }
    
    new g_speed[33]
    new key = 1
    switch(key)
    {
        case 1:
        {
            g_speed[id] > 4 ? client_print(id, print_chat, "Mejoria maxima") : g_speed[id]++
            g_puntos[id] >= puntos1[g_speed[id]] ? g_puntos[id] -= puntos1[g_speed[id]] : client_print(id, print_chat, "no te alcanza los puntos")
        }
        show_menu_pija(id)
    }
    
    /////////////
    
    set_pev(id, pev_maxspeed, mejoria[g_speed[id]]
}

01101101 · 01-22-2010 , 08:22 Re: [ES] Esta bien como "optimise" el code?

Quote:

Originally Posted by Raddish

Creo que entendi, aver si decime si esta bien esto. no me compila pero we

es la primera ves q uso eso de

variable > numero ? mayor : menor

u.u

PHP Code:


			
/* Script generated by Pawn Studio */

#include <amxmodx>
#include <amxmisc>

#define PLUGIN    "New Plugin"
#define AUTHOR    "Unknown"
#define VERSION    "1.0"

new g_puntos[33]
new g_speed[33]

public plugin_init()
{
    register_plugin(PLUGIN, VERSION, AUTHOR)
    // Add your own code here
}

public asd(id)
{
    new Float:mejoria[] = { 220.0, 250.0, 280.0, 310.0, 350.0 }
    new const puntos1[] = { 1, 3, 6, 12, 24 }
    
    new g_speed[33]
    new key = 1
    switch(key)
    {
        case 1:
        {
            g_speed[id] > 4 ? client_print(id, print_chat, "Mejoria maxima") : g_speed[id]++
            g_puntos[id] >= puntos1[g_speed[id]] ? g_puntos[id] -= puntos1[g_speed[id]] : client_print(id, print_chat, "no te alcanza los puntos")
        }
        show_menu_pija(id)
    }
    
    /////////////
    
    set_pev(id, pev_maxspeed, mejoria[g_speed[id]]
}

No entendiste, eso era pseudo codigo.

Un ejemplo practico seria esto

PHP Code:


			
#include <amxmodx>
#include <amxmisc>
#include <fakemeta>

new g_speed[33]
new g_puntos[33]
new Float:mejoria[] = { 220.0, 250.0, 280.0, 310.0, 350.0 }
new valor[] = { 2, 4, 6, 10, 50 }

public plugin_init() 
{
    register_plugin("","","")
    
    // registras prethink etc
}

public prethink(id)
{
    set_pev(id, pev_maxspeed, mejoria[g_speed[id]])
}

public menu(id, key)
{
    switch(key)
    {
        case 1:
        {
            if(g_puntos[id] >= valor[g_speed[id]])
            {
                g_puntos[id] -= valor[g_speed[id]]
                g_speed[id]++
            }
            else client_print(id, print_chat, "No tienes puntos")
        }
    }
}

fechu · 01-22-2010 , 08:25 Re: [ES] Esta bien como "optimise" el code?

perdon que pregunte .
q es ese plugin?

SylvertCs · 01-22-2010 , 08:25 Re: [ES] Esta bien como "optimise" el code?

raddish no era q te ibas del pawn para siempre?

01101101 · 01-22-2010 , 08:26 Re: [ES] Esta bien como "optimise" el code?

Quote:

Originally Posted by fechu

perdon que pregunte .
q es ese plugin?

El ejemplo que yo le di es un pseudo plugin que sube la speed.

Quote:

Originally Posted by Raddish

NO OPTIMIZADO:

INTENTO DE OPTIMIZADO:

Fixed

Raddish · 01-22-2010 , 08:31 Re: [ES] Esta bien como "optimise" el code?

Aver, probe con el expmod ese q hise xd,

PHP Code:


			
#include <amxmodx>
#include <amxmisc>
#include <hamsandwich>
#include <colorchat>
#include <fakemeta>
#include <fvault>

#define MAX_LVL 10
#define MAX_EXP 1000
#define STARTER_EXP 1

#define TASK_SHOWHUD 2100

#define PLUGIN_NAME "Experience Mod"
#define PLUGIN_VERSION "1.0"
#define PLUGIN_AUTHOR "Raddish"

#define MESSAGE_PREFIX "[EXP MOD]"

//---------------------
//      HABILITIES
//----------------------

new health[] = { 100, 200, 300, 400, 500, 600, 700, 800, 900, 1000 }
new armor[] = { 50, 100, 150, 200, 250, 300, 350, 400, 450, 500 }

//----------------------

new const Nivel_List[MAX_LVL] = { 100, 200, 300, 400, 500, 600, 700, 800, 900, 1000 }

new const vault[] = "Experience Mod"

new Nivel[33], Experience[33], g_msgsync

public client_putinserver(id)
{
    LoadExp(id)
    set_task(1.0, "ShowHUD", id+TASK_SHOWHUD, _, _, "b")
}

public client_disconnect(id) remove_task(id+TASK_SHOWHUD)

public plugin_init()
{
    register_plugin(
    PLUGIN_NAME,
    PLUGIN_VERSION,
    PLUGIN_AUTHOR)
    
    g_msgsync = CreateHudSyncObj()
    
    register_cvar("Experience_mod", PLUGIN_VERSION, FCVAR_SERVER|FCVAR_SPONLY)
    register_cvar("killer_exp", "50")
    RegisterHam(Ham_Killed, "player", "fw_PlayerKilled")
    RegisterHam(Ham_Spawn, "player", "fw_PlayerSpawn", 1)
    
}

public fw_PlayerKilled(victim, attacker, shouldgib)
{
    if (attacker == victim || is_user_connected(attacker))
    {
        if (Experience[attacker] >= MAX_EXP)
        {
            Experience[attacker] = MAX_EXP
            ColorChat(attacker, GREEN, "^x04%s^x01 Has llegado al limite de experiencia^x04 %d", MESSAGE_PREFIX, MAX_EXP)
            return PLUGIN_HANDLED
        }
        Experience[attacker] += get_cvar_num("killer_exp")
        SaveExp(attacker)
    }
    return PLUGIN_CONTINUE
}

public fw_PlayerSpawn_Post(id)
{
    if (!is_user_alive(id))
        return;
    
    set_task(0.5, "give_habilitie", id)
    
    check_nivel(id)
}

public check_nivel(id)
{
    Nivel[id] = 0
    while(Experience[id] >= Nivel_List[Nivel[id]])
    {
        Nivel[id]++
    }
}

SaveExp(id)
{
    static name[32];
    get_user_name(id, name, sizeof(name) - 1);
    
    new data[16];
    num_to_str(Experience[id], data, sizeof(data) - 1);
    
    fvault_set_data(vault, name, data);
    
    check_nivel(id)
}

LoadExp(id)
{
    new name[32]
    get_user_name(id, name, charsmax(name))
    
    new data[16]
    if(fvault_get_data(vault, name, data, charsmax(data)))
    {
        Experience[id] = str_to_num(data)
    }
    else
    {
        Experience[id] = STARTER_EXP
    }
}

public Message(id)
    if (is_user_connected(id))
        ColorChat(id, GREEN, "^x04%s^x01 This server is using %s^x04 v%s^x01 By^x04 %s", MESSAGE_PREFIX, PLUGIN_NAME, PLUGIN_VERSION, PLUGIN_AUTHOR)
    
public give_habilitie(id)
{
    set_pev(id, pev_health, health[Nivel[id]])
    set_pev(id, pev_armorvalue, armor[Nivel[id]])
    
}

public ShowHUD(id)
{
    if (!is_user_connected(id))
        return;
    
    static name[32]
    get_user_name(id, name, charsmax(name))
    
    set_hudmessage(255, 255, 255, 0.01, 0.85, 1, 0.0, 3.0, 2.0, 1.0, -1)
    ShowSyncHudMsg(id, g_msgsync, "[Health: %d] | [Armor: %d]^n[Experience: %d/%d] | [Nivel: %d/%d]", get_user_health(id), get_user_armor(id), Experience[id], MAX_EXP, Nivel[id], MAX_LVL)
}

Raddish · 01-22-2010 , 08:32 Re: [ES] Esta bien como "optimise" el code?

Quote:

Originally Posted by SylvertCs

raddish no era q te ibas del pawn para siempre?

Yo nunca dije que me iba para siempre solo que ya me aburria

01101101 · 01-22-2010 , 08:37 Re: [ES] Esta bien como "optimise" el code?

Hay un monton de errores ahi que me dan paja corregir.
Lo unico que cambie porque fui lo primero que note (porque esta en la primer linea lol) es unos defines asquerosos que usabas para poner maximos.

Ahi lo corregi, igual el code sigue desastroso.

PHP Code:


			
#include <amxmodx>
#include <amxmisc>
#include <hamsandwich>
#include <colorchat>
#include <fakemeta>
#include <fvault>

#define STARTER_EXP 1

#define TASK_SHOWHUD 2100

#define PLUGIN_NAME "Experience Mod"
#define PLUGIN_VERSION "1.0"
#define PLUGIN_AUTHOR "Raddish"

#define MESSAGE_PREFIX "[EXP MOD]"

//---------------------
//      HABILITIES
//----------------------

new health[] = { 100, 200, 300, 400, 500, 600, 700, 800, 900, 1000 }
new armor[] = { 50, 100, 150, 200, 250, 300, 350, 400, 450, 500 }

//----------------------

new const Nivel_List[] = { 100, 200, 300, 400, 500, 600, 700, 800, 900, 1000 }

new const vault[] = "Experience Mod"

new Nivel[33], Experience[33], g_msgsync

public client_putinserver(id)
{
    LoadExp(id)
    set_task(1.0, "ShowHUD", id+TASK_SHOWHUD, _, _, "b")
}

public client_disconnect(id) remove_task(id+TASK_SHOWHUD)

public plugin_init()
{
    register_plugin(
    PLUGIN_NAME,
    PLUGIN_VERSION,
    PLUGIN_AUTHOR)
    
    g_msgsync = CreateHudSyncObj()
    
    register_cvar("Experience_mod", PLUGIN_VERSION, FCVAR_SERVER|FCVAR_SPONLY)
    register_cvar("killer_exp", "50")
    RegisterHam(Ham_Killed, "player", "fw_PlayerKilled")
    RegisterHam(Ham_Spawn, "player", "fw_PlayerSpawn", 1)
    
}

public fw_PlayerKilled(victim, attacker, shouldgib)
{
    if (attacker == victim || is_user_connected(attacker))
    {
        if (Experience[attacker] >= Nivel_List[sizeof Nivel_List - 1])
        {
            Experience[attacker] = Nivel_List[sizeof Nivel_List - 1]
            ColorChat(attacker, GREEN, "^x04%s^x01 Has llegado al limite de experiencia^x04 %d", MESSAGE_PREFIX, Nivel_List[sizeof Nivel_List - 1])
            return PLUGIN_HANDLED
        }
        Experience[attacker] += get_cvar_num("killer_exp")
        SaveExp(attacker)
    }
    return PLUGIN_CONTINUE
}

public fw_PlayerSpawn_Post(id)
{
    if (!is_user_alive(id))
        return;
    
    set_task(0.5, "give_habilitie", id)
    
    check_nivel(id)
}

public check_nivel(id)
{
    Nivel[id] = 0
    while(Experience[id] >= Nivel_List[Nivel[id]])
    {
        Nivel[id]++
    }
}

SaveExp(id)
{
static name[32];
get_user_name(id, name, sizeof(name) - 1);

new data[16];
num_to_str(Experience[id], data, sizeof(data) - 1);

fvault_set_data(vault, name, data);

check_nivel(id)
}

LoadExp(id)
{
new name[32]
get_user_name(id, name, charsmax(name))

new data[16]
if(fvault_get_data(vault, name, data, charsmax(data)))
{
Experience[id] = str_to_num(data)
}
else
{
Experience[id] = STARTER_EXP
}
}

public Message(id)
if (is_user_connected(id))
ColorChat(id, GREEN, "^x04%s^x01 This server is using %s^x04 v%s^x01 By^x04 %s", MESSAGE_PREFIX, PLUGIN_NAME, PLUGIN_VERSION, PLUGIN_AUTHOR)

public give_habilitie(id)
{
    set_pev(id, pev_health, health[Nivel[id]])
    set_pev(id, pev_armorvalue, armor[Nivel[id]])
    
}

public ShowHUD(id)
{
    if (!is_user_connected(id))
        return;
    
    static name[32]
    get_user_name(id, name, charsmax(name))
    
    set_hudmessage(255, 255, 255, 0.01, 0.85, 1, 0.0, 3.0, 2.0, 1.0, -1)
    ShowSyncHudMsg(id, g_msgsync, "[Health: %d] | [Armor: %d]^n[Experience: %d/%d] | [Nivel: %d/%d]", get_user_health(id), get_user_armor(id), Experience[id], Nivel_List[sizeof Nivel_List - 1], Nivel[id], sizeof Nivel_List)
}