Why can't I save what's read from the file to a variable?

Solar2D General Questions/Discussion
#21

Yes, read my code, see how it works, see it working, experiment, learn.

http://github.com/roaminggamer/RG_FreeStuff/blob/master/AskEd/2016/03/ssk_core/ssk_core/extensions/io.lua

if( io.readFile ) then print("ERROR! io.readFile() exists already") else function io.readFile( fileName, base ) local base = base or system.DocumentsDirectory local fileContents if( io.exists( fileName, base ) == false ) then return nil end local fileName = fileName if( base ) then fileName = system.pathForFile( fileName, base ) end local f=io.open(fileName,"r") if (f == nil) then return nil end fileContents = f:read( "\*a" ) io.close(f) return fileContents end end if( io.writeFile ) then print("ERROR! io.writeFile() exists already") else function io.writeFile( dataToWrite, fileName, base ) local base = base or system.DocumentsDirectory local fileName = fileName if( base ) then fileName = system.pathForFile( fileName, base ) end local f=io.open(fileName,"w") if (f == nil) then return nil end f:write( dataToWrite ) io.close(f) end end

Sorry, but I gave you an alternate solution above and you rejected it as, “I tried that and it didn’t work…”.  So, the only alternative is for me to show you a working example which I’ve done.  

Unfortunately, I don’t have the time right now to take your code, run in locally, debug it, and give it back to you fixed, which I guess is the third option… :unsure:  

#22

@SonicX278
Inside your M.readFile you are setting _number as a local variable. This replaces the _number parameter you are passing when you call the function and you loose the reference.

But _number is neither a table nor a function: it’s a value, as correctly noted by @roaminggamer so i think you’d do better by returning the value from the M.readFile instead of passing it’s reference as a parameter

#23

I totally get wanting to learn and accomplish things, but at the same time you have to ask, “What is my goal here, learning or making a game?”

If it is learning, then that is awesome.  However, you can still learn by starting with a working base and tweaking till you break it or understand how it works.

#24

@RoamingGamer – I have to head off to church soon so ill take a hard look at the ssk after.

@g.sciacchitano – Did you mean something like this?

function M.readFile( \_fileName, \_number ) \_path = system.pathForFile( \_fileName, system.DocumentsDirectory ) \_file = io.open( \_path, "r" ) for line in \_file:lines() do \_number = tonumber(line) end io.close( \_file ) return \_number end

 local number NumberZ.readFile( "playerScore.txt", number ) print(number)

–SonicX278

#25

@sonic,

No that is wrong.  He noticed you were creating a local which overwrote _number, but you still can’t pass in _number as a reference.  It isn’t a table or object.

Listen, if you really want to learn this stuff focus on LUA then Corona.  Grab the PIL, read it, write the examples, you’ll learn that way for sure:

http://www.lua.org/pil/

Also, grab Brian’s book(s) and follow them to learn more:

http://www.burtonsmediagroup.com/books/

#26

I ended up doing this. It works now. 

function M.readFile( \_fileName ) \_path = system.pathForFile( \_fileName, system.DocumentsDirectory ) \_file = io.open( \_path, "r" ) for line in \_file:lines() do \_number = tonumber(line) end io.close( \_file ) \_file = nil \_path = nil return \_number end

 local number = NumberZ.readFile( "playerScore.txt" ) print(number)

I removed the the local variable from the readFile function and added “return _number”.

Hmm. Weird. @RoamingGamer – Your first method ended up working. But only after i removed the “local _number” from it and testing it and it worked. Then i added it in again and it ended working. 

Thanks @RoamingGamer!

–SonicX278 

#27

Also its not that i didn’t understand how this works and how to do it. It wasn’t working until i restarted my simulator. ( I have my PC on for weeks at a time and usually on a load )

Which results in my sim nitnworkkmg properly sometimes.

–SonicX278

#28

Hey.  I noticed you didn’t take my advice from the other post/thread re: nil-ing  and that there is a coding issue that could bite you later if you continue to do it.  You make some globals then nil them.   

This is better and more correct:

function M.readFile( \_fileName ) local \_path = system.pathForFile( \_fileName, system.DocumentsDirectory ) local \_file = io.open( \_path, "r" ) local \_number for line in \_file:lines() do \_number = tonumber(line) end io.close( \_file ) return \_number end
#29

So removing the nil’s and adding the _number variable will be a better practice of doing this?

–SonicX278

#30

Not creating globals for what is essentially local work, and using locals is a better practice, “Yes.”

Note: The locals fall out of scope at the end of the function and don’t need to be nil’d.

#31

Understood. Thanks!

–SonicX278