- Укорачивать и оптимизировать код
- Не использовать лишнее (например, зачем создавать переменную, если она может не понадобиться)
- Не использовать часто одинаковый код (создавайте функции)
- Не злоупотреблять такими циклами, как OnGameFrame, OnPlayerRunCmd (даже избегать их)
- Не создавать "утечку памяти" (закрывать все ненужные Handle и т.д.)
- Понимать порядок выполнения команд и событий
- Делать нужные проверки (например, перед выдачей оружия игроку: он точно на сервере? жив? и т.д.)
- Не делать ненужные/лишние/повторные проверки в условиях
- Указывать массивам правильный размер (не создавать много лишних ячеек)
- И др.. (скажите, если я что-то упустил)

// Так делать глупо:
if (GetClientTeam(client) == 2) ...
else if (GetClientTeam(client) == 3) ...

// Зачем повторно получать команду игрока, если мы уже сделали это?
// Сохраните её в переменной и уже с ней работайте:
new team = GetClientTeam(client);
if (team == 2) ...
else if (team == 3) ...

// Если вам нужно сделать только одно сравнение, то создавать переменную нет смысла:
if (GetClientTeam(client) == 2) ...

// Готовим переменную для хранения строки
decl String:name[45];

// Вот так можно обезопасить строку.
// Это полезно, если хотим работать с переменной, но не уверены, есть ли в ней значение.
name[0] = '\0';   

// Команда для примера:
GetEntPropString(index, Prop_Data, "m_iName", name, 45);

// И ниже уже можно работать с этой переменной, не беспокоясь
if (name[0] == '\0') .. // строка пустая

new x = 5;

// так плохо:

if (x == 2)
{
     // код
}
if (x == 5)
{
     // код
}

// так хорошо:

if (x == 2)
{
     // код
}
else if (x == 5)
{
     // код
}
else if (x == 7)
{
     // код
}
else
{
     // код
}