On Oct 13, 10:58 am, Wojciech Jaczewski <w...@o...pl> wrote:
> Andrzej Jarzabek wrote:
>
> > Ale nie wiadomo, co konkretnie dany mutex synchronizuje.
>
> Synchronizuje ten fragment, wokół którego jest użyty. To w kodzie widać.

Przecież defnicja mutexa jest gdzie indziej niż kod, gdzie jest
używany.

> > Z zasady nie lubię skrótów typu 'inc'. Zrobiłbym ++counter albo
> > counter.increment(), ale raczej nie traciłbym czasu na zastanawianie się
> > nad tym zbyt długo.
>
> Wg mnie słówko atomic, bądź synchronized jest konieczne - aby wyraźnie
> oznaczyć, że to jest operacja z synchronizacją między wątkami. I że w razie
> problemów z wydajnością na wielu procesorach, należy się przyjrzeć także
> temu fragmentowi.

Według mnie jeśli klasa nazywa się ThreadSafeCounter i można ją tylko
np. inkrementować, resetować i sprawdzać wartość, to oczywistym jest,
że te wszystkie operacje są thread-safe.

> > Ale co ci tutaj dają komentarze? Czy może są to komentarze typu:
> > /* Wiem, że to wygląda bez sensu, ale uwierz mi, tak właśnie ma być */
> > ?
>
> Np. "Filtr pasmowo-przepustowy wokół częstotliwości 2400Hz, ale ze
> współczynnikami zespolonymi, tak aby w jego widmie częstotliwościowym były
> niezerowe wartości wyłącznie dla częstotliwości dodatnich".

Nie znam się na tym, ale zasadniczo wydaje mi się, że tak jak liczba
zespolona może być wyrażona odpowiednim typem, tak możesz mieć typ
generyczny Współczynnik (być może zdefiniowany wewnątrz klasy filtra
czy przestrzeni nazw 'Filtr', żeby było jasne, że chodzi o
współczynnik filtra), możesz mieć konkretną klasę
WspółczynnikZespolony parametryzującą szablon FiltrPasmowoPrzepustowy,
który ma konstruktor przyjmujący argument typu Częstotliwość i tak
dalej.