xscc Posted January 8, 2011 Share Posted January 8, 2011 Hello, I have a problem with this code: #include <WindowsConstants.au3> #include <StaticConstants.au3> $coord = PixelSearch( 0, 0, 20, 300, 0xFF0000) While 1 If Not @error Then MsgBox(0, "X and Y are:", $coord[0] & "," & $coord[1]) Else MsgBox (0, "Error", "errore") EndIf WEnd When I execute this code the console display: C:\Users\user\Desktop\autoit\Pixelsearch.au3 (6) : ==> Subscript used with non-Array variable.: MsgBox(0, "X and Y are:", $coord[0] & "," & $coord[1]) MsgBox(0, "X and Y are:", $coord^ ERROR >Exit code: 1 Time: 2.016 Link to comment Share on other sites More sharing options...
Moderators Melba23 Posted January 8, 2011 Moderators Share Posted January 8, 2011 xscc,Put the $coord = PixelSearch( 0, 0, 20, 300, 0xFF0000) line inside the loopAt the moment you are only testing the error from PixelSearch on the first pass - after that you are testing the error from MsgBox, which is usually 0, while $coord is probably not an array as the PixelSearch failed. #include <WindowsConstants.au3> #include <StaticConstants.au3> While 1 $coord = PixelSearch(0, 0, 20, 300, 0xFF0000) If Not @error Then MsgBox(0, "X and Y are:", $coord[0] & "," & $coord[1]) Else MsgBox(0, "Error", "errore") EndIf WEndNow you will get successive error MsgBoxes until the pixel appears in the area. 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...
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