Authors CurseForge

hey all,

sorry to bug again but i cant seem to get the addon i'm working on to register that my variables should be numbers. not strings. but for some reason when i load the addon it loads with an error saying it cant do the arithmetic for string value. when i looked over all the code i diddnt find any links to strings so i might just not be typing it up right i dunno :/

function LvlWizRev_EstXpTime()
local lvltime = LevelTime
local UXP=UnitXP("PLAYER")
ET = UXP / lvltime * 3600
print (ET)
--if ET==0 then ET="inf."end
--LvlWizRev_Tooltip:AddLine("|cFF6495EDEstimated time to level up: "..ET.."|r")

end

this is the function i'm having issues with and it tells me that lvltime is the problem "string"

    elseif (event == "TIME_PLAYED_MSG") then
          -- Remember play time
          AllTime = SecondsToTime(a1);
       LevelTime = SecondsToTime(a2); 

is what's in my Onevent function that grabs the initial data

and these are my session variables(note some of them actually dont do anything i need to clean these up lol):

local LevelTime =0
local lvltime = 0
local LWR_currXP = 0
local LWR_exptogo = 0
local LWR_Init = nil
local WML = 0
local NUMMOBS = 0
local TPL = 0
local TX = 0; --(total amount of xp needed to level)--
local UXP = 0; --(current amount of xp aquired)--
local PT = 0 --(current amount of played time)--
local lxp = {} --(current player XP)--
local mkc=0  --(kills needed on specific mob to level up)--
local lastkillXP=0 --(last kill xp reward)--
local TL= {} --(level of the current target selected)--
local LWR_prevXP=0
PLT = 0
local prev_update = 0
local prev_update_delay = 4
local LvlWizRev_Update_Freq = 1

then finally it all is supposed to wrap in nice and neatly into my LvlWizRev_ShowTooltip() function which works for everything else. i've been over this bit of code trying to tweak and change it but i usually end up with a string error or cant do an upvalue on nil, so i'm lost lol any help is greatly appreciated! :)

LevelTime = SecondsToTime(a2);

SecondsToTime returns a string. SecondsToTime(3600) returns "1 Hr" (in enUS). So you should set lvltime to a2.

thanks for the help guys, i switched the LevelTime call to just = a2 then made a secondary call in the function to convert to SecondsToTime() to get a time format. i also found a way to add more lines into it for a more precise look :)

i guess i shouldof figured SecondsToTime() was a string lol :P

just out of curiosity, is it possible to pull a number out of a string and use the number value in math? i tried like in the following but i never get an exact number, just a lower number

X = string.find("some number of 45", "%d+")

but when i call that it doesnt return the number 45 it returns something less which makes it no good for use in math if i need the number in question to be pulled

FYI: string.match() (and string.find() for that matter) always returns a string. So if you need to do arithmetic on the returned value, you need to pass it to tonumber() (e.g. x = tonumber(x)).

ahh i see... another thing that is puzzling me that i was looking up in the lua refrence manual and i cant seem to find... what is _, ? is that like ome unused variable?

Quote from OrionShock

the underscore _ is used as the common "i don't care about that return" variable. Just remember to use a local _ not the global _ ( IE [[ local _, a = someFunc() ]] vs [[ _, a = sumfunc() ]] )

I've adopted using $_ in my PHP programming. lol

understood... thanks for the info guys! :)

i thought i'd post my finished line of code here for research purposes for anyone else that might be a bit confounded as i was.

in Levelwiz_revamped.lua:

        elseif (event == "CHAT_MSG_COMBAT_XP_GAIN") then
    local _,_,_,killEXP,_ = string.find(a1, LWR_STR_COMBATXPNOTE)
    if killEXP then LWR_prevXPkill = tonumber(killEXP) end
    if LWR_prevXPkill < 0 then LWR_prevXPkill = 0 end
 end

in my localization.lua

LWR_STR_COMBATXPNOTE    = "(.*) dies, you gain (%d+) experience."

it may very well be possible that i have more then one _ in the beginning of my call handler but after a lot of testing this is the string that worked for what i was needing called.

credit goes to the author of titanpanel XP of whom i got the idea that it was possible to seperate xp gains from kill gains and based the majority of the code off of.

thanks again for all your guys help! with this line completed i can officially call my addon ready!:D (even if it is just a rehash :P )

If you don't care about the first two things string.find returns, why not just use string.match?

LWR_STR_COMBATXPNOTE    = ".* dies, you gain (%d+) experience."

local killEXP = string.match(a1, LWR_STR_COMBATXPNOTE)