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

0

  1. 1
  2. 2
  3. 3
  4. 4
  5. 5
  6. 6
  7. 7
  8. 8
  9. 9
  10. 10
  11. 11
  12. 12
  13. 13
  14. 14
  15. 15
  16. 16
  17. 17
  18. 18
  19. 19
  20. 20
  21. 21
else {
                                             if(codeName == SummaryConstants.TYPE_COST) {
                                                return new Cost(rtOBJ);
                                             }
                                             else {
                                                if(codeName == SummaryConstants.TYPE_EQUIPMENT) {
                                                   return new Equipment(rtOBJ);
                                                }
                                             }
                                          }
                                       }
                                    }
                                 }
                              }
                           }
                        }
                     }
                  }
               }
            }
         }

не стал утомлять еще и километром вложенных if ов..
а говнокод в том, что здесь можно было использовать switch

Запостил: maxt maxt, (Updated )

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

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

    Ну или не обязательно использовать else, можно ведь
    if (...) return ...;
    if (...) return ...; и т.д.
    Ответить

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

      Можно еще переформатировать:
      if () {
} else if {

} else if {

} else {

};
      Ответить

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

    А если codeName - строка, то свич нельзя использовать...
    Ответить

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

    Ну мой рекорд по вложенности ты не побил 😀
    Ответить

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

    а теперь представьте, жто написано на дельфи - отступтов два пробела, вместо скобок слова.

    ужос
    Ответить

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

      Если оператор один, то в Delphi операторные скобки не ставятся, так что строк там будет в 2 раза меньше, не надо гнать.
      Ответить

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

    Можно было бы вместо строковых констант использовать Emum, который имел бы абстрактный метод типа createSomeObject(), а каждая константа переопределяла бы его на свой лад, возвращая конкретный подтип. Таким образом константы были бы "умные" и не нуждались бы вообще в стороннем коде, создающем на их основании конкретные типы.
    Ответить

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

    да тут классический случай макаронного кода, когда всё можно было бы по маленьким функциям раскидать
    Ответить

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

    А если
    if (check_something) { continue; }
    пойдёт в таком случае, что бы не заморачиватся с вложеностью ?
    Ответить

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

Помни, guest, за тобой могут следить!

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

    ↑
    8
    ↓