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

    −2

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    if(spForActiveActivityDetect.getBoolean( "active", false) == false) {
                        
                        Intent cabinetManagerAct = new Intent(this, EmployeeCabinetActivity.class );
                        finish();
                        startActivity(cabinetManagerAct );
    
                    }else {
    
                   //...
                  }

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


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

    Запостил: Sergdev, 04 Сентября 2018

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

    • Переведи на "PHP".
      Ответить
      • Перевёл бы тебе за щеку, но... 1. csrf verification error
        Ответить
        • Что не с rss-а в этот раз пизданул?
          Ответить
          • Пизданул бы с меня, если бы не "1. csrf verification error"
            Ответить
    • Педерасты с гейдева скоро все сюда перебегут.
      Лови палочку, лошара.
      Ответить
      • Перебежал бы тебе в ебальник, но... 1. csrf verification error
        Ответить
    • fixed
      Ответить
    • А почта ходит?
      Ответить
    • Не, ну а что.
      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 // значение данного параметра по умолчанию)
          Ответить

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