Как упростить условие?

Есть функция
Код:

public fw_TakeDamage(victim, inflictor, attacker, Float:damage)
{
	if ( is_valid_player( attacker ) && get_user_weapon(attacker) == CSW_AK47 && g_HasAk[attacker] )
	{
		SetHamParamFloat(4, damage + get_pcvar_float( cvar_dmgmultiplier ) )		
	}
	else if ( is_valid_player( attacker ) && get_user_weapon(attacker) == CSW_M4A1 && g_HasAk[attacker] )
	{
		SetHamParamFloat(4, damage + get_pcvar_float( cvar_dmgmultiplier ) )
	}
	else if ( is_valid_player( attacker ) && get_user_weapon(attacker) == CSW_MP5NAVY && g_HasAk[attacker] )
	{
		SetHamParamFloat(4, damage + get_pcvar_float( cvar_dmgmultiplier ) )
	}
	
	return HAM_HANDLED
}

пробовал вместо этого писать ==>>
Код:

public fw_TakeDamage(victim, inflictor, attacker, Float:damage)
{
	if (is_valid_player( attacker ) && get_user_weapon(attacker) == CSW_AK47, CSW_M4A1, CSW_MP5NAVY && g_HasAk[attacker])
	{
		SetHamParamFloat(4, damage + get_pcvar_float( cvar_dmgmultiplier ) )		
	}
	
	return HAM_HANDLED
}

она работает, но иногда в логах ошибки на 3 строку, что не так?
Цикл не надо!

Как загнать в одну переменную CSW_AK47, CSW_M4A1, CSW_MP5NAVY и проверить не используя цикл?

можно общие проверки вынести в начало
если уж очень хочется проверить за один раз оружие, то можно через case или через битсумму

mazdan,
Я так понял вот так?
Код:

if(is_valid_player( attacker ) && g_HasAk[attacker])
	{
		new weapon = get_user_weapon(attacker) 
		if(weapon == CSW_AK47 || weapon == CSW_M4A1 || weapon == CSW_MP5NAVY)
		{
			SetHamParamFloat(4, damage + get_pcvar_float( cvar_dmgmultiplier ) ) 
		} 
	}

Почему вот здесь работает, и не выдает ошибок ?
Код:

new plrClip, plrAmmo
new plrWeapId = get_user_weapon(id, plrClip , plrAmmo)	
	
if (plrWeapId == CSW_AK47, CSW_M4A1, CSW_MP5NAVY && g_HasAk[id])

На сколько правильно использовать Код:

if (peremennay == CSW_AK47, CSW_M4A1, CSW_MP5NAVY)

и допускается ли это вообще?

navigator19, mazdan тебе правильно посоветовал использовать битсуммы, только вот жалко, что он не показал как реализовать это тебе

Код:

new const SHOT_BITSUM = (1<<CSW_AK47)|(1<<CSW_M4A1)|(1<<CSW_MP5NAVY)

new plrClip, plrAmmo
new plrWeapId = get_user_weapon(id, plrClip , plrAmmo) 

if (((1<<plrWeapId) & SHOT_BITSUM) && g_HasAk[id])

Algalon, А константу можно глобальную объявить, если проверка будет использоваться не только в одной функции?

navigator19, кончено можно, можно даже без константы, это я для наглядного примера показал

Algalon, я не уверен что битсумма тут будет быстрее чем сравнения т.к. тут дополнительно операции << и & вместо 1-3 условий. Нужно тестировать.
+ если делать с битсуммами, то тогда нужно экономить и на записи в переменные, иначе это как-то бессмысленно (у меня в примере переменная для дела)
А в варианте лишняя переменная plrWeapId + проверка массива выполняется в самом конце, хотя логичнее дешевую проверку сделать в начале.
т.е. результат будет теоретически быстрееКод:

 if(is_valid_player( attacker ) && g_HasAk[attacker] && (1<<get_user_weapon(id, plrClip , plrAmmo) ) & SHOT_BITSUM)  )

mazdan, Algalon, Так а все таки, такая запись
Код:

if (peremennay == CSW_AK47, CSW_M4A1, CSW_MP5NAVY)

в корне не правильная?
Ведь она иногда работает

У меня аналогичный вопрос
Код:

get_ucp_from_team(from, alive) // gets alive user with ucp
{
	new UcpUsers[33], c;
	
	for( new id = 1; id < 33; id ++ )
	{
		if( is_user_alive(id) == alive && get_user_team(id) == from && gbUCP[id])
		{
			UcpUsers[c++] = id;
		}
	}
	
	if©
	{
		return UcpUsers[ random© ];
	}
	
	return 0;
}

Как добавить в рандом стим игроков,

Код:

if( is_user_alive(id) == alive && get_user_team(id) == from && gbUCP[id] || is_user_steam(id))

такой код вроде баганый, довольно часто срабатывает выдача плента двум игрокам

Так тут видимо сама выдача больше 1 раза происходит, либо хз что. Так-то выходит, что больше одного id эта функция не вернет в любом случае.

А вообще так лучше сделать:
Код:

if( ( is_user_alive(id) == alive ) && ( get_user_team(id) == from ) && ( gbUCP[id] || is_user_steam(id) ) )

Просто на всякий случай)

AndrewZ,
Выдача такого вида
Код:

public logevent_SpawnedBomb()
{
	new bombCarrier = get_loguser_index();
	
	if( !gbUCP[bombCarrier] || !is_user_steam(bombCarrier) )
	{
		new UcpPlayer = get_ucp_from_team(1, 1); // maybe players here are not alive
		//new SteamPlayer = get_steam_from_team(1, 1);
		
		if(UcpPlayer)
		{
			fm_transfer_user_gun(bombCarrier, UcpPlayer, CSW_C4, "weapon_c4")
		}
	}
}

ну и напоследок, может что добавить?
Код:

stock get_loguser_index()
{
    new loguser[80], name[32]
    read_logargv(0, loguser, 79)
    parse_loguser(loguser, name, 31)
 
    return get_user_index(name)
}

Ну выходит, что logevent_SpawnedBomb() срабатывает несколько раз.. Не иначе. Либо как-то криво работает fm_transfer_user_gun и не удаляет бомбу у старого игрока.
Если первый вариант - то можно попробовать костыль, что-то вроде глобальной булевой переменной, которая будет true после первого срабатывания, и false по ивенту HLTV, например.

До моего кривого добавления проверки на стим, иногда плент оказывался не у того игрока,толи не забирало,толи передавало не тому,довольно редко