c# timer component creation

Question

0.00/5 (No votes)

See more:

i am very new to C# i used the following code for my component class when i add this is to my windows application and calling start function the timer is not elapsing and the count value is not displaying what may the problem can anyone clear it.

C#

using System;
using System.Collections.Generic;
using System.Text;
using System.Timers;
using System.ComponentModel;
namespace timerer
{
    public class timercomp:Component
    {
        public int count=0;

        System.Timers.Timer myTimer = new System.Timers.Timer();

        public timercomp(IContainer container)
        {
            container.Add(this);
             }
             public int start()
             {

                myTimer.Enabled = true;
                 myTimer.Interval += 1000;
                 myTimer.Elapsed += new ElapsedEventHandler(OnTimer);
                myTimer.AutoReset = true;
                 return count;
             }
        public void OnTimer(Object source, ElapsedEventArgs e)
        {
            myTimer.Enabled = false;
            myTimer.Interval += 1000;
            count = count + 1;
            start();

        }

    }
}

Posted 13-Sep-12 4:47am

suresh varman

Add a Solution

Comments

[no name] 13-Sep-12 10:50am

Where is it that you call MyTimer.Start()?

[no name] 13-Sep-12 10:52am

And you do realize that everytime that your timer event fires, you are adding a new event handler right? And return count makes no sense?

2 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)

BobJanova · Accepted Answer · 2012-09-13T05:38:00

Solution 2

If you want a timer, you should just use one of the existing timers (System.Windows.Forms.Timer or System.Threading.Timer). Your component doesn't make sense and is completely unnecessary.

Posted 13-Sep-12 5:38am

BobJanova

Philip Stuyck · Accepted Answer · 2012-09-13T05:36:00

C#

using System;
using System.Collections.Generic;
using System.Text;
using System.Timers;
using System.ComponentModel;
namespace timerer
{
    public class timercomp:Component
    {
        public int count=0;
 
        System.Timers.Timer myTimer = new System.Timers.Timer();
 
        public timercomp(IContainer container)
        {
            container.Add(this);
            myTimer.Elapsed += new ElapsedEventHandler(OnTimer);
        }
        
        public int start()
        {
 
                myTimer.Enabled = true;
                 myTimer.Interval += 1000;
                 myTimer.AutoReset = true;
                 return count;
        }
        
        public void OnTimer(Object source, ElapsedEventArgs e)
        {
            myTimer.Enabled = false;
            myTimer.Interval += 1000;
            count = count + 1;
            start();
 
        }
 
    }
}

//somewhere in your code you need
timercomp comp = new timercomp();
timercomp.start();

don't know why you return count in start, when it is called outside the code above it will always return 0. You should see it incrementing using the debugger though.
Note your intervals are getting bigger and bigger, don't know if that was your intention.