[dalegato]

Hola, les pido un favor chiquito si no me optimizarian esto por favor.

PHP Code:

        if (g_l4d[attacker])
        {
            new weapon = get_user_weapon(attacker)
            
            switch (weapon)
            {
                case CSW_M4A1: damage *= 2.0
                case CSW_P90: damage *= 2.0
                case CSW_MAC10: damage *= 2.0
                case CSW_SG552: damage *= 2.0
            }
            SetHamParamFloat(4, damage)
        }
        
        if (g_ninja[attacker])
        {
            new weapon = get_user_weapon(attacker)
            
            switch (weapon)
            {
                case CSW_KNIFE: damage *= 25.0
            }
            SetHamParamFloat(4, damage)
        }
        
        if (g_sniper[attacker])
        {
            new weapon = get_user_weapon(attacker)
            
            switch (weapon)
            {
                case CSW_AWP: damage *= 90.0
            }
            SetHamParamFloat(4, damage)
        } 


[matrix123]

new weapon = get_user_weapon(attacker)
if (g_l4d[attacker]) 
        {  
             
            switch (weapon) 
            { 
                case CSW_M4A1: damage *= 2.0 
                case CSW_P90: damage *= 2.0 
                case CSW_MAC10: damage *= 2.0 
                case CSW_SG552: damage *= 2.0 
            }  
        } 
         
        if (g_ninja[attacker]) 
        {  
             
            switch (weapon) 
            { 
                case CSW_KNIFE: damage *= 25.0 
            } 
           
        } 
         
        if (g_sniper[attacker]) 
        { 
             
            switch (weapon) 
            { 
                case CSW_AWP: damage *= 90.0 
            } 
        }  
        SetHamParamFloat(4, damage)

toy en la net sino te ayudaria mas xD

[dalegato]

Originally Posted by matrix123 new weapon = get_user_weapon(attacker) if (g_l4d[attacker])         {                            switch (weapon)             {                 case CSW_M4A1: damage *= 2.0                 case CSW_P90: damage *= 2.0                 case CSW_MAC10: damage *= 2.0                 case CSW_SG552: damage *= 2.0             }          }                   if (g_ninja[attacker])         {                            switch (weapon)             {                 case CSW_KNIFE: damage *= 25.0             }                     }                   if (g_sniper[attacker])         {                           switch (weapon)             {                 case CSW_AWP: damage *= 90.0             }         }          SetHamParamFloat(4, damage) toy en la net sino te ayudaria mas xD

No cambia nada por lo que veo ya que solo sacaste

PHP Code:

new weapon = get_user_weapon(attacker) 


y lo pusiste arriba de g_l4d xD

[rak]

PHP Code:

// global
new const Float:Damage_Mods[] = { 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 2.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 
1.0, 1.0, 1.0, 1.0, 90.0, 1.0, 1.0, 1.0, 2.0, 1.0, 1.0, 1.0, 1.0, 2.0, 1.0, 25.0, 2.0 }

// takedamage
static weapon
weapon = get_user_weapon(attacker) //EDIT3: NO USAR ''NEW'' EN TAKEDAMAGE!
if (g_l4d[attacker] || g_ninja[attacker] || g_sniper[attacker]) 
{  
    damage *= Float:Damage_Mods[weapon]
}  
SetHamParamFloat(4, damage) 


como odio q usen muchas variables...
creo q funcionaria..

EDIT: nose si es mejor usar una constante enves de poner 1.0 todo el tiempo... por las dudas te lo dejo

PHP Code:

// global
const Float: Default = 1.0

new const Float:Damage_Mods[] = { Default, Default, Default, Default, Default, Default, 2.0, Default, 
Default, Default, Default, Default, Default, Default, Default, Default, Default, 90.0, Default, 
Default, Default, Default, Default, Default, Default, Default, 2.0, Default, 25.0, 2.0 } // me había faltado un default xD 


EDIT2: I WIN e.e

EDIT4: estoy al pedo por eso hago tantos edit y sigo agregando cosas para algo tan simple e.e cualquier cosa mira esto..
amxconst.inc

