Pallaris13th
1.
2.
3.
4.
5.
6.
7.
8.
9.
10.
11.
12.
13.
14.
15.
16.
17.
18.
19.
20.
21.
22.
23.
24.
25.
26.
27.
set
{
lock (_selectedItem)
{
if (_selectedItem == value)
return;
_selectedItem = value;
}
OnPropertyChanged("selectedItem");
if (_selectedItem == "")
return;
Debug.WriteLine("set_selectedItem: Try to start bg work");
if (bgWorker.IsBusy)
{
Debug.WriteLine("bgWorker is busy, cancel async");
bgWorker.CancelAsync();
}
Debug.WriteLine("set_selectedItem: Startint bg work");
while (bgWorker.IsBusy) Thread.Sleep(100);
bgWorker.RunWorkerAsync(_selectedItem);
Debug.WriteLine("set_selectedItem: Bg work started");
}
Вот прям печальный сеттер в ui-потоке. Во-первых, он огромный, нужен отдельный метод в tViewModel, чтобы перезапускать BackgroundWorker с нужным параметром. Во-вторых, судя по коду, он всегда вызывается в UI-потоке, стало быть lock не нужен. В-третьих, блокировка UI-потока, бесконечно ожидая чего-либо - прям вообще плохо.
Ну как мне кажется, ты преувеличиваешь "огромность" сетера. Если убрать отладку, то по сути будет вот что:
1.
2.
3.
4.
5.
6.
7.
8.
9.
10.
11.
12.
13.
14.
15.
set
{
lock (_selectedItem)
_selectedItem = value;
OnPropertyChanged("selectedItem");
if (_selectedItem == "")
return;
if (bgWorker.IsBusy)
bgWorker.CancelAsync();
while (bgWorker.IsBusy) Thread.Sleep(100);
bgWorker.RunWorkerAsync(_selectedItem);
}
Не такой он уж и "огромный". Конечно, конструкция "while (bgWorker.IsBusy) Thread.Sleep(100);" в данном контексте - это тупизм, но я писал уже вечером, поэтому немного протупил, чего уж там.
Что даст вынос кода из сеттера? Время выполнения не уменьшит.
lock - нужен, из разных потоков обращаемся к этим данным.
PallarisЯ так понимаю, BackgroundWorker используется просто, чтоб изучать асинхронность? Рекомендую сразу же начать с Task.
Нет, в асинхронность я умею. Хотел познакомиться именно с этим классом. Обычно если MS пишет какую-либо обёртку, как правило - она хороша, использовать её приятно. Кто ж знал, что не описана типовая, в общем-то ситуация: перезапуск уже работающего воркера.
PallarisХороший тон общения на форуме - написать, как решил проблему. У кого-то может возникнуть похожий вопрос в будущем
Судя по количеству отзывов не подумал, что это кому-то интересно. Я сделал вот так:
1.
2.
3.
4.
5.
6.
7.
8.
9.
10.
11.
12.
13.
14.
15.
16.
17.
if (bgWorker.IsBusy)
{
bgWorker.CancelAsync();
System.Threading.ThreadStart startPoint = new System.Threading.ThreadStart(delegate()
{
while (bgWorker.IsBusy) System.Threading.Thread.Sleep(50);
lock (_selectedItem) bgWorker.RunWorkerAsync(this);
});
Thread waitingThread = new Thread(startPoint);
waitingThread.Start();
}
else
{
bgWorker.RunWorkerAsync(this);
}
