Оптимизация кода

@leoniduak · Регистрация: 24.04.2017

Author24 — интернет-сервис помощи студентам

Всем привет написал простую программу, в которой две кнопки запускают / останавливают два таймера.
Вопрос как можно оптимизировать код? как не создавать, например, 2 разных таймера и 2 разных метода которые почти одинаковые. Пробовал как то применить перегрузку методов, но что то не выходит

C#

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 Threating_My_Version
{
    public partial class Form1 : Form
    {
        int increase;
       
        public Form1()
        {
            InitializeComponent();
            BackColor = Color.Aquamarine;
        }
 
        private void btn_left_Click(object sender, EventArgs e)
        {
            Off_On_Left();
        }
 
        private void btn_right_Click(object sender, EventArgs e)
        {
            Off_On_Right();
        }
 
        private void timer1_Tick(object sender, EventArgs e)
        {
            increase++;
           
            label_left.Text = increase.ToString();
        }
 
        private void timer2_Tick(object sender, EventArgs e)
        {
            increase++;
            label_right.Text = increase.ToString();
        }
 
        public void Off_On_Left()
        {
            timer1.Enabled = !timer1.Enabled;
        }
 
        public void Off_On_Right()
        {
            timer2.Enabled = !timer2.Enabled;
        }       
    }
}

@ViterAlex · 17.07.2017, 22:39

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

как не создавать, например, 2 разных таймера и 2 разных метода которые почти одинаковые

Очень правильный ход мыслей. Можно объединить в юзерконтрол кнопку и таймер. Сделать у этого контрола свойство Increase и событие Tick. Но это громоздко и куча ненужного кода.
Проще дополнить класс кнопки таймером:

C#

public class ButtonWithTimer : Button
{
    private Timer _timer = new Timer() { Interval = 500 };
    public int Increase { get; set; }
 
    public int Interval
    {
        get
        {
            return _timer.Interval;
        }
        set
        {
            _timer.Interval = value;
        }
    }
 
    public event EventHandler Tick;
 
    protected override void OnClick(EventArgs e)
    {
        base.OnClick(e);
        _timer.Tick -= _timer_Tick;
        _timer.Tick += _timer_Tick;
        _timer.Enabled = !_timer.Enabled;
    }
 
    private void _timer_Tick(object sender, EventArgs e)
    {
        Increase++;
        OnTick();
    }
 
    protected virtual void OnTick()
    {
        Tick?.Invoke(this, EventArgs.Empty);
    }
}

Теперь каждая кнопка при нажатии будет запускать или останавливать таймер, а таймер при каждом тике будет делать инкремент и вызывать событие Tick, но уже у нашей кнопки. В принципе, такое бинарное поведение подводит к использованию CheckBox {Appearance = Appearance.Button}

@leoniduak · 19.07.2017, 10:12 **[ТС]**

Спасибо,
А что значит этот код ?

C#

1	Tick?.Invoke(this, EventArgs.Empty);

@ViterAlex · 19.07.2017, 10:18

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

А что значит этот код ?

null-оператор. Это значит, что если на событие подписан метод, то вызвать этот метод с данными параметрами. Можно записать более развёрнуто:

C#

1
2
3

var handle = Tick;
if (handle != null)
  handle.Invoke(this, EventArgs.Empty);

@leoniduak · 19.07.2017, 10:37 **[ТС]**

А, все. Понял)

@leoniduak 0 / 0 / 0 Регистрация: 24.04.2017 Сообщений: 37
	19.07.2017, 10:37 [ТС]	5
	А, все. Понял) 0

Опции темы