4 replies to this topic

[Ark223]

Because my topic wasn't accepted in Library section till this day, so I post here.

Like in the title, skillshot database for everyone! If something is incorrect, please notify me.

 

Patch 8.5

https://github.com/A...hotDatabase.lua

Edited by Ark223, 07 March 2018 - 07:45 .

[ReverseMisaya]

Thanks for the data.

 

Maybe it's just me, but wouldnt it be better to have the table index based on champion names instead of spell names?

If I dont know the spell names this is a pain to find them...

 

Also, from a development point of view this seems better:

local heroSpellData = spells[myHero.charName]

versus

--first some other method to find the spellnames...

local qData = spells[qSpellName]
local wData = spells[wSpellName]
local eData = spells[eSpellName]
local rData = spells[rSpellName]

_______

 

Also, you might want to localize the table before newbie scripters copy-paste this directly in their script and start complaining about performance.

Spells = { 

should be

local Spells = { 

[Ark223]

 

Thanks for the data.

 

Maybe it's just me, but wouldnt it be better to have the table index based on champion names instead of spell names?

If I dont know the spell names this is a pain to find them...

 

Also, from a development point of view this seems better:

local heroSpellData = spells[myHero.charName]

versus

--first some other method to find the spellnames...

local qData = spells[qSpellName]
local wData = spells[wSpellName]
local eData = spells[eSpellName]
local rData = spells[rSpellName]

_______

 

Also, you might want to localize the table before newbie scripters copy-paste this directly in their script and start complaining about performance.

Spells = { 

should be

local Spells = { 

My table set won't be changed. It lets you to make for example evade, which will detect spells based on skillshot table.

[Ark223]

Could somebody close that topic or remove, cause it's already in Library section.

[Aristo]

Best Answer

Could somebody close that topic or remove, cause it's already in Library section.

alright.