My Coding does NOTHING when I press the hotkey

[thatboy]

HotKeySet('h', 'Hotkey1')

Func Hotkey1()
    ConsoleWrite('The Hotkey was pressed' & @CRLF)
EndFunc     ;==>Hotkey1

While 1
    Sleep(50)
WEnd

Func MC()
    Sleep(10)
    MouseClick("LEFT", 702. 953, 1, 1)
EndFunc

 

 

When I hit the Hotkey, it confirms that I pressed the Hotkey, but it does not Mouseclick.  I'm new to this.  What have I done wrong?  Thanks!

[water]

Where in your code do you call function MC?

[jchd]

41 minutes ago, thatboy said:

but it does not Mouseclick.

Why should it? You don't ask for that.

BTW do you realize that discussing automating games is forbidden here?

[thatboy]

9 minutes ago, water said:

Where in your code do you call function MC?

 Am I supposed to do "Func Mouseclick" Instead?

[thatboy]

10 minutes ago, jchd said:

Why should it? You don't ask for that.

BTW do you realize that discussing automating games is forbidden here?

 

I'm not using it to play Video Games, but I appreciate your very helpful input, bud.

[water]

18 minutes ago, thatboy said:

Am I supposed to do "Func Mouseclick" Instead?

Did you try? If yes, does it work?

[thatboy]

1 minute ago, water said:

Did you try? If yes, does it work?

 

I actually took out the "Hotkeyset" line.

And I took out "Func"

 

My goal is for it to do it on its own on an infinite loop.  This is just a small fraction of the code.  I'm trying to run this script successfully, before I add anything else.  And I've yet to successfully run a script.

Now I Just have this:

 

#include <AutoItConstants.au3>

While 1
    Sleep(50)
WEnd

$mouse_Speed(10)
MouseClick("LEFT", 702. 953, 1, 1)

 

I Run this script and nothing happens. 

 

[Jos]

Read that source again and try to understand when you will encounter the MouseClick() function.  

[water]

Grab your original script and update the name of the function that should be called when you press the hotkey.

[jchd]

If you insist on invoking function MC, then you need to invoke it by yourself, like in:

HotKeySet('h', Hotkey1) ; no quote needed since functions are first-class citizens

Func Hotkey1()
    ConsoleWrite('The Hotkey was pressed' & @CRLF)
    MC()                    ; invoke this holly function
EndFunc     ;==>Hotkey1

While 1
    Sleep(50)
WEnd

Func MC()
    Sleep(10)
    MouseClick("LEFT", 702, 953, 1, 1)      ; typo fixed
EndFunc

 

Edited May 6, 2022 by jchd

[thatboy]

27 minutes ago, jchd said:

If you insist on invoking function MC, then you need to invoke it by yourself, like in:

HotKeySet('h', Hotkey1) ; no quote needed since functions are first-class citizens

Func Hotkey1()
    ConsoleWrite('The Hotkey was pressed' & @CRLF)
    MC()                    ; invoke this holly function
EndFunc     ;==>Hotkey1

While 1
    Sleep(50)
WEnd

Func MC()
    Sleep(10)
    MouseClick("LEFT", 702. 953, 1, 1)
EndFunc

 

Thanks a lot.  However, this code returns an error:

 

 HotKeySet('h', Hotkey1)

Func Hotkey1()
    ConsoleWrite('The Hotkey was pressed' & @CRLF)
    MC()
EndFunc     ;==>Hotkey1

While 1
    Sleep(50)
WEnd

Func MC()
    Sleep(10)
    MouseClick("LEFT", 702. 953, 1, 1)
EndFunc

 

Error:

The Hotkey was pressed

MouseClick("LEFT", 702. 953, 1, 1)
MouseClick("LEFT", ^ ERROR
>Exit code: 1

[Musashi]

MouseClick("LEFT", 702. 953, 1, 1)

Just a small typo by @jchd (dot instead of comma).

@thatboy : You could have easily spotted this yourself if you took a look at the help or the error message in the console output .

Use

Func MC()
    Sleep(10)
    MouseClick("LEFT", 702, 953, 1, 1)
EndFunc

 

Edited May 6, 2022 by Musashi

[Musashi]

By the way: A hotkey to terminate the script would be beneficial :

HotKeySet('h', Hotkey1) ; no quote needed since functions are first-class citizens
HotKeySet("{ESC}", _Terminate)

Func Hotkey1()
    ConsoleWrite('The Hotkey was pressed' & @CRLF)
    MC()                    ; invoke this holly function
EndFunc     ;==>Hotkey1

While 1
    Sleep(50)
WEnd

Func MC()
    Sleep(10)
    MouseClick("LEFT", 702, 953, 1, 1)
EndFunc

Func _Terminate()
    MsgBox(BitOR(4096, 64), "Message :", "Script terminated" & @CRLF)
    Exit
EndFunc

 

[jchd]

18 minutes ago, Musashi said:

Just a small typo by @jchd (dot instead of comma)

Not mine: it comes verbatim from the OP, I was aware but was distracted before posting and forgot to fix it.

[thatboy]

 

Yeah, that's my dumb ass who did the typo.

 

Thanks Guys!    Finally got my first script to work! 

 

 

[Musashi]

59 minutes ago, jchd said:

1 hour ago, Musashi said:

Just a small typo by @jchd (dot instead of comma)

Not mine: it comes verbatim from the OP, I was aware but was distracted before posting and forgot to fix it.

True, I hope you accept my sincere apology .

@thatboy : Why did you mark your buggy script from your initial question as the Solution? You should mark the one from @jchd (he could still edit the typo though ).

 

[thatboy]

16 minutes ago, Musashi said:

True, I hope you accept my sincere apology .

@thatboy : Why did you mark your buggy script from your initial question as the Solution? You should mark the one from @jchd (he could still edit the typo though ).

 

 

Was an accident.  My first time using this site.  How do I change it?

[jchd]

20 minutes ago, Musashi said:

(he could still edit the typo though ).

Done.

[thatboy]

Fixed the Solution Marking.  Thanks again mfs 

[Werty]

8 minutes ago, thatboy said:

mfs 

I had to google that one, and it didnt turn out good, google says either...

"M*therf*ckers" or "Middle Finger Salute", but maybe google is wrong. 😛

Edited May 6, 2022 by Werty