error? why it isn't works?

[Unc3nZureD]

hi. i've got a simple script. I think the source is good but it don't want to work. (some errors are posible) so the script:

expandcollapse popup

msgbox(0, "Help", "Hotkeys:" & @CRLF & " " & @CRLF & "F1 - RPGmenu" & @CRLF & "F2 - RTV" & @CRLF & "F3 - RPGrank" & @CRLF & "F6 - Votekick" & @CRLF & "F5 - Voteban" & @CRLF & "Esc - Exit script")

HotKeySet("{ESC}", "Terminate")
HotKeySet("{F1}", "rpgmenu")
HotKeySet("{F2}", "RTV")
HotKeySet("{F3}", "rpgrank")
HotKeySet("{F5}", "votekick")
HotKeySet("{F6}", "voteban")

Func rpgmenu()
    Send("{y}")
    sleep(200)
    Send("rpgmenu")
    Send("{ENTER}")
EndFunc

Func RTV()
    Send("{y}")
    sleep(200)
    Send("rtv")
    Send("{ENTER}")
EndFunc

Func rpgrank()
    Send("{y}")
    sleep(200)
    Send("rpgrank")
    Send("{ENTER}")
EndFunc

Func voteban()
    Send("{y}")
    sleep(200)
    Send("votekick")
    Send("{ENTER}")
EndFunc

Func votekick()
    Send("{y}")
    sleep(200)
    Send("voteban")
    Send("{ENTER}")
EndFunc

is there any error in the source? if i start it it starts for 1ms and it exit.

Edited June 23, 2010 by Unc3nZureD

[AlmarM]

Add

While 1
     Sleep(100)
WEnd

[jchd]

Your script does nothing except defining function and intercepting hotkeys you define.

Place a waiting loop right after the block of HotKeySet(...) calls.

While 1
    Sleep(10)
WEnd

[Unc3nZureD]

Add

While 1
     Sleep(100)
WEnd

woohoo!

ty

[nf67]

You need to add a while loop so the program doesn't run through all the code immediately (and exits when done).

I suggest you add it between the funcs and HotKeySets, because if you add it before there, your script will never reach HotKeySet (because it remains in the loop).

(...)

HotKeySet("{F5}", "votekick")
HotKeySet("{F6}", "voteban")

While 1
;Optional: You could add Sleep(XXX) here so the script won't be too heavy for your memory.
WEnd


Func rpgmenu()
    Send("{y}")
    sleep(200)

(...)

Edited June 23, 2010 by nf67

[jchd]

nf67,

A Sleep(<something small>) is more than recommended: it's necessary to avoid hogging a core to 100% uselessly. With a small Sleep, the core usage will be low (in the few %).

[trancexx]

nf67,

A Sleep(<something small>) is more than recommended: it's necessary to avoid hogging a core to 100% uselessly. With a small Sleep, the core usage will be low (in the few %).

Just as an additional information, in this particular case sleep value doesn't have to be small at all, meaning any sleep would be good. This is due to the fact that sleeps larger than 10 are split into chunks of sleeps 10 internally by AutoIt (for it not to fall asleep). Seems.

[jchd]

That's new to me. I'll dig further someday, thanks Trancexx (you deserve an uppercase!).

[EnderWiggin]

That's new to me. I'll dig further someday, thanks Trancexx (you deserve an uppercase!).

Uppercase isn't always an upgrade. Proper nouns come and go but lowercase is forever.