How to make a fixed alarm ?

Question

2.00/5 (1 vote)

See more:

I would like to make the same result but by using (date time) in if condition and count the rest time to alarm.

Thanks for the reply.

C#

using system.media;

//code in time tick
   private void timer1_Tick(object sender, EventArgs e)
        {
            label2.Text = DateTime.Now.ToString("hh:mm:ss") + " " + DateTime.Now.ToString("tt");

        }
//code in form_load 
   private void Form1_Load(object sender, EventArgs e)
        {
            soundplay snplayer= new soundplay (alarm_test.properties.resources.alarm);
            if (label2.text=="06:00:00 AM")
                snplayer.play();
        }

Posted 16-Aug-14 3:52am

Peter M. Adeeb

Updated 16-Aug-14 4:57am

André Kraak

v3

Add a Solution

Comments

OriginalGriff 16-Aug-14 10:57am

Don't "bump" your question: adding information is fine, but just changing it to get it back to the top of the queue is rude and unhelpful.
Bear in mind it is a weekend, so there are less people here than normal - have some patience!

Richard MacCutchan 16-Aug-14 11:23am

There is no point in putting that test in your Form1_Load method, since that only gets executed once and that is before the timer tick will take. Add the test in the timer method, and just check the TimeOfday value of the DateTime object, not some string that is set elsewhere.

Peter M. Adeeb 18-Aug-14 7:36am

great, when i put the code in form1_load it doesn't work well but, when i put it in time tick it's working well , thanks

Sergey Alexandrovich Kryukov 16-Aug-14 14:20pm

Condition related to time based on comparison two strings, instead of DateTime? Very bad...
—SA

1 solution

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)

Ravi Bhavnani · Answer 1 · 2014-08-16T09:04:00

Solution 1

There are a couple of things wrong with your code.

Setting the text of label2 should be simplified to:

label2.Text = DateTime.Now.ToString("hh:mm:ss tt");

As others have already pointed out, move the if test to the timer tick handler, as in:
```
if (DateTime.Now.TimeOfDay == TimeSpan.FromHours(6)) {
    snplayer.play();
}
```

Warning: You may still find sndplayer.play() will never execute! Can you guess why?

/ravi

Posted 16-Aug-14 9:04am

Ravi Bhavnani

Updated 16-Aug-14 9:05am

v2

Comments

Peter M. Adeeb 18-Aug-14 7:39am

I think you didn't define (sndplayer)
the miss code is
soundplay snplayer= new soundplay (alarm_test.properties.resources.alarm);
thanks for the replay ,

Ravi Bhavnani 18-Aug-14 7:46am

No, I assumed you would define sndplayer outside the scope of my answer.

The problem is, it's very unlikely that your timer will fire when it's EXACTLY 6:00:00.000 am. For this reason, you should play the sound the first time the time (to the nearest minute only) is 6:00, and then disable the timer.

/ravi