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

    +79.2

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    if("USD".equals(currency)) {
model.put(CURRENCY, String.valueOf(RBSConstants.CURRENCY_USD));
} else if("EUR".equals(currency)) {
model.put(CURRENCY, String.valueOf(RBSConstants.CURRENCY_EUR));
} else if("RUB".equals(currency)) {
model.put(CURRENCY, String.valueOf(RBSConstants.CURRENCY_RUB));
} else if("UAH".equals(currency)) {
model.put(CURRENCY, String.valueOf(RBSConstants.CURRENCY_UAH));
} else if("NOK".equals(currency)) {
model.put(CURRENCY, String.valueOf(RBSConstants.CURRENCY_NOK));
}

    5 валют это тебе не 55 и не стопицот, так что херячим ифами

    Запостил: shomeser, 13 Октября 2009

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

    • ava guest 13.10.2009 18:52 # 0

      эм...а case разве не применимо?
      Ответить

    • ava guest 14.10.2009 07:20 # −2

      Не вижу особой беды в подобном. Названия валют тока в константы убрать.
      Ответить

      • ava shomeser 14.10.2009 09:16 # 0

        а если валют 150?
        Ответить

        • ava guest 14.10.2009 13:07 # 0

          а если бартер?
          Ответить

        • ava guest 17.10.2009 20:08 # 0

          А с механизмом Reflection никто не знаком? Пара-тройка строк и хоть 5 валют, хоть 150...
          Ответить

          • ava guest 19.10.2009 08:00 # 0

            По-моему, фиговое применение reflection. Как бэ золотой молоток.
            Ответить

            • ava guest 23.10.2009 17:16 # 0

              Можно еще танком кур давить для паштета.
              Ответить

    • ava guest 21.10.2009 10:25 # 0

      Лучше создать статический Map, тогда уж.
      А еще лучше использовать enum (Java 1.5)
      Ответить

    • ava guest 16.11.2009 21:44 # 0

      > А еще лучше использовать enum (Java 1.5)

      Ололо, в жабе энумы появились только к пятой версии? Прискорбно...

      Авось к версии десятой реализуют свитчи по стринговым литералам...
      Ответить

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