confucion Posted February 4, 2011 Share Posted February 4, 2011 Is this usage correct? While 1 $tmp = Int("abc") If @error = 1 Then ExitLoop Wend The loop is exited upon first run? Link to comment Share on other sites More sharing options...
jvanegmond Posted February 4, 2011 Share Posted February 4, 2011 (edited) Is this usage correct? While 1 $tmp = Int("abc") If @error = 1 Then ExitLoop Wend The loop is exited upon first run? Usage is correct. Behavior is also correct. Int tries to turn "abc" into an integer. Since it can't make any sense of it (there are no numbers, could you make sense of it?) it will error. Upon error your script exits. Edited February 4, 2011 by Manadar github.com/jvanegmond Link to comment Share on other sites More sharing options...
Moderators Melba23 Posted February 4, 2011 Moderators Share Posted February 4, 2011 confucion,Why should it not exit? The string "abc" is not an integer and so the function will return 0 and set @error to 1 just as the Help file says. If you are referring to the remainder of the Help file line where it says "if not an integer, float or string" then you need to understand that the function will always fail unless the value is in integer, float or string format. Look at the results from this:ConsoleWrite(Int(1) & @CRLF) ;integer format ConsoleWrite(Int(1.0) & @CRLF) ; float format ConsoleWrite(Int("1") & @CRLF) ; string formatYou can see that they all return 1 saying that the item is an integer.All clear? M23 Any of my own code posted anywhere on the forum is available for use by others without any restriction of any kind Open spoiler to see my UDFs: Spoiler ArrayMultiColSort ---- Sort arrays on multiple columnsChooseFileFolder ---- Single and multiple selections from specified path treeview listingDate_Time_Convert -- Easily convert date/time formats, including the language usedExtMsgBox --------- A highly customisable replacement for MsgBoxGUIExtender -------- Extend and retract multiple sections within a GUIGUIFrame ---------- Subdivide GUIs into many adjustable framesGUIListViewEx ------- Insert, delete, move, drag, sort, edit and colour ListView itemsGUITreeViewEx ------ Check/clear parent and child checkboxes in a TreeViewMarquee ----------- Scrolling tickertape GUIsNoFocusLines ------- Remove the dotted focus lines from buttons, sliders, radios and checkboxesNotify ------------- Small notifications on the edge of the displayScrollbars ----------Automatically sized scrollbars with a single commandStringSize ---------- Automatically size controls to fit textToast -------------- Small GUIs which pop out of the notification area Link to comment Share on other sites More sharing options...
confucion Posted February 4, 2011 Author Share Posted February 4, 2011 Because when I execute the code above, my CPU is stuck at 100% and the script doesn't quit indicating it is in infinite loop. Have you tried running the above code? Link to comment Share on other sites More sharing options...
jvanegmond Posted February 4, 2011 Share Posted February 4, 2011 Because when I execute the code above, my CPU is stuck at 100% and the script doesn't quit indicating it is in infinite loop. Have you tried running the above code? You're right. It can convert any string format. If there are no numbers available then 0 is assumed. One of the things it fails at if you try to pass a pointer type: While 1 $tmp = Int(Ptr(0)) If @error = 1 Then ExitLoop Wend github.com/jvanegmond Link to comment Share on other sites More sharing options...
Moderators Melba23 Posted February 4, 2011 Moderators Share Posted February 4, 2011 confucion,That caught us all out! Int failing on a string was reported as a bug (#1592) and dismissed as "No Bug". The trick is to do this:$sString = "abc" While 1 If Not (IsNumber($sString) And Int($sString)) Then ExitLoop WendNow you only test for an integer if the string is recognised as a number. M23 Any of my own code posted anywhere on the forum is available for use by others without any restriction of any kind Open spoiler to see my UDFs: Spoiler ArrayMultiColSort ---- Sort arrays on multiple columnsChooseFileFolder ---- Single and multiple selections from specified path treeview listingDate_Time_Convert -- Easily convert date/time formats, including the language usedExtMsgBox --------- A highly customisable replacement for MsgBoxGUIExtender -------- Extend and retract multiple sections within a GUIGUIFrame ---------- Subdivide GUIs into many adjustable framesGUIListViewEx ------- Insert, delete, move, drag, sort, edit and colour ListView itemsGUITreeViewEx ------ Check/clear parent and child checkboxes in a TreeViewMarquee ----------- Scrolling tickertape GUIsNoFocusLines ------- Remove the dotted focus lines from buttons, sliders, radios and checkboxesNotify ------------- Small notifications on the edge of the displayScrollbars ----------Automatically sized scrollbars with a single commandStringSize ---------- Automatically size controls to fit textToast -------------- Small GUIs which pop out of the notification area Link to comment Share on other sites More sharing options...
confucion Posted February 4, 2011 Author Share Posted February 4, 2011 Just found out I was using Int incorrectly to validate if the variable value was a string or number.Use IsNumber instead in these situations.While 1 If IsNumber("abc") = 0 Then ExitLoop WendThis works. 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