TrashBoat Posted March 7, 2019 Posted March 7, 2019 Im trying to iterate through a 2d array created by ProcessList() function that returns a 2d array containing names and pid's, i wanted to filter through the names and delete entries that have "svchost.exe" and this is how i did it Func _Sanitize($array) For $x = 1 To UBound($array) - 1 ConsoleWrite($array[$x][0] & @CRLF) If $array[$x][0] == "svchost.exe" Then _ArrayDelete($array, $x) EndIf Next EndFunc ;==>_Sanitize And i get this error message Quote Array variable has incorrect number of subscripts or subscript dimension range exceeded.: If $array[$x][0] == "svchost.exe" Then If ^ ERROR What is wrong?
Subz Posted March 7, 2019 Posted March 7, 2019 You need to go backwards in your loop not forward using Step - 1, although you could also just use: #include <Array.au3> Local $aProcesses = ProcessList("svchost.exe") _ArrayDisplay($aProcesses) TrashBoat 1
BrewManNH Posted March 7, 2019 Posted March 7, 2019 If you're going to delete elements of an array in a loop, you have to run the loop in reverse. Instead of For $x = 1 To UBound($array) - 1 Use this For $x = UBound($array) - 1 To 1 Step - 1 TrashBoat 1 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
TrashBoat Posted March 7, 2019 Author Posted March 7, 2019 Oh it makes sense, i get it now because i am deleting entries my array is getting smaller and the for loop will fail because it wont have those entries at the bottom anymore. Thanks for the help.
Dionysis Posted March 7, 2019 Posted March 7, 2019 (edited) What @Subz and @BrewManNH said, and change this If $array[$x][0] == "svchost.exe" Then ; AutoIT uses only single "=" for "equals" ;change it to this If $array[$x][0] = "svchost.exe" Then Edited March 7, 2019 by Dionysis
TrashBoat Posted March 7, 2019 Author Posted March 7, 2019 @Dionysis Yeah im learning c++ and got that habit from there, but it seems to work anyway.
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