Говнокод

Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!

Jawa / Говнокод #692

0

1
2
3
4
5
6
7
8
9

if(included.equals(INCLUDED_ALL) || fieldNames.indexOf(field.getName()) != -1) {
          if ((field.getDocumentMapping() != null && field.getDocumentMapping().trim().length() > 0)
                  || (isCase.booleanValue()
                      && ((field.getWorkflowMapping() != null && field.getWorkflowMapping().trim().length() > 0)
                          || (field.getContentMapping() != null && field.getContentMapping().trim().length() > 0))
                     ) {
    // тут еще насрано
}}
             ) {

классический унылый говнокод, весь проект в таком стиле..

Запостил: guest, 11.03.2009 (Updated 24.03.2018)

Комментарии (1) RSS

guest 17 лет ago # 0

В одной книге по рефакторингу я вычитал рекомендации избегать в том числе и локальных переменных, не являющимися необходимыми. Автор кода, похоже, тоже это место читал, плюс IntelliSence, - вот и последствия.
Ответить

Добавить комментарий Отменить ответ

Семь раз отмерь — один отрежь, guest!

А не использовать ли нам bbcode?

↑

8

↓