Image Search Problem?

[Reptos]

Why is it giving me and error when i do the ImageSearch, The bmp is in the same location as the Script so why is it doing this?

 

Error Below

$result = _ImageSearch('gift.bmp',1,$x,$y,100)
$result = _ImageSearch('gift.bmp',1,^ ERROR

Script Below 

#include <ImageSearch.au3>


HotKeySet("{ENTER}","start")





While 1
   Sleep(1)
   WEnd





Func start()
   While 1
     $result = _ImageSearch('gift.bmp',1,$x,$y,100)
      If $result = 1 Then
         MouseClick("Left",$x,$y,1,10)
      EndIf

   WEnd
EndFunc

 

 

Edited December 31, 2015 by Melba23
Added tags

[BrewManNH]

$x and $y aren't declared.

[Reptos]

$x and $y aren't declared.

Okay so i did 

$x = 0
$y = 0

Still having same error

[BrewManNH]

What error are you getting?

[Reptos]

What error are you getting?

$result = _ImageSearch("gift.bmp",1,$x,$y,100)
$result = _ImageSearch("gift.bmp",1,^ ERROR

[BrewManNH]

That's the entire error message?

[Reptos]

That's the entire error message?

Sorry 

 

"C:\Users\Owner\Desktop\Autoit Stuff\Ourworld\BMP\nestin.au3" (17) : ==> Variable used without being declared.:
$result = _ImageSearch("gift.bmp",1,$x,$y,100)
$result = _ImageSearch("gift.bmp",1,^ ERROR
>Exit code: 1    Time: 2.769

 

[Jos]

So what about showing the script that gets the error with the lines defining $x and $y  as you clearly have done something wrong?

Jos

 

[Reptos]

This is the whole Script : 

#include <ImageSearch.au3>

HotKeySet("{ENTER}","start")




While 1
   Sleep(1)
   WEnd



$x = 0
$y = 0

Func start()
  While 1
      WinActivate("ourWorld")
     $result = _ImageSearch("gift.BMP",1,$x,$y,100)
      If $result = 1 Then
         MouseClick("Left",$x,$y,1,10)
      EndIf

   WEnd
EndFunc

 

Edited December 31, 2015 by Melba23
Removed quote and added tags

[Jos]

So did you try to understand how your script is running and what would be a logical location to put those 2 lines?
Actually the whole script doesn't make any sense looking at the logic.

Jos

[Reptos]

Jos, Why don't you help me understand instead of always trying to be a smart ass?

Edited December 31, 2015 by Melba23
Removed quote

[Reptos]

Obviously i don't understand so why ask me... That is kinda why i'm here FOR HELP! Come'on Jos use your brain.

Edited December 31, 2015 by Melba23
Removed quote

[BrewManNH]

Scripts are ran from top to bottom, unless there's a function call in between.

In your script, starting from the top, it goes like this.

; step 1 add the contents of the ImageSearch UDF and execute anything that's not contained inside a function
#include <ImageSearch.au3>
; step 2 Set the hotkey
HotKeySet("{ENTER}", "start")


; Step 3 Start the While loop
While 1
; Step 4 Sleep for 10ms (the minimum amount you can sleep for)
    Sleep(1)
; Step 5 Go back to the top of the While loop (Step 3)
WEnd
; Step 6 There is no step 6 because you're stuck inside the While loop
$x = 0
$y = 0
; Function called by the HotKey
Func start()
    ; Step 1 Start the While loop
    While 1
        ; Step 2 Wait for the windows titled "ourWorld" to be the active window - You may never leave this step if the window never becomes active
        WinActivate("ourWorld")
        ; Step 3 Search for the image "gift.bmp" and assign the return to $result
        $result = _ImageSearch("gift.BMP", 1, $x, $y, 100)
        ; Step 4 Check to see if $result is -1
        If $result = 1 Then
            ; Step 5 If $result = -1 then click the left mouse button at $x and $y
            MouseClick("Left", $x, $y, 1, 10)
        EndIf
        ; step 6 Go back to step 1 forever.
    WEnd
EndFunc   ;==>start

2 errors in your script, maybe 3, First $x and $y aren't declared because they're after the first While loop, Second you have no way to get out of the function call because it's running another endless loop there. 3rd optionally, you don't put a timeout on the WinWaitActive so it could be sitting there forever before the imagesearch ever runs. A 4th optional error is that you have no way of stopping the script from running for as long as the computer is on.

[Reptos]

Thank you for actually taking to time to explain! I think i got it now

Edited December 31, 2015 by Melba23
Removed quote

[Reptos]

The error im getting now is 

expandcollapse popup

if $result[0]="0" then return 0
if $result^ ERROR





; step 1 add the contents of the ImageSearch UDF and execute anything that's not contained inside a function
#include <ImageSearch.au3>
; step 2 Set the hotkey
HotKeySet("{ENTER}", "start")


; Step 3 Start the While loop

; Step 6 There is no step 6 because you're stuck inside the While loop

          $x = 0
         $y = 0

While 1
   Sleep(1)
   WEnd

; Function called by the HotKey

Func start()
    ; Step 1 Start the While loop
    While 1
        ; Step 2 Wait for the windows titled "ourWorld" to be the active window - You may never leave this step if the window never becomes active
        WinActivate("ourWorld")

        ; Step 3 Search for the image "gift.bmp" and assign the return to $result
        $result = _ImageSearch("gift.BMP", 1, $x, $y, 100)
        ; Step 4 Check to see if $result is -1
        If $result = 1 Then
            ; Step 5 If $result = -1 then click the left mouse button at $x and $y
            MouseClick("Left", $x, $y, 1, 10)
        EndIf

        ; step 6 Go back to step 1 forever.
    WEnd

EndFunc   ;==>start

 

Edited December 31, 2015 by Melba23
Removed quote and added tags

[Melba23]

Reptos,

When you post code please use Code tags - see here how to do it.  Then you get a scrolling box and syntax colouring as you can see in the other posts.

And when you reply, please use the "Reply to this topic" button at the top of the thread or the "Reply to this topic" editor at the bottom rather than the "Quote" button - we know what we wrote and it just pads the thread unnecessarily.

Thanks for your cooperation,

M23

 

 

[BrewManNH]

Please use code tags when posting code, it looks like "<>" in the tool bar of the reply screen.

Also, don't quote what I posted, I know what I wrote and it's harder to follow the thread with unnecessary quoting.

 

That error you posted looks like it's coming from the UDF and not your script. Try adding a local declaration in front of your $result variable, as you may be overwriting an array inside the UDF because you don't declare your variables properly.

[Jos]

Obviously i don't understand so why ask me... That is kinda why i'm here FOR HELP! Come'on Jos use your brain.

I am sorry, but you have been in this mode since the day to signed up 10 days ago, and are not showing any initiative to understand what you are doing and seems to keep on asking the same thing.

Start reading and learning and you will get help.

Jos

[Reptos]

I am sorry, but you have been in this mode since the day to signed up 10 days ago, and are not showing any initiative to understand what you are doing and seems to keep on asking the same thing.

Start reading and learning and you will get help.

Jos

Jos you're those type of people that think they are Mr know it all. I came here to get help. I'm new at all of this literally. You stated " are are not showing any initiative " please choose your words wisely because that did not make any since. Maybe you need to "Start Reading" and learn how to type correctly. Now my intention was not to come here and argue it was simply to come and learn things that i did not know. Some people have certain learning deference's, you may learn quicker than i do that is why i keep asking for help maybe on the same topic. I'm done arguing with you for now have a great day!

[Jos]

Good to see you are taking the time to read and properly articulate your sentences. As to the content of it:  It is clear you do not know what you are saying but will not hold it against you.

Good luck with your scripting endeavors.

Jos