Subscript used with non-Array variable

[Medic873]

Okay here is my code

#include <IE.au3>
$ErrorCount = 0;

If WinExists("Courseware - Windows Internet Explorer") Then
WinActivate ("Courseware - Windows Internet Explorer")
WinSetState("Courseware - Windows Internet Explorer", "", @SW_MAXIMIZE)
while 1
    Sleep (10000)
    $coord = PixelSearch(941, 697, 988, 730, 0x003399)
    MouseClick("left", $Coord[0], $Coord[1], 1)

    if @error Then
    $ErrorCount = $ErrorCount + 1
    endif
    if $ErrorCount = 6 Then
        MouseClick("left", 973, 721, 1)
    endif


wend
Else
    MsgBox(16, "window Not Found", "Please start web browser")
endif

I cant figure out why this wouldnt be working

line nine im having a subscript error same as the title lol thank you for all of the help i have already recived.

[JohnOne]

lol

Just curious, whats so funny ?

Regarding your script, maybe you couls give us some indication as to what line 9 is and a hint as to how much vital code you have ommitted.

[Medic873]

$coord = PixelSearch(941, 697, 988, 730, 0x003399)

that is line nine

and im not to sure why it wont work

it works the first time but when repeating it dosnt process it correctly.

[JohnOne]

The only reason that is not an array is because it cannot find the color

MouseClick("left", $Coord[0], $Coord[1], 1)

if @error Then

$ErrorCount = $ErrorCount + 1

endif

I dont think MouseClick sets @error so it most probably pointless there

If you are trying to catch the Pixelsearch error then the check has to be imediately after it.

[ShawnW]

I don't have a way to test it since I am not sure what courseware is or what you are looking for but try moveing the MouseClick line to only happen if there is no error. If PixelSearch fails and sets @error to 1, the value returned to $coord might not be an array. Therefore when you access the subscript of $coord in the MouseClick line the program will error.

[ShawnW]

Also just a fyi, On single command If... statements you don't need to multi-line them and add the EnfIf. You can turn...

If @error Then
    $ErrorCount = $ErrorCount + 1
EndIf

...into...

If @error Then $ErrorCount = $ErrorCount + 1

[bo8ster]

Added check after PixelSearch. You could validate the $coord after PixelSearch is called because if it is invalid and you do '$coord[0]' it will fail, which is what you are seeing.

include < IE.au3 >
$ErrorCount = 0;

If WinExists("Courseware - Windows Internet Explorer") Then
    WinActivate("Courseware - Windows Internet Explorer")
    WinSetState("Courseware - Windows Internet Explorer", "", @SW_MAXIMIZE)
    While 1
        SetError(0) ; if you wanted to make sure it is reset every time.
        Sleep(10000)
        $coord = PixelSearch(941, 697, 988, 730, 0x003399)
        If Not @error Then
            MouseClick("left", $coord[0], $coord[1], 1)
        EndIf

        If @error Then
            $ErrorCount = $ErrorCount + 1
        EndIf
        If $ErrorCount = 6 Then
            MouseClick("left", 973, 721, 1)
        EndIf

    WEnd
Else
    MsgBox(16, "window Not Found", "Please start web browser")
EndIf