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

    • эм...а case разве не применимо?
      Ответить
    • Не вижу особой беды в подобном. Названия валют тока в константы убрать.
      Ответить
      • а если валют 150?
        Ответить
        • а если бартер?
          Ответить
        • А с механизмом Reflection никто не знаком? Пара-тройка строк и хоть 5 валют, хоть 150...
          Ответить
          • По-моему, фиговое применение reflection. Как бэ золотой молоток.
            Ответить
            • Можно еще танком кур давить для паштета.
              Ответить
    • Лучше создать статический Map, тогда уж.
      А еще лучше использовать enum (Java 1.5)
      Ответить
    • > А еще лучше использовать enum (Java 1.5)

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

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

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