Variable ain't declared? code ain't working, why?

[Scriptie]

Hello, I been using AutoIT for several years ago, got some basic knowledge in the basic programming.

I just want this code to work, the mouse is going to click 10 times, each time will be a minus 35 pixels on width.

To do this, my code is this:

sleep(5000)

reset()

Func reset()
$startX = 857
$startY = 675
$count = 0
start1()
EndFunc

start1()
Func start1()
If $count =< 10 Then
Mouseclick("left",$startX,$startY)
$startX = $startX - 35
$count = $count + 1
start1()
Else

Exit

EndIf
EndFunc

I don't understand what the fault is?

At first I put a sleep.

Then I tell it to go to reset function, right?

In reset function I reclare my variables. It end the function with going to the start1 function, where it simply just watches so the count is 10 or smaller, and if it is, it will click on a location. After that it will minus the width with 35 px, and the count will make an addition of 1.

Then it will loop this, until it's up on 11 at count, at that time it will exit the script.

So, what is really wrong here? I even tried to put the variables on top of the sleep but nothing really happends then.

Thanks in advance.

Edited March 9, 2014 by Scriptie

[FireFox]

Hi,

The variables declared in your reset function are Local (without declaration scope).

Either declare them in the global scope or add Global before them in the function (but in the last case be sure that you call the reset function first).

And secondly, it's <= and not =<.

Br, FireFox.

[mikell]

Why don't you use a For/Next loop ?

Global $startX, $startY

reset()
start1()

Func reset()
   $startX = 857
   $startY = 675
EndFunc

Func start1()
   For $i = 0 to 10
      $startX -= $i*35
      Mouseclick("left", $startX, $startY)
      Sleep(100)
   Next
EndFunc

[TechCoder]

Like any new adventure, the LAST thing you should do is 'sleep'!  (think of visiting a new place, new city, new country!)

First thing you would want to do is 'look around' a bit and see what might be 'interesting' - if nothing else, at least set some bearings and/or 'rules'..........

So, what 

mikell

suggested is quite valid - first, set up some 'rules' (i.e., parameters) and THEN do some 'looking around' - in this case, check out "from 1 to 10" - and, possibly, then (maybe even AS you are counting, if it makes sense...) do some 'sleep'

but, IMHO, a program should NEVER begin with SLEEP!

[mikell]

 

but, IMHO, a program should NEVER begin with SLEEP!

This can be useful sometimes, for instance when the program 'starts with Windows' and you want to delay a little its execution

[TechCoder]

Granted, waiting for Windows to start is a great example of 'a good time to have a long sleep'.........  

However, instead of sleep(), I would rather consider alternatives like WinWait - i.e., watch some 'trigger'.  

You might use a sleep(x) (some small number) in a loop if that is what you need (for instance, you require Internet to be active, so do a "While $not_connected" or something), I just prefer to be more pro-active and look for the event, not just sleep() as a first command.

But, that is why there are many options in the code - each person has their own preference - none 'wrong', just different ways of looking at things.

[BrewManNH]

I rewrote a tool that when it was updated needed to be restarted. It also used _Singleton because it wouldn't work if there were more than one instance running. Occasionally when you exited the first instance it would restart before the first instance had time to shutdown completely. I put a small sleep at the start of the script so that the first instance would be sure to be closed by the time the _Singleton function was accessed. It may have been hacky to do it that way, but it works and it works well.

[mikell]

TechCoder,

The only reason I can see why you should put a heading loop instead of a Sleep is when you are in a hurry and want your program to start as soon as possible

Otherwise I don't understand why using Sleep in such conditions would be a bad practice