FinalVersion Posted February 11, 2010 Share Posted February 11, 2010 #include <File.au3> Local $String, $Array, $Max $String = FileReadLine(@ScriptDir & "\Data.txt", 1) $Array = StringSplit($String, ".") $Max = UBound($Array) For $i = 1 To $Max Step +1 MsgBox(0, "", $Array[$i]) Next So i spotted this in the console C:\Users\Scott\Documents\AutoIt\Arrays\Hmm.au3 (10) : ==> Array variable has incorrect number of subscripts or subscript dimension range exceeded.: MsgBox(0, "", $Array[$i]) MsgBox(0, "", ^ ERROR Obviously it's not the end of the world, the script still runs fine, just curious about this. [center][+] Steam GUI [+][+] Clipboard Tool [+][+] System :: Uptime [+][+] StarCraft II Mouse Trap [+][/center] Link to comment Share on other sites More sharing options...
Moderators Melba23 Posted February 11, 2010 Moderators Share Posted February 11, 2010 Finalversion,Ah, this old chestnut again! Arrays in AutoIt start at element [0], so a 5 element array has [0], [1], [2], [3], [4]UBound, on the other hand, gives you the total number of elements, a 5 element array will return 5.Can you see where this is going yet? You have set your For...Next loop as 1 To $Max - when it should be 0 To $Max - 1.I know your first reaction will be: "Why the hell does UBound return 5!" - you are not alone! However, you are warned - from the Help file for UBound: "Remember that the value returned by UBound is one greater than the index of an array's last element!". Personally I would have called UBound something else, like NumElements, but then I did not write it. If it is any consolation, I still have to think very carefully about the upper and lower limits of array elements and often get my For...Next counter wrong on the first pass. You might find this topic of interest if you want to see a bit more confusion...... 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...
FinalVersion Posted February 11, 2010 Author Share Posted February 11, 2010 Yeah I remember reading that, so should this work? For $i = 1 to $Max -1 Step +1 [center][+] Steam GUI [+][+] Clipboard Tool [+][+] System :: Uptime [+][+] StarCraft II Mouse Trap [+][/center] Link to comment Share on other sites More sharing options...
omikron48 Posted February 11, 2010 Share Posted February 11, 2010 For $i = 0 to $Max -1 You skipped the [0] element, not unless that was intentional. Also, You don't need to specify the step as being +1. It is +1 by default. You only use Step when you need a different Step value. Link to comment Share on other sites More sharing options...
enaiman Posted February 11, 2010 Share Posted February 11, 2010 Yeah I remember reading that, so should this work?Would you like someone else to test the script for you?? ... by the way - with icecream on top or without? SNMP_UDF ... for SNMPv1 and v2c so far, GetBulk and a new example script wannabe "Unbeatable" Tic-Tac-Toe Paper-Scissor-Rock ... try to beat it anyway :) Link to comment Share on other sites More sharing options...
FinalVersion Posted February 11, 2010 Author Share Posted February 11, 2010 Ok it's cleared up. [center][+] Steam GUI [+][+] Clipboard Tool [+][+] System :: Uptime [+][+] StarCraft II Mouse Trap [+][/center] 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