__oFFmaN :3
Умник
Здрасте :3
Я в разочеровании и недаумении
Постоянно вижу бредо коды типо:
Ну не глупо ли это? зачем столько столько лишних строк?
Не проще ли использовать "#include <dutils>" и немножко больше переменных ?
Код станет красивее и более оптимизированым :df:
К примеру рассмотрим вариант высше
Команду можно уменьшить в трое(добавив ещо 2 перменные) :3
Смотрите разницу:
Сейчас вы скажете "это увеличит нагрузку на мод" Да,не спорю)))
Но а ты лучше все свои new string[100500]; для
Укороти!
Критикуйте,Коментируйте,Мне Интересно ваше мнение(может я не прав?) хмм...
---------- Post added at 22:20 ---------- Previous post was at 20:34 ----------
Нуже,Люди Выскажите свое мнение :3
Обматерите меня,или назовите няшкой $)
Я в разочеровании и недаумении
Постоянно вижу бредо коды типо:
PHP:
new Float:x, Float:y, Float:z;
new cmd[256];
new tmp[256], idx;
cmd = strtok(cmdtext, idx);
PHP:
if(strcmp("/ts", cmd, true, 10) == 0)
{
tmp = strtok(cmdtext, idx);
if(!strlen(tmp))
{
SendClientMessage(playerid, -1, "{FF0000}Используйте: {FFFFFF}/ts [ID авто] [Цвет 1] [Цвет 2]");
return 1;
}
new auto = strval(tmp);
tmp = strtok(cmdtext, idx);
if(!strlen(tmp))
{
SendClientMessage(playerid, -1, "{FF0000}Используйте: {FFFFFF}/ts [ID авто] [Цвет 1] [Цвет 2]");
return 1;
}
new col1 = strval(tmp);
tmp = strtok(cmdtext, idx);
if(!strlen(tmp))
{
SendClientMessage(playerid, -1, "{FF0000}Используйте: {FFFFFF}/ts [ID авто] [Цвет 1] [Цвет 2]");
return 1;
}
new col2 = strval(tmp);
if(auto > 611 || auto < 400)
{
SendClientMessage(playerid, -1, "{FF0000}[Oшибка]:{FFFFFF} ID машин находятся в интервале: {00FFFF}400 - 611");
return 1;
}
GetPlayerPos(playerid, Float:x, Float:y, Float:z);
CreateVehicle(auto, Float:x-4, Float:y, Float:z+2, 60, col1, col2, -1);
return 1;
}
Не проще ли использовать "#include <dutils>" и немножко больше переменных ?
Код станет красивее и более оптимизированым :df:
К примеру рассмотрим вариант высше
Команду можно уменьшить в трое(добавив ещо 2 перменные) :3
PHP:
new cmd[256];
new tmp1[256], tmp2[256], tmp3[256], idx;
cmd = strtok(cmdtext, idx);
tmp1 = strtok(cmdtext, idx);
tmp2 = strtok(cmdtext, idx);
tmp3 = strtok(cmdtext, idx);
PHP:
if(strcmp("/ts", cmd, true, 10) == 0)
{
new Float:x, Float:y, Float:z;
if(!strlen(tmp1) || !strlen(tmp2) || !strlen(tmp3)) return SendClientMessage(playerid, -1, "{FF0000}Используйте: {FFFFFF}/ts [ID авто] [Цвет 1] [Цвет 2]");
if(strval(tmp1) > 611 || strval(tmp1) < 400) return SendClientMessage(playerid, -1, "{FF0000}[Oшибка]:{FFFFFF} ID машин находятся в интервале: {00FFFF}400 - 611");
GetPlayerPos(playerid,x,y,z);
return CreateVehicle(strval(tmp1),x-4,y,z+2,60,strval(tmp2),strval(tmp3),0);
}
Но а ты лучше все свои new string[100500]; для
PHP:
format(string,sizeof string,"Привет %s ",pName(playerid);
SendClientMessage(playerid,COLOR_LIME,string);
Критикуйте,Коментируйте,Мне Интересно ваше мнение(может я не прав?) хмм...
P.S. Лично я думаю что 2 переменные лучше чем эти бредо коды,проще,и ты не будеш мучется при написании команды....
---------- Post added at 22:20 ---------- Previous post was at 20:34 ----------
Нуже,Люди Выскажите свое мнение :3
Обматерите меня,или назовите няшкой $)
Последнее редактирование:
