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

0

  1. 1
  2. 2
  3. 3
  4. 4
  5. 5
  6. 6
@Column(name = "IS_DEFAULT", unique = false, nullable = true, insertable = true, updatable = true, length = 1)
private String isDefault;

......

if (b.getIsDefault().equals("N"))...

Created by: Sudharsan Veluru

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

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

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

        СУТЬ, конечно, в том, что выбран неподходящий тип, но это неинтересно )
        Ответить

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

          Похоже, что БД - Oracle. Там нет ни перечислений, ни булеанов, и их часто имитируют varchar2 с констрэйнтом length=1 и ограничением на принимаемые значения. Если использовать char, то JDBC будет при запросах конвертить переданные символы в числа, что Oracle очень огорчит. Поэтому, возможно, String и к месту.
          Ответить

  • ava roman-kashitsyn # Ссылка на оригинал 0

    По моему, самое большое говно здесь в том, что
    b.getIsDefault().equals("N")
    вместо 
    "N".equals(b.getIsDefault())
    при условии 
    nullable = true
    Ну и метод 
    getIsDefault()
    доставляет.
    Ответить

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

      собсна, это и хотел показать 🙂
      ну и у меня вызывает отторжение сама идея проверять булеан через equals
      Ответить

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

        Это понятно, звучит просто криво ) Можно, конечно, было извернуться и добавить 
        boolean isDefault() { return "N".equals(isDefault); }
        , но и тут есть свои проблемы... Кругом засада, короче.
        Ответить

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

Семь раз отмерь — один отрежь, guest!

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

    ↑
    8
    ↓