Пишу планировщик задач — как остановить проигрывание события?

Question

[Максим К]

Пишу простой планировщик задач - "для себя", если есть примеры готовых решений, которые можно повторить - с удовольствием гляну. Пишу с использованием WinForms.

Есть таймер в котором каждую секунду берется значение часы и минуты + берется значение часы и минуты из файла и если они совпали, то проигрывается звуковой файл.

private void timer1_Tick(object sender, EventArgs e)
        {
            int h = DateTime.Now.Hour;
            int m = DateTime.Now.Minute;
            int s = DateTime.Now.Second;

            string time = "";

            if (h < 10)
            {
                time += "0" + h;
            }
            else
            {
                time += h;
            }

            time += ":";

            if (m < 10)
            {
                time += "0" + m;
            }
            else
            {
                time += m;
            }

            time += ":";

            if (s < 10)
            {
                time += "0" + s;
            }
            else
            {
                time += s;
            }

            toolStripStatusLabel1.Text = time;


            string str = null;
            string[] strArr = null;
            str = dataGridView1[1, 0].Value.ToString();
            char[] splitchar = { ':' };
            strArr = str.Split(splitchar);

            if (h.ToString() == strArr[0] && m.ToString() == strArr[1]) 
            {
                    playSimpleSound(); 
            }
        }

Пока время совпадает, каждую секунду проигрывается этот файл, но надо что бы при нажатии кнопки стоп - файл переставал проигрываться

private void button1_Click(object sender, EventArgs e)
        {
            stopSimpleSound();
        }

Но нажимая кнопку, звук все равно продолжает играть каждую секунду. Как правильно воспроизводиться звук пока не нажата кнопка? Таймер должен тика дальше, т.к. далее будет список задач.

Comments

[Василий Банников]

1. Именуйте всё нормально, а не button1, timer1
2. Почистите код в методе timer1_tick

1. Зачем сокращать часы/минуты/секунды до h/m/s?
2. Три раза вызывается DateTime.Now
3. Какая-то совсем не очевидная логика по формированию строчки со временем.
Кажется, что вам нужен DateTime.Now.ToString("HH:mm:ss");
4. Самое главное: совсем не очевидное условие

if (h.ToString() == strArr[0] && m.ToString() == strArr[1])

Кажется, оно тут вообще не нужно, но я не уверен.

Подумал я немного над вашим timer_tick и придумал такое:

private void timer1_Tick(object sender, EventArgs e)
{
    var now = DateTime.Now.TimeOfDay;
    toolStripStatusLabel1.Text = now.ToString("hh:mm:ss");

    var alarmTimeText = dataGridView1[1, 0].Value.ToString();
    var alarmTime = TimeSpan.ParseExact(alarmTimeText, "hh:mm:ss", null, TimeSpanStyles.None);

    if (now == alarmTime) 
    {
        playSimpleSound(); 
    }
}

[Максим К]

Василий Банников, Проверил - выдает ошибку в строке :

var alarmTime = TimeSpan.ParseExact(alarmTimeText, "hh:mm:ss", null, System.Globalization.TimeSpanStyles.None);

Все компилируется, но при запуске сразу вываливается ошибка ((

************** Текст исключения **************
System.FormatException: Входная строка имела неверный формат.
в System.Globalization.TimeSpanFormat.FormatCustomized(TimeSpan value, String format, DateTimeFormatInfo dtfi)
в System.Globalization.TimeSpanFormat.Format(TimeSpan value, String format, IFormatProvider formatProvider)
в System.TimeSpan.ToString(String format)
в PlaningTimeWork.Form1.timer1_Tick(Object sender, EventArgs e) в H:\C# проекты\PlaningTimeWork\PlaningTimeWork\PlaningTimeWork\Form1.cs:строка 94
в System.Windows.Forms.Timer.OnTick(EventArgs e)
в System.Windows.Forms.Timer.TimerNativeWindow.WndProc(Message& m)
в System.Windows.Forms.NativeWindow.Callback(IntPtr hWnd, Int32 msg, IntPtr wparam, IntPtr lparam)

[Василий Банников]

Максим К, а что лежит в alarmTimeText?

[Максим К]

Василий Банников, 2 числа через : - 12:00

[Василий Банников]

Максим К, Часы и минуты? Тогда надо формат "hh:mm"

Answer 1

[Василий Банников]

В комментарии я написал, что у вас не так в коде - надо бы его переписать, чтобы вам самому было понятнее, что происходит там.

Смотрите, что у вас происходит в в методе playSimpleSound() и stopSimpleSound()
Ошибка где-то там.

И отделите в кои то веки логику представления от бизнес-логики

Comments

[Максим К]

вроде и Троелсена читаю, а с ооп сложно пока, но постараюсь исправить

[Василий Банников]

Максим К, может стоит пока отложить создание GUI-приложений на потом и начать с консольных и веб-приложений? Это всё-таки проще.
Ну и придумывание имён для переменных и методов - это не ООП и в принципе не супер сложно.