Runner Posted January 22, 2014 Share Posted January 22, 2014 I can't figure for the life of me why I keep getting a zero result for line 10 in the following code?Local $arr[15] Local Const $sFilePath = "C:\IRTest2.txt" Local $hFileOpen = FileOpen($sFilePath, 0) ; handle for file $sFileRead = FileRead($hFileOpen) FileClose($hFileOpen) Local $arr = StringSplit($sFileRead, "," & @LF) ConsoleWrite($arr[3] - $arr[1])IRTest2.txt file is this:"01348","0","01355","0","01473","1","01498","0","01533","1","01558","0","01593","1" If I change the ' - ' to '&' in line 10 I get this output:> "01355""01348". So the variables have the right values.But this works:Local $arr[20] $arr[11] = "01348" $arr[13] = "01355" ConsoleWrite($arr[13] - $arr[11])Output for this one is 7. So what am I missing? I've tried using Number($arr[3]... ,etc still returns zero. Link to comment Share on other sites More sharing options...
Solution DW1 Posted January 22, 2014 Solution Share Posted January 22, 2014 (edited) Remove the quotes from the file when you read it (or elsewhere in the script) so that the variable isn't forced to be treated as a string. For example, replace your FileRead with: $sFileRead = StringReplace(FileRead($hFileOpen), '"', '') EDIT: If it is still not clear why your second test example worked but the first script reading the file failed, consider that keeping the quotes when you read the file would actually make the second example look like the following snippet and also fail (when you treat the strings as numbers in the example, they are equal to zero): Local $arr[20] $arr[11] = '"01348"' $arr[13] = '"01355"' ConsoleWrite($arr[13] - $arr[11]) Edited January 22, 2014 by DW1 AutoIt3 Online Help Link to comment Share on other sites More sharing options...
BrewManNH Posted January 22, 2014 Share Posted January 22, 2014 Your numbers have quotes around them in the array. They're seen as strings because of them, and string minus string equals 0. If I posted any code, assume that code was written using the latest release version unless stated otherwise. Also, if it doesn't work on XP I can't help with that because I don't have access to XP, and I'm not going to.Give a programmer the correct code and he can do his work for a day. Teach a programmer to debug and he can do his work for a lifetime - by Chirag GudeHow to ask questions the smart way! I hereby grant any person the right to use any code I post, that I am the original author of, on the autoitscript.com forums, unless I've specifically stated otherwise in the code or the thread post. If you do use my code all I ask, as a courtesy, is to make note of where you got it from. Back up and restore Windows user files _Array.au3 - Modified array functions that include support for 2D arrays. - ColorChooser - An add-on for SciTE that pops up a color dialog so you can select and paste a color code into a script. - Customizable Splashscreen GUI w/Progress Bar - Create a custom "splash screen" GUI with a progress bar and custom label. - _FileGetProperty - Retrieve the properties of a file - SciTE Toolbar - A toolbar demo for use with the SciTE editor - GUIRegisterMsg demo - Demo script to show how to use the Windows messages to interact with controls and your GUI. - Latin Square password generator Link to comment Share on other sites More sharing options...
Runner Posted January 22, 2014 Author Share Posted January 22, 2014 But it works in the last example? Why? Link to comment Share on other sites More sharing options...
DW1 Posted January 22, 2014 Share Posted January 22, 2014 But it works in the last example? Why? As I added to my post, because you are not keeping quotes in the second script, forcing the variable to be read as a string. See my edited post above. AutoIt3 Online Help Link to comment Share on other sites More sharing options...
Runner Posted January 22, 2014 Author Share Posted January 22, 2014 Excellent! I've got it now.Thanks for the added edit that made it clear. Link to comment Share on other sites More sharing options...
Recommended Posts
Create an account or sign in to comment
You need to be a member in order to leave a comment
Create an account
Sign up for a new account in our community. It's easy!
Register a new accountSign in
Already have an account? Sign in here.
Sign In Now