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

    +75

    1. 1
    2. 2
    3. 3
    if ((getProductId() != null) && (getProductId().equals(new String("5")))) {
  //do it
}

    ProductId читается один раз при старте приложения, а таких проверок около 50-и.

    Запостил: lotik, 10 Июня 2010

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

    • ava Lure Of Chaos 10.06.2010 13:14 # 0

      вот мне интересно, а что внутри getProductId()? небось соединение с БД или еще какая нибудь ересь
      Ответить

      • ava lotik 10.06.2010 13:52 # 0

        public String getProductId() {
        return <прочтенное значение из property файла при старте>
        }
        Ответить

    • ava Dummy00001 10.06.2010 17:58 # 0

      к слову, а javac такое говно может соптимизировать нафиг?
      Ответить

    • ava Анонимус 11.06.2010 01:48 # +2

      >>new String("5")
      а это -- что бы стринг-пул случайно не сработал, и что б сборщик мусора не отдыхал
      Ответить

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