/* Id of weapons in CS */
#define CSW_P228		1
#define CSW_SCOUT		3
#define CSW_HEGRENADE		4
#define CSW_XM1014		5
#define CSW_C4			6
#define CSW_MAC10		7
#define CSW_AUG			8
#define CSW_SMOKEGRENADE	9
#define CSW_ELITE		10
#define CSW_FIVESEVEN		11
#define CSW_UMP45		12
#define CSW_SG550		13
#define CSW_GALI		14
#define CSW_GALIL		14
#define CSW_FAMAS		15
#define CSW_USP			16
#define CSW_GLOCK18		17
#define CSW_AWP			18
#define CSW_MP5NAVY		19
#define CSW_M249		20
#define CSW_M3			21
#define CSW_M4A1		22
#define CSW_TMP			23
#define CSW_G3SG1		24
#define CSW_FLASHBANG		25
#define CSW_DEAGLE		26
#define CSW_SG552		27
#define CSW_AK47		28
#define CSW_KNIFE		29
#define CSW_P90			30
#define CSW_VEST		31
#define CSW_VESTHELM	32

[Kiske]

Si lo estás haciendo adentro del ZP usa la variable g_currentweapon[index]

[dalegato]

Originally Posted by rak PHP Code: // global new const Float:Damage_Mods[] = { 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 2.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0,  1.0, 1.0, 1.0, 1.0, 90.0, 1.0, 1.0, 1.0, 2.0, 1.0, 1.0, 1.0, 1.0, 2.0, 1.0, 25.0, 2.0 } // takedamage static weapon = get_user_weapon(attacker) //EDIT3: NO USAR ''NEW'' EN TAKEDAMAGE! if (g_l4d[attacker] || g_ninja[attacker] || g_sniper[attacker])  {       damage *= Float:Damage_Mods[weapon] }   SetHamParamFloat(4, damage)  como odio q usen muchas variables... creo q funcionaria.. EDIT: nose si es mejor usar una constante enves de poner 1.0 todo el tiempo... por las dudas te lo dejo PHP Code: // global const Float: Default = 1.0 new const Float:Damage_Mods[] = { Default, Default, Default, Default, Default, 2.0, Default,  Default, Default, Default, Default, Default, Default, Default, Default, Default, 90.0, Default,  Default, Default, Default, Default, Default, Default, Default, 2.0, Default, 25.0, 2.0 }

y ahora te hago una pregunta, como se que en el modo ninja me va a tomar el dmg: 25.0, y no 2.0 o 9.0??

[rak]

si g_ninja[attacker] es verdadero y ataca con el cuchillo le va a tomar el damage 25.0... lo que tiene es q si g_sniper[attacker] ataca con el cuchi tambien le va a tomar 25.0 pero si ataca con el awp le toma 90.0.. mientras que g_ninja[attacker] o g_sniper[attacker] o g_l4d[attacker].. no tengan ningun arma en comun.. nunca toman el valor del otro

[dalegato]

Originally Posted by rak si g_ninja[attacker] es verdadero y ataca con el cuchillo le va a tomar el damage 25.0... lo que tiene es q si g_sniper[attacker] ataca con el cuchi tambien le va a tomar 25.0 pero si ataca con el awp le toma 90.0.. mientras que g_ninja[attacker] o g_sniper[attacker] o g_l4d[attacker].. no tengan ningun arma en comun.. nunca toman el valor del otro

Entonces donde seteo el modo sniper, le voy a tener que sacar el cuchi. o si no me toma el damage
de ninja y eso no quiero.

Y porque no poner new en takedamage?

[rak]

Originally Posted by dalegato Entonces donde seteo el modo sniper, le voy a tener que sacar el cuchi. o si no me toma el damage de ninja y eso no quiero. Y porque no poner new en takedamage?

1. claro
2. takedamage se llama muchas veces.. estas creando una variable todo el tiempo y aumentas( tampoco es tanto xD ) el consumo.. es preferible usar un static dado que cuando se llama takedamage ya existe weapon y no la vuelve a crear por que se almacena en la memoria.. y ahi modifico algo por que no te va a compilar xD

Edit: listo, igual fijate lo que dijo kiske.. no hay nececidad de crear weapon.. existe g_currentweapon[attacker]

[dalegato]

Originally Posted by rak 1. claro 2. takedamage se llama muchas veces.. estas creando una variable todo el tiempo y aumentas( tampoco es tanto xD ) el consumo.. es preferible usar un static dado que cuando se llama takedamage ya existe weapon y no la vuelve a crear por que se almacena en la memoria.. y ahi modifico algo por que no te va a compilar xD Edit: listo, igual fijate lo que dijo kiske.. no hay nececidad de crear weapon.. existe g_currentweapon[attacker]

A bueno gracias me sacaste una duda.