13.05.2013, 08:59
Чтение файла внутри main() плохая идея, для этого существует OnFilterScriptInit.
не стоит так делать, конечный "потребитель" не должен этим заморачиваться, этим должен заморачиваться ваш скрипт.
а зачем index передавать по ссылке если он используется только для проверки?
null - очень, просто ужас какое плохое имя для переменной!
И внутри функция можно было бы использовать не i, а смело использовать playerid, код станет более читаемый.
Оформление кода почему-то хромает от одного вида к другому. Как и типы функций.
Часть кода можно упростить, избавиться от ненужных конструкций, например:
Quote:
Originally Posted by Seregamil1
Если вы допустили кол-во координат, которое не делится на 4 - зоны попросту не создадутся.
|
pawn Code:
search(i,&index)
null - очень, просто ужас какое плохое имя для переменной!
И внутри функция можно было бы использовать не i, а смело использовать playerid, код станет более читаемый.
Оформление кода почему-то хромает от одного вида к другому. Как и типы функций.
Часть кода можно упростить, избавиться от ненужных конструкций, например:
pawn Code:
search(playerid, index) {
for (new i; i != index; i++) {
if (InTheGangZone(playerid, pos[i][0], pos[i][1], pos[i][2], pos[i][3])) {
save(playerid);
escape(playerid);
return 1;
}
}
return 0;
}