Botman Posted April 24, 2013 Share Posted April 24, 2013 So from my first post I had not read the rules too attentively and realized i had posted something against the rules. This time i just want to test it on a random site so i did it on a camping reservations site. This is my script: WinWaitActive ( "Discover Camping Reservation System - Mozilla Firefox" ); While 1=1 MouseMove ( 474, 824 [, 10] ) Sleep (10000 MouseClick ( "Left" ) WEnd The message that pops up is this: Line 3 (File "C:\Users\Sacre_000\Desktop\BotScripts\ClickerTest.exe"): Error: Error parsing function call. could someone please help give me some pointers on what i did wrong in writing my script. Link to comment Share on other sites More sharing options...
dynamicfusion Posted April 24, 2013 Share Posted April 24, 2013 Your while should read While 1 <statements> You dont need while 1=1. The logic of your bot is basically looping infinitely and moving the mouse to those coordinates forever every 10 seconds, and then clicking. Is this what you want it to do? Link to comment Share on other sites More sharing options...
MouseSpotter Posted April 24, 2013 Share Posted April 24, 2013 Nice title: First time making bot, please helpAre you sure you have read the rules? Link to comment Share on other sites More sharing options...
water Posted April 24, 2013 Share Posted April 24, 2013 Your while should readWhile 1 <statements>You dont need while 1=1. The logic of your bot is basically looping infinitely and moving the mouse to those coordinates forever every 10 seconds, and then clicking. Is this what you want it to do?While 1 is the same as While 1=1 or While "a" = "a". You just need an expression that returns True. My UDFs and Tutorials: Spoiler UDFs:Active Directory (NEW 2022-02-19 - Version 1.6.1.0) - Download - General Help & Support - Example Scripts - WikiExcelChart (2017-07-21 - Version 0.4.0.1) - Download - General Help & Support - Example ScriptsOutlookEX (2021-11-16 - Version 1.7.0.0) - Download - General Help & Support - Example Scripts - WikiOutlookEX_GUI (2021-04-13 - Version 1.4.0.0) - DownloadOutlook Tools (2019-07-22 - Version 0.6.0.0) - Download - General Help & Support - WikiPowerPoint (2021-08-31 - Version 1.5.0.0) - Download - General Help & Support - Example Scripts - WikiTask Scheduler (NEW 2022-07-28 - Version 1.6.0.1) - Download - General Help & Support - Wiki Standard UDFs:Excel - Example Scripts - WikiWord - Wiki Tutorials:ADO - WikiWebDriver - Wiki Link to comment Share on other sites More sharing options...
water Posted April 24, 2013 Share Posted April 24, 2013 You have to remove the brackets from MouseMove ( 474, 824 [, 10] )Brackets denote optional parameters. I wouldn't use MouseMove and MouseClick statements. This gives unreliable scripts because they depend on screen resolution and window positions. To automate FireFox use the FireFox UDF. My UDFs and Tutorials: Spoiler UDFs:Active Directory (NEW 2022-02-19 - Version 1.6.1.0) - Download - General Help & Support - Example Scripts - WikiExcelChart (2017-07-21 - Version 0.4.0.1) - Download - General Help & Support - Example ScriptsOutlookEX (2021-11-16 - Version 1.7.0.0) - Download - General Help & Support - Example Scripts - WikiOutlookEX_GUI (2021-04-13 - Version 1.4.0.0) - DownloadOutlook Tools (2019-07-22 - Version 0.6.0.0) - Download - General Help & Support - WikiPowerPoint (2021-08-31 - Version 1.5.0.0) - Download - General Help & Support - Example Scripts - WikiTask Scheduler (NEW 2022-07-28 - Version 1.6.0.1) - Download - General Help & Support - Wiki Standard UDFs:Excel - Example Scripts - WikiWord - Wiki Tutorials:ADO - WikiWebDriver - Wiki Link to comment Share on other sites More sharing options...
Moderators Melba23 Posted April 24, 2013 Moderators Share Posted April 24, 2013 Botman,So from my first post I had not read the rules too attentively and realized i had posted something against the rulesAnd you still have not read them properly - you seem to have missed the bit that says:"Do not repost the same question if the previous thread has been locked - particularly if you merely reword the question to get around one of the prohibitions listed above"And before you argue that your first thread had not been locked at that point, it was obvious that it would be as soon as we got around to it.You can use AutoIt for many more things than game-bots and I suggest that you think of doing that, because any further infringements of the rules on your part will result in sanctions. M23 Any of my own code posted anywhere on the forum is available for use by others without any restriction of any kind Open spoiler to see my UDFs: Spoiler ArrayMultiColSort ---- Sort arrays on multiple columnsChooseFileFolder ---- Single and multiple selections from specified path treeview listingDate_Time_Convert -- Easily convert date/time formats, including the language usedExtMsgBox --------- A highly customisable replacement for MsgBoxGUIExtender -------- Extend and retract multiple sections within a GUIGUIFrame ---------- Subdivide GUIs into many adjustable framesGUIListViewEx ------- Insert, delete, move, drag, sort, edit and colour ListView itemsGUITreeViewEx ------ Check/clear parent and child checkboxes in a TreeViewMarquee ----------- Scrolling tickertape GUIsNoFocusLines ------- Remove the dotted focus lines from buttons, sliders, radios and checkboxesNotify ------------- Small notifications on the edge of the displayScrollbars ----------Automatically sized scrollbars with a single commandStringSize ---------- Automatically size controls to fit textToast -------------- Small GUIs which pop out of the notification area Link to comment Share on other sites More sharing options...
Recommended Posts