[YamiKaitou]

alan_el_more, No. He is correct.

[alan_el_more]

Originally Posted by YamiKaitou alan_el_more, No. He is correct.

why?
i read this tut: https://forums.alliedmods.net/showthread.php?t=91207

[Arkshine]

Actually you can do it but it should be : SetHamParamFloat(4, ( damage *= dmg ) )

but there is no need to update 'damage' since the new value will be passed using SetHamParamFloat. So, he's right to use directly damage* dmg.

[YamiKaitou]

Because the *= operation is for assigning a value. Using the * operator is enough. Plus, I believe your method would pass true (or 1) to the function and not the actual value

[joaquimandrade]

Originally Posted by alan_el_more why? i read this tut: https://forums.alliedmods.net/showthread.php?t=91207

Both work.

The difference is that yours make a redundant operation. It saves the value of the final damage in the variable damage

[AlejandroSk]

Can someone help me whit this?

L 05/04/2009 - 18:54:16: Invalid player id 0
L 05/04/2009 - 18:54:16: [AMXX] Displaying debug trace (plugin "zp_extra_goldenak.amxx")
L 05/04/2009 - 18:54:16: [AMXX] Run time error 10: native error (native "get_user_weapon")
L 05/04/2009 - 18:54:16: [AMXX]    [0] zp_extra_goldena.sma::fw_TakeDamage (line

[Arkshine]

The reason is the killer can be world ( 0 as id, eg: suicide ) or an entity > max_clients ; and you don't check it.

But, instead of get_user_weapon, you should read_data(1). ( like you can see here)

[IneedHelp]

Originally Posted by AlejandroSk Can someone help me whit this? L 05/04/2009 - 18:54:16: Invalid player id 0 L 05/04/2009 - 18:54:16: [AMXX] Displaying debug trace (plugin "zp_extra_goldenak.amxx") L 05/04/2009 - 18:54:16: [AMXX] Run time error 10: native error (native "get_user_weapon") L 05/04/2009 - 18:54:16: [AMXX] [0] zp_extra_goldena.sma::fw_TakeDamage (line

Check if the attacker is a player.

EDIT:
Oh, arkshine, you answered first.

[AlejandroSk]

How can i check if the attacker is a player?

[alan_el_more]

PHP Code:

#define is_valid_player(%1) (1 <= %1 <= 32)

if(is_valid_player[attacker])