AHK nice cm script

[mrdruids]

i was doing z & Lbutton:: i liked that if you let off z with left mouse still held it would stick... but this is simpler. just hold z and u stand still and spam. my skills for this are
if you hold z and right mouser you can stand spam while moving
1-crystal-shards 2-nova-snap 3-explosive blast-chain 4-energy/electric-pinpoint/shock L-energy twister-wicked R-teleport-wormhole
also wheelup potion
wheeldown for shield

forgive the messiness, im still learning and i only learn as needed



#NoEnv
SendMode Input
SetWorkingDir %A_ScriptDir%

#SingleInstance Force



;SKILLS

z::
{
Send {Shift Down}
Click
Loop
{
if not GetKeyState("z","P")
Break
Click
Send 2
sleep 10
send 1
sleep 10
send 2
sleep 10
send 3
sleep 10
send 2
sleep 10
send 1
sleep 10
send 2
sleep 10
send 1
sleep 10


}
Send {Shift Up}
}

return

;ELITE HERE

f1::
{
Send {enter}
sleep 25
send E
sleep 25
send {enter}
}
return

;KEYWARDEN HERE

f2::
{
Send {enter}
sleep 25
send KW
sleep 25
send {enter}
}
return

;UBERING

f3::
{
Send r
sleep 25
send in ubers brb
sleep 25
send {enter}
}
return

wheelup::send q

wheeldown::send 4




f9::ExitApp

[timmy04]

thank you for this script. i love that it incorporates shift.

BUT this script sometimes loops for me (~once every 2 minutes maybe) . Meaning that even when i stop pressing the macro button it keeps pressing shift, 123 and Click buttons. The only way to fix this is to manually press shift and pause the script, then unpause it (i added a pause function).
The problem must be related to shift because i have tried many AHK scripts without shift and they always work flawlessly.

Any idea why this would happen?

This is the slightly modified script

#NoEnv
SendMode Input
SetWorkingDir %A_ScriptDir%

#SingleInstance Force



;SKILLS

a::
{
Send {Shift Down}
Loop
{
if not GetKeyState("a","P")
Break
  Click
  Send 1 
  Send 2
  Send 3
  Sleep 40

}
Send {Shift Up}
}

return

;ELITE
f3::
{
Send {enter}
sleep 25
send E
sleep 25
send {enter}
}
return


;KEYWARDEN
f4::
{
Send {enter}
sleep 25
send KW
sleep 25
send {enter}
}
return

;GOBLIN
f5::
{
Send {enter}
sleep 25
send Gob
sleep 25
send {enter}
}
return

;wheelup::send q

;wheeldown::send 4


F1::
Suspend
Pause,,1

P.S.: Can you explain why you chose to use "Click" twice and Send {1,2,3} so many times with low break time? I am not an expert at AHK scripting but it seems redundant to me.