К UI Алекса нельзя даже близко подпускать.

Покажу это на примере его файла AY_menu_property.lua

Начнем с шапки, где он (как ему кажется) инициирует меню:

Код:

local AY_menu = {} local Funcs = {} local function init()    for _, menu in ipairs(Menus) do     if menu.name == "PropertyMenu" then       AY_menu = menu       menu.cleanup = Funcs.AY_cleanup       menu.buttonComm  = Funcs.AY_menu.buttonComm       menu.buttonBroadcast  = Funcs.AY_menu.buttonBroadcast       menu.buttonShowMap  = Funcs.AY_menu.buttonShowMap       menu.buttonDetails  = Funcs.AY_menu.buttonDetails       menu.buttonExtend  = Funcs.AY_menu.buttonExtend       menu.checkboxSelected  = Funcs.AY_menu.checkboxSelected       menu.hotkey  = Funcs.AY_menu.hotkey       menu.onShowMenu  = Funcs.AY_onShowMenu       menu.display  = Funcs.AY_menu.display       menu.onUpdate  = Funcs.AY_onUpdate       menu.onRowChanged  = Funcs.AY_onRowChanged       menu.onSelectElement  = Funcs.AY_onSelectElement       menu.onCloseElement  = Funcs.AY_onCloseElement       menu.addContainerRow  = Funcs.AY_menu.addContainerRow       menu.createSection  = Funcs.AY_menu.createSection       menu.addContainerButtonScript  = Funcs.AY_menu.addContainerButtonScript       menu.setButtonsForSection  = Funcs.AY_menu.setButtonsForSection       menu.broadcastAvailable  = Funcs.AY_menu.broadcastAvailable       break     end   end end

Вот это все - просто синтаксический бред, который не выдает кучу ошибок по единственной причине: он на самом деле нифига не работает. Обратите внимание, что перепутан даже синтаксис в разных строчках. Кое-где перед ванильным именем вставлено "Funcs.AY_menu.", а кое-где - "Funcs.AY_". А ведь это - абсолютно разные вещи. Происходит это из-за того, что автор вообще не понимает, что делает.

Но как я уже и говорил, эта чушь не срабатывает. Потому что дальше автор полностью скопипастил ванильный код, в котором переменной init назначена другая функция - ванильная, которая и срабатывает при инициации скрипта:

Код:

local function init()    Menus = Menus or { }    table.insert(Menus, menu)    if Helper then       Helper.registerMenu(menu)    end end

Казалось бы - да и фиг-то с ним, КАК именно это работает, лишь бы работало... Ан нет.

Дело в следующем: во-первых, в таблицу Menus меню собственности длбавляется еще раз, поскольку ванильный init предназначен тупо для записи нового меню в таблицу, и он не проверяет, есть ли там уже меню с таким именем.

Во-вторых, при регистрации меню хэлпером еще раз происходит развешивание скриптов. А те, которые уже были навешены на ванильное меню, никто не снимает. Как минимум, это засирает таблицы ненужными данными, из-за чего замедляется работа других меню. Ведь чем больше таблица - тем дольше ищется в ней нужное значение.

И в третьих (вот это - самое главное) ванильное меню целиком получает нескомпилированный код вместо ванильного байткода из xpl-файла. В результате все меню начинает работать медленнее, ибо компиляция на то и сделана.

И все бы ничего, но вот, к примеру, меню собственности устроено так, что в нем некоторые функции выполняются бешеное кол-во раз. и вот достаточно "тяжеленькая" функция menu.addContainerRow, которую автор не менял, но совершенно без нужды заменил байткод на некомпилированный, проворачиваясь на каждом объекте, имеющем подчиненных (а это как минимум - ВСЕ станции), тормозит работу меню мама не горюй!

Помнится, Алекс где-то сетовал на медленную работу меню собственности при большом ее кол-ве... Так вот виноват больше не Егософт, а его собственный горе-фикс.

Вот как-то так...

Попался мне на глаза вот этот мод. Мысль собрать всех продавцов кораблей в одно меню я нашел удачной, но вот исполнение - из рук вон плохим.

Очевиднейшие недостатки:

1. Сортировка верфей по фракциям их владельцев - абсолютно неожиданное и крайне спорное решение. Гораздо логичнее сортировать их по кластерам.
2. Крайне низкая информативность.
3. Отсутствует разделение малых и крупных верфей.

Менее очевидные недостатки (которые выясняются при пользовании модом):

1. Глючит при строительстве нескольких кораблей. Странно, что автор этого не замечает, но это видно даже на его собственной второй картинке на Нексусе, где продублировался его Harpy Missile Cruiser с одним процентом прогресса.
2. Отсутствует обновление прогресса строительства кораблей при отображении меню.
3. Показ неизвестных игроку верфей - злостный чит (злостное имхо ).

Еще менее очевидные недостатки (которые выясняются при ковырянии внутри и исследовании дебаг-лога):

1. Чтобы верфь отображалась в меню без использования вышеозначенного чита, игроку требуется просканировать верфь минимум на один процент, в то время как логичнее сделать проверку, известно ли плеернейму ее название, т.е. к верфи необходимо всего лишь один раз подлететь достаточно близко, чтобы она на карте отобразилась не как "неизвестная станция", а со своим реальным названием.
2. Автор какого-то лешего забиндил автопилот, который конфликтует с ванильным биндингом, из-за чего в дебаг лезут ошибки. На самом деле автопилот этому меню нужен как рыбе зонтик, ибо есть опция "показать на карте", откуда можно уже и автопилот включить, ежели уж так приспичило, что вряд ли...

В общем, понял я, что переписывать надо с нуля.