E2 Lua Error Help

[Officer Tibbles]

I'm working on a calculator that is based in my OS, and I got the code written out, but there's a problem.

I get these Lua errors..

entities\gmod_wire_expression2\core\number.lua:109: attempt to perform arithmetic on a nil value

Timer Error: entities\gmod_wire_expression2\core\number.lua:109: attempt to perform arithmetic on a nil value

The part of my code that does the calculations is as follows

if(Word=="-mult"&Key==13){
A=ceil((A-1)/30)*30+0
Busy=1
PrintColor=18
Print="Multiplication                Enter Numbers"
Word=""
Mult=1
}

if(Key==13&Mult==1){
A=ceil((A-1)/30)*30+0
PrintColor=666
Busy=1
Print = Answer
AN = Number * Number
Number = Word:toNumber()
Answer = toString(AN)
}

So, yea help will be appreciated.

[ZeikJT]

It seems like you are trying to multiply by a number that is nil.
I honestly thought it was impossible for that to happen in E2 as all number variables that get defined are supposed to start with a value of 0.

Make sure all the variables you are working with have valid values. Does Number have a value? It could be there when the multiplication gives an error. It might be this part too:
A=ceil((A-1)/30)*30+0

Which by the way is weird, why would you add 0? You don't need it.

I'll see if I can determine anything.

EDIT: Yeah, it seems that if a string is "" it can be converted to nil.

[Bull]

AN = Number * Number
Number = Word:toNumber()

Shouldn't those two be switched?

And you second comparison is run too early.

if(Word=="-mult"&Key==13){
A=ceil((A-1)/30)*30+0
Busy=1
PrintColor=18
Print="Multiplication Enter Numbers"
Word="" # Here you empty the string
Mult=1 # Here you set mult to 1
}

if(Key==13&Mult==1){ # So this comparison returns true
A=ceil((A-1)/30)*30+0
PrintColor=666
Busy=1
Print = Answer
AN = Number * Number
Number = Word:toNumber() # and this tries to convert a empty string to a number.
(Since you emptied the string earlier) Which gives the error probably.
Answer = toString(AN)
}

Why do you set the Print string to answer before calculating it?

[Whodunnit]

word = "" might count as nil ?

[Officer Tibbles]

AN = Number * Number
Number = Word:toNumber()

Shouldn't those two be switched?

And you second comparison is run too early.

if(Word=="-mult"&Key==13){
A=ceil((A-1)/30)*30+0
Busy=1
PrintColor=18
Print="Multiplication Enter Numbers"
Word="" # Here you empty the string
Mult=1 # Here you set mult to 1
}

if(Key==13&Mult==1){ # So this comparison returns true
A=ceil((A-1)/30)*30+0
PrintColor=666
Busy=1
Print = Answer
AN = Number * Number
Number = Word:toNumber() # and this tries to convert a empty string to a number.
(Since you emptied the string earlier) Which gives the error probably.
Answer = toString(AN)
}

Why do you set the Print string to answer before calculating it?

Wow, Thanks. I'll try that. I always thought that declaring Word as "" then it meant that nothing is changed to it.


And Zeik... The whole

A=ceil((A-1)/30)*30+0

Is just a formula, so if I wanted it to skip lines, I would change the +0 to +1, But I just Copy / Paste that thing in there.

[Syranide]

I've committed a fix that prevents toNumber() from returning nil, it returns 0 instead.