Hashtable

Spotkałem ostatnio w kodzie bardzo denerwujące mnie wyrażenie. Metoda jako jeden z parametrów przyjmowała Hashtable, który stanowił zbiór różnych parametrów wykorzystywanych przez tę metodę. Około osiem kolejnych linijek kodu stanowił komentarz informujący co ta hashtabela zawierać powinna.

Super, fajnie, ale po co?

Czy nie lepiej napisać jest anemiczną klasę, która znacznie czytelniej przekaże parametry do tejże metody? Co w przypadku, gdy zrobiliśmy literówkę w kluczu dla tej hashtabeli. Musimy wtedy poprawić go wszędzie, również w komentarzach, a narzędzia refactoringu mogą nam w tym nie pomóc…

Strasznie nie lubię odwoływania się w kodzie do różnych elementów przez napis w cudzysłowach. Bardzo łatwo o pomyłkę, a jeszcze trudniej utrzymać takie rozwiązanie. Cały ten komentarz rozpoczynający metodę mógłby zostać przeniesiony do klasy anemicznej w postaci kodu, czyli czegoś, co musi być poprawne. Moim zdaniem kod stałby się przez to czytelniejszy i łatwiejszy w utrzymaniu!

Skomentuj

Wprowadź swoje dane lub kliknij jedną z tych ikon, aby się zalogować:

Logo WordPress.com

Komentujesz korzystając z konta WordPress.com. Wyloguj /  Zmień )

Zdjęcie na Google

Komentujesz korzystając z konta Google. Wyloguj /  Zmień )

Zdjęcie z Twittera

Komentujesz korzystając z konta Twitter. Wyloguj /  Zmień )

Zdjęcie na Facebooku

Komentujesz korzystając z konta Facebook. Wyloguj /  Zmień )

Połączenie z %s