Solved

Do Until with a timer issues...

Posted on 2008-09-30
2
183 Views
Last Modified: 2010-04-21
I am working on some code that contains a Do Until statement that continually pings a computer until it comes online or the timer reaches 5 seconds. The reason I want it to wait and keep trying is that a previous script attempted to power on the computer. The problem I'm running into, is that the timer doesn't seem to do anything unless I put a msgbox in the loop statement. Only then does the timer do anything with the counter variable. I'd much rather I didn't have to continually click OK until it's done. Ideas?
Private Sub BackgroundWorker4_DoWork(ByVal sender As System.Object, ByVal e As System.ComponentModel.DoWorkEventArgs) Handles BackgroundWorker4.DoWork
 
            ifonline1 = 0
 
            counter = 0
 
            InitializeTimer()
            MsgBox("Attempting to power on " & strcomputer)
            Do Until ifonline1 = 1 Or counter >= 5
 
                On Error Resume Next
                Dim Ping As New Net.NetworkInformation.Ping
                Dim PingOptions As New Net.NetworkInformation.PingOptions
                PingOptions.Ttl = 64
                Dim PingReply As Net.NetworkInformation.PingReply = Ping.Send(strcomputer, 200)
                If PingReply.Status = Net.NetworkInformation.IPStatus.Success Then
                    ifonline1 = 1
                    flag4 = "ON"
                Else
                    ifonline1 = 0
                    flag4 = "OFF"
                End If
 
            Loop
            MsgBox(flag4)
            If ifonline1 = 1 Then
                Dim Host As Net.IPHostEntry = Net.Dns.GetHostEntry(strcomputer)
                Dim firstaddress As Net.IPAddress = Host.AddressList(0)
                ip = firstaddress.ToString
                
            End If
            flag1 = pc3
            flag2 = ip
            flag3 = rownum
            worker.ReportProgress(prog1)
        Next
 
End Sub
 
Private Sub InitializeTimer()
        ' Run this procedure in an appropriate event.
        counter = 0
        Timer1.Interval = 1000
        Timer1.Enabled = True
    End Sub
 
    Private Sub Timer1_Tick(ByVal sender As System.Object, ByVal e As System.EventArgs) Handles Timer1.Tick
 
        If counter >= 6 Then
            ' Exit loop code.
            Timer1.Enabled = False
            counter = 0
        Else
            'MsgBox(counter)
            ' Run your procedure here.
            ' Increment counter.
            counter = counter + 1
        End If
 
    End Sub

Open in new window

0
Comment
Question by:jcamping
2 Comments
 
LVL 85

Accepted Solution

by:
Mike Tomlinson earned 95 total points
ID: 22610619
I would just get rid of the Timer altogether and add the timing logic directly into your loop.

Here is the big change:

            Dim endTime As DateTime = DateTime.Now.AddSeconds(5)
            Do Until ifonline1 = 1 OrElse DateTime.Now.Subtract(endTime).TotalMilliseconds > 0

See full code below:
        Private Sub BackgroundWorker4_DoWork(ByVal sender As System.Object, ByVal e As System.ComponentModel.DoWorkEventArgs) Handles BackgroundWorker4.DoWork
            ifonline1 = 0
            MsgBox("Attempting to power on " & strcomputer)
 
            Dim endTime As DateTime = DateTime.Now.AddSeconds(5)
            Do Until ifonline1 = 1 Or DateTime.Now.Subtract(endTime).TotalMilliseconds > 0
 
                On Error Resume Next
                Dim Ping As New Net.NetworkInformation.Ping
                Dim PingOptions As New Net.NetworkInformation.PingOptions
                PingOptions.Ttl = 64
                Dim PingReply As Net.NetworkInformation.PingReply = Ping.Send(strcomputer, 200)
                If PingReply.Status = Net.NetworkInformation.IPStatus.Success Then
                    ifonline1 = 1
                    flag4 = "ON"
                Else
                    ifonline1 = 0
                    flag4 = "OFF"
                End If
            Loop
            MsgBox(flag4)
            If ifonline1 = 1 Then
                Dim Host As Net.IPHostEntry = Net.Dns.GetHostEntry(strcomputer)
                Dim firstaddress As Net.IPAddress = Host.AddressList(0)
                ip = firstaddress.ToString                
            End If
            flag1 = pc3
            flag2 = ip
            flag3 = rownum
            worker.ReportProgress(prog1)
        Next
     End Sub

Open in new window

0
 

Author Closing Comment

by:jcamping
ID: 31501791
That works great! Thanks!
0

Featured Post

Free Tool: Site Down Detector

Helpful to verify reports of your own downtime, or to double check a downed website you are trying to access.

One of a set of tools we are providing to everyone as a way of saying 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

You can of course define an array to hold data that is of a particular type like an array of Strings to hold customer names or an array of Doubles to hold customer sales, but what do you do if you want to coordinate that data? This article describes…
Since upgrading to Office 2013 or higher installing the Smart Indenter addin will fail. This article will explain how to install it so it will work regardless of the Office version installed.
Get people started with the utilization of class modules. Class modules can be a powerful tool in Microsoft Access. They allow you to create self-contained objects that encapsulate functionality. They can easily hide the complexity of a process from…
This lesson covers basic error handling code in Microsoft Excel using VBA. This is the first lesson in a 3-part series that uses code to loop through an Excel spreadsheet in VBA and then fix errors, taking advantage of error handling code. This l…

713 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