[Khaine]

#csharp #programowanie

Mirki mam pewien problem z asyncami, bo osobiście nigdy ich nie używałem w ten sposób (robiłem jawnego Taska jak już, i zwykle raczej fire and forget, więc nie musiałem tego synchronizować).

Mam sobie rzecz którą próbuję przerobić na asynchroniczną wersję w ramach testu czy będzie szybciej czy wolniej, ale coś chyba nie do końca ogarniam temat, bo zdaje się, że robi mi się deadlock i nie wiem czemu.

Mam zwieszkę na Task.WaitAll. Co jest nie tak? Bebechów nawet macie po co analizować - synchronicznie to na pewno wszystko dobrze działa, nawet jeśli dało się to ładniej napisać xD

https://pastebin.com/7dy7fWhq

[Khaine]

@kuite22: Czyli mówisz, że ta funkcja nie powinna być async, tylko powinienem zrobić async zaślepkę taką jak jest wewnątrz niej?

[Khaine]

@kuite22: Faktycznie, zmieniłem na coś takiego i zadziałało:

private void #!$%@?()
{
string tags =

[Khaine]

@kuite22: A czemu to nie zadziałało wcześniej? Czego nie zrozumiał?

[Khaine]

@kuite22: No cóż, asynchronicznie się udało ugrać dwie sekundy, więc się opłaca ¯\_(ツ)_/¯

5800 ms -> 3800 ms.

[Khaine]

@kuite22: Swoją drogą jak czytam ten temat na stacku to nadal nie wiem czemu u mnie to nie zadziałało. Koleś to trochę dziwnie zrobił, bo on miał funkcję async na której jeszcze w dodatku robił start new, więc task w tasku jakby.

Jak wejdę sobie do tej listy tasków w momencie dojścia do WaitAll, to taski są jeszcze nierozpoczęte. Jak dojdzie do linii WaitAll, to ewidentnie wszystkie te taski się

[Khaine]

@kuite22 wrócę zaraz do domu to sprawdzę.

[Khaine]

@kuite22:

Task.WaitAll(Task.WhenAll(tasks.ToArray()));

Takie coś też nie działa. WhenAll tworzy taska, który się zakończy jak zakończą się rzeczy które są do niego przypisane. Ten górny task widać też nie łapie.

Chyba trzeba na Stacku się zapytać.

[Khaine]

@kuite22: Ale to nic nie da, bo on utworzy taska który trzeba awaitować. A nie mogę go awaitować, bo Execute nie jest asynciem. Więc aby go awaitować w funkcji nie będącej asynciem muszę nadal użyć WaitAll.

[Khaine]

@kuite22: https://stackoverflow.com/questions/51801460/async-await-and-task-waitall

Może się dowiemy o co chodzi.

[Khaine]

@kuite22: Cóż, znalazłem jeszcze kilka sposobów aby to zrównoleglić aby działało, ale nadal nie wiadomo czemu owrappowanie funkcji synchronicznej przez Task.Run wewnątrz funkcji async daje taki smutny efekt.

Działa w momencie gdy zrobię prymitywnego asynca na samym ReadImages i wtedy odpalam go bezpośrednio do tej listy (bez pośrednika w postaci Task.Run). Ot dałem na końcu await Task.CompletedTask; i działa. Działa także zmiana zwykłego foreacha na asynchroniczny:

Parallel.ForEach(parsedTags, currentTag => ReadImages(currentTag));