int GetOppositeTeam(int iTeam)
{
if(iTeam == 2)
{
if(g_bDisplayEnable && g_bMode)
{
if(g_bRoundMode)
{
SetTeamScore(3, GetTeamScore(3)-1);
}
else SetTeamScore(3, GetTeamScore(3)+1);
}
return 0;
}
if(iTeam == 3)
{
if(g_bDisplayEnable && g_bMode)
{
if(g_bRoundMode)
{
SetTeamScore(2, GetTeamScore(2)-1);
}
else SetTeamScore(2, GetTeamScore(2)+1);
}
return 1;
}
return -1;
}
можно упростить до
int GetOppositeTeam(int iTeam)
{
if(iTeam > 1 && g_bDisplayEnable && g_bMode)
{
static int team;
team = 5 - iTeam;
if(g_bRoundMode) SetTeamScore(team, GetTeamScore(team)-1);
else SetTeamScore(team, GetTeamScore(team)+1);
}
return iTeam - 2;
}
Если кто-то гранату кинет и за секунду убьёт больше одного противник, то таймер
Timer_Check_Frags может не закончить раунд, если фрагов у одной из команд (
g_iFrags[]) станет больше, чем выставлено в лимите
g_iLimit и бой продолжится, потому что у тебя
== вместо
>= в проверке.
А вообще эту проверку нужно было использовать в событии
Event_PlayerDeath, а не в том таймере.
Функция
SetHUDColor хороша: совершенно нет никаких проверок на ошибки в параметрах.
Если введут любые символы отличные от цифр то
StringToInt тебе выдаст 0 (ноль). Если параметр окажется больше, чем 255, или меньше нуля, то у тебя скорее всего будет ошибка в логах, чем текст в панельке.
В функции
SetHUDPosition точно так же нет проверок валидности параметров. Параметры положения на экране могут иметь значения только -1 для расположения по центру и от 0 до 1.
И ещё ты в
CreateConVar не делаешь ограничение на величину задаваемого размера квара для булевых параметров и лимита убийств (
его можно спокойно поставить равным или меньше нуля).
