I need some help with my script please. Thank u

[dannymcf2]

Hey!

Im just going to say that im new to autoit and this is my first successful script !  

anyway i have a little problem that does not

make sense at all to me.... Maybe you can help me out? Anyone please??

What i have the SCRIPT do is this : 

The script finds a text box (on the screen) then it clicks the text box, then it sends "guess what...."

but after it has sent "guess what...." FOR SOME REASON IT SCROLLS DOWN THEN ENDS THE SCRIPT WITH AN ERROR.... (see below for error...)

"C:\Users\yolo\Desktop\bot\best.au3" (47) : ==> Illegal text at the end of statement (one statement per line).:
mousewheel("down"), 3)
mousewheel("down")^ ERROR

HERE IS MY SCRIPT BELOW... That im having problems with.

func ns3()
     $pixel = PixelSearch(416, 295, 882, 315, 0x707070)
    if IsArray($pixel) = true then
    MouseClick("left", $pixel[0], $pixel[1])
       tn()
          EndIf
       EndFunc

    func tn()
       send ("guess{SPACE}what.....")
       sleep ($interval)
       sleep ($interval)
       findcombox()
    EndFunc

So what happens is it searches for the text box, then if it's true then mouse click then Func tn() runs

 

Then func tn sends "guess what...."

then sleeps

THEN IS SCROLLS DOWN AND AN ERROR COMES UP?!!?!?!? what the heck ?!

oh BTW i have a few "Scroll func's" (See Below)

Func scroll()
     mousewheel("down", 2)
  EndFunc

func scroll3t()
   mousewheel("down"), 3)
EndFunc

func scrollup()
   mousewheel("up"), 20)
EndFunc

scroll () - scrolls twice down

scroll3t() - scrolls 3 times

scrollup() - YOU GUESSED IT! scrolls up

SO?! What the heck is happening?! im so confused why it's scrolling down twice for no reason.... it's not like i have the scroll() in the tn() ??

 

Edited May 15, 2015 by dannymcf2

[Danp2]

You have too many closing parentheses in some of your lines of code.

[SadBunny]

True. Every opening parenthesis should have exactly one closing parenthesis. Change mousewheel("down"), 3) to mousewheel("down", 3) and you should be fine..

Also, next time you open a topic, please choose a more descriptive title. The forum is called "General Help and Support", meaning that virtually all posts on this forum are posts where someone needs some help with a script  Try to hint at the sort of problem you are experiencing, and it will be more likely to get a good answer quickly.

/edit: lastly, as a final tip: don't write "THEN IS SCROLLS DOWN AND AN ERROR COMES UP?!!?!?!?", but rather "Then it scrolls down and an error comes up." ALLCAPS is considered yelling and generally frowned upon by people who try to communicate in a grown-up manner, and as such is better avoided in general. It is unnecessarily irritating.

Edited May 15, 2015 by SadBunny

[dannymcf2]

Okay thanks a lot guys, I'll try harder next time, thanks again.

When I'm home I'll change it.

i spent 2 hours writing my script so it's just I made a mistake right ? Like in places of my script I forgot to Endif haha and end Funcs.

 

i want to thank autoit for creating this great program! I freakin love it

[Danp2]

 

 Func scroll()
     mousewheel("down", 2)
  EndFunc

func scroll3t()
   mousewheel("down"), 3)
EndFunc

func scrollup()
   mousewheel("up"), 20)
EndFunc

These could be written as a single function with default parameters for direction and count.

[dannymcf2]

Guys, is there a way to show a debug in the console or something like that? 

Ni need to know what my script is doing .

Edited May 15, 2015 by dannymcf2

[dannymcf2]

Hey danp2,

ive split these Funcs up because...

in some Funcs I need mouse to scroll up twice and sometimes them to scroll 3 times or more and so on, so I decided to make multiple Funcs because I simply don't know what else i should do.also I made a little script where it scans small areas of the screen for a colour, and if it finds it, its continues to the next Func and if it doesn't find the colour it searches for another part of the screen area and...... Its keeps doing that until it finds something  I thought of the idea when was at work and as soon as I got home I worked on it for like 2 hours and it actually worked! I think I should reward my self

 

[Danp2]

For debugging, check out the ConsoleWrite command in the help file.

in some Funcs I need mouse to scroll up twice and sometimes them to scroll 3 times or more and so on, so I decided to make multiple Funcs because I simply don't know what else i should do.

Again, this could all be handled with a single function. Perhaps like this:

Func Scroll($direction = 'down', $count = 1)

Then your code would use the values of $direction and $count to control the action to be performed. Here are some examples of calling this function:

scroll() ; uses the defaults of down and 1

scroll('up') ; overrides the direction

scroll('up', 12) ; overrides both default values

 

[Zedna]

"C:\Users\yolo\Desktop\bot\best.au3"

Don't forget to read forum rules because all game related posts are forbidden here.

https://www.autoitscript.com/forum/topic/169097-forum-rules/