Говнокод #24718 — Java — Говнокод.ру

Говнокод: по колено в коде.

Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!

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

−2

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

g0_1494084791211 05.09.2018 00:05 # 0

Переведи на "PHP".
Ответить
- Konardyan 05.09.2018 00:10 # −104
  
  показать все, что скрытоПеревёл бы тебе за щеку, но... 1. csrf verification error
  Ответить
  - guest8 05.09.2018 00:24 # −999
    
    показать все, что скрытоvanished
    Ответить
    - rss 05.09.2018 00:26 # −102
      
      показать все, что скрытоПизданул бы с меня, если бы не "1. csrf verification error"
      Ответить
Konardyan 05.09.2018 00:11 # −100

показать все, что скрытоПедерасты с гейдева скоро все сюда перебегут.
Лови палочку, лошара.
Ответить
- guest8 05.09.2018 00:21 # −999
  
  показать все, что скрытоvanished
  Ответить
admin 05.09.2018 08:42 # 0

fixed
Ответить
- admin 05.09.2018 08:43 # 0
  
  123
  Ответить
admin 05.09.2018 08:43 # 0

А почта ходит?
Ответить
- AnaILoIIipop 05.09.2018 08:52 # 0
  
  Тебе за щеку.
  Ответить
- Konardyan 05.09.2018 19:12 # −102
  
  показать все, что скрытоОтправил почту тебе за щеку, проверь.
  Ответить
  - guest8 05.09.2018 20:57 # −999
    
    показать все, что скрытоvanished
    Ответить
    - kir_rik 05.09.2018 21:10 # 0
      
      ... сношает твою узкую трубочку...
      Ответить

Steve_Brown 05.09.2018 13:59 # +1

Не, ну а что.

if (getInt("size") == 0)
  autoSize();
if (getString("user") == "admin")
  admin = true;
if (getBoolean("maximized") == false)
  minimize();

Ответить

Sergdev 05.09.2018 14:20 # 0

есть смысл
Ответить

Ph03niX 27.09.2018 20:03 # 0

А что тут не так? Это как-бы андроид код. Тут getSharedPreferences. Вот это "false" в скобках - значение по умолчанию. Тут говнокода нет. Разве что не понятно зачем пустой else, но тут автор по моему обратил внимание на проверку false. Всё нормально. Разве что можно было сократить и после открытой скобки после if поставить "!" и тогда бы не пришлось писать == false
Ответить
- guest8 27.09.2018 23:38 # −999
  
  показать все, что скрытоvanished
  Ответить
  - Ph03niX 29.09.2018 18:51 # 0
    
    Это да. Но суть то в том, что если автор думал, что тут идёт двойная проверка на false, то он ни фига не угадал. Здесь идёт проверка параметра типа Boolean на false, тот false обязателен в скобках(необязательно false, можно и true, зависит от того, что должно быть по дефолту, ибо это дефолтное значение параметра). То есть он может быть "true", если пользователь или программа изменила значение по умолчанию, но... Когда идёт получение значение параметра, то пишется его дефолтное значение... getBoolean("name" // имя параметра, false // значение данного параметра по умолчанию)
    Ответить

Говнокод: по колено в коде.

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

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

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