Вид для печати
Привет. Делаю систему по покупке оружия не используя куча диалогов. Интересны пожелания по упрощению/улучшению кода. Сама система на начальном этапе разработки. Код не маленький, так что через github...
https://github.com/m1n1vv/Pawn/blob/...eapons_0.2.inc
P.S. Пожалуйста без ассемблера
Отделяй блоки кода друг от друга, а то читать не приятно :)
Открыть/закрыть
Ну тут на вкус и цвет товарищей нет. Но на время сделаю.Цитата:
Сообщение от Anton Styazhkin
Отделяй блоки кода друг от друга, а то читать не приятно :)
Провёл беглый аудит кода, вот что удалось найти:
- Массив weapons можно разбить на два массива... Скажем, на player_weapons и player_ammo. Во-первых, доступ к одномерному массиву быстрее, чем к двухмерному. Во-вторых, в массиве player_weapons будут храниться только числа меньше 256, а значит можно упаковать его и сэкономить память. Двойной профит.
- Упаковать строки в BuyWeapons.
- Что заставило ваc хранить названия оружий в массиве, но использовать switch в mn_PlayerWeapon?
- В BuyWeapons можно кэшировать "weaponid-22" вместо того, чтобы 3 раза подряд вычислять одно и то же.
PHP код:mn_weaponid{playerid} = weaponid;
weaponid -= 22;
// ...
format(string, sizeof string, str,
mn_buystat_weap_name[weaponid],
mn_buystat_caliber[weaponid],
// ...
- Кстати об оружии, что это за "22", "25", "33", "34", разбросанные по всему скрипту? Под всё это есть константы:
PHP код:// Из a_samp.inc
#define WEAPON_COLT45 (22)
#define WEAPON_SILENCED (23)
#define WEAPON_DEAGLE (24)
#define WEAPON_SHOTGUN (25)
#define WEAPON_SAWEDOFF (26)
#define WEAPON_SHOTGSPA (27)
#define WEAPON_UZI (28)
#define WEAPON_MP5 (29)
#define WEAPON_AK47 (30)
#define WEAPON_M4 (31)
#define WEAPON_TEC9 (32)
#define WEAPON_RIFLE (33)
#define WEAPON_SNIPER (34)
#define WEAPON_ROCKETLAUNCHER (35)
- Подсчёт размера строки в BuyWeapons оставляет желать лучшего. И да, там в формуле ошибка.
Чтобы наглядно продемонстрировать недостаток вашего метода подсчёта, я не стану говорить, где эта ошибка (или даже ошибки?) - пусть этот пункт аудита будет таким же информативным, как ваша формула.
Если будет время, загляните сюда: http://pro-pawn.ru/showthread.php?13388
А зачем берутся цифры в скобки если можно и без них или это очередной элемент удобства?Цитата:
PHP код:#define WEAPON_ROCKETLAUNCHER (35)
static
mn_player_weapons[7],
mn_player_ammo[7];
PHP код:GetPlayerWeaponData(playerid, i, mn_player_weapons[i], mn_player_ammo[i]);
- Не до конца понял
- Осталось от первой версии https://github.com/m1n1vv/Pawn/blob/master/buy_weapons :rofl:
- Сделал
- Подумаю над этим
- Он был точный, но оформлен не красиво. Сейчас более-менее сделал:
PHP код:sizeof fmt_str+(15+18+2+12+5)-(2*5+14+5)
Ок, половина работы сделана. Теперь прочитайте ещё раз 1-й пункт в моём предыдущем посте (на этот раз до конца).Цитата:
Сообщение от m1n1vv
http://pro-pawn.ru/showthread.php?13962Цитата:
Сообщение от m1n1vv
Не угадали. Предложение прочитать ту статью про подсчёт всё ещё в силе.Цитата:
Сообщение от m1n1vv
Вместо вот этого:
я бы использовал "GetWeaponName".Код:mn_buystat_weap_name[][] =
{
"9mm",
"Silenced 9mm",
"Desert Eagle",
"Shotgun",
"Sawnoff Shotgun",
"Combat Shotgun",
"Micro SMG/Uzi",
"MP5",
"AK-47",
"M4",
"Tec-9",
"Country Rif le",
"Sniper Rif le"
},
А так всё очень даже неплохо :)
И да, стиль написания конечно, вроде
читать мягко говоря сложновато. Хотя тут, как уже говорилось, кому как удобнее.Код:if (0 == response)
Если ты про mn_player_weapons[7] и mn_player_ammo[7] с char, то CrashDetect ругается ну и само действие не выполняется.Цитата:
Сообщение от Daniel_Cortez
Там же formatЦитата:
Сообщение от Daniel_Cortez
- - - Добавлено - - -
Но там же надо обращаться к playerid или killeridЦитата:
Сообщение от Nexius_Tailer
В смысле? Там нет ничего подобного: http://wiki.sa-mp.com/wiki/GetWeaponNameЦитата:
Сообщение от m1n1vv
НевнимательныйЦитата:
Сообщение от ziggi
Что лучше?
илиPHP код:static stock Switch_library(playerid)
{
switch(mn_weaponid{playerid})
{
case WEAPON_COLT45, WEAPON_SILENCED: mn_ammunition{playerid} = 17;
case WEAPON_DEAGLE, WEAPON_SHOTGSPA: mn_ammunition{playerid} = 7;
case WEAPON_SHOTGUN, WEAPON_RIFLE, WEAPON_SNIPER: mn_ammunition{playerid} = 1;
case WEAPON_SAWEDOFF: mn_ammunition{playerid} = 2;
case WEAPON_UZI, WEAPON_M4, WEAPON_TEC9: mn_ammunition{playerid} = 50;
case WEAPON_MP5, WEAPON_AK47: mn_ammunition{playerid} = 30;
}
switch(mn_weaponid{playerid})
{
case WEAPON_COLT45: mn_price[playerid] = 1720;
case WEAPON_SILENCED: mn_price[playerid] = 2135;
case WEAPON_DEAGLE: mn_price[playerid] = 2476;
case WEAPON_SHOTGUN: mn_price[playerid] = 1459;
case WEAPON_SAWEDOFF: mn_price[playerid] = 1928;
case WEAPON_SHOTGSPA: mn_price[playerid] = 2836;
case WEAPON_UZI: mn_price[playerid] = 1838;
case WEAPON_MP5: mn_price[playerid] = 2382;
case WEAPON_AK47: mn_price[playerid] = 3718;
case WEAPON_M4: mn_price[playerid] = 3967;
case WEAPON_TEC9: mn_price[playerid] = 2593;
case WEAPON_RIFLE: mn_price[playerid] = 4391;
case WEAPON_SNIPER: mn_price[playerid] = 5628;
}
switch(mn_weaponid{playerid})
{
case 22, 23, 24: mn_slot{playerid} = 2;
case 25, 26, 27: mn_slot{playerid} = 3;
case 28, 29, 32: mn_slot{playerid} = 4;
case 30, 31: mn_slot{playerid} = 5;
case 33, 34: mn_slot{playerid} = 6;
}
return 1;
}
PHP код:static stock GetWeaponInfo(playerid)
{
static
weaponid;
static const
slot[] =
{
2, 2, 2, 3, 3, 3, 4, 4, 5, 5, 4, 6, 6
},
ammunition[] =
{
17, 17, 7, 1, 2, 7, 50, 30, 30, 50, 50, 1, 1
},
price[] =
{
1720,
2135,
2476,
1459,
1928,
2836,
1838,
2382,
3718,
3967,
2593,
4391,
5628
};
weaponid = mn_weaponid{playerid}-22;
mn_slot{playerid} = slot[weaponid];
mn_ammunition{playerid} = ammunition[weaponid];
mn_price[playerid] = price[weaponid];
return 1;
}