nazarpunk
WarCraft 3

ZinC: Сон AOE

Добавлен , опубликован
Алгоритмы, Наработки и Способности
Способ реализации:
Zinc
Тип:
Способность
Версия Warcraft:
1.26+

Сон AOE

Исходный код
MUI: да
Импорт: иконка
Утечки: нет
Даммики: 1 (Нейтрально пассивный)
Требования: JNGP

Видео

Установка

  • импортировать SpellSleepAOE.zn
  • скопировать юнита dumy dummy
  • скопировать способности: SUsl Сон (герой), CUsl Сон (дамми)
Скачать - 30 КБ 419
,
29 янв 2019
3 121
`
ОЖИДАНИЕ РЕКЛАМЫ...
2
Показать
28
2
Показать
Для определения того, жив ли юнит, лучше использовать UnitAlive или GetWidgetLife.
7
Показать
22
7
Показать
не актуально создавать массив когда вызываете getRange
1
Показать
28
1
Показать
pro100master:
не актуально создавать массив когда вызываете getRange
Поддерживаю, бредово, либо сделать через вычисления (100 + level * 100), либо через инициализацию библиотеки.
Ещё стоит добавить, что лучше не использовать строчные приказы.
С Zink не так сильно знаком, но на JASS я бы написал так.
Вариант из ресурса 
            while(true){
                u = FirstOfGroup(g);
                if (u == null) { break; }
                DummyCastBuff(GetTriggerUnit(), u);
                GroupRemoveUnit(g, u);
            }
Мой вариант 
            loop
                set u = FirstOfGroup(g);
                exitwhen u == null
                call GroupRemoveUnit(g, u)
                call DummyCastBuff(GetTriggerUnit(), u)
            endloop
Потому что мне кажется, что while(true) превращается в exitwhen (not true), но это не точно.
2
Показать
30
2
Показать
PT153:
Для определения того, жив ли юнит, лучше использовать UnitAlive или GetWidgetLife.
Я как-то привык по старинке. Можно вынести функцию isUnitAlive вверх и позволить людям выбирать реализацию.
pro100master:
не актуально создавать массив когда вызываете getRange
С удовольствием написал бы
constant integer Range[] = [100, 200, 300];
но увы, пускать новичков ниже блока onInit не сильно хочется.
PT153:
С Zink не так сильно знаком, но на JASS я бы написал так.
Не так уж страшно он компилится 
            loop
                set u=FirstOfGroup(g)
                if (u==null)then
                    exitwhen true
                endif
                call SpellSleepAOE___DummyCastBuff(GetTriggerUnit(),u)
                call GroupRemoveUnit(g,u)
            endloop
А в идеале хотелось бы написать 
while(u = FirstOfGroup(g) != null){
	DummyCastBuff(GetTriggerUnit(), u);
	GroupRemoveUnit(g, u);
}
0
Показать
28
0
Показать
NazarPunk:
С удовольствием написал бы
Я думаю, что реализация с вычислением лучше всего. А так, этот массив можно сделать глобальным, а нужные значения записывать в инициализаторе библиотеки.
Не так уж страшно он компилится
Хорошо.
Я как-то привык по старинке. Можно вынести функцию isUnitAlive вверх и позволить людям выбирать реализацию.
UnitAlive нужно сначала определить, ибо она есть в common.ai. Потому способ с GetWidgetLife лучше в данной ситуации: не нужно что-то определять и работает быстрее GetUnitState.
И приказ лучше написать численный, а не строчный, но это уже мелочи.
0
Показать
30
0
Показать
Вышла новая версия! Прокрутить к ресурсу
  • Переписан код
  • Добавлен гитхаб
  • Добавлена иконка
  • Добавлена проверка коллизии
Чтобы оставить комментарий, пожалуйста, войдите на сайт.