Great it is up :D
Let see my reviewing...
Shadow Thunder
+The spell is original
+Some nice spell effects there, models and icons fit well.
+The spell is Mui (it seems)
-The use of custom value could interfiere with other spell systems.
-In my opinion you had too many variables, and the code and variable names looked complex.
-At some points the flames was still there after casting it. I dont know where in the trigger, but it made the target un targettable by that unit until another hero override the effect.
Curse of Attraction
+Spell is original
+Indeed matching the theme
-It is not mui
-The effects were alittle boring
-The knockback leaks 2 points every time the "pick unit in unit group" runs.
Shield of Just
+Nice imported shield, the icon, sure, even though it was a wow-one, the yellow blockshield and the floatingtext was really nice.
+Spell seems to work Mui, well coded Gui
-Uses custom values, as i said before, it could interfiere...
-Spell is abusable. If an enemy would cast the shield on himself, and you use the attack bug (pressing stop multiply times at a very small intervall) the shield would register it as a block, and get removed pretty instantly.
Dance of Fire
+fit the theme well, referring to the last effect explosion
+well coded
+perfectly mui
+the idea is original
-the spell could be better actually, by using vjass. Also Handle vars are not that popular anymore, since the system is less efficient than newer ones.
-the spell lacked special effects indeed. The last explosion seems too small. The fire line was also kind of narrow.
+/-if the caster didnt move after the cast no fire arised either. Was that intended?
Global Time Shift
+Original idea
+Matched the theme well
+Well coded jass
+/- Mui? I wonder what would happend if you cast it once, waited ~2seconds and casted it again.
-Is the multiply struct methods really needed? It would have been easier to use struct create or struct allocation instead.
-The spell lacks effects and sounds. it made it alittle boring.
Nuclear Missile
+Idea was fairly original, though, I've never seen it in warcraft;)
+Massing of effects, in this case actually necessary.
+Fits the theme
+Well coded Gui, and mui
-In my opinion this would be great in a cinematic map. However, in a playable map it seems to imbalanced.
-I could not think of anything more than, like most of the other spells, Gui is inefficient:p
Ember
+Spell fits the theme
+Seems Mui
+Effects fit - bot the intial and the final.
-The greatest flaw here would be: You can move the flames and break the circle... Which is actually a very bad thing. Add locust to the flames next time.
[] There wasnt alot to say about it. Could be positive, coud be negative
Seal of Rightneousness
+Spell uses nice effects, I was surprised, I thought they were imported.
+Fits the theme well
+Is Mui
-inefficient Gui. An example, in the Seal release trigger, it could greatly be optimized by a outer loop and an inner loop instead of writing the inner loop multiply times.
-Too long tooltip. It didnt display all words at the end of the tooltip due to this, and it was hard to understand what the spell did.
Touch of Nature
+Original
+Fit the theme, fit the hero, fitting spells
[]Autocast is a nice idea, if it stacks. I didnt check if it did though
-Could be abusable (in a dumb way perhaps, but still), using unit is attacked is a bad option since you can bug it by pressing stop multiply times (this would be for the enemy I think)
-Way too long tooltip here as well.
-Spell is not Mui, but however, Mpi.
-The spell code is way too inefficient. It could be greatly reduced by adding the constant for the level of the spell in a formula instead of checking if the spellevel is 1,2 or 3 and so on.
Fade Away Aura
+The spell is original
+The spell fits the theme, I guess, referring to the final destruction of the units.
[] To be honest I dont really want to be patronizing in any way. The spell was not in my taste at all. When testing it, I got debug messages on the screen saying you leaked locations (and groups) (in jassnewgenpack this is). The effects and code could have been alot better.
----
Results
None of the entries was perfect. But the one who gets my vote is Dreadnought[dA] (with Dance of Fire)
Even though it lacked with the effects and used an old attachment system, it gains my vote. Also, that he managed to do a pretty spell without using vjass is good too. it was also easy to understand, as tooltip and as a code.
Quote:
FROST VORTEX
+ Really nice coding, some awesome effects, easy to implement...
- Delayed effect does not fit well to your spell.
|
Hm? It had a minor effect for x seconds, then a huge blast+shatterpossibility. That is a delayed effect.