Тоже стоит понимать, что ради компактности ты жертвуешь процессорным временем :)
Тут действует принцип "
китайского кода": да, код более объёмный, но все вариации прописаны сразу, за счёт чего не происходит лишних обращений к переменным и прочих действий.
По-хорошему, здесь стоит только от лишней переменной избавится, как это сделал ты
PHP код:
case D_REG_SEX: {
if(!response) return Kick(playerid);
switch(listitem) {
case 0: {
pInfo[playerid][Sex] = 1;
pInfo[playerid][Skin] = Random_Skin_Male[random(sizeof(Random_Skin_Male)];
}
case 1: {
pInfo[playerid][Sex] = 2;
pInfo[playerid][Skin] = Random_Skin_Female[random(sizeof(Random_Skin_Female)];
}
}
CreateNewAccount(playerid);
return true;
}
В остальном код оптимален (нет повторных вызовов ни для listitem, ни для pInfo)