Timer

[nwhta]

вот озадачился, написать прогу "таймер"
выкладываю коды свои которые написал, если что не правильно, вы меня пожалуйста поправьте(пробую многое чтобы опыт нарастить, надеюсь вы мне не откажите в помощи)

using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
using System.Windows.Forms;

namespace Ringer
{
    public partial class Form1 : Form
    {
        int min = 0, sek = 0, milisek = 0;

        public Form1()
        {
            InitializeComponent();
            timer1.Interval = 1000;

        }

        private void button1_Click(object sender, EventArgs e)
        {
            
            if (timer1.Enabled)
            {
                timer1.Enabled = false;
                button1.Enabled = false;
            }
            else
            {
                timer1.Enabled = true;
                button1.Enabled = false;
            }



        }

        private void timer1_Tick(object sender, EventArgs e)
        {
           int min=0, sek=0, mili=0;
            while (button1.Enabled)
            {
                 for (min = 0; min <= 59; min++)
                {
                    for (sek = 0; sek <= 59; sek++)
                    {
                       for (mili = 0; mili <= 59; mili++)
                        {
                             Console.Write("00:00:00" + label1);
                        }
                    }
                }
            }


        }

        private void button2_Click(object sender, EventArgs e)
        {
            timer1.Enabled = false;

            
                
        }
    }
}

[GetMax]

вы меня пожалуйста поправьте

1. Не нужно объявлять несколько раз переменные с одним названием. Это будут разные переменные по уровню видимости. Достаточно один раз объявить переменные на уровне класса и работать с ними в разных методах если это необходимо.
2. В методе таймера сдавать переменные не нужно. Читайте п1.
3. Циклы в методе таймера не используются. Работа таймера итак происходит "в цикле" с некоторым установленным интервалом.
4. Не нужно смешивать Console и WinForms. Это я про вывод результата на экран.
5. Миллисекунды у вас считаться не будут, потому что интервал таймера установлен в 1000, а это 1 секунда.
Мой вариант таймера

 public partial class Form1 : Form
    {
        int hour, min, sek;
        public Form1()
        {
            InitializeComponent();
           
        }

        private void button1_Click(object sender, EventArgs e)//старт
        {
            if (!timer1.Enabled)
            {
                button1.Enabled = false;
                button2.Enabled = true;
                timer1.Enabled = true;
            }
        }

        private void timer1_Tick(object sender, EventArgs e)
        {
            if (button2.Enabled)
            {
                sek++;
                if (sek > 59)
                {
                    sek = 0;
                    min++;
                }

                if (min > 59)
                {
                    min = 0;
                    hour++;
                }

                if (hour > 23)
                {
                    hour = 0;
                }
            }

            label1.Text = hour.ToString() + ":" + min.ToString() + ":" + sek.ToString();
        }

        private void Form1_Load(object sender, EventArgs e)
        {
            timer1.Interval = 1000;
            timer1.Enabled = false;
            button2.Enabled = false;
            label1.Text = "0:0:00";
            hour = 0;
            sek = 0;
            min = 0;
        }

        private void button2_Click(object sender, EventArgs e)//стоп
        {
            timer1.Enabled = false;
            button1.Enabled = true;
            button2.Enabled = false;
           
        }
    }

p.s. Надеюсь нигде не ошибся в объяснениях.

[Stilet]

А в чем смысл сего таймера? Т.е. я имею ввиду что твои четыре цикла в тике просто загрузят на 100% проц. Что нужно? - секундомер?
В таймерах для измерения времени циклы не ставят - таймер сам по себе цикл.

Console.Write("00:00:00" + label1);

Вот ума не приложу, зачем мешать консольные и оконные методы... Что это дает?

[nwhta]

Сообщение от GetMax

1. Не нужно объявлять несколько раз переменные с одним названием. Это будут разные переменные по уровню видимости. Достаточно один раз объявить переменные на уровне класса и работать с ними в разных методах если это необходимо.
2. В методе таймера сдавать переменные не нужно. Читайте п1.
3. Циклы в методе таймера не используются. Работа таймера итак происходит "в цикле" с некоторым установленным интервалом.
4. Не нужно смешивать Console и WinForms. Это я про вывод результата на экран.
5. Миллисекунды у вас считаться не будут, потому что интервал таймера установлен в 1000, а это 1 секунда.
Мой вариант таймера

 public partial class Form1 : Form
    {
        int hour, min, sek;
        public Form1()
        {
            InitializeComponent();
           
        }

        private void button1_Click(object sender, EventArgs e)//старт
        {
            if (!timer1.Enabled)
            {
                button1.Enabled = false;
                button2.Enabled = true;
                timer1.Enabled = true;
            }
        }

        private void timer1_Tick(object sender, EventArgs e)
        {
            if (button2.Enabled)
            {
                sek++;
                if (sek > 59)
                {
                    sek = 0;
                    min++;
                }

                if (min > 59)
                {
                    min = 0;
                    hour++;
                }

                if (hour > 23)
                {
                    hour = 0;
                }
            }

            label1.Text = hour.ToString() + ":" + min.ToString() + ":" + sek.ToString();
        }

        private void Form1_Load(object sender, EventArgs e)
        {
            timer1.Interval = 1000;
            timer1.Enabled = false;
            button2.Enabled = false;
            label1.Text = "0:0:00";
            hour = 0;
            sek = 0;
            min = 0;
        }

        private void button2_Click(object sender, EventArgs e)//стоп
        {
            timer1.Enabled = false;
            button1.Enabled = true;
            button2.Enabled = false;
           
        }
    }

p.s. Надеюсь нигде не ошибся в объяснениях.

Хм...........очень интересно Спасибо

[nwhta]

Сообщение от Stilet

А в чем смысл сего таймера? Т.е. я имею ввиду что твои четыре цикла в тике просто загрузят на 100% проц. Что нужно? - секундомер?
В таймерах для измерения времени циклы не ставят - таймер сам по себе цикл.

Вот ума не приложу, зачем мешать консольные и оконные методы... Что это дает?

Извиняюсь, это ошибка, я просто учусь))

[Stilet]

это ошибка

Ну не то чтобы ошибка, но к ошибкам может привести.

[phomm]

Мда...
я бы сказал, что тут всё неправильно.
1 в событии нажатия кнопки Вы делаете её неактивной

private void button1_Click(object sender, EventArgs e)
...
button1.Enabled = false;

и кликуть по ней уже будет нельзя по сути никогда (а Вам по ней , надо кликнуть насколько видно из кода, в обработчике клика стоит иф - т.е. проверка условия, каковая имеет смысл только при нескольких кликах). Что исправить - сказать нельзя никак, ибо неизвестно чего вы хотите добиться.
2. обработчик тика - сущий бред - тройной цикл с обрамляющим вайлом вообще как козни сатаны тут смотрится.
3. Console.Write("00:00:00" + label1); верх киберпреступного гения. в виндовсформс проекте выводить что-то на консоль (такое возможно конечно, но не для Вашего уровня), пишется же сущность , которая ввергнет любого в ужас, некая операция "сложения" (а само оно тут неприменимо, не числа ибо) строки и объекта, что совершенно неприемлемо в данном языке.

По факту (как я понимаю задачу, это таймер с возможностью паузы), надо выкинуть всё, кроме public Form1() и сделать

DateTime start;
int ticks = 0;
public Form1()
{
    InitializeComponent();
    start = new DateTime();
    timer1.Interval = 100;
    button1.Text = "Start/pause";
    button2.Text = "Stop";
}
private void button1_Click_1(object sender, EventArgs e)
{
    timer1.Enabled = !timer1.Enabled;
}
private void timer1_Tick(object sender, EventArgs e)
{
    ticks++;
    timeshow();
}
private void timeshow()
{
    label1.Text = string.Format("{0:HH:mm:ss:fff}", start.AddMilliseconds(timer1.Interval * ticks));
}
private void button2_Click(object sender, EventArgs e)
{
    ticks = 0;
    timer1.Enabled = false;
    timeshow();
}

Интервал таймера надо подобрать, ибо винда не умеет работать в реальном времени с высокой точностью, около 100 миллисекунд - нормально.

упд. Ух ёклмн, пока набирал пост уже пачка ответов приехала... надо блин приучать себя набирать в ворде и перед постингом обновлять тему, сколько раз уже обжигался.

[nwhta]

Сообщение от phomm

Мда...
я бы сказал, что тут всё неправильно.
1 в событии нажатия кнопки Вы делаете её неактивной

private void button1_Click(object sender, EventArgs e)
...
button1.Enabled = false;

и кликуть по ней уже будет нельзя по сути никогда (а Вам по ней , надо кликнуть насколько видно из кода, в обработчике клика стоит иф - т.е. проверка условия, каковая имеет смысл только при нескольких кликах). Что исправить - сказать нельзя никак, ибо неизвестно чего вы хотите добиться.
2. обработчик тика - сущий бред - тройной цикл с обрамляющим вайлом вообще как козни сатаны тут смотрится.
3. Console.Write("00:00:00" + label1); верх киберпреступного гения. в виндовсформс проекте выводить что-то на консоль (такое возможно конечно, но не для Вашего уровня), пишется же сущность , которая ввергнет любого в ужас, некая операция "сложения" (а само оно тут неприменимо, не числа ибо) строки и объекта, что совершенно неприемлемо в данном языке.

По факту (как я понимаю задачу, это таймер с возможностью паузы), надо выкинуть всё, кроме public Form1() и сделать

DateTime start;
int ticks = 0;
public Form1()
{
    InitializeComponent();
    start = new DateTime();
    timer1.Interval = 100;
    button1.Text = "Start/pause";
    button2.Text = "Stop";
}
private void button1_Click_1(object sender, EventArgs e)
{
    timer1.Enabled = !timer1.Enabled;
}
private void timer1_Tick(object sender, EventArgs e)
{
    ticks++;
    timeshow();
}
private void timeshow()
{
    label1.Text = string.Format("{0:HH:mm:ss:fff}", start.AddMilliseconds(timer1.Interval * ticks));
}
private void button2_Click(object sender, EventArgs e)
{
    ticks = 0;
    timer1.Enabled = false;
    timeshow();
}

Интервал таймера надо подобрать, ибо винда не умеет работать в реальном времени с высокой точностью, около 100 миллисекунд - нормально.

упд. Ух ёклмн, пока набирал пост уже пачка ответов приехала... надо блин приучать себя набирать в ворде и перед постингом обновлять тему, сколько раз уже обжигался.

Спасибо за поддержку

[phomm]

Рад, что это способствует Вашему росту.

Однако, код оформлять Вы почти сразу научились, а вот из цитаты желательно удалять большую часть содержания, (или вообще всю, сделав только обращение к соотв. форумчанину), или же оставив конкретные небольшие кусочки, если Вам надо на соотв. элементы поста ответить. Иначе получается "оверквотинг", что есть нарушение на данном и большинстве других форумов.

[nwhta]

Сообщение от phomm

}
private void timer1_Tick(object sender, EventArgs e)
{
ticks++;
timeshow();
}
private void timeshow()
{
label1.Text = string.Format("{0:HH:mm:ss:fff}", start.AddMilliseconds(timer1.Interv al * ticks));
}

А можно в подробностях эти 2 метода разобрать?(и коды соответственно)