do loop until 10 times

[Alexxander]

Hi all i an new to Autoit

i want to make a script that search for a pic and click it 

and repeate that for 10 times i had readed the help doc and some postes in the forums but no luck

the search and click itself is working but i cant get the loop to work

any ideas ?

#include <ImageSearch.au3>

$x = 0
$y = 0


$NUM = 0
While $NUM <= 10
   
Func start()
  
  Do
$Search = _ImageSearch('pic.bmp', 0, $x, $y, 0)

if $Search = 0 Then sleep (1000)
Until $Search = 1
If $Search = 1 Then
 MouseMove($x, $y,10)
 MouseClick("left", $x, $y, 1)
 EndIf
EndFunc


WEnd

when i run this code it give the following error

C:UsersAdministratorDownloadsImageSearchImageSearchtut.au3 (10) : ==> "While" statement has no matching "Wend" statement.:

Func start()

Edited July 7, 2013 by alexander95

[water]

Which program do you try to automate?

Often there are faster and more reliable ways to do what you want to do.

[JohnOne]

Because have a function inside a while wend loop.

It is not allowed.

[Alexxander]

Which program do you try to automate?

Often there are faster and more reliable ways to do what you want to do.

 

Because have a function inside a while wend loop.

It is not allowed.

 

Please guys just tell me how to fix it

i want to use this method 

thank u

[water]

Alexander95,

this is no "I have a problem please solve it for me" forum. We ask questions for a reason. So please give us some answers and we will see how we can help you.

[Melba23]

alexander95,

JohnOne has told you what is wrong with your script; water asked for more information so he could perhaps offer some more focused help. And you complain?

M23

[Alexxander]

 

this is no "I have a problem please solve it for me" forum. We ask questions for a reason. So please give us some answers and we will see how we can help you.

 

bro i want to learn this method 

there is no program i want to run on it i just wanna learn it

now i am seeing the autoit help doc i used all the loop methods but they are giving errors 

•  For...Next
•  While...WEnd
•  Do...Until
•  For...In...Next

 

any ideas ?

Edited July 7, 2013 by alexander95

[bleh]

You have 2 "ifs" and only one Endif. Either you put another EndIf in or you use ElseIf. (I think)

And your func starts in the middle of a loop.

[Alexxander]

alexander95,

JohnOne has told you what is wrong with your script; water asked for more information so he could perhaps offer some more focused help. And you complain?

M23

 

i am not complaining he is doing a favor for me and i am so thankful

but the thing is that i want to learn this type of automation 

thank u guys

[water]

To learn AutoIt please have a look at the wiki. There is a tutorial section which is a good place to start.

[Alexxander]

You have 2 "ifs" and only one Endif. Either you put another EndIf in or you use ElseIf. (I think)

And your func starts in the middle of a loop.

 

i made them to i still have the following error

 

C:UsersAdministratorDownloadsImageSearchImageSearchtut.au3 (12) : ==> "Do" statement has no matching "Until" statement.:

Func start()

 

thank u for attempting to help me guys it is  so apriciated

[bleh]

If you use "Do", you will have to have "Until" somewhere. Somewhere where AutoIT can find it, not hidden in another func or something.

Do <- This does the following command over and over again

Until <- Until a certain criteria is matched.

For example:

Do

   $variable = $variable + 1

Until $variable = 4

This would add 1 to the variable "Until" the variable equals 4.

I hope you understand what i mean.

Edited July 7, 2013 by bleh

[JohnOne]

Move the function outside of the loop, and call the function from inside the loop.

Add a Sleep() in the loop too.

[TheSaint]

Hopefully this is clear as mud.

But I'll let you fill in the missing dots, etc.

While x < y

      .....

      Start()

      ......

Wend


Func Start()

     ........
     Do
         .....
         .....
     Until x

     ........

EndFunc

You really need to try some tutorials and carefully read the Help file though.

And probably put in the sleep that JohnOne mentioned.

Edited July 7, 2013 by TheSaint

[PKG]

Use this code.

#include <ImageSearch.au3>
$x = 0
$y = 0
$NUM = 0
While $NUM <= 10
    start()
    Sleep(10)
WEnd
Func start()
    Do
        $Search = _ImageSearch('pic.bmp', 0, $x, $y, 0)
        If $Search = 0 Then Sleep(1000)
    Until $Search = 1
    If $Search = 1 Then
        MouseMove($x, $y, 10)
        MouseClick("left", $x, $y, 1)
    EndIf
EndFunc   ;==>start

[James]

bro i want to learn this method 

Bro, you can't use a method which doesn't work. Read what people are saying, if your native language isn't English, use Google Translate.

[spudw2k]

You'll have to increment $NUM else it will never reach 10 and you will loop forever.

Also, just FYI, starting at 0 and looping until <= 10 is actually 11 rounds...assuming were talking integers here.

[Blue_Drache]

Does this have anything to do with your adfly bot?

'?do=embed' frameborder='0' data-embedContent>>

[Alexxander]

thank you guys for help really appreciated ...