Spell Kamikaze Slash

Magoiche

Member
Kamikaze Slash v1.3
by Magoiche



GUI/JASS: vJASS
MUI/MPI: MUI and MPI
Leakless?: Yep
Lagless?: Yep
Requirements: NewGen
Import Difficult: Medium

Description: The caster enters in a milesecond of frenzy hiting the enemy with all its power. The hit is so powerfull that even the caster receives damage.

Level 1 - The caster receives 15% of the target current HP in damage and the target receives the same damage plus 5% of the caster current HP.
Level 2 - The caster receives 30% of the target current HP in damage and the target receives the same damage plus 10% of the caster current HP.
Level 3 - The caster receives 45% of the target current HP in damage and the target receives the same damage plus 15% of the caster current HP.
Level 4 - The caster receives 60% of the target current HP in damage and the target receives the same damage plus 20% of the caster current HP.


Screenshots:

Code:
JASS:

//////////////////////////////////////////////
////            Kamikaze Slash            ////
////          by Magoiche Saika           ////
//////////////////////////////////////////////
////NOTE: Everything that you can't change////
////       in this script you can in the  ////
////              Object Editor.          ////
//////////////////////////////////////////////

scope KamikazeSlash initializer Initial

globals
    private constant integer KamikazeSlashRawData = 'A000'
    // Raw Data of the Kamikaze Slash spell
    private constant real TargetHPPercentage = 0.15
    // Percentage of HP per Spell Level that damages the Caster
    private constant real CasterHPPercentage = 0.05
    // Percentage of the Caster current HP that is added to the Damage for the Caster
    private constant attacktype KamikazeSlashAttackType = ATTACK_TYPE_HERO
    // Attack Type of the spell
    private constant damagetype KamikazeSlashDamageType = DAMAGE_TYPE_NORMAL
    // Damage Type of the spell
    private constant weapontype KamikazeSlashWeaponType = null
    // Weapon Type of the spell
    private constant string KamikazeSlashEffectsAttachmentPoint = "chest"
    // Attachment Point for the effects
    private constant integer KamikazeSlashNumberOfEffects = 4
    // Number of Active Effects
    private string array KamikazeSlashEffects
    // Dont Change the Line Above please
endglobals

private function Effects takes nothing returns nothing
    set KamikazeSlashEffects[0] = "Abilities\\Spells\\Other\\Stampede\\StampedeMissileDeath.mdl"
    set KamikazeSlashEffects[1] = "Objects\\Spawnmodels\\Human\\HumanBlood\\BloodElfSpellThiefBlood.mdl"
    set KamikazeSlashEffects[2] = "Objects\\Spawnmodels\\Other\\BeastmasterBlood\\BeastmasterBlood.mdl"
    set KamikazeSlashEffects[3] = "Objects\\Spawnmodels\\Orc\\Orcblood\\BattrollBlood.mdl"
endfunction

// To change an effect just change the path. The path is the text
// between the ""
// To implement new effects you have to make a line bellow the line 37,
// the line should be like this: set Effects[4] = "Patch to the Effect"
// then put the number of effects in the Line 27
// REMEMBER: There must be two "\\". If you jsut copy the patch from the
// object editor there will be only one "\" so remember to put two '\\'

// Dont change anything after this line if don't know what you are doing.

private function Conditions takes nothing returns boolean
    return GetSpellAbilityId() == KamikazeSlashRawData
endfunction

private function Actions takes nothing returns nothing
    local unit Caster = GetTriggerUnit()
    local unit Target = GetSpellTargetUnit()
    local integer LoopAIndex = 0
    local real HPDamage = TargetHPPercentage * GetUnitAbilityLevel( Caster, KamikazeSlashRawData ) * GetWidgetLife( Target )
    call UnitDamageTarget( Caster, Caster, HPDamage, true, false, KamikazeSlashAttackType, KamikazeSlashDamageType, KamikazeSlashWeaponType )
    set HPDamage = HPDamage + ( CasterHPPercentage * GetUnitAbilityLevel( Caster, KamikazeSlashRawData ) ) * GetWidgetLife( Caster )
    call UnitDamageTarget( Caster, Target, HPDamage, true, false, KamikazeSlashAttackType, KamikazeSlashDamageType, KamikazeSlashWeaponType )
    loop
        exitwhen LoopAIndex > KamikazeSlashNumberOfEffects
        call DestroyEffect( AddSpecialEffectTarget( KamikazeSlashEffects[LoopAIndex], Caster, KamikazeSlashEffectsAttachmentPoint ) )
        call DestroyEffect( AddSpecialEffectTarget( KamikazeSlashEffects[LoopAIndex], Target, KamikazeSlashEffectsAttachmentPoint ) )
        set LoopAIndex = LoopAIndex + 1
    endloop
    set Caster = null
    set Target = null
endfunction

private function Initial takes nothing returns nothing
    local trigger trig = CreateTrigger()
    call Effects()
    call TriggerRegisterAnyUnitEventBJ( trig, EVENT_PLAYER_UNIT_SPELL_EFFECT )
    call TriggerAddCondition( trig, Condition( function Conditions ) )
    call TriggerAddAction( trig, function Actions )
    set trig = null
endfunction

endscope


Changelog
v1.0 - Created and Submited the Spell
v1.1 - Made the spell more configurable
v1.2 - Made the spell even more configurable and removed some leaks
v1.3 - Enhanced the spell. Special thanks to Romek

Instructions of how to import inside the map!​
 

Magoiche

Member
How could i say?...
The spell is based on Storm Bolt and its have the Meat Wagon Missle.
When you cast the spell you will hear something like a "Flesh Explosion"
So it will be weird if there is another sound.
At least i think it will be.
But anyway. I WILL DO IT! ò.ó

EDIT: Added v1.1
 
> hiting the enemies
That was a bit misleading - I thought it was an Omnislash-type ability (if you know what I mean).

>
JASS:

    set Effect[0] = FirstEffect
    set Effect[1] = SecondEffect
    set Effect[2] = ThirdEffect
    set Effect[3] = FourthEffect

and
>
JASS:

   private constant integer NumberOfEffects = 4

What if I would want to add two effects more? Then I have to put myself "FifthEffect" and "Sixth Effect" and edit the array Effect to suit my needs. You could add this to the configuration somehow to make things a lot more easier. And, I think you aren't using NumberOfEffects anywhere (or I'm blind).

Other than that, good spell. :) +rep
 

Romek

Super Moderator
Staff member
You could set those 'Effects' at map init, so you don't do it every time the function runs.
And those aren't fully configurable.

> exitwhen LoopAIndex >= NumberOfEffects - 1
exitwhen LoopAIndex > NumberOfEffects

> GetUnitState( Target, UNIT_STATE_LIFE )
GetWidgetLife(Target)

> ...ndex], Caster, "chest" ) )...
Make the "chest" configurable



Also, your variable names don't follow any sort of naming convention.
Constants are usually in ALL CAPS. Locals are usually all small. No caps at all.
 

Magoiche

Member
Thanks! +rep

Added v1.2 - Its even more configurable now.

And Romek...

I like the way i call the variables.
I don't have to change its names because one is local and other is global
They all are usefull to the trigger so i name them all the same.
Simple!
 

Romek

Super Moderator
Staff member
I like the way i call the variables.
I don't have to change its names because one is local and other is global
They all are usefull to the trigger so i name them all the same.
Simple!
You'll get muddled up easily like that.
And it's a simple naming convention which people follow for a reason.
It's not just there to annoy people.
When I see a variable in capitals, I assume it's a constant. Same with some capitals (globals) and all lowercase (locals).
Also, if you have 2 variables, a local and global with a common name, the local will get priority, and you might get some annoying bugs. ^_^

Edit:
Why are you setting the Effect array in a different scope?
"Effect" is a very common variable name, and it should definitely be private.

Though this wouldn't be a problem if you just put it into one scope.. o_O
 

Magoiche

Member
Searching an translator for muddled.
And... sorry! I didn't say that to botter you. i.i

About the effects.
If i put it in the same scope i will have to set the whole trigger to run on map init.
Or there is no probme about that?

EDIT:

I GET IT!

I just put a call for the stup function on the initializer function.
It worked well. Thanks man!
 
General chit-chat
Help Users
  • No one is chatting at the moment.
  • tom_mai78101 tom_mai78101:
    Hmm, do I need to start posting some of the Science and Technology news to this new section in the future?
  • tom_mai78101 tom_mai78101:
    Or do we need to move the news to their respective sections once the threads are old enough, read to be put into the News Archive?
  • The Helper The Helper:
    The 2nd thing
  • The Helper The Helper:
    I have been moving stuff from the current Headline news like a year old and stuff over when I see it in the logs or members online but mostly I have been moving stuff from the Archives
  • The Helper The Helper:
    It is possible to automate it so that instead of moving to Archive if it has the Science, technology or sci tech tag that it gets archived there instead. I am considering a new Weird News Tag
  • tom_mai78101 tom_mai78101:
    Please do. I'll try to tag the news appropriately when I get the chance.
  • The Helper The Helper:
    The new Discourse forum software we are moving to next does alot with the tags and also hashtags
  • The Helper The Helper:
    OK I found the source of your burrito post stuff it is from Reddit and possibly a facebook post too but definitely reddit
  • The Helper The Helper:
    Happy Monday!
  • The Helper The Helper:
    Happy Tuesday :)
  • Blackveiled Blackveiled:
    Happy Tuesday
  • The Helper The Helper:
    Blackveiled! What is up?
  • The Helper The Helper:
    I saw XXXConan on here the other day I dont know if you remember him
  • The Helper The Helper:
    blasts from the past
  • Blackveiled Blackveiled:
    I remember him, buncha old faces
  • jonas jonas:
    He's a software engineer at a hiring platform now I think
  • The Helper The Helper:
    I wanna know if his mom is still single LOL
  • tom_mai78101 tom_mai78101:
    I need context.
    +1
  • The Helper The Helper:
    Back in the day conan was a young one and his mom was single and I used to tease him that I was going to date his mom and be his daddy - yeah sounds bad now but it was funny at the time
  • The Helper The Helper:
    his mom is my age
  • The Helper The Helper:
    i know right it was pretty funny
  • thewrongvine thewrongvine:
    its pretty funny
    +1
  • tom_mai78101 tom_mai78101:
    Thanks for the context.
    +1
  • The Helper The Helper:
    Happy Hump Day!!!

    Staff online

    Members online

    Affiliates

    Hive Workshop NUON Dome World Editor Tutorials

    Network Sponsors

    Apex Steel Pipe - Buys and sells Steel Pipe.
    Top