[C#]Wątki

[Zdziwiony 0]

Witam. Oto mój kod:

MyProgressBar.IsIndeterminate = true;
if(CheckBox.IsChecked == true)
           {
               Thread thr = new Thread(funkcja);
               thr.Start();           
           }
if(CheckBox2.IsChecked == true)
           {
               Thread thr = new Thread(funkcjanr2);
               thr.Start();           
           }
//wątek dalej się wykonuje....
thr.Join(); // jeżeli tego użyje zawiesi się całe okno
MyProgressBar.IsIndeterminate = false;

 

Jak widać wyżej za pomocą tego kodu chce wykonać jakieś czynności w osobnych wątkach, jednak przed tym "uruchamiam" sobie progressbara, a po wykonaniu tych wątków chcę go zatrzymać. Niestety nie mogę tego zrobić ponieważ progressbar zatrzyma się chwile po wywołaniu jakiegoś wątku...(Wszystko przez to, że główny wątek nadal będzie sobie działał). Natomiast jeżeli użyje metody thr.Join(); to całe okno zawiesi się i progressbar w ogóle nie zadziała. Ma ktoś jakiś pomysł na to jak rozwiązać ten problem ?

 

Proszę o pomoc.

[misiakufal 0]

na końcu wątku daj sobie:

 

           this.Invoke((MethodInvoker)delegate
           {
               MyProgressBar.IsIndeterminate = false;
           });

 

zakładam ze this dziedziczy po Control.

 

Ale tak w zasadzie żeby to wygladało poprawnie to;

Nazywaj obiekty i zmienne normalnie, a nie CheckBox1 Checkbox2 itp...

Ponieważ oba warunki mogą być jednocześnie spełnione, to mogą sie odpalić oba wątki naraz. Wtedy przed zakończeniem danego wątka musisz sprawdzić czy trzeba zatrzymać pbara, czy jezscze czekamy na drugi wątek.

 

Zrób sobie zmienną int threadsCount = 2;

a potem funkcję

private void stopProgressBarIfNecessary()
{
  threadsCount--;
  if(threadsCount==0)
  {
      this.Invoke((MethodInvoker)delegate
      {
          MyProgressBar.IsIndeterminate = false;
      });
  }
}

 

PS. Z Pbara zakładam ze piszesz to w WPF, a "this.Invoke" jest dla WinFormsów, wiec poszukaj w google jak sie dobrać do GUI z wątka w WPFie.

[BlackBishop 1]

mówię z góry, że nie znam c# ale rozsądek podpowiada, że po wyjściu ze scope {} thr ginie i join potem jest robiony na zniszczonym obiekcie

[misiakufal 0]

BlackBishop

 

tak, metoda z thr.Join jest w ogóle niekompilowalna, bo wyszedł poza scope thr... ale to nie jest podstawą problemu

[Zdziwiony 0]

na końcu wątku daj sobie:

 

           this.Invoke((MethodInvoker)delegate
           {
               MyProgressBar.IsIndeterminate = false;
           });

 

zakładam ze this dziedziczy po Control.

 

Ale tak w zasadzie żeby to wygladało poprawnie to;

Nazywaj obiekty i zmienne normalnie, a nie CheckBox1 Checkbox2 itp...

Ponieważ oba warunki mogą być jednocześnie spełnione, to mogą sie odpalić oba wątki naraz. Wtedy przed zakończeniem danego wątka musisz sprawdzić czy trzeba zatrzymać pbara, czy jezscze czekamy na drugi wątek.

 

Zrób sobie zmienną int threadsCount = 2;

a potem funkcję

private void stopProgressBarIfNecessary()
{
  threadsCount--;
  if(threadsCount==0)
  {
      this.Invoke((MethodInvoker)delegate
      {
          MyProgressBar.IsIndeterminate = false;
      });
  }
}

 

PS. Z Pbara zakładam ze piszesz to w WPF, a "this.Invoke" jest dla WinFormsów, wiec poszukaj w google jak sie dobrać do GUI z wątka w WPFie.

Tak pisze w wpf, i mam coś takiego:

            #region obsluga GUI
           Dispatcher.Invoke(new Action(delegate { TabCore.IsEnabled = false; }));
           //progressbar
           Dispatcher.Invoke(new Action(delegate { MyProgressBar.IsIndeterminate = true; }));
           #endregion

Jedyną rzeczą jaką mi teraz do szczęścia potrzeba to sprawdzenie czy trzeba zatrzymać progressbara, czy też nie.

Ale nie za bardzo rozumiem tutaj twoją metodę tego sprawdzania ;/ (W swoim programie mam większą ilość checkboxów, a i mam w planach dodać ich jeszcze więcej i wydaje mi się, że w takim przypadku ta metoda zawiedzie)

BlackBishop

 

tak, metoda z thr.Join jest w ogóle niekompilowalna, bo wyszedł poza scope thr... ale to nie jest podstawą problemu

Rzeczywiście , pisałem z pamięci i nie zauważyłem ;p

[.Mad. 0]

MyProgressBar.IsIndeterminate = true;
await twojaFunkcja();
MyProgressBar.IsIndeterminate = false;

Poczytaj o async await w .NET 4.5.

[SmiercionosnyOddechSmokaPL 1]

mówię z góry, że nie znam c# ale rozsądek podpowiada, że po wyjściu ze scope {} thr ginie i join potem jest robiony na zniszczonym obiekcie

Czystonz ciekawości pytam: masz na myśli zmienne automatyczne?

[misiakufal 0]

Jeśli masz większą ilość chekboxów, to zorganizuj je w kolekcje, tak samo jak wątki, moze opakuj je w obiekty, rejestruj które wątki się zakończyły i usuwaj je z kolekcji, jak ona będzie pusta, to zatrzymuj pbara.

[Zdziwiony 0]

MyProgressBar.IsIndeterminate = true;
await twojaFunkcja();
MyProgressBar.IsIndeterminate = false;

Poczytaj o async await w .NET 4.5.

Dzięki!

Ta funkcja jest dużo prostsza od bawienia się w wątki

 

Zmodyfikowany kod:

        private async void Start_Click(object sender, RoutedEventArgs e)
       {        
           //zablokowanie GUI i włączenie progressbara
           TabCore.IsEnabled = false;
           MyProgressBar.IsIndeterminate = true;
           if (CheckFunkcja.IsChecked == true)
           {
               //czekaj na wykonanie wątku
               await Task.Run(() => Funkcja()); 
           }
           //włączenie GUI i wyłączenie progressbara
           TabCore.IsEnabled = true;
           MyProgressBar.IsIndeterminate = false;
       }

 

Mam tylko wątpliwości, czy powinienem dopisywać tutaj

private [b]async[/b] void

, czy może powinienem stworzyć osobną funkcję, i po kliknięciu przycisku ją wywoływać ?

[.Mad. 0]

Jeżeli usuniesz "async" z podanego kodu, to przecież program się nie skompiluje. Tylko metody/funkcje typu "async" mogą używać wewnątrz await. Pamiętaj o jednej istotnej rzeczy, jeżeli tworzysz swoją metodę asynchroniczną, która w środku korzysta ze słowa "await", to nigdy nie zwracaj void. Przykładowo:

 

Źle

private async void MojaMetoda()

Dobrze

private async Task MojaMetoda()

Więcej tutaj http://msdn.microsoft.com/en-us/magazine/jj991977.aspx. Tutaj może pojawić się pytanie - czy w takim razie powinienem dodać Task do obsługi zdarzenia Start_Click? Z jednej strony tak, ale z drugiej nie jest to możliwe (program się nie skompiluje). WPF i WindowsForms zostały napisane dużo wcześniej i nie są w pełni zgodne z obecnymi standardami. To jeden z wyjątków gdy stosuje się "async void".

 

Jeżeli chcesz wew. swojej metody tylko odpalić Task'a i go zwrócić to wystarczy (bez async):

private Task MojaMetoda()

Natomiast w obsłudze kliknięcia piszesz:

await twojaFunkcja();

Możesz też odpalić kilka Task-ów na raz i poczekać na wszystkie.

await Task.WhenAll(task1, task2);

Jeżeli bardziej jesteś zainteresowany programowaniem desktopowym (WPF itp.) to jak będziesz miał czas to poczytaj o Reactive Extensions (jak już zapoznasz się z Taskami oraz async await).

[Zdziwiony 0]

Dzięki za wyjaśnienie