![]()
truth troll
Silenced by truth troll
![]() ![]() offline
Опыт:
7,396Активность: |
Тестирование спелла
И снова здравствуйте. Вот написал свой первый триггерный спелл.
Принцип действия: Создан на основе "стремительности" (Мастер Клинка), только при атаке герой не наносит доп. урон, а оглушает противника... » Spell01a Код:
Прошу оценить спелл, а также указать на возможные ошибки и помочь в его оптимизации... карта на версии 1.24 Отредактировано truth troll, 07.08.2009 в 08:59. |
![]() |
#1
+0/−0
Профиль |
Приват |
Поиск |
Цитата |
IP: Записан
|
![]()
Hellfim
Новичок
![]() ![]() offline
Опыт:
79,707Активность: |
DragonSpirit, советую прочитать код =)
Hellfim добавил: truth_troll, для начала надо нормально называть переменные. |
![]() |
#2
+0/−0
Профиль |
Приват |
Поиск |
Цитата |
IP: Записан
|
![]()
truth troll
Silenced by truth troll
![]() ![]() offline
Опыт:
7,396Активность: |
Hellfim, Чем тебе не нравится как я их назвал? Подозреваю ты имеешь ввиду, что код с такими названиями переменных плохо читается... А какие именно переменные не удачно названы?
а где комент DragonSpirit? О_о |
![]() |
#3
+0/−0
Профиль |
Приват |
Поиск |
Цитата |
IP: Записан
|
![]()
Hellfim
Новичок
![]() ![]() offline
Опыт:
79,707Активность: |
truth_troll, ну по мне, так все переменные надо называть вразумительно, например переменные из функции Spell01b
И да, названия функций тоже очень хромают, если у тебя 100 спеллов, то представь как ты будешь ориентироваться. Он самоудалил свои комментарии =) |
![]() |
#4
+0/−0
Профиль |
Приват |
Поиск |
Цитата |
IP: Записан
|
![]()
truth troll
Silenced by truth troll
![]() ![]() offline
Опыт:
7,396Активность: |
Hellfim,
Передела, по твоему совету, первый пост... Теперь переменные все нормально названы? Цитата:
Не фига себе! 100 спеллов! Я загнусь под ними! =)) Нет, 100 спеллов не будет максиму 20... Цитата:
Жаль... надеюсь ничего интересного. truth troll добавил: Меня интересует замечание по самому коду, а не по его оформлению... |
||
![]() |
#5
+0/−0
Профиль |
Приват |
Поиск |
Цитата |
IP: Записан
|
![]()
ZeToX2007
![]() ![]() offline
Опыт:
7,009Активность: |
truth troll, Лучше используй GetWidgetLife чем GetWidgetLife. эээ непонял а спел срабатывает, когда юнит атакует или наносит повреждения ? и кстати TriggerSleepAction работает неточно =/ можешь проверить.
|
![]() |
#6
+0/−0
Профиль |
Приват |
Поиск |
Цитата |
IP: Записан
|
![]()
DragonSpirit
у - уходи
![]() ![]() offline
Опыт:
22,625Активность: |
мб ты чутка промахнулся? ^_^ truth_troll, твой спелл рабочий,я писал что он может работать только один раз из за действия,
и наверное я был не прав,раз меня Hellfim поправил =) |
![]() |
#7
+0/−0
Профиль |
Приват |
Поиск |
Цитата |
IP: Записан
|
![]()
truth troll
Silenced by truth troll
![]() ![]() offline
Опыт:
7,396Активность: |
Цитата:
ZeToX2007,я так понял ты имеешь ввиду: лучше GetWidgetLife() чем GetUnitState()... Можешь объяснить почему? Цитата:
Сам триггер Spell01a срабатыват на "Юнит - атакован", а его действие создаёт триггер Spell01b. Который срабатывает на "Юнит - получает урон"... Цитата:
В принципе можно использовать таймер, но будет ли это оправданное действие? DragonSpirit, Ну да, Hellfim правильно заметил... Т.к. call DestroyTrigger(GetTriggeringTrigger) для динамически созданого триггера... |
|||
![]() |
#8
+0/−0
Профиль |
Приват |
Поиск |
Цитата |
IP: Записан
|
![]()
Hellfim
Новичок
![]() ![]() offline
Опыт:
79,707Активность: |
truth_troll, оправданно будет. Про GetWidgetLife(...) смотри соседнюю тему, там свдн дал точные цифры скорости работы этих функций. |
![]() |
#9
+0/−0
Профиль |
Приват |
Поиск |
Цитата |
IP: Записан
|
![]()
Garret
Недоброе творится
![]() ![]() offline
Опыт:
3,893Активность: |
Не очень понял, зачем там творится какое-то чудо со стуном - почему бы просто не использовать дамми каст? |
![]() |
#10
+0/−0
Профиль |
Приват |
Поиск |
Цитата |
IP: Записан
|
![]()
truth troll
Silenced by truth troll
![]() ![]() offline
Опыт:
7,396Активность: |
Цитата:
думаю ответ находится в первом посте =) Цитата:
А вообще учусь... =))) Hellfim, Понятно... Возможно ли такое: call TimerStart(Timer1, 2.00, false, function Func00 (Unit))? Отредактировано truth troll, 09.08.2009 в 03:02. |
||
![]() |
#11
+0/−0
Профиль |
Приват |
Поиск |
Цитата |
IP: Записан
|
![]()
Hellfim
Новичок
![]() ![]() offline
Опыт:
79,707Активность: |
truth_troll, функция, которая вызывается таймером не может принимать аргументы. Ты должен сделать переменные глобальными. |
![]() |
#12
+0/−0
Профиль |
Приват |
Поиск |
Цитата |
IP: Записан
|
![]()
truth troll
Silenced by truth troll
![]() ![]() offline
Опыт:
7,396Активность: |
Hellfim, Это, что опять придется использовать массивы? Ведь если я создам глобальные переменные (не массив), то они могут измениться в течении отсчёта таймера и действия проведутся не над нужными мне значениями...
|
![]() |
#13
+0/−0
Профиль |
Приват |
Поиск |
Цитата |
IP: Записан
|
![]()
Hellfim
Новичок
![]() ![]() offline
Опыт:
79,707Активность: |
truth_troll, не массивы, а глобальные переменные. Это не так страшно. |
![]() |
#14
+0/−0
Профиль |
Приват |
Поиск |
Цитата |
IP: Записан
|
![]()
truth troll
Silenced by truth troll
![]() ![]() offline
Опыт:
7,396Активность: |
Hellfim, Я понял о чём это ты, вроде... =) Ты предлагаешь использовать Scope и создавать private глобалки? Но у меня чего-то не получается =((( Вот переделал свой код спелла. Покамись заменил один Wait на таймер, создал scope и объявил приват глобалки... И с этими изменениями карта вылетает, явно я делаю что-то не так =(((
» не работает!!! Код:
|
![]() |
#15
+0/−0
Профиль |
Приват |
Поиск |
Цитата |
IP: Записан
|
![]()
Hellfim
Новичок
![]() ![]() offline
Опыт:
79,707Активность: |
» code
Возможно код не работает, ибо я немного запутался =) + тебе надо создать дамми-юнита и дать ему абилку на основе молота бурь (Горного короля). |
![]() |
#16
+0/−0
Профиль |
Приват |
Поиск |
Цитата |
IP: Записан
|
![]()
truth troll
Silenced by truth troll
![]() ![]() offline
Опыт:
7,396Активность: |
Hellfim, Спс, твой код мне очень помог... завтра попытаюсь всё довести до ума и закрою тему...
|
![]() |
#17
+0/−0
Профиль |
Приват |
Поиск |
Цитата |
IP: Записан
|
![]()
bee
vjass.optimizer
![]() ![]() offline
Опыт:
16,615Активность: |
Hellfim, уверен? |
![]() |
#18
+0/−0
Профиль |
Приват |
Поиск |
Цитата |
IP: Записан
|
![]()
Hellfim
Новичок
![]() ![]() offline
Опыт:
79,707Активность: |
В мануале написано, что не могут, хз. Но да, тема закрыта. |
![]() |
#19
+0/−0
Профиль |
Приват |
Поиск |
Цитата |
IP: Записан
|
![]()
ShadoW DaemoN
![]() ![]() offline
Опыт:
37,078Активность: |
Hellfim, нет, сэр, вы не правы:
Надо мануал читать лучше, так как инициализаторы областей были введены в JassHelper версии 0.9.9.9 |
![]() |
#20
+0/−0
Профиль |
Приват |
Поиск |
Цитата |
IP: Записан
|