[Webinar] Streamline your web hosting managementRegister Today

x
?
Solved

VBScript Loop Error

Posted on 2010-11-10
4
Medium Priority
?
633 Views
Last Modified: 2012-05-10
I'm receiving a Microsoft VBScript Compilation error " loop without do"  at line 93.  I don't understand considering I have the two loop statements in the Function getCurrentFiles().
I've attached the script I'm using for your review and comments.  Thank you - Daniel
' Author: Dan Figueroa
 ' Version: 1.0
 ' Description: Uses the FileMaker Server 9 command line to search for
 ' all non-opened databases and opens them
 '
 ' This script does not have full Error handling.
 ' For instance, it will break if there are spaces in the FM file names.
 '
WScript.Echo "Setting variables "

 Const WshFinished = 1
 q = Chr(34) ' the " character, needed to wrap around paths with spaces

 '--------------------------------------------------------------------------------------------
 ' Change these variables to match your setup

 theAdminUser = "administrator"
 theAdminPW = "happysoftware"
 pathToSAtool = "C:\Program Files\FileMaker\FileMaker Server\Database Server\fmsadmin.exe"

 '--------------------------------------------------------------------------------------------


 SAT = "cmd /c " & q & pathToSAtool & q & " " ' watch the trailing space
 
WScript.Echo "Finding utility"

 callFMS = SAT
 If Len(theAdminUser) > 0 Then
 callFMS = callFMS & " -u " & theAdminUser
 End If
 If Len(theAdminPW) > 0 Then
 callFMS = callFMS & " -p " & theAdminPW
 End If
 
WScript.Echo "entering passwords "
 listfiles = callFMS & " list files -s"
 closeFiles = callFMS & " close file "

 ' hook into the Windows shell
 Set sh = WScript.CreateObject("wscript.shell")

 ' get list of files and open them
 fileIDs = getCurrentFiles()
 fileCount = UBound(fileIDs)

 ' loop through the files and open them
 WScript.Echo "loop through the files and open them "
 
 If fileCount > 0 Then
      Do Until fileCount = 0
      	   WScript.Echo "files to be opened "
           'fullCommand = openFiles & fileIDs(0) & " -y"
           'Set oExec = sh.Exec(fullCommand)
           ' give FMS some time and then requery the list of files
           Do Until oExec.Status = WshFinished
               WScript.Sleep 50
           Loop
           fileIDs = getCurrentFiles()
           fileCount = UBound(fileIDs)

      Loop
 End If

Function getCurrentFiles()

 tempCount = 0
 Dim tempArray()
 Set oExec = sh.Exec(listfiles)

 ' in case there are no files...
 If oexec.StdOut.AtEndOfStream Then Redim temparray(0)

 ' read the output of the command
 Do While Not oExec.StdOut.AtEndOfStream
     strText = oExec.StdOut.ReadLine()
     strText = Replace(strtext, vbTab, "")
     Do Until InStr(strtext, " ") = 0
           strText = Replace (strtext, " ", " ")
     Loop
     If InStr(strText, "ID File Clients Size Status Enabled Extended Privileges") > 0 Then
         ' do nothing
         Redim temparray(0)
     Else
 		 tempFile = Split(strtext, " ")
 		 status = LCase(tempFile(4))
         'If status = "closed" Then
 		 If status = "normal" Then
 		 tempCount = tempCount + 1
 		 Redim Preserve tempArray(tempCount)
 		 tempArray(tempCount - 1) = tempFile(1) & ".fp7"
 		 End If
  Loop

 	 getCurrentFiles = tempArray
 	 WScript.Echo "write to event log "
 	 WshShell.LogEvent 2, "WARNING: These file-ids: " & getCurrentFiles & "  are not open " & Time() & " on " & Date()
 End Function

Open in new window

0
Comment
Question by:danfiggolf
  • 2
4 Comments
 
LVL 6

Expert Comment

by:r3nder
ID: 34107999
you need
Do[{while | Until} condition]
[statements]
[Exit Do]
[statements]
Loop

Example:
Do Until str = "Cool"
System.Console.WriteLine("What to do?")
str = System.Console.ReadLine()
Loop
End Sub

Thanks R3nder


0
 
LVL 3

Expert Comment

by:lecretia
ID: 34108028
Try moving lines 78, 79 and 80 into their own function and call that function in their place.

e.g.

Function getCurrentFiles()

 tempCount = 0
 Dim tempArray()
 Set oExec = sh.Exec(listfiles)

 ' in case there are no files...
 If oexec.StdOut.AtEndOfStream Then Redim temparray(0)

 ' read the output of the command
 Do While Not oExec.StdOut.AtEndOfStream
     strText = oExec.StdOut.ReadLine()
     strText = Replace(strtext, vbTab, "")
     replacetxt()
     If InStr(strText, "ID File Clients Size Status Enabled Extended Privileges") > 0 Then
         ' do nothing
         Redim temparray(0)
     Else
              tempFile = Split(strtext, " ")
              status = LCase(tempFile(4))
         'If status = "closed" Then
              If status = "normal" Then
              tempCount = tempCount + 1
              Redim Preserve tempArray(tempCount)
              tempArray(tempCount - 1) = tempFile(1) & ".fp7"
              End If
  Loop

        getCurrentFiles = tempArray
        WScript.Echo "write to event log "
        WshShell.LogEvent 2, "WARNING: These file-ids: " & getCurrentFiles & "  are not open " & Time() & " on " & Date()
 End Function

Function replacetxt()
     Do Until InStr(strtext, " ") = 0
           strText = Replace (strtext, " ", " ")
     Loop
End Function
0
 
LVL 3

Expert Comment

by:lecretia
ID: 34108066
r3nder's method is the correct way. Please ignore my offering.
0
 
LVL 65

Accepted Solution

by:
RobSampson earned 2000 total points
ID: 34108375
You need an End If between lines 92 and 93, so this
              End If
  Loop


becomes this
              End If
       End If
  Loop


Regards,

Rob.
0

Featured Post

Free Tool: ZipGrep

ZipGrep is a utility that can list and search zip (.war, .ear, .jar, etc) archives for text patterns, without the need to extract the archive's contents.

One of a set of tools we're offering as a way to say thank you for being a part of the community.

Question has a verified solution.

If you are experiencing a similar issue, please ask a related question

This is pretty cool.  The purpose of this VB Script is to help you document where JAR (Java ARchive) files and specifically java class files are located so that you can address issues seen with a client or that you can speak intelligently with a dev…
Over the years I have built up my own little library of code snippets that I refer to when programming or writing a script.  Many of these have come from the web or adaptations from snippets I find on the Web.  Periodically I add to them when I come…
Stellar Phoenix SQL Database Repair software easily fixes the suspect mode issue of SQL Server database. It is a simple process to bring the database from suspect mode to normal mode. Check out the video and fix the SQL database suspect mode problem.
Free Data Recovery software is an advanced solution from Kernel Tools to recover data and files such as documents, emails, database, media and pictures, etc. It supports recovery from physical & logical drive after a hard disk crash, accidental/inte…
Suggested Courses

612 members asked questions and received personalized solutions in the past 7 days.

Join the community of 500,000 technology professionals and ask your questions.

Join & Ask a Question