Можешь показать где, чтобы я пофиксил?
#define mapfor(%0,%1) \
for(new _mpidx = 0, %1; \
_mpidx < Map_Internal_LoopStep(%0, _mpidx) && ((%1 = Map_GetKeyByIndex(%0, _mpidx)) != -1); \
_mpidx++)
_mpidx захардкожен как фиксированное имя. Если написать два вложенных mapfor, оба объявят одну и ту же переменную, внешний цикл сломается молча без единой ошибки компилятора.
native bool:Map_Get(PawnMap:mapid, key, const AnyTag:data_array[], size = sizeof(data_array));
const на выходном буфере семантически неверен. Функция пишет данные в массив, а const говорит компилятору что массив не тронут.
Map_Set сделан правильно, там данные только читаются и const уместен. Здесь его можно было бы убрать.
native Map_FindKeyByField(PawnMap:mapid, AnyTag:offset, PawnMapType:type, {Float, _}:...);
native bool:Map_SortByField(PawnMap:mapid, offset, PawnMapSortOrder:order, PawnMapType:typeid = MAP_TYPE_INT);
В
FindKeyByField offset принимает
AnyTag, в
SortByField тот же параметр просто
int без тега. Одна и та же концепция, разное оформление в рамках одного API, это непоследовательно и сбивает при чтении инклюда.
--
Для павн экосистемы у тебя выше среднего однозначно. Большинство плагинов в этой среде пишутся без нормального API-дизайна вообще, здесь же видно что ты думал об удобстве использования, теги везде расставлены,
INVALID_MAP_ID и
INVALID_MAP_KEY_ID как константы а не голые -1, ALS хук на OnGameModeInit сделан правильно, проверка версий между бинарником и инклюдом,это уже достаточно хорошая практика, которую мало кто делает.
Замечания которые я указал это скорее точечные вещи но не системные проблемы. const на выходном буфере и непоследовательность тегов в offset, правки на пять минут, не переработка архитектуры. Проблема с вложенным mapfor реальная, но проявится только в специфичном сценарии.
Определённый плюс.
