Addskill Level for players

[Sigoles]

Hello, My script dont working 100%
When I put for example: /addskill Player, level, 1

If player are level 1, he will be level 9 wtf?
The correctly was level 2, because 1+1

Well look my script please and help me to check what is wrong?

local function getSkillId(skillName)
    if skillName == "club" then
        return SKILL_CLUB
    elseif skillName == "sword" then
        return SKILL_SWORD
    elseif skillName == "axe" then
        return SKILL_AXE
    elseif skillName:sub(1, 4) == "dist" then
        return SKILL_DISTANCE
    elseif skillName:sub(1, 6) == "shield" then
        return SKILL_SHIELD
    elseif skillName:sub(1, 4) == "fish" then
        return SKILL_FISHING
    else
        return SKILL_FIST
    end
end

local function getExpForLevel(level)
    level = level - 1
    return ((50 * level * level * level) - (150 * level * level) + (400 * level)) / 3
end

function onSay(player, words, param)
    if not player:getGroup():getAccess() then
        return true
    end

    if player:getAccountType() < ACCOUNT_TYPE_GOD then
        return false
    end

    local split = param:split(",")
    if split[2] == nil then
        player:sendCancelMessage("Insufficient parameters.")
        return false
    end

    local target = Player(split[1])
    if target == nil then
        player:sendCancelMessage("A player with that name is not online.")
        return false
    end

    -- Trim left
    split[2] = split[2]:gsub("^%s*(.-)$", "%1")

    local count = 1
    if split[3] ~= nil then
        count = tonumber(split[3])
    end

    local ch = split[2]:sub(1, 1)
    for i = 1, count do
        if ch == "l" or ch == "e" then
            target:addExperience(getExpForLevel(target:getLevel() + 1) - target:getExperience(), false)
        elseif ch == "m" then
            target:addManaSpent(target:getVocation():getRequiredManaSpent(target:getBaseMagicLevel() + 1) - target:getManaSpent())
        else
            local skillId = getSkillId(split[2])
            target:addSkillTries(skillId, target:getVocation():getRequiredSkillTries(skillId, target:getSkillLevel(skillId) + 1) - target:getSkillTries(skillId))
        end
    end
    return false
end

 

[Nazubal27]

I don't have a solution for you but I have a great tip that every programmer needs to know:
If things aren't working you really need to see whats happening during that function call, best way to do this is print everything out.

Trying printing things out to make sure the values are what you expect by using: player:sendTextMessage(MESSAGE_INFO_DESCR, getExpForLevel(target:getLevel() + 1))
Try printing out:
target:getExperience()
getExpForLevel(target:getLevel() + 1)
and getExpForLevel(target:getLevel() + 1) - target:getExperience()



Trying replacing your for loop with this just to debug it

for i = 1, count do
        if ch == "l" or ch == "e" then
            player:sendTextMessage(MESSAGE_INFO_DESCR, target:getExperience() .. "")
            player:sendTextMessage(MESSAGE_INFO_DESCR, getExpForLevel(target:getLevel() + 1) .. "")
            player:sendTextMessage(MESSAGE_INFO_DESCR, (getExpForLevel(target:getLevel() + 1) - target:getExperience()) .. "")
            target:addExperience(getExpForLevel(target:getLevel() + 1) - target:getExperience(), false)
        elseif ch == "m" then
            target:addManaSpent(target:getVocation():getRequiredManaSpent(target:getBaseMagicLevel() + 1) - target:getManaSpent())
        else
            local skillId = getSkillId(split[2])
            target:addSkillTries(skillId, target:getVocation():getRequiredSkillTries(skillId, target:getSkillLevel(skillId) + 1) - target:getSkillTries(skillId))
        end
    end