Timer control waiting time

Question

0.00/5 (No votes)

See more:

Iam Trying to Show values in Text Box, But Loops gets faster and exists out.
I have given Wait Time. But It doesn't respond.

C#

for (int i = j; i <= ds.Tables[0].Rows.Count; i++)
           {
                   string Sname = ds.Tables[0].Rows[i][SB].ToString();
                   txt_TextBox1.Text = Sname.ToString();
                   txt_TextBox1.Focus();
              // Delay Time Which I have taken from Input User- 8000 MilliSeconds
               Task.Delay(Convert.ToInt32(Stimer));
             }

What I have tried:

I have made Text box focused. But it exists fast from the loop.

Posted 18-Aug-16 2:52am

vinodh muthusamy

Updated 19-Aug-16 1:00am

Add a Solution

Comments

Andy Lanng 18-Aug-16 10:27am

Why are you setting the delay in a loop. It doesn't matter how many times you set it to [8000], it'll still be 8000 from the start, not from when you set it.
On reflection, I may be wrong

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)

Karthik_Mahalingam · Answer 1 · 2016-08-18T18:49:00

Follow this example

C#

async Task MyDelay(int time)
       {
           await Task.Delay(time);
       }

       private async void button1_Click(object sender, EventArgs e)
       {
           int delayinMilliSec = 1000; // 1 second delay
           for (int i = 0; i < 10; i++)
           {
               txt_TextBox1.Text = i.ToString();
               txt_TextBox1.Focus();
               await MyDelay(delayinMilliSec);
           }

       }

referred from Thread.Sleep vs. Task.Delay [^]

In your code

C#

for (int i = j; i <= ds.Tables[0].Rows.Count; i++)

<= results in index exception,

C#

string Sname = ds.Tables[0].Rows[i][SB].ToString();

always use

Philippe Mori · Answer 2 · 2016-08-18T04:36:00

The call to ToString() after Sname is completly useless as you already have a string.

It does not make much sense to give the focus to a control inside a loop.

Your variables are not properly named according to .NET convention. Variables should start with a lowercase letter.

Your variable name are not descriptive.

Usually, it is better to use int.TryParse (Int32.TryParse Method (System)[^]) to convert a string to an integer.

It does not make much sense to directly have a delay inside UI thread as the application will stop responding. It would be a good idea to read the documentation: Task.Delay Method (TimeSpan) (System.Threading.Tasks)[^]. You should probably use await keyword as in the example so that the UI thread won't be blocked.

Also check the version that take an integer as it give some extra information about the use of await: Task.Delay Method (Int32) (System.Threading.Tasks)[^].

See also that code: TechNet Wiki[^]

Some extra links:
Don't deadlock with async and await[^]
Will it block? Debunking async/await pitfalls - Journey of Code[^]

Ralf Meier · Answer 3 · 2016-08-19T01:00:00

Try it like this :

C#

private int cnt = 0;

private void TimerMethod()
{
    string sname = ds.Tables[0].Rows[i][SB].ToString();
    txt_TextBox1.Text = sname;

    cnt += 1;
    if (cnt > ds.Tables[0].Rows.Count) { cnt = 0; }
}

... and call this Timer-Method by the Timer.Tick (assign the method to this Event)
Also you should set the delay-Time for the actualisation inside the Timer-Intervall (the Property of the Timer)

Additional:
If you want to have your TextBox active you should use TextBox.Select instead of TextBox.Focus.

Timer control waiting time

3 solutions

Solution 2

Solution 1

Solution 3

Add your solution here

Preview 0