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

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

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