While timer working, why form can not move properly ?

Question

0.00/5 (No votes)

See more:

I write a simple code with Timer. but when I start timer and then when I try to move form with mouse dawn or when I try to click a Button response with delayed.

What I am doing Wrong ?

What I have tried:

VB

Public Class Form3
    Dim _RGB As Integer
    Dim renk As Color
    Private Sub Timer1_Tick(sender As Object, e As EventArgs) Handles Timer1.Tick
        Dim hiz As Double
        hiz = -0.4

        For i = 256 To 0 Step hiz
            _RGB = RGB(i, 0, 0) 'VScrollBar1.Value, 0, 0) ', VScrollBar2.Value, VScrollBar3.Value)
            renk = ColorTranslator.FromOle(_RGB)
            'RectangleShape1.FillColor = renk
            PictureBox1.BackColor = renk
            PictureBox1.Refresh()
        Next i

        For i = 256 To 0 Step hiz
            _RGB = RGB(0, i, 0) 'VScrollBar1.Value, 0, 0) ', VScrollBar2.Value, VScrollBar3.Value)
            renk = ColorTranslator.FromOle(_RGB)
            'RectangleShape1.FillColor = renk
            PictureBox2.BackColor = renk
            PictureBox2.Refresh()
        Next i

        For i = 256 To 0 Step hiz
            _RGB = RGB(0, 0, i) 'VScrollBar1.Value, 0, 0) ', VScrollBar2.Value, VScrollBar3.Value)
            renk = ColorTranslator.FromOle(_RGB)
            'RectangleShape1.FillColor = renk
            PictureBox3.BackColor = renk
            PictureBox3.Refresh()
        Next i



    End Sub

    Private Sub Button1_Click(sender As Object, e As EventArgs) Handles Button1.Click

        If Timer1.Enabled = False Then
            Timer1.Start()
        Else
            Timer1.Stop()
        End If
    End Sub
End Class

Posted 25-Dec-19 8:48am

Nicomendox

Updated 27-Jan-20 16:00pm

Add a Solution

3 solutions

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

RickZeeland · Answer 1 · 2019-12-25T09:44:00

Solution 1

Your tight loops are preventing the user interface from responding.
Regularly use the Application.DoEvents Method (System.Windows.Forms) | Microsoft Docs[^]

Posted 25-Dec-19 9:44am

RickZeeland

Patrice T · Answer 2 · 2019-12-25T12:43:00

Quote:
What I am doing Wrong ?

Your code is executing in the GUI thread, it means that when Timer1_Tick is running, anything else will wait until the end of tick.
Note that setting the speed of animation by artificially adding some workload is bot a good idea either, it worsen the problem.

What is the reason of Timer1_Tick the way you did it ?

Doug- VisualBasic VB.NET · Answer 3 · 2020-01-27T16:01:00

Inside every for - next loop use the application.doevents, So you would need to put it in 3 times.

For i = 256 To 0 Step hiz
          Application.Doevents
         _RGB = RGB(i, 0, 0) 'VScrollBar1.Value, 0, 0) ', VScrollBar2.Value, VScrollBar3.Value)
            renk = ColorTranslator.FromOle(_RGB)
            'RectangleShape1.FillColor = renk
            PictureBox1.BackColor = renk
            PictureBox1.Refresh()
        Next i

This allows the system to monitor for mouse movements or clicks or other needed to respond events. You do not show how long your timer.ticks are for. You may also consider putting a longer time between ticks to also allow the system to respond. Depending on how long this takes to run, and the length of you timer ticks, I would also add timer.stop as the first line of the event, and add timer.start as the last line. This will prevent multiple events running, and really using up the resources.

Application.doevents can have draw backs in some instances, I do not see how this would be one.

Another approach would be to use a separate thread for your timer events. But, that would depend on the overall desired affect and the size of the program.

While timer working, why form can not move properly ?

3 solutions

Solution 1

Solution 2

Solution 3

Add your solution here

Preview 0