Quinch Posted January 30, 2012 Share Posted January 30, 2012 What would be the best way to do it? I have a loop set up thusly - Func Inet_Cleanup() For $x = 1 To UBound($inet)-1 $inet_info = InetGetInfo($inet[$x], -1) ;If the download was complete and successful If $inet_info[2] = True And $inet_info[3] = True Then InetClose($inet[$x]) _ArrayDelete($inet, $x) $x = 1 ;If the download was complete and unsuccessful ElseIf $inet_info[2] = True And $inet_info[3] = False Then ConsoleWriteError("Download error: "&_ArrayToString($inet_info)&@CRLF) InetClose($inet[$x]) _ArrayDelete($inet, $x) $x = 1 EndIf Next EndFunc - but I eventually hit an array range exceeded error. Link to comment Share on other sites More sharing options...
ZacUSNYR Posted January 30, 2012 Share Posted January 30, 2012 (edited) Which makes sense. When you're going from 0 to MaxIndex and deleting, you're lowering the the index value of the array on each loop. Easiest solution, go backwards. #include <Array.au3> Dim $asInet[5] = ["1", "a", "2", "b", "3"] _ArrayDisplay($asInet) For $i = UBound($asInet) -1 To 0 Step -1 ConsoleWrite($asInet[$i] & @CRLF) _ArrayDelete($asInet, $i) Next If Not IsArray($asInet) Then ConsoleWrite("No longer an Array!") Edited January 30, 2012 by ZacUSNYR Link to comment Share on other sites More sharing options...
hannes08 Posted January 30, 2012 Share Posted January 30, 2012 (edited) Go from last to first element. For $x = UBound($inet)-1 To 1 Step -1 .... Next Edited January 30, 2012 by hannes08 Regards,Hannes[spoiler]If you can't convince them, confuse them![/spoiler] Link to comment Share on other sites More sharing options...
Quinch Posted January 30, 2012 Author Share Posted January 30, 2012 Ah - so the Ubound($inet) is evaluated once, not every time the loop runs? Link to comment Share on other sites More sharing options...
Moderators Melba23 Posted January 30, 2012 Moderators Share Posted January 30, 2012 Quinch,Yes, the limits are set by the For statement and are not revaluated during the loop. 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...
Quinch Posted January 30, 2012 Author Share Posted January 30, 2012 Alright, that's one mystery solved, then. Thanks for the help. 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