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

0

  1. 1
  2. 2
  3. 3
  4. 4
  5. 5
  6. 6
  7. 7
  8. 8
  9. 9
  10. 10
if(spForActiveActivityDetect.getBoolean( "active", false) == false) {
                    
                    Intent cabinetManagerAct = new Intent(this, EmployeeCabinetActivity.class );
                    finish();
                    startActivity(cabinetManagerAct );

                }else {

               //...
              }

Равно ли getBoolen из false false ?


по мне, заслуженное звание: топ - говенная проверка из говен, так как особенно выносит мозг

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

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

  • Педерасты с гейдева скоро все сюда перебегут.
    Лови палочку, лошара.
    Ответить
    • Перебежал бы тебе в ебальник, но... 1. csrf verification error
      Ответить
  • Не, ну а что.
    if (getInt("size") == 0)
      autoSize();
    if (getString("user") == "admin")
      admin = true;
    if (getBoolean("maximized") == false)
      minimize();
    Ответить
  • А что тут не так? Это как-бы андроид код. Тут getSharedPreferences. Вот это "false" в скобках - значение по умолчанию. Тут говнокода нет. Разве что не понятно зачем пустой else, но тут автор по моему обратил внимание на проверку false. Всё нормально. Разве что можно было сократить и после открытой скобки после if поставить "!" и тогда бы не пришлось писать == false
    Ответить
    • В общем, похоже на классику: if(x == false) вместо более короткого if(!x). По идее компилятор должен сгенерировать одинаковый код.
      Ответить
      • Это да. Но суть то в том, что если автор думал, что тут идёт двойная проверка на false, то он ни фига не угадал. Здесь идёт проверка параметра типа Boolean на false, тот false обязателен в скобках(необязательно false, можно и true, зависит от того, что должно быть по дефолту, ибо это дефолтное значение параметра). То есть он может быть "true", если пользователь или программа изменила значение по умолчанию, но... Когда идёт получение значение параметра, то пишется его дефолтное значение... getBoolean("name" // имя параметра, false // значение данного параметра по умолчанию)
        Ответить

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

Переведи на "PHP", guest!

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


    8