Witki mi opadły

Jeszcze raz o nazewnictwie, bo dziś trafiłem w kodzie na coś takiego:

Listing 1. Potworek

findPxxSxxInPxxSxxListByCxxIdSxxsIdSxxCxxxxIdUxxxxTp

wyiksowałem rzeczy, które są biznesowo istotne, ale nie straciło to nic ze swojego uroku. Metoda jest prywatna w danej klasie, bo takie potworki czasami trafiają się w Spring Data, i przyjmuje trzy parametry. Zwraca jakiś tam obiekt biznesowy.

Ja się pytam, co ćpał człowiek, który to napisał.

5 myśli na temat “Witki mi opadły

  1. Wiemy ze jest zle. Teraz napisz jak powinno byc dobrze 🙂 Nigdy nie wiesz czy jakas poczatkujaca osoba tu nie trafi.

  2. To zależy czy jest to część logiki domenowej (wtedy możliwe, że jest to nieodpowiedni model) czy zapytanie (query). W tym drugim przypadku można zastanawiać się nad sensownością wyciągania tak zagnieżdżonych atrybutów jednak sama metoda nie wydaje się niczym złym (i nie „psuje” nam innych metod). O ile oczywiście nazwy pól są klarowne i wiemy co wyciągamy. Jestem ciekawe jaka jest Twoja opinia w tym temacie.

Napisz odpowiedź

Twój adres email nie zostanie opublikowany. Pola, których wypełnienie jest wymagane, są oznaczone symbolem *

To create code blocks or other preformatted text, indent by four spaces:

    This will be displayed in a monospaced font. The first four 
    spaces will be stripped off, but all other whitespace
    will be preserved.
    
    Markdown is turned off in code blocks:
     [This is not a link](http://example.com)

To create not a block, but an inline code span, use backticks:

Here is some inline `code`.

For more help see http://daringfireball.net/projects/markdown/syntax