Как передать таймер EventHandler?

Question

Я хочу иметь ComboBox, который после SelectedIndexChanged меняет Timer.Interval. Мой код в основном выглядит так:

public Form1()
{
    InitializeComponent();
    Timer AutoRefresh = new Timer();
    AutoRefresh.Tick += new EventHandler(AutoRefresh_Tick);
    var RefreshIntervals = new[] { "4 hours", "2 hours", "1 hour", "15 minutes", "10 seconds" };
    comboBox1.DataSource = RefreshIntervals;
    comboBox1.SelectedIndexChanged += new EventHandler(comboBox1_SelectedIndexChanged);
}

void comboBox1_SelectedIndexChanged(object sender, EventArgs e)
{
    if (AutoRefresh.Enabled == true)
        AutoRefresh.Enabled = false;
    if (comboBox1.SelectedText == "4 hours")
        AutoRefresh.Interval = 14400000;
    else if (comboBox1.SelectedText == "2 hours")
        AutoRefresh.Interval = 7200000;
    else if (comboBox1.SelectedText == "1 hour")
        AutoRefresh.Interval = 3600000;
    else if (comboBox1.SelectedText == "15 minutes")
        AutoRefresh.Interval = 900000;
    else if (comboBox1.SelectedText == "10 seconds")
        AutoRefresh.Interval = 10000;
    AutoRefresh.Enabled = true;
}

Теперь, очевидно, это не работает, потому что comboBox1_SelectedIndexChanged() не имеет ссылки на переменную Timer.

Как я могу изменить свой код для передачи AutoRefresh в comboBox1_SelectedIndexChanged()?

Вероятно, самое время указать, что я все еще новичок в C #. Пожалуйста, будьте добры.

Answer 1

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

Timer AutoRefresh;   
public Form1()
{
  InitializeComponent();
 AutoRefresh = new Timer();
  AutoRefresh.Tick += new EventHandler(AutoRefresh_Tick);

 resh.Interval = 10000;
  AutoRefresh.Enabled = true;

}

Answer 2

Одним из способов может быть объявление объекта Time как члена класса.Тогда вы сможете получить к нему доступ внутри события.

, также вы должны удалить 'throw new NotImplementedException ();'из вашего события, потому что это утверждение выдает исключение

Answer 3

Класс SqlDependency может хорошо работать для вас, на основании вашего комментария:

... Программа ... запрашивает базу данных SQL, которая заполняетDataTable для отображения.Я хочу использовать этот комбинированный список и таймер для автоматического обновления таблицы данных ...

Answer 4

Я произвел некоторый рефакторинг и обнаружил еще одну ошибку в коде. Этот метод работает нормально в моем проекте

void comboBox1_SelectedIndexChanged(object sender, EventArgs e)
    {
        if (AutoRefresh.Enabled)
            AutoRefresh.Enabled = false;
        var selectedItem = comboBox1.SelectedItem.ToString();

        switch (selectedItem)
        {
            case "4 hours":
                AutoRefresh.Interval = 14400000;
                break;
            case "2 hours":
                AutoRefresh.Interval = 7200000;
                break;
            case "1 hour":
                AutoRefresh.Interval = 3600000;
                break;
            case "15 minutes":
                AutoRefresh.Interval = 900000;
                break;
            case "10 seconds":
                AutoRefresh.Interval = 10000;
                break;
        }
        AutoRefresh.Enabled = true;
    }

Вы должны использовать свойство SelectedItem insteed SelectedText

Answer 5

namespace WindowsFormsApplication1
{
    using System;
    using System.Windows.Forms;

    public partial class Form1 : Form
    {
        /// <summary>
        /// The default constructor.
        /// I used the designer, so the InitializeComponent method contains the timer and combobox initialization.
        /// </summary>
        public Form1()
        {
            InitializeComponent();
        }

        /// <summary>
        /// Occurs when the combo box selection changes.
        /// </summary>
        /// <param name="sender">The sender object, i.e., the combobox</param>
        /// <param name="e">The event arguments.</param>
        private void comboBox1_SelectedIndexChanged(object sender, EventArgs e)
        {
            if (autoRefresh.Enabled == true)
            {
                autoRefresh.Enabled = false;
            }

            // so I can easily change the time scale when debugging, I'm using a multiplier
            const int multiplier = 10000;

            // notice I'm using SelectedItem because the event triggered was index changed, not text changed
            var selectedInterval = comboBox1.SelectedItem.ToString();

            // if a valid entry wasn't selected, then we'll disable the timer
            var enabled = true;

            switch (selectedInterval)
            {
                case "4 hours":
                    autoRefresh.Interval = 1440 * multiplier;
                    break;
                case "2 hours":
                    autoRefresh.Interval = 720 * multiplier;
                    break;
                case "1 hour":
                    autoRefresh.Interval = 360 * multiplier;
                    break;
                case "15 minutes":
                    autoRefresh.Interval = 90 * multiplier;
                    break;
                case "10 seconds":
                    autoRefresh.Interval = 1 * multiplier;
                    break;
                default:
                    // we didn't choose a valid entry, or picked blank line
                    enabled = false;
                    break;
            }

            autoRefresh.Enabled = enabled;
        }

        /// <summary>
        /// Occurs every time the timer reaches its interval
        /// </summary>
        /// <param name="sender">The sender object, i.e., the timer.</param>
        /// <param name="e">The event arguments.</param>
        private void AutoRefresh_Tick(object sender, EventArgs e)
        {
            // to ensure the next timer triggers at the desired interval
            // stop the timer while doing the operations in this method (they could be lengthy)
            // and then restart the timer before exiting
            autoRefresh.Enabled = false;

            // give a visual indicator of the timer ticking.
            // Here is where you would do your DB operation.
            MessageBox.Show(string.Format("Hello! time={0}:{1}", DateTime.Now.ToLongTimeString(), DateTime.Now.Millisecond));

            autoRefresh.Enabled = true;
        }
    }
}

Answer 6

Я нашел одну ошибку в вашем коде

void comboBox1_SelectedIndexChanged(object sender, EventArgs e)
{
  throw new NotImplementedException(); // remove this line

Answer 7

Вам нужно поместить таймер в поле в вашем классе.