Оптимизации Pawn Кода

[Дим]

Оптимизации Pawn Кода
Автор: eFFect aka YoYo.
Дополнил: zindo

Для начала, немного советов.

Совет #1:
Используйте в скрипте/моде как можно меньше таймеров. Каждый таймер способствует постепенному нарастанию грыжи у сервера.

Совет #2:
Функции которые не используются в таймерах, помещайте в stock а не в public

Совет #3:
При создании массивов типа:

Code: Select all

new string[256];

Подумайте заранее, какая самая большая строка может использоваться в массиве, и максимально уменьшите число в квадратных скобках.
Макс. значение для SendClientMessage - 144.

Совет #4:
Соблюдайте табуляцию/выравнивание кода (лесенка) и не используйте функций типа tabsize.

Совет #5:
В некоторых случаях лучше использовать функцию напрямую, чем заводить для неё переменную.
Пример:

Code: Select all

new var1; 
var1 = GetPlayerSkin(playerid);
if(var1 == 0) return SendClientMessage(playerid, 0xFF0000FF, "У вас скин CJ");

Лучше сделать так:

Code: Select all

if(GetPlayerSkin(playerid) == 0) return SendClientMessage(playerid, 0xFF0000FF, "У вас скин CJ");

Совет #6:
Если в переменной может быть значение только 0 или 1, используйте тип переменной bool.
Пример:

Code: Select all

PlayerInfo[playerid][pDriving];
/* Права на машину, они либо есть, либо их нет. 
Поэтому pDriving в enum'е лучше пометить типом bool:pDriving */
// С обычными переменными аналогично    

Совет #7:
Старайтесь меньше использовать такие константы как MAX_PLAYERS, MAX_VEHICLES и т.п.
Лучше сделать так:

Code: Select all

#undef MAX_PLAYERS
#define MAX_PLAYERS 20    

где 20 - новое значение константы

Совет #8:
Старайтесь использовать стандартные функции, в большинстве случаев они быстрее чем их аналоги написанные сторонними скриптерами.
Яркий пример:

Code: Select all

PlayerToPoint и IsPlayerRangeOfPoint

Совет #9:
В некоторых случаях лучше использовать switch нежели if/else
Пример:

Code: Select all

if(var1 == 1) print("1");
else if(var1 == 2) print("2");
else if(var1 == 3) print("3");
else print("4"); 

Лучше записать так:

Code: Select all

switch(var1)
{
    case 1: print("1");
    case 2: print("2");
    case 3: print("3");
    default: print("4");
} 

Совет #10:
Если возможно, лучше использовать имя клавиши, нежели её код.
Пример: Клавиша Alt: KEY_WALK/1024 Лучше записать KEY_WALK.

Совет #11:
В большинстве случаев, использование паблика OnPlayerUpdate не оправдывает само себя, поэтому лучше с ним не злоупотреблять.

Совет #12:
Никогда не используйте цикл в цикле.
Пример:

Code: Select all

for(new i = 0; i < MAX_PLAYERS; i++)
{
    for(new z = 0; z < MAX_PLAYERS; z++) // Этот цикл выполнится 500 по 500 (т.е. 500 раз по 500)
    {
    }
} 

Совет #13:
Если вы часто используете один и тот же код, лучше создать для него функцию.

Пользуясь этими советами, вы добьётесь большей эффективности скриптинга. Как в быстродействии, так и в визуальном восприятии кода.

А теперь пара приёмов "оптимизированного" кода.

1. Структура switch:

Code: Select all

switch(var1)
{
    case 0: var1 = 1;
    case 1: var1 = 1;
    case 2: var1 = 1;
    case 3: var1 = 2;
    case 4: var1 = 2;
    ...
}

Лучше записать так:

Code: Select all

switch(var1)
{
    case 0..2: var1 = 1;
    case 3,4: var1 = 2;
}

2. Часто в функциях/командах используется:

Code: Select all

SendClientMessage(playerid, 0xFFFFFFFF, "Текст");
return 1;

"Функцию" return, можно использовать в составе с любой функцией которая возвращает 1 или 0 (истина или ложь).
В нашем примере можно сделать так:

Code: Select all

return SendClientMessage(playerid, 0xFFFFFFFF, "Текст");

2.1. Если функция возвращает true/else по условию:

Code: Select all

if(!strcmp("text", "text", true)) return 1;
else return 0;

Можно записать так:

Code: Select all

!strcmp("text", "text", true)?true:false;

3. Использование псевдодинамических массивов для оптимизации циклов типа:

Code: Select all

for(new i; i<MAX_PLAYERS; i++) { ... }

4. Использование массива вместо функции GetPlayerName (либо ее аналога, GetPlayerNameEx) для узнавания никнейма игрока.

Code: Select all

new PlayerName[MAX_PLAYERS][MAX_PLAYER_NAME];

Далее в OnPlayerConnect'e записываем никнейм игрока в массив:

Code: Select all

GetPlayerName(playerid, PlayerName[playerid], MAX_PLAYER_NAME);

При каждом изменении ника по сценарию мода также добавляем эту строку (например в команду /setname)
При выходе игрока нужно очистить его ячейку в массиве:

Code: Select all

PlayerName[playerid][0] = 0;

Пример использование:

Code: Select all

format(str, 128, "Ник игрока под идом 5 '%s'", PlayerName[5]);

Для этого создаем глобальный двумерный массив: