"Do until IsArray() = true" is not working as intented.

[kakarotsaiyan]

(Here is the code) Everything works fine except it keeps sending "4" and doesn't stop when there is a pixel change. What am i doing wrong?

While 1
                Send("al")
                Send("{ENTER}")
                Sleep(1000)
                $crystal = PixelSearch( 385, 635, 400, 650, 0xFAA61A)
                If IsArray($crystal) = True Then
                   Send("al")
                   Send("{ENTER}")
                   Sleep(1000)
                EndIf
                Send("4")
                Send("{ENTER}")
                Sleep(1000)
                $success = PixelSearch( 385, 555, 400, 575, 0x00FF00)
                If IsArray($success) = False Then
                   Do
                      Send(" 4")
                      Send("{ENTER}")
                      Sleep(1000)
                   Until IsArray($success) = True
                EndIf
            WEnd

[water]

Welcome to AutoIt and the forum!

Using Pixelsearch and Send is not very reliable (as you have noticed) as it depends on screen size, window position, screen resolution etc. Plus your script might interfere with user interaction.
Please tell us which application you try to automate. Most of the time there are more reliable ways to do what you want to do

[kakarotsaiyan]

hey thanks, i'm using it for chrome for my website. i been using send for a few scripts as controlsend doesn't work for chrome. Pixelsearch is what i use for my previous scripts as i need to detect color changes. could you help me with these? 

[water]

To automate Chrome i suggest to use the WebDriver UDF:

 

[kakarotsaiyan]

k thank you, i use that now- what a relief but does the window needs to be active for pixel search and pixel get color if i use that UDF? and could u help me with my code that i mentioned?

[Earthshine]

you won't need pixelsearch if you use that udf. that is the help

Edited July 24, 2018 by Earthshine

[kakarotsaiyan]

oh, k i'll try it and get back to u guys.