Make MouseClick click only once

[Sakkz]

I'm trying to make it click only once but it just keeps clicking, I'm new and I used examples to come up with this code but idk what is wrong

$left = 353
$bottom = 972
$right = 513
$top = 855
$color = 0xDD2E44

Global $Paused, $Runner
HotKeySet("{f7}", "TogglePause")
HotKeySet("{f8}", "Terminate")

Func TogglePause()
    $Paused = Not $Paused
    While $Paused
        Sleep(10)
        ToolTip('Script is "Paused"', 0, 0)
    WEnd
    ToolTip("")
EndFunc

Func Terminate()
    Exit 0
EndFunc

Sleep(2500)

while(true)
    $pix = PixelSearch($right, $bottom, $left, $top, $color)
    if Not @error Then
        MouseClick("left", $pix[0], $pix[1], 1, 0)
    EndIf
WEnd

 

Edited June 9, 2022 by Jos
added codebox

[Jos]

43 minutes ago, Sakkz said:

it click only once but it just keeps clicking,

Right ..  don't you think this has something to do with the While...Wend loop you have put in your script?
 

[Musashi]

 

41 minutes ago, Sakkz said:

I'm trying to make it click only once but it just keeps clicking

Then you should not run PixelSearch in a loop. Furthermore, the order of the parameters when calling Pixelsearch is wrong (take a look at the help).

 

[Nine]

@Musashi

The order provided within PixelSearch can be mixed :

Quote

The search direction varies as follows:
    Left-to-Right - left < right
    Right-to-Left - right < left
    Top-to-Bottom - top < bottom
    Bottom-to-Top - bottom < top
 

[Musashi]

2 hours ago, Nine said:

The order provided within PixelSearch can be mixed :

I am aware of that, but you are right - "wrong order of parameters" is not a correct statement. .

According to the help, Param1 is the left coordinate and Param3 the right one. The search direction can be reversed, if one sets the value of Param1(left) greater than the value of Param3(right). (Analog for Param2(top) and Param4(bottom)).

I just wanted to make sure, @Sakkz got this point right.

In the end, it's probably just a matter of personal preference whether you set :

$left = 353, $bottom = 972, $right = 513, $top = 855, $color = 0xDD2E44
$pix = PixelSearch($right, $bottom, $left, $top, $color)

or :

$left = 513, $top = 972, $right = 353, $bottom = 855, $color = 0xDD2E44
$pix = PixelSearch($left, $top, $right, $bottom, $color)

which fits at least better to the help, regarding the naming of the variables.

 

[Sakkz]

To explain what exactly I'm trying to do. In a discord group I'm in, the owner does these random giveaways which you have to react within like 5 seconds. (The winner gets their idea used for the owner to use as a rp character) and so I just want it to loop PixelSearch till it finds it and only clicks it once instead of it acting like an autoclicker. But if this sorta thing isn't allowed then I apologize

[Jos]

2 minutes ago, Sakkz said:

But if this sorta thing isn't allowed then I apologize

That is something you can determine yourself by taking the time to read or forum rules !