Simple Looping Mouseclick

[PlasticJesus]

Hi all,

I have been trying to get a simple looping mouseclick script to work depending on the quantity you put into a popup box. I would also like the escape key to be a fallback to exit the loop as well as the count. I am getting errors when trying to run the .exe once compiled.

AutoIt Error
Line 4 (File "...\gomouse.exe):
Error: Unknown function name.

Here is the file

$Main = GUICreate("Enter Desired Clicks", 100, 100, 810, 200)
$Input1 = GUICtrlCreateInput("0", 40, 40, 33, 21)
$Go = GUICtrlCreateButton("Go", 40, 64, 33, 17, 0)
GUISetState(SW_SHOWNORMAL)

HotKeySet("F1", "_Mouse1")
HotKeySet("{ESC}", "_Escape")

While 1
   $nMsg = GUIGetMsg()
   Switch $nMsg
      Case $GUI_EVENT_CLOSE
         Exit
      Case $Go
         _Mouse1()
   EndSwitch
WEnd

Func _Mouse1()
   Local $Times
   Do
      $Times += 1
      MouseClick("left")
      Sleep(1000)
   Until $Times = GUICtrlRead($Input1) Or $Escape = 1
EndFunc

Func _Escape()
   $Escape = 1
EndFunc

[water]

Welcome to AutoIt and the forum!
Is this script for learning purpose or do you want to solve a real problem? As you do not specify a mouse position it clicks whereever the mouse is at the moment you run your script.

[water]

Your error is caused by the missing @ in @SW_SHOWNORMAL.

[PlasticJesus]

Ok, caught a couple mistakes but I still can't seem to get it completed. Once compiled now it does not stop the mouse click loop if it reached the number entered in the GUI box, it also does not exit if I press the window close button however pressing escape closes the window and program. Not sure what I am still missing to finish this script. Any help is appreciated!

#include <GUIConstantsEx.au3>

$Main = GUICreate("Enter Desired Clicks", 100, 100, 810, 200)
$Input1 = GUICtrlCreateInput("0", 40, 40, 33, 21)
$Go = GUICtrlCreateButton("Go", 40, 64, 33, 17, 0)
GUISetState()
Local $Times = 0
Local $Escape = 0
HotKeySet("F1", "_Mouse1")
HotKeySet("{ESC}", "_Escape")

While 1
   $nMsg = GUIGetMsg()
   Switch $nMsg
      Case $GUI_EVENT_CLOSE
         Exit
      Case $Go
         _Mouse1()
   EndSwitch
WEnd

Func _Escape()
   $Escape = 1
EndFunc

Func _Mouse1()
   Do
      $Times += 1
      MouseClick("left")
      Sleep(1000)
   Until $Times = GUICtrlRead($Input1) Or $Escape = 1
EndFunc

[water]

Could you please anwer my question in my first post?

[Jos]

It works one time as you do not reset $Escape=0 when you have hit ESC, so the second time it will Escape right away.

This however doesn't close your script but returns to its GUI.

Jos

 

 

 

[water]

BTW: Could you please enclose AutoIt code in code tags (the "<>" icon in the editor)? That greatly enhances readability

[SadBunny]

• Clicking the close button will not work while the script is in the loop inside _Mouse1(), as you are not polling GUIGetMsg there. The message will get registered as soon as you leave the loop. I don't know if you meant it like that.
• Like Jos said, reset your $Escape, otherwise all runs after the first will only have exactly 1 loop iteration.
• Even then, $Times is a global variable and just keeps on counting upwards. So if you reset $Escape, the script will stay in the loop forever after the first run. Meaning: also reset your $Times counter before you enter that loop.

[PlasticJesus]

Could you please anwer my question in my first post?

​Hi Water,

The mouse function should be clicking statically on the screen where the user selects it's location.

[water]

I can't imagine where such a script could be useful Clicking n times with a break of 1 second. Why wouldn't the user click himself?
Is this for a game?

[PlasticJesus]

I can't imagine where such a script could be useful Clicking n times with a break of 1 second. Why wouldn't the user click himself?
Is this for a game?

​Yes, needing to open a specific amount of items prior to running out, if it runs out then you start to purchase.

[Melba23]

PlasticJesus,

Welcome to the AutoIt forum.

Unfortunately you appear to have missed the Forum rules on your way in. Please read them now - particularly the bit about not discussing game automation - and then you will understand why you will get no help and this thread will now be locked.

See you soon with a legitimate question I hope.

M23