Голос за Enein`а.У него красивее.

Оба спелла ничо так, но у енейна не очень понравились маленькие эффекты от разрывов огня (как по мне, не очень смотрится) и долгий кастинг. У bee пришлось по душе абсолютно всё. За него голос дам.

Голос за пчёлку aka Bee т.к. у еника aka Enein не понравилось то что не кастует спел если навести в кучку юнитов и обезательно должно быть не ближе 750 радиуса. Вообще оба спела хорошие, но нужно же за кого-то проголосовать.

Енеин не используй переменные с названием TIME и DATE, решил поменять константы, сжасс парсер их не переваривает.

сжасс никому не нужен, например

Ну и собственно поставил 32 осколка, (undef TIME помогло), один огонек с каждого выстрела спеллом первого уровня не удаляется.

DoctorGester добавил:
Enein, мне не нужен, но парсер то стоит.

DoctorGester добавил:
P.S. голосовать не буду, оба хороши в принципе, ожидал большего.

точно, 31 максимум

Enein, скачал твой карта, запуск "тест" тоже там не работает вар3 кроме версия 1.21 и 1.22

там используется гетспеллтаргетХ/У

Enein,

это не английский, он незнает английского
чтобы работало на бородатых патчах, надо изменить всего две строки

Clamp:

Спел энейна тоже хороший, но в реальной игре он никакой. с точки зрения играбельности, 7 секунд каста- невероятно долго

Не очень то согласен с мнением Clamp, так как если считать что это игровая способность, то ее легко можно применять хоть и время каста 7 секунд. Как бы дальнее заклинание, с требованием небольшого расчета)
Голосую за Enein а

Люди вы так рассуждаете ... конешн - "нельзя использовать в мультиплеере" , но почему нельзя сделать урон огромным и радиус чуть по шире красота то от этого не измениться . ни каст ,ни чего . Голос отдал за Enein , действителльно хороший спел ... , а у bee нехватка эффектов ( большое время каста позволило Enein всунуть дофига эффектов в один спел )

Enein:

надо изменить всего две строки

давай сделать? если не сделал код, я отдаю голос bee :)

Идея — 15/30:
Разновидность бурана с претензией на оригинальность. Да, спел вполне юзабельный, однако не хватает оригинальности, и он довольно короткий по продолжительности. Выдумки автора хватило лишь на оглушение и нанесение урона в качестве дополнительного воздействия. Было бы неплохо добавить сюда эффект уменьшения брони у войск и какой-нибудь быстродействующий DoT для реалистичности.

Внешний вид — 20/40:
Довольно странно видеть большие снаряды, падающие из облака. По сути заклинания это должны быть капли, а не метеориты. Я считаю, что следует уменьшить масштаб снарядов. Далее — облако не разрастается из маленькой тучи, а появляется совершенно внезапно. Следует создавать ядовитые облака постепенно (допустим, с интервалом в 0.334 секунды — время проигрывания анимации Birth у модели). Небольшой пинок автор получает за лень в редакторе объектов, так как описание на всех уровнях заклинания одинаковое.

Код — 20/30:
При первом осмотре код порадовал больше остальных частей — он написан аккуратно, с отступами и комментариями. Однако при более детальном изучении выявилось следующее:

1. Проверку на допустимость целей можно было вынести в отдельную функцию.
2. Почему бы сразу не считать в радианах полярные координаты? То есть не Cos((i * 60.)*.0174532), а Cos(i * 10.47192)?
3. Множественные вызовы GetOwningPlayer(c) в методе create структуры spelldata. Я считаю, что можно добавить в структуру дополнительный элемент типа player.
4. При вызове UnitDamageTarget(…) не используются константы AT, DT, WT.
5. call spelldata.create(GetTriggerUnit(), x, y), вместо x и y можно сразу подставить вызовы функций GetLocationX и GetLocationY.
6. В методе fall структуры missile совершенно бесполезно вызываются GetUnitX и GetUnitY, так как снаряд в процессе меняет лишь свою высоту, а координаты x и y постоянны.
7. Диапазоны высоты следовало вынести в секцию настройки заклинания.
8. Дамми-юнит, который кидает молот бурь, создается и удаляется каждый раз, хотя можно использовать 1 дамми для всех запущенных заклинаний.
9. (-1.)*<переменная> можно заменить на -<переменная> (например, (-1.)*MissileAoE можно заменить на –MissileAoE).

Насчет дерева ифов сказать ничего конкретного не могу, так как подзабыл уже.

Идея — 20/30:
Неплохая задумка, своеобразная вариация на тему огненного столба. Однако ничего экстраординарного я не увидел. К тому же, заклинание довольно неюзабельное из-за длительного применения. В качестве дополнения можно сделать прыгающие снаряды и разлетающиеся тушки подгоревшего мяса (которые наносят урон при попадании в юнита).

Внешний вид — 32/40:
Замечаний по поводу внешнего вида, используемых эффектов и описания заклинания нет. Все выглядит гармонично. Для разнообразия можно было бы добавить какой-нибудь эффект при столкновении снаряда с землей. Если заклинание не отвечает требованиям (дистанция ближе 750 единиц), то у кастера продолжает играть анимация channel.

Код — 13/30:
Код написан менее дружелюбно. При детальном осмотре код разочаровал, несмотря на кажущуюся серьезность.

1. В функции EnumDestInCircle каждый раз создается новый квадрат, хотя можно брать и двигать один и тот же квадрат и двигать его функцией MoveRectTo.
2. Строки с ресурсами не вынесены в секцию констант.
3. При количестве осколков больше 10 заклинание будет некорректно создавать эффекты.
4. Статический массив групп, который перезаписывается, если запустить вторую копию заклинания, не дожидаясь окончания первой.
5. Нет зависимости скорости работы от периода таймера.
6. Вычисление высоты можно было вынести в отдельную функцию и вообще там можно численные значения свести к минимуму, а в комментарии написать оригинальную формулу.
7. Не учитывает высоту ландшафта (учитывает, но только на первом этапе применения заклинания).
8. Почему бы сразу не считать в радианах полярные координаты?
9. Для удобочитаемости можно вынести каждый этап в отдельную функцию.

MUI есть, но два спелла одновременно будут глючить по причине, указанной в пункте 4. То есть спелл применять двоим и больше можно, но это будет неочем. "MUI есть, но он глючно реализован" ~ "MUI нет". Я думаю, что Enein просто поленился доработать работу с группами.

спсасиб за понятный пост, как всегда все ясно

bee добавил:

call spelldata.create(GetTriggerUnit(), x, y), вместо x и y можно сразу подставить вызовы функций GetLocationX и GetLocationY.

да... это фэйл

Если Павел не расценит как флуд,

Проверку на допустимость целей можно было вынести в отдельную функцию.

Можно узнать, а это вынесение нужно для чего?
Эстетический фактор, удобность для оценивателя кода или это для производительности?
Ответьте, пожалуйста думаю причину вопроса описывать не надо.