Таймер и WPF

@Di-man · Регистрация: 08.01.2013

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

Есть приложения на WPF, и есть задача, которую нужно выполнять раз в 1 секунду. Для этого я использую system.threading.timer (или можно system.timers.timer), работает всё замечательно, но в методе таймера мне нужно создавать событие, а как раз на это событие подписывается контрол из WPF. Дело в том, что как вы знаете UI поток и методы этих таймеров выполняются в разных потоках. Знаю, можно было бы использовать dispatchtimer, но не хочется забивать пользовательский поток такими частыми процедурами. Пробовал вызов Invoke и BeginInvoke, ошибок нет, но событие не срабатывает. Подскажите, как быть?

Tulosba · 24.03.2013, 00:27

Приведите код с Invoke/BeginInvoke, чтобы прояснить почему не срабатывает.

@Casper-SC · 24.03.2013, 09:36

Сообщение от Di-man

Пробовал вызов Invoke и BeginInvoke, ошибок нет, но событие не срабатывает. Подскажите, как быть?

Покажи весь код класса, иначе сам подумай, как тебе тут помогать будут.

Добавлено через 4 минуты

Сообщение от Di-man

ошибок нет, но событие не срабатывает

Ещё возможен вариант, что надо сделать такие действия, чтобы увидеть ошибку: Отладка -> Исключения. Далее проставить везде галочки где надо (отсеишь).

@Di-man · 24.03.2013, 12:34 **[ТС]**

Вот мой класс

C#

namespace vkontakte.src.vk.message
{
    public class NewMessages
    {
        private string jString;
 
        public int count { get; set; } 
     
        // Событие: новое сообщение
        public event EventHandler<NewMessageEventArgs> NewMessage;
 
        // Событие: новых сообщений нет
        public event EventHandler<EventArgs> EmptyMessage;
 
        // Задержка (интервал вызова функции)
        private int delay = 800;   // 1000
 
 
        //Таймер
        private Timer timer;
 
      
        public NewMessages()
        {
            TimerCallback method = new TimerCallback(checkNewMessage);
 
            timer = new Timer(method);
            timer.Change(0, delay);  
        }
 
 
        void checkNewMessage(object sender)
        {
 
            // Строка с json
            jString = "...";
 
            // Количество сообщений
            count = int.Parse(Regex.Match(jsonString, @"(?<=resp\D*)\d+").Value);
 
            switch (count)
            {
            // --- Новых сообщений нет --- 
                case 0:
 
                    ////////////////////////////////////////////////////////////////////////////////////////
                    OnEmptyMessage(EventArgs.Empty);
                    ////////////////////////////////////////////////////////////////////////////////////////
 
                    break;
 
            // --- Есть новые сообщения --- 
                default:
 
                    ////////////////////////////////////////////////////////////////////////////////////////
                    OnNewMessage(new NewMessageEventArgs(messages.messages.Count, messages, count);
                    ////////////////////////////////////////////////////////////////////////////////////////
 
                    break;
            }            
 
        }
 
        // Событие получения нового сообщения
        protected virtual void OnNewMessage(NewMessageEventArgs e)
        {
            if (NewMessage != null)
                NewMessage(this, e);
        }
 
        // Событие новых сообщений нет
        protected virtual void OnEmptyMessage(EventArgs e)
        {
            if (EmptyMessage != null)
                EmptyMessage(this, e);
        }
        
    }
}

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

Пока нашёл такое решение:

C#

switch (count)
            {
            // --- Новых сообщений нет --- 
                case 0:
 
                    ////////////////////////////////////////////////////////////////////////////////////////
                    App.Current.Dispatcher.BeginInvoke(new Action( () => OnEmptyMessage(EventArgs.Empty) ));
                    ////////////////////////////////////////////////////////////////////////////////////////
 
                    break;
 
            // --- Есть новые сообщения --- 
                default:
                    ////////////////////////////////////////////////////////////////////////////////////////
                    App.Current.Dispatcher.BeginInvoke(new Action(() => OnNewMessage(new NewMessageEventArgs(messages.messages.Count, messages, count))));
                    ////////////////////////////////////////////////////////////////////////////////////////
 
                    break;
            }

Насколько это правильно, и может быть можно сделать лучше?

Psilon · 24.03.2013, 20:46

Di-man, ну как вам сказать, если учесть, что можно сделать в одну строчку и вот это

C#

switch (count)
            {
            // --- Новых сообщений нет --- 
                case 0:
 
                    ////////////////////////////////////////////////////////////////////////////////////////
                    App.Current.Dispatcher.BeginInvoke(new Action( () => OnEmptyMessage(EventArgs.Empty) ));
                    ////////////////////////////////////////////////////////////////////////////////////////
 
                    break;
 
            // --- Есть новые сообщения --- 
                default:
                    ////////////////////////////////////////////////////////////////////////////////////////
                    App.Current.Dispatcher.BeginInvoke(new Action(() => OnNewMessage(new NewMessageEventArgs(messages.messages.Count, messages, count))));
                    ////////////////////////////////////////////////////////////////////////////////////////
 
                    break;
            }

заменить на

C#

1
2

Action action = count == 0 ? () => OnEmptyMessage(EventArgs.Empty) : () => OnNewMessage(new NewMessageEventArgs(messages.messages.Count, messages, count))
App.Current.Dispatcher.BeginInvoke(action);

Хотя я бы не стал делать 2 разных события, проще сделать так:

C#

class Message
{
   public virtual bool IsEmpty {get {return false; }}
   string content;
}
 
class EmptyMessage : Message
{
   public override bool IsEmpty {get {return true;}}  
}

@Di-man · 24.03.2013, 20:52 **[ТС]**

Спасибо, но так не получится:

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

C#

1
2

Action action = count == 0 ? () => OnEmptyMessage(EventArgs.Empty) : () => OnNewMessage(new NewMessageEventArgs(messages.messages.Count, messages, count))
App.Current.Dispatcher.BeginInvoke(action);

Потому что там выполняется много условий, приведён не весь код.

И я не особо понял как и где и зачем делать два класса? У меня один. Просто два разных событий. У них разное количество параметров и разное назначение

Psilon · 24.03.2013, 21:26

Di-man, я ваш же код переписал, что может не получиться? Это эквивалентное преобразование...
Два класса, чтобы сделать одно событие, это намного удобнее... Подробнее можете тут почитать:
http://occs.cs.oberlin.edu/~jw... bjPattern/
http://babylon.wiki-wiki.ru/b/... D0%B0_Null

@Di-man 0 / 0 / 0 Регистрация: 08.01.2013 Сообщений: 9
		1
	Таймер и WPF 23.03.2013, 22:38. Показов 4119. Ответов 6 Метки нет (Все метки) Есть приложения на WPF, и есть задача, которую нужно выполнять раз в 1 секунду. Для этого я использую system.threading.timer (или можно system.timers.timer), работает всё замечательно, но в методе таймера мне нужно создавать событие, а как раз на это событие подписывается контрол из WPF. Дело в том, что как вы знаете UI поток и методы этих таймеров выполняются в разных потоках. Знаю, можно было бы использовать dispatchtimer, но не хочется забивать пользовательский поток такими частыми процедурами. Пробовал вызов Invoke и BeginInvoke, ошибок нет, но событие не срабатывает. Подскажите, как быть? 0

Tulosba :) 4773 / 3267 / 497 Регистрация: 19.02.2013 Сообщений: 9,046
	24.03.2013, 00:27	2
	Приведите код с Invoke/BeginInvoke, чтобы прояснить почему не срабатывает. 0

@Casper-SC 4432 / 2092 / 404 Регистрация: 27.03.2010 Сообщений: 5,657 Записей в блоге: 1
	24.03.2013, 09:36	3
	Сообщение от Di-man Пробовал вызов Invoke и BeginInvoke, ошибок нет, но событие не срабатывает. Подскажите, как быть? Покажи весь код класса, иначе сам подумай, как тебе тут помогать будут. Добавлено через 4 минуты Сообщение от Di-man ошибок нет, но событие не срабатывает Ещё возможен вариант, что надо сделать такие действия, чтобы увидеть ошибку: Отладка -> Исключения. Далее проставить везде галочки где надо (отсеишь). 0

Psilon Master of Orion 6098 / 4954 / 905 Регистрация: 10.07.2011 Сообщений: 14,522 Записей в блоге: 5
	24.03.2013, 21:26	7
	Di-man, я ваш же код переписал, что может не получиться? Это эквивалентное преобразование... Два класса, чтобы сделать одно событие, это намного удобнее... Подробнее можете тут почитать: http://occs.cs.oberlin.edu/~jw... bjPattern/ http://babylon.wiki-wiki.ru/b/... D0%B0_Null 0

Опции темы