Spell Steal v1.2

[Moderator]

05:42, 26th Aug 2012
Magtheridon96:

The code's fine, but it would be nice if you were to cache things like (Last created floating text) into variables so that you don't have to repeat the call over and over again.

 

[Almia]

Reserved~
Also i found out that this is easy to make!
Check it out!

edit
This spell is pain in the object editor!

 

[CoLd Bon3]

WOW! This is neat! Amazing job there, Radamantus!

 

[Almia]

Thank You

 

[baassee]

Noooooooooessss replace most arrays with non-arrayed variables. Arrays takes too much space to be worth it.

Why you use customscript for LoadInteger only but not when you save all the other variables into the hashtable? Don't know what their native name is?

 

[Almia]

Its okay for the arrays.
Its okay to use Custom Script and that hashtable action.
Nobody cares.

 

[baassee]

No it isn't sorry to disappointed ya.

And you still didn't answer the question - why you use custom script to avoid the hashtable BJ only with integers? Why not do it on all Hashtable related while you're at it.

 

[Almia]

I just got confused with integers and abilities thats why,
Also there is no script for these
LoadAbility

 

[baassee]

• Hashtable - Save SS_Casted as (Key Boolean) of SS_RegisteredKey in SS_Hash
• Hashtable - Save SS_Current as 5 of SS_Key[1] in SS_Hash
• Hashtable - Save SS_SSLevel as 6 of SS_Key[1] in SS_Hash
• Hashtable - Save True as (Key On) of SS_Key[1] in SS_Hash

And there's a few more concerning if condition and variables.

 

[Almia]

ahh the one SS_Casted right?
I made so that the text Default String wont appear
while the true is to dont spam the warning

 

[Magtheridon96]

Abilities arent actual legit objects you know.
They're integers.

 

[Almia]

I dont get it,what do you mean by meaningful names and run a trigger?
Do you mean not using custom script?

 

[defskull]

TempAbility -> OriginalAbility
TempAbility2 -> StolenAbility

Users would not understand what the heck is TempAbility1/2.
When they read variable name Original/Stolen - Ability, they would understand it a bit (compared to TempAbility1/2)

Run a trigger, is you do the saving inside another trigger (fires off by running trigger action).
You're making the user of your trigger to be harder to configure your trigger.

Look at some of my system (Signature), it uses this method, saving the values in another trigger (Item Weight (setup save) / Item Durability (setup save) / Frictional Knockback (event save) / etc)

You can learn something from it, apparently it is called as "encapsulation", one of programming-style.

 

[Almia]

oh thank you!

 

[doomhammer99]

Nice Radamantus..You've done a great job....

 

[~DARK~]

I Love Your Spells They Are Nice

 

[FRENGERS]

Holy shet babe @.@

 

[doomhammer99]

• Lightning Dash Cast

[EVENT]Events[/Event]

 

[Lolznessmann]

Ok so silly question but how do i import this spell? I get the error(s) that
Line 142: Expected a variable name
set udg_SS_OriginalAbility = 'AOww'
Same for lines 144,148,150,154,156,160,162,166,168
And for Line 184: Expected a name
call SaveInteger(udg_SS_Hash,udg_SS_OriginalAbility,1,udg_SS_StolenAbility

Thx in advance

 

[Daffa]

Do you check preference?

 

[Tasyen]

You need to define 2 variables with variable Editor, do this inside the target map:
name : type
SS_OriginalAbility : Ability
SS_StolenAbility : Ability