1

Имеется класс:

  public class ConvertProcess
  {
    public ConvertState State { get; set; }
    ...
  }

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

Теперь, пишу поверх этого дела форму. Сделал простой вариант - добавил событие изменения State - на нужных мне состояниях добавил dialog.Show() и dialog.Close() соответственно.

Проблема - т.к. события срабатывают в том же потоке, в котором идёт конвертация, диалог отображается, но отрисовка в нём забинденных свойств не отрабатывает, т.к. поток занят.

Как это правильнее сделать?

ПС: если диалог просто вызвать в новом потоке, то он не виснет, но завершить его привязкой к событию уже нельзя, т.к. нельзя обращаться к WPF в разных потоках.

4 ответа 4

3

В обработчике события вам нужно просто перенаправлять вызов в UI поток c помощью диспетчера:

// обработчик события
private void ConvertProcess_StateChanged(object sender, StateChangedEventArgs e)
{
    if (Application.Current.Dispatcher.CheckAccess())
    {
        // мы уже в UI потоке, просто вызываем метод
        StateChanged(e.State);
    }
    else
    {
        // мы не в UI потоке, перенаправляем вызов метода
        Application.Current.Dispatcher.Invoke(StateChanged, e.State);
    }
}

private void StateChanged(State newState)
{
    ...
}

Т.о. процесс конвертации ничего не будет знать о UI и потоках (как и положено), а логика по "маршрутизации" будет находиться на стороне UI.

6
  • уже предлагал такое, но тоже не подходит - тут
    – Stack
    15 янв 2016 в 13:26
  • Именно эвент так завернуть таки не подумал, попробую.
    – Monk
    15 янв 2016 в 13:37
  • @Monk Именно эвент так завернуть таки не подумал, попробую" -- я же давал ссылку на такой способ, а вы сказали: "А разница в чем? Фактически, снова говорю, выполни вот это в другом потоке, а вот это - в своём" -- я так понял, что такой способ вам не подходит.
    – Stack
    15 янв 2016 в 14:51
  • @Stack я по ссылке подумал, что речь идёт об Invoke на каждый чих, когда надо обновить форму. А тут явно событие обёрнуто. В любом случае, я пока ещё не проверил, работает ли такой вариант.
    – Monk
    15 янв 2016 в 15:37
  • Завелось, но тут есть одна особенность - учитывая, что эвенты срабатывают в разных потоках, брать надо не Dispather потока текущего, ибо к нему всегда есть доступ, а Application.Current.Dispatcher.
    – Monk
    16 янв 2016 в 5:45
1

Вы делаете не совсем верно.

Ваша модель действительно должна/имеет право бежать в любом потоке. Но отображаться должны данные VM, а не модели, а VM должна бежать в UI-потоке. Кроме того, бизнес-логика (а решение начать взаимодействие с пользователем — это именно бизнес-логика) должна происходить на уровне VM, а не модели.

Таким образом, вы должны сделать следующее:

  1. Вы заводите VM, соответствующую вашему процессу.
  2. Вы заводите экземпляр этой VM в UI-потоке.
  3. Вы показываете UI в UI-потоке, привязываясь к данным VM.
  4. Вы запускаете модель, которая занимается конвертацией. (VM бежит в своём потоке.)
  5. Модель каким-то образом сигнализирует об изменении своих данных. (Любой способ подойдёт: event, INotifyPropertyChanged, IProgress, IObservable, что угодно.)
  6. VM ловит нотификации, перебрасывает их в UI-поток (любым способом, например через Dispatcher.BeginInvoke или SynchronizationContext.Post) и модифицирует свои данные.
  7. UI подхватывает изменения данных и показывает их.

Обновление: для вашего случая (в начале работы приложения возможна длительная конвертация) подойдёт такая идея:

Когда приложение стартует, MainVM создаёт и запускает InitializationVM. InitializationVM выясняет у модели, нужна ли конвертация. Если конвертация нужна, она запускается (в другом потоке, но это наверное подробности работы модели), и InitializationVM показывает свой UI. Когда конвертация отработала, InitializationVM убивает свой UI, сообщает MainVM, что она отработала, MainVM стартует основную логику приложения (и свой UI).

Если невозможно по каким-то причинам узнать заранее, нужна ли конвертация (хотя в принципе должно быть возможно), есть смысл открывать UI для конвертации только по приходу первого event'а от неё.


Когда вы перебрасываете изменение данных в в UI-поток, я бы очень не рекомендовал синхронные, блокирующие функции наподобие Dispatcher.Invoke или SynchronizationContext.Send — это практически прямой путь к deadlock'ам. Используйте Dispatcher.InvokeAsync или SynchronizationContext.Post.

14
  • А как сделать именно инициализацию ВМ\Формы по событию в модели? Т.е. не держать же мне всю жизнь приложения где то невидимую форму с моделью, которые будут ждать оповещения от модели, только чтобы отобразиться? M - V - VM у меня в целом в голове держится, я именно этот дурацкий кейс не могу понять, как закрывать.
    – Monk
    15 янв 2016 в 13:49
  • @Monk: По-всякому. Для начала, VM. VM первична, она существует до модели, запускает модель, и подписывается на её события. Теперь View. Когда вам нужно показать View, вы делаете это либо прямо из VM (простой, но идеологически не вполне чистый путь), либо через отдельный менеджер, который умеет для данной VM создать для неё View (а также уничтожить).
    – VladD
    15 янв 2016 в 13:52
  • @Monk: Поскольку VM первична, то жизнь приложения — это и есть жизнь (главной) VM. Именно она создаёт, запускает, останавливает и убивает модели.
    – VladD
    15 янв 2016 в 13:53
  • 1
    @Monk: Когда приложение стартует, MainVM создаёт и запускает ConvertationVM. ConvertationVM запускает конвертацию, и показывает свой UI. Когда конвертация отработала, ConvertationVM убивает свой UI, сообщает MainVM, что она отработала, MainVM стартует основную логику приложения (и свой UI). Как-то так.
    – VladD
    15 янв 2016 в 13:59
  • 1
    @Stack: Да, чересчур частая доставка данных в UI-поток — плохо. Я предпочитаю Rx/IObservable, там можно просто попросить доставлять состояние не очень часто и сразу в нужном потоке: eventsFromModel.Throttle().ObserveOn(SynchronizationContext.Current).
    – VladD
    15 янв 2016 в 15:43
1

события срабатывают в том же потоке, в котором идёт конвертация

Чтобы передать данные в основной поток, надо использовать SynchronizationContext

using System.Threading;

static void log(string msg) {
   Console.WriteLine(msg + " " + Environment.CurrentManagedThreadId);
}

class Convert {
    public event EventHandler Changed = delegate { };
    public void Run() {
        log("1");
        var c = SynchronizationContext.Current; // запомнить основной поток
        new Thread(() => {   // запустить отдельный поток
            log("2");
            c.Send(o => {    // вызвать в основном потоке 
                log("3");
                Changed(this, EventArgs.Empty);
            }, null);
        }).Start();
    }
}

var c = new Convert();
c.Changed += (s, e) => log("4");
c.Run()

Результат

msg  threadId
1    6
2    16     
3    6
4    6

Другой способ:

void Changed(object sender, EventArgs e) {
  // если метод был вызван из не-UI-потока, то транслируем его в UI-поток
  if(!Dispatcher.CheckAccess()) {
    Dispatcher.BeginInvoke(new EventHandler(Changed), sender, e);
    return;
  }
  // выполняется в UI-потоке
  textbox.Text = "hello";
}
4
  • 1
    В таком случае, процесс конвертации почему то должен подстраиваться под UI и его проблемы(добавлять какую то синхронизацию, делать свой запуск в отдельном потоке зачем-то), тогда как он простой и однопоточный. Мне не нравится идея, что надо менять рабочий процесс конвертации, чтобы можно было его отобразить. Почему я и задаю этот вопрос.
    – Monk
    15 янв 2016 в 13:16
  • @Monk "Мне не нравится идея, что надо менять рабочий процесс конвертации" -- есть другой способ. тут
    – Stack
    15 янв 2016 в 13:18
  • А разница в чем? Фактически, снова говорю, выполни вот это в другом потоке, а вот это - в своём. Т.е. снова процессу конвертации говорю, какие данные мне понадобятся в UI.
    – Monk
    15 янв 2016 в 13:20
  • @Monk "Фактически, снова говорю, выполни вот это в другом потоке, а вот это - в своём" -- если хотите чтобы передача данных между потоками была скрыта, то можно использовать BackgroundWorker. пример тут
    – Stack
    15 янв 2016 в 13:24
0

Можно попробовать:

  • Создать форму в главном потоке
  • По клику на кнопку - запустить нужных механизм конвертации в другом потоке
  • Добавить делегат и подписаться на события потока.
1
  • Ситуация такая, что процесс конвертации просто лежит где то внутри Core.Init() условного, который надо сделать перед запуском. Т.е. никаких форм нигде нет и "ядро" о них не знает. Соответственно, конвертации по клику тоже нет - само запустится, завершится, и только потом я могу показать основное окно программы. Может что-то плохо описываю =(
    – Monk
    15 янв 2016 в 13:18

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.