Jawa / Говнокод #692 Ссылка на оригинал

0

  1. 1
  2. 2
  3. 3
  4. 4
  5. 5
  6. 6
  7. 7
  8. 8
  9. 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 guest, (Updated )

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

  • ava guest # Ссылка на оригинал 0

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

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

Где здесь C++, guest?!

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

    ↑
    8
    ↓