Wpis z mikrobloga

10/100 dni z książką

“Funkcje powinny coś wykonywać lub odpowiadać na jakieś pytanie, ale nie powinny robić tych dwóch operacji jednocześnie. [...] Jako przykład weźmy następującą funkcję:

Public boolean set(String attribute, String value);
Lepszym rozwiązaniem jest oddzielenie polecenia od zapytania, dzięki czemu niejasność nie występuje:

if(attributeExists(“username”) {
setAttribute(“username”, “unclebob”);
}



#feaoftruss #czystykod #programowanie #programista15k #webdev #gamedev

Podobało się? To zaplusuj i zapisz się do wołania (link w stopce)

************

[Chcesz być wołany?]
  • 23
  • Odpowiedz
jest masa funkcji które coś robią i coś zwracają i nie uważam że to nie jest czytelne.

@zibizz1: gdyby tu tylko chodziło o to czy coś jest czytelne czy nie :) Ponadto CQRS, to może być ciekawa lektura.
  • Odpowiedz
@ZasilaczKomputerowy: Jedynym, który sie tutaj "#!$%@?" jesteś Ty. Pamiętaj, że Twoje "jak coś się da przeczytać, nie budzi wątpliwości i co najważniejsze działa to znaczy że jest ok" to też Twoje zdanie które każdy może mieć inne.

@nilphilus @zibizz1 zawsze można sprawdzić warunek w funkcji i ją nazwać "setIfExists" lub "trySet", wtedy nie trzeba za każdym razem powtarzać sprawdzania a i nazwa sugeruje, że to nie zwykły setter ( ͡°
  • Odpowiedz