Won't recognise my loop

[brawling83]

Hi,

I'm running a script that had A Do Until statement in it. However every time I ran it, I got an error message saying:

"Error: "Until" statement with no matching "Do" statement."

I tried replacing the loop with a While..Wend statement but then got this error message:

"Error: "Wend" statement with no matching "while" statement."

Why is it not recognising my loops, and is there anyway of me linking them manually?

Thanks for the help!

[GEOSoft]

Hi,

I'm running a script that had A Do Until statement in it. However every time I ran it, I got an error message saying:

"Error: "Until" statement with no matching "Do" statement."

I tried replacing the loop with a While..Wend statement but then got this error message:

"Error: "Wend" statement with no matching "while" statement."

Why is it not recognising my loops, and is there anyway of me linking them manually?

Thanks for the help!

That error is often misleading. Instead of looking for the missing Do or While, look for something like a missing End statement. Usually it's EndIf but it could also be EndSwitch or EndSelect.

[someone]

We can't pinpoint whats going on in the code unless you show it. Essentially the code would be,

Do

code here

code here

Until code here

for anything more clear we need to see your code.

[brawling83]

Would it be around the line that I am being pointed to?

In all the error messages it has pointed me to the line it thinks has a missing statement

[Jos]

Run Tidy on your code and it will tell you exactly where your problem is located.

Jos

[TomZ]

I have got a hint for you in order to: 1) fix this problem, 2) fix this problem in the future and 3) code cleaner

Consider this code:

While 1
If 1 Then
Do
Select 1
Case 2
MsgBox(0, "", "")
Until 0
EndIf
WEnd

Now, that isn't terribly easy to read. Now, take a look at this:

While 1
    If 1 Then
        Do
            Select 1
                Case 2
                    MsgBox(0, "", "")
        Until 0
    EndIf
WEnd

Not only is this easier to read and understand, we can also immediately see we're missing something: EndSelect.

You should try indenting your script like so, it will indicate you where your error is, and if you do so consistently in the future, your scritps will be easier to read, easier to modify later on, and small errors can be found more easially. It has become a second nature to me.

Edited July 7, 2008 by TomZ