Simple Search

[Liner]

Working on autoit and c++.

My first autoit script is relatively simple, I'm trying to get it to:

When Mouse1 is pressed, move to a different position. In this case, anything that is bright green in color.

My script so far:

Func _IsPressed($hexKey)
; $hexKey must be the value of one of the keys.
; _IsPressed will return 0 if the key is not pressed, 1 if it is.
; $hexKey should entered as a string, don't forget the quotes!
; (yeah, layer. This is for you)
  
  Local $aR, $bO
  
  $hexKey = '0x' & $hexKey
  $aR = DllCall("user32", "int", "GetAsyncKeyState", "int", $hexKey)
  If Not @error And BitAND($aR[0], 0x8000) = 0x8000 Then
     $bO = 1
 EndIf
 EndFunc
; I DID NOT WRITE THIS USERDEF.

$Color = "0x48B748"
$Search = PixelSearch (1000, 1000, 1000, 1000, $Color, 10)
If _IsPressed('01') = 1 then
sleep (50)
Mouseclick ("left", $Search)

IsPressed is a user defined function that I got from somewhere else. The function definition code I cannot vouch for, I don't know if it works, but from the research I've done it looks like it should. TBH, I'm assuming it is.

Here's the sequence I have layed out:

Set the color

Set search parameters

if mouse1 is pressed, then

wait 50

on mouseclick, move to position determined by Search.

When I build, I get no errors.

Running this script, I notice 2 things:

Nothing happens. Mouse1 click with the color on the screen does nothing.

The autoit icon does not appear in the taskbar. I'm guessing this means the script is not running, did not run at all, or only ran for a fraction of a second.

There's probably an obvious logic error, but I can't see it. Any ideas?

[kaotkbliss]

couple of things

your script is running for only a fraction of a second because there is no loop to keep it running (so as soon as it checks and finds nothing, it exits)

pixelsearch is only searching a single pixel on the screen. it works like:

PixelSearch(x of top left corner of search area,y of top left corner of search area, x of bottom right corner, y of bottom right corner, color)

pixelsearch, if found returns an array so the line

Mouseclick ("left", $Search)

needs to be changed to

Mouseclick ("left", $Search[0],$Search[1])

and unless there is more to the code, you are missing an endif at the end

[PsaltyDS]

Try:

$Color = 0x48B748

Without the quotes. You want a literal INT32, not a string.

The function _IsPressed() is included in the Misc.au3 standard UDF that ships with AutoIt. If you don't know that, you haven't bothered with the help file much yet...

[Liner]

The function _IsPressed() is included in the Misc.au3 standard UDF that ships with AutoIt. If you don't know that, you haven't bothered with the help file much yet...

I obviously didn't discover it in the help file, but I used it first. It wasn't a command listed in the yellow cursor function list either. I got it off of a forum. He said someone wrote it, if that person was someone from autoit, I didn't know.

I'd never have figured out the quote problem. But regardless, tbh I rarely resort to the help files, perhaps I need to make it a habit :/

kaotkbliss - I was planning to add more to the code, I didn't include endif. And of course, it does needs more very helpful

Edited March 11, 2010 by Liner

[kaotkbliss]

One problem I found with the help file, if you don't know the exact name of the function and don't type it in exactly, you aren't going to find it under the search tab. ispressed will not return the function for _ispressed

[Liner]

One problem I found with the help file, if you don't know the exact name of the function and don't type it in exactly, you aren't going to find it under the search tab. ispressed will not return the function for _ispressed

That makes me feel better, ha

I'm updating the code right now, so I'll edit this post when I'm done.