12 replies to this topic

[theeboris]

Hello I have a problem. If I use

if http.request("http://lightos.webuda.com/lightos/version.inf") >= version then
version = false
end

I got the error 'startup:20: attempt to compare nil with number'. How can I fix this?

[Bubba]

http.request does not return a value. You'll need to use events to capture the http_success or http_failure, as well as the web function table.

http.request("http://www.google.com/")
local e = {os.pullEvent()}
if e[1] == "http_failure" then
   print("Uh oh! Couldn't connect to google!")
   return
end
local content = e[3].readAll()
e[3].close()
print(content)

I also noticed that you are trying to compare the result of an http request (a string) directly to a number. You need to convert the string into a number with tonumber(content).

[theeboris]

I have

local version = http.get("http://lightos.webuda.com/lightos/version.inf")
write(version.readAll())
a = version.readAll()
version.close()
if tonumber(a) >= 011 then
uptodate = false
en

now I got 'startup:23: attempt to compare nil with number'

[bjornir90]

I think it's because a is nil

[remiX]

Remove the write(version.readAll()) and then it will work

[theeboris]

Thanks remix! It works.

[Bubba]

Just in case you were wondering why it didn't work, you used http.get as opposed to http.request which is the one I used. They return different values.

[theeboris]

Thanks bubba. Now it understan it.

[remiX]

Bubba, on 04 March 2013 - 05:33 AM, said:

Just in case you were wondering why it didn't work, you used http.get as opposed to http.request which is the one I used. They return different values.

No, the problem is that doing readAll() causes it go be at the very last line (which is nil if read) and then he does another readAll() which returns nil.
It's like doing readLine() and then readLine() again - the second readLine() will read the second line and not the first.

[Bubba]

remiX, on 04 March 2013 - 06:01 AM, said:

Bubba, on 04 March 2013 - 05:33 AM, said:

Just in case you were wondering why it didn't work, you used http.get as opposed to http.request which is the one I used. They return different values.

No, the problem is that doing readAll() causes it go be at the very last line (which is nil if read) and then he does another readAll() which returns nil.
It's like doing readLine() and then readLine() again - the second readLine() will read the second line and not the first.

Ah yes. My mistake. Please ignore my response: foolish http noob here.

[remiX]

Bubba, on 04 March 2013 - 06:27 AM, said:

remiX, on 04 March 2013 - 06:01 AM, said:

Bubba, on 04 March 2013 - 05:33 AM, said:

Just in case you were wondering why it didn't work, you used http.get as opposed to http.request which is the one I used. They return different values.

No, the problem is that doing readAll() causes it go be at the very last line (which is nil if read) and then he does another readAll() which returns nil.
It's like doing readLine() and then readLine() again - the second readLine() will read the second line and not the first.

Ah yes. My mistake. Please ignore my response: foolish http noob here.

Not only http Also with fs/io api for opening files for reading

[Bubba]

remiX, on 04 March 2013 - 06:29 AM, said:

Not only http Also with fs/io api for opening files for reading

Nah I'm perfectly fine with the fs api. I just missed the fact that he used f.readAll() twice (didn't read through it very carefully).
Thank you for being so quick to confirm my idiocy though XD

[oeed]

You may want to make your title a bit more precise in future... '[Error]' isn't exactly descriptive.