1. Java / Говнокод #692

    +130

    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, 11 Марта 2009

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

    • ava guest 13.03.2009 18:11 # +1

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

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