TomaSzz Posted May 25, 2010 Share Posted May 25, 2010 (edited) also,i dunno wuts wrong in this thing... as u see i put in var's to be sent from input boxes..but even if leave some inputboxes empty..it still sends buttons according to the number at the end of each var... wat i want it to send is only the input boxes that have number entered..thnx for all help,rly apreciate it Send($skill1) Sleep($s1) Send($skill2) Sleep($s2) Send($skill3) Sleep($s3) Send($skill4) Sleep($s4) Send($skill5) Sleep($s5) Send($skill6) Sleep($s6) Send($skill7) Sleep($s7) Send($skill8) Sleep($s8) Send($skill9) Sleep($s9) Send($skill0) Sleep($s0) Edited May 25, 2010 by TomaSzz Link to comment Share on other sites More sharing options...
TomaSzz Posted May 26, 2010 Author Share Posted May 26, 2010 no1? Link to comment Share on other sites More sharing options...
Affe Posted May 26, 2010 Share Posted May 26, 2010 You have given us nothing to go by. Post the rest of your code. [center][/center] Link to comment Share on other sites More sharing options...
TomaSzz Posted May 26, 2010 Author Share Posted May 26, 2010 $coord = PixelSearch(50, 106, 950, 334, 0x7F7F7F, 4) If Not @error Then MouseClickDrag("left", $coord[0], $coord[1], $coord[0] + 30, $coord[1] + 30) EndIf Send($skill1) Sleep($s1) Send($skill2) Sleep($s2) Send($skill3) Sleep($s3) Send($skill4) Sleep($s4) Send($skill5) Sleep($s5) Send($skill6) Sleep($s6) Send($skill7) Sleep($s7) Send($skill8) Sleep($s8) Send($skill9) Sleep($s9) Send($skill0) Sleep($s0) Link to comment Share on other sites More sharing options...
Affe Posted May 26, 2010 Share Posted May 26, 2010 Your $skill variables have no value, according to the code you have posted. If you're using the Send() function to send keypresses, you need to give them a value. Send("e") ;sends the keypress "e" Send("{F4}") ;sends the keypress F4 Send("{ALTDOWN}{F4}{ALTUP}") ;holds down the alt key, presses the F4 key, releases the alt key [center][/center] Link to comment Share on other sites More sharing options...
TomaSzz Posted May 26, 2010 Author Share Posted May 26, 2010 (edited) but did u read first post? i hav input boxes...and i input in boxes number to be send from inputbox and if i leave my other input boxes the program still sends empty input boxes too...i only want data to be sent from inputboxes where is number written in it..or at least anythin..uknow.. as far as i understand it now i must enter every input box... Edited May 26, 2010 by TomaSzz Link to comment Share on other sites More sharing options...
PsaltyDS Posted May 26, 2010 Share Posted May 26, 2010 Let me take a wild guess at it: Your $skill variables contain the control IDs of input controls instead of the value read from them. Valuater's AutoIt 1-2-3, Class... Is now in Session!For those who want somebody to write the script for them: RentACoder"Any technology distinguishable from magic is insufficiently advanced." -- Geek's corollary to Clarke's law Link to comment Share on other sites More sharing options...
TomaSzz Posted May 26, 2010 Author Share Posted May 26, 2010 erm i never used controlID so hm.. well i hav it like this expandcollapse popup$skill1 = GUICtrlCreateInput("", 135, 160, 20, 20) $skill2 = GUICtrlCreateInput("", 135, 190, 20, 20) $skill3 = GUICtrlCreateInput("", 135, 220, 20, 20) $skill4 = GUICtrlCreateInput("", 135, 250, 20, 20) $skill5 = GUICtrlCreateInput("", 135, 280, 20, 20) $skill6 = GUICtrlCreateInput("", 135, 310, 20, 20) $skill7 = GUICtrlCreateInput("", 135, 340, 20, 20) $skill8 = GUICtrlCreateInput("", 135, 370, 20, 20) $skill9 = GUICtrlCreateInput("", 135, 400, 20, 20) $skill0 = GUICtrlCreateInput("", 135, 430, 20, 20) $s1 = GUICtrlCreateInput("", 165, 160, 50, 20) $s2 = GUICtrlCreateInput("", 165, 190, 50, 20) $s3 = GUICtrlCreateInput("", 165, 220, 50, 20) $s4 = GUICtrlCreateInput("", 165, 250, 50, 20) $s5 = GUICtrlCreateInput("", 165, 280, 50, 20) $s6 = GUICtrlCreateInput("", 165, 310, 50, 20) $s7 = GUICtrlCreateInput("", 165, 340, 50, 20) $s8 = GUICtrlCreateInput("", 165, 370, 50, 20) $s9 = GUICtrlCreateInput("", 165, 400, 50, 20) $s0 = GUICtrlCreateInput("", 165, 430, 50, 20) and then those in case $coord = PixelSearch(50, 106, 950, 334, 0x7F7F7F, 4) If Not @error Then MouseClickDrag("left", $coord[0], $coord[1], $coord[0] + 30, $coord[1] + 30) EndIf Send($skill1) Sleep($s1) Send($skill2) Sleep($s2) Send($skill3) Sleep($s3) Send($skill4) Sleep($s4) Send($skill5) Sleep($s5) Send($skill6) Sleep($s6) Send($skill7) Sleep($s7) Send($skill8) Sleep($s8) Send($skill9) Sleep($s9) Send($skill0) Sleep($s0) Link to comment Share on other sites More sharing options...
PsaltyDS Posted May 26, 2010 Share Posted May 26, 2010 You want, for example: $sVal = GuiCtrlRead($skill5) Send($sVal) Or... Send(GuiCtrlRead($skill1)) Valuater's AutoIt 1-2-3, Class... Is now in Session!For those who want somebody to write the script for them: RentACoder"Any technology distinguishable from magic is insufficiently advanced." -- Geek's corollary to Clarke's law Link to comment Share on other sites More sharing options...
TomaSzz Posted May 26, 2010 Author Share Posted May 26, 2010 yey .. u solved it the second one fits wat i want... tytytytytyty..ild kiss u but im not gay ... Link to comment Share on other sites More sharing options...
TomaSzz Posted May 26, 2010 Author Share Posted May 26, 2010 ok another issue :| it wont go on other pixelsearch's just stays at first and keeps sending those send , sleep expandcollapse popupWhile 1=1 $coord = PixelSearch(50, 106, 950, 334, 0x7F7F7F, 4) If Not @error Then MouseClickDrag("left", $coord[0], $coord[1], $coord[0] + 60, $coord[1] + 60) EndIf sleep(300) Send(GuiCtrlRead($skill1)) Sleep(GuiCtrlRead($s1)) Send(GuiCtrlRead($skill2)) Sleep(GuiCtrlRead($s2)) Send(GuiCtrlRead($skill3)) Sleep(GuiCtrlRead($s3)) Send(GuiCtrlRead($skill4)) Sleep(GuiCtrlRead($s4)) Send(GuiCtrlRead($skill5)) Sleep(GuiCtrlRead($s5)) $coord = PixelSearch(584, 273, 952, 600, 0x7F7F7F, 4) If Not @error Then MouseClickDrag("left", $coord[0], $coord[1], $coord[0] + 60, $coord[1] + 60) EndIf sleep(300) Send($skill1) Send(GuiCtrlRead($skill1)) Sleep(GuiCtrlRead($s1)) Send(GuiCtrlRead($skill2)) Sleep(GuiCtrlRead($s2)) Send(GuiCtrlRead($skill3)) Sleep(GuiCtrlRead($s3)) Send(GuiCtrlRead($skill4)) Sleep(GuiCtrlRead($s4)) Send(GuiCtrlRead($skill5)) Sleep(GuiCtrlRead($s5)) $coord = PixelSearch(61, 434, 952, 600, 0x7F7F7F, 4) If Not @error Then MouseClickDrag("left", $coord[0], $coord[1], $coord[0] + 60, $coord[1] + 60) EndIf sleep(300) Send(GuiCtrlRead($skill1)) Sleep(GuiCtrlRead($s1)) Send(GuiCtrlRead($skill2)) Sleep(GuiCtrlRead($s2)) Send(GuiCtrlRead($skill3)) Sleep(GuiCtrlRead($s3)) Send(GuiCtrlRead($skill4)) Sleep(GuiCtrlRead($s4)) Send(GuiCtrlRead($skill5)) Sleep(GuiCtrlRead($s5)) $coord = PixelSearch(50, 106, 415, 600, 0x7F7F7F, 4) If Not @error Then MouseClickDrag("left", $coord[0], $coord[1], $coord[0] + 60, $coord[1] + 60) EndIf sleep(300) Send(GuiCtrlRead($skill1)) Sleep(GuiCtrlRead($s1)) Send(GuiCtrlRead($skill2)) Sleep(GuiCtrlRead($s2)) Send(GuiCtrlRead($skill3)) Sleep(GuiCtrlRead($s3)) Send(GuiCtrlRead($skill4)) Sleep(GuiCtrlRead($s4)) Send(GuiCtrlRead($skill5)) Sleep(GuiCtrlRead($s5)) WEnd Link to comment Share on other sites More sharing options...
Affe Posted May 26, 2010 Share Posted May 26, 2010 (edited) ok another issue :| it wont go on other pixelsearch's just stays at first and keeps sending those send , sleep expandcollapse popupWhile 1=1 $coord = PixelSearch(50, 106, 950, 334, 0x7F7F7F, 4) If Not @error Then MouseClickDrag("left", $coord[0], $coord[1], $coord[0] + 60, $coord[1] + 60) EndIf sleep(300) Send(GuiCtrlRead($skill1)) Sleep(GuiCtrlRead($s1)) Send(GuiCtrlRead($skill2)) Sleep(GuiCtrlRead($s2)) Send(GuiCtrlRead($skill3)) Sleep(GuiCtrlRead($s3)) Send(GuiCtrlRead($skill4)) Sleep(GuiCtrlRead($s4)) Send(GuiCtrlRead($skill5)) Sleep(GuiCtrlRead($s5)) $coord = PixelSearch(584, 273, 952, 600, 0x7F7F7F, 4) If Not @error Then MouseClickDrag("left", $coord[0], $coord[1], $coord[0] + 60, $coord[1] + 60) EndIf sleep(300) Send($skill1) Send(GuiCtrlRead($skill1)) Sleep(GuiCtrlRead($s1)) Send(GuiCtrlRead($skill2)) Sleep(GuiCtrlRead($s2)) Send(GuiCtrlRead($skill3)) Sleep(GuiCtrlRead($s3)) Send(GuiCtrlRead($skill4)) Sleep(GuiCtrlRead($s4)) Send(GuiCtrlRead($skill5)) Sleep(GuiCtrlRead($s5)) $coord = PixelSearch(61, 434, 952, 600, 0x7F7F7F, 4) If Not @error Then MouseClickDrag("left", $coord[0], $coord[1], $coord[0] + 60, $coord[1] + 60) EndIf sleep(300) Send(GuiCtrlRead($skill1)) Sleep(GuiCtrlRead($s1)) Send(GuiCtrlRead($skill2)) Sleep(GuiCtrlRead($s2)) Send(GuiCtrlRead($skill3)) Sleep(GuiCtrlRead($s3)) Send(GuiCtrlRead($skill4)) Sleep(GuiCtrlRead($s4)) Send(GuiCtrlRead($skill5)) Sleep(GuiCtrlRead($s5)) $coord = PixelSearch(50, 106, 415, 600, 0x7F7F7F, 4) If Not @error Then MouseClickDrag("left", $coord[0], $coord[1], $coord[0] + 60, $coord[1] + 60) EndIf sleep(300) Send(GuiCtrlRead($skill1)) Sleep(GuiCtrlRead($s1)) Send(GuiCtrlRead($skill2)) Sleep(GuiCtrlRead($s2)) Send(GuiCtrlRead($skill3)) Sleep(GuiCtrlRead($s3)) Send(GuiCtrlRead($skill4)) Sleep(GuiCtrlRead($s4)) Send(GuiCtrlRead($skill5)) Sleep(GuiCtrlRead($s5)) WEnd How can you tell that it doesn't try the other PixelSearch() functions? You don't have it do anything if it doesn't find the pixel, so if the color isn't found in that given area, nothing will happen, and it will just move on to the next portion of the code, which are all identical Send() and Sleep() functions. You should also consider modulating your code. Put repeated code into a function and call that instead of copying/pasting the same thing over and over again. Variables that will remain the same (your $skill values) should be read into Global variables to decrease processing time. These aren't major issues in your current code now, but as your code expands it could become a problem, and good coding practices are always best to learn early. Commenting code also helps a lot if you ever plan to return to it at a later date, and especially if you don't plan to return (that's when you forget what you were doing!). Also While 1=1 Should be While 1 if you are intending to create an infinite loop. 1=1 may work (didn't try it), but that isn't how it was intended. Edited May 26, 2010 by Affe [center][/center] Link to comment Share on other sites More sharing options...
Recommended Posts
Create an account or sign in to comment
You need to be a member in order to leave a comment
Create an account
Sign up for a new account in our community. It's easy!
Register a new accountSign in
Already have an account? Sign in here.
Sign In Now