Loop not exiting.

[Bekaarfaltu]

WinActivate ("LMS — Mozilla Firefox")

 Global $j = 0, $k = 0
Do
   MouseClick("left", 415, 205, 1)
   Sleep(2000)
   Do
      MouseClick("left", 1271, 858, 1)
      Sleep(10000)
      $j = $j + 1
   Until $j = 2 ; Increase the value of $j until it equals the value of 2.
   $k = $k + 1
Until $k = 2 ; Increase the value of $k until it equals the value of 2.
Exit

The problem is that the second loop keeps on running.

I'm obviously missing something simple.

I'm new to this. Please help.

[water]

Welcome to AutoIt and the forum!

You have to reset $j when you start the inner loop again:

WinActivate ("LMS — Mozilla Firefox")

Global $j = 0, $k = 0
Do
   MouseClick("left", 415, 205, 1)
   Sleep(2000)
   $j = 0
   Do
      MouseClick("left", 1271, 858, 1)
      Sleep(10000)
      $j = $j + 1
   Until $j = 2 ; Increase the value of $j until it equals the value of 2.
   $k = $k + 1
Until $k = 2 ; Increase the value of $k until it equals the value of 2.
Exit

 

Edited September 20, 2021 by water

[JockoDundee]

@wateris correct.

Why not just:

WinActivate ("LMS — Mozilla Firefox")

For $k = 0 To 1 
   MouseClick("left", 415, 205, 1)
   Sleep(2000)
   For $j = 0 To 1
      MouseClick("left", 1271, 858, 1)
      Sleep(10000)    
   Next
Next
Exit

 

Edited September 20, 2021 by JockoDundee

[Bekaarfaltu]

Thank you @water. Just the value of $j needed to be reset to 0. Such a simple solution!

@JockoDundee Your solution is much cleaner and simpler. For...Next loops seem better. Thanks!

 

[Musashi]

32 minutes ago, Bekaarfaltu said:

@JockoDundee Your solution is much cleaner and simpler. For...Next loops seem better. Thanks!

Unfortunately, it seems not possible to mark more than one post as "Solution" .

[JockoDundee]

3 hours ago, Musashi said:

Unfortunately, it seems not possible to mark more than one post as "Solution"

@waterquickly dissolved the problem, leaving only a solution

[water]

I would vote for multiple solutions. There is more than one way to skin a cat
As a user I would like to select the most elegant/efficient code from a bunch of "solutions".

Just my 2 cents worth

[JockoDundee]

2 hours ago, water said:

I would vote for multiple solutions.

I would agree.

In this case though, I’m not worthy, since at the very moment that water posted $j = 0, my draft response showed something like:

Quote

The program should exit in 44 seconds…

 

[water]

Your updated version not only fixed the problem but is much cleaner and elegant. So the OP can learn how good code looks like in addition.
Every post that solves the OPs problem should be a candidate to be marked by the OP as "Solution".