Есть ли утечки, в этом простом триггере?

вот

а полностью избежать всех утечек, почти невозможно

С чего бы это?

обезопасить себя по максимуму, от подобного, на GUI

Перейти на jass / vjass / zinc.

всё возможно
утечка, это если ты создал объект, а потом не удалил, когда он больше не нужен
хуже всего если это происходит вызовом каждые 0.01 сек
в триггерах должны быть действие в категории Точка, с ними взаимодействуй

к моему удивлению, у меня в версии 1.31 такой категории триггеров тупо нет 0_0
хотя она мне и никогда не была нужна, очень давно в 99% случаев юзаю тупа координаты
вот для работы с точкой:

native Location					takes real x, real y returns location
native RemoveLocation		takes location whichLocation returns nothing
native MoveLocation			takes location whichLocation, real newX, real newY returns nothing
native GetLocationX			takes location whichLocation returns real
native GetLocationY			takes location whichLocation returns real

Taurenus:

есть две, но не значительные, особенно если способность Добыть будет вызываться всего раз в минуту
точка и область

PROSHELDOTU:

Потому что её добавили в JNGP пробросив нативку. У меня всё руки не доходят все нативки добавить в JNGP.

nazarpunk:

та и пофиг ) код рулез

2 утечки - region centered и target point of ability, их надо в отдельные переменные вешать, а потом вычищать.

EugeAl:

Звук вроде как тоже считается утечкой не? После проигрывания надо удалить его.

gellin, если он локальный, но на гуи звуки глобальные все. Глобалки с именами gg_sound или gg_snd, что то из этого. Функция в джассе StartSound, которая тут из гуи пишется через PlaySoundBJ, не returns sound

Чтоб избежать утечек здесь надо две переменные.
Точка (Location) и Регион (Rect).
Сначала задаешь точку и в нее заносишь Target point of ability being cast.
Потом создаешь Регион на основе этой точки.
Как я понял ты хочешь проверить есть ли на точке разрушаемый объект, то есть руда. Не совсем корректный способ ты выбрал, ибо если при такой проверке там будет не только руда, то может такой код и не сработать.
Чтобы сделать корректно надо в уже созданном регионе Pick every destructible in rect and do multiple actions. После чего снизу берешь if then else и делаешь проверку Destructible-type на свою руду и снизу делаешь нужный тебе код. Косяк такого подхода в том, что на каждую руду будет срабатывать проверка, чтобы сработало только на одну из нескольких, тебе лучше сделать логическую (boolean) переменную и через ее проверку когда первый раз отберешь нужную декорацию переключать логическую и таким образом будет срабатывать только на одну руду из допустим трех.

Забыл добавить. В конце всего чтобы почистить утечки надо сделать два кастомных скрипта.
custom script: call RemoveLocation(udg_название точки)
custom script: call RemoveRect(udg_название региона)

Если что спрашивай тут же в комментариях вопросы.

Real60Hz, желательно искать не рандомную, а ближайшую. И лучше это делать кодом, тогда можно вообще не париться с Location.

nazarpunk:

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

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

Real60Hz:

Недавно отвечал на вопрос. Там всё реализовано.