[0x00]

Let's say that I want to do a check just like the code above, how I am supposed to do so?

It simple doesnt compile and give me a shit load of errors, even if they're all in the same line.

PHP Code:

if(!StrEqual(temp, "tf_weapon_bat", false) || !StrEqual(temp, "tf_weapon_bat_wood", false) || 
!StrEqual(temp, "tf_weapon_bat_fish", false) || !StrEqual(temp, "tf_weapon_shovel", false) || 
!StrEqual(temp, "tf_weapon_fireaxe", false) || !StrEqual(temp, "tf_weapon_bottle", false) || 
!StrEqual(temp, "tf_weapon_sword", false) || !StrEqual(temp, "tf_weapon_fists", false) || 
!StrEqual(temp, "tf_weapon_lunchbox", false) || !StrEqual(temp, "tf_weapon_wrench", false) || 
!StrEqual(temp, "tf_weapon_bonesaw", false) || !StrEqual(temp, "tf_weapon_club", false) || 
!StrEqual(temp, "tf_weapon_knife", false) || !StrEqual(temp, "tf_weapon_invis", false) || 
!StrEqual(temp, "tf_weapon_robot_arm", false) || !StrEqual(temp, "tf_weapon_stickbomb" false))
{
        // something here
} 


[databomb]

StrContains(temp, "tf_weapon") ?

[0x00]

Nope, if you havent noticed there's just melee weapons on my list.. any kind of weapon uses tf_weapon~ meaning it won't filter anything.

[databomb]

Long live TFC!

In that case 0x00, have you tried checking if the active weapon slot position is the melee position?

Also, that last StrEqual is missing a comma!! And if you do go with StrEqual approach then I should suggest truncating the first part "tf_weapon_" before you do all the checks to save CPU cycles.

[Monkeys]

As suggested, use the weaponslot.
If that isn't equal for all of them, go with a StrEqual from position temp[9]. (If there are only a few exceptions, like all melee + 2 secondary, you can just check slot || the two names, ofcourse)

[AtomicStryker]

Besides, there has to be a smarter way to determine if a weapon is a melee weapon. I'd say check the ammo, or something else in the netprops.

[FoxMulder]

Originally Posted by AtomicStryker Besides, there has to be a smarter way to determine if a weapon is a melee weapon. I'd say check the ammo, or something else in the netprops.

I thought of this but then The Sandman came into mind

I think checking to see if the active weapon is in the 0 slot would suffice.

PHP Code:

new iWeapon = GetEntPropEnt(client, Prop_Send, "m_hActiveWeapon");
if(GetPlayerWeaponSlot(client, 0) == iWeapon)
{
   //looks like player is most likely holding a melee weapon
} 


[Monkeys]

I decided to look through your list, and isn't lunchbox NOT a melee item?

[Fyren]

Originally Posted by 0x00 It simple doesnt compile and give me a shit load of errors, even if they're all in the same line.

I suggest fixing the error that's on the line given to you by the compiler.

[Cookies.net]

the last strequal is missing a ,

PHP Code:

!StrEqual(temp, "tf_weapon_stickbomb" false) 


should be

PHP Code:

!StrEqual(temp, "tf_weapon_stickbomb", false) 


no compiler errors then here