Говнокод #2917 — PHP — Говнокод.ру

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

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

PHP / Говнокод #2917

+166

$limit = (isset($this->params['pass'][1]) == 'news')?null:5;

балина, и это работает :D

Запостил:

lener, 02 Апреля 2010

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

Abductio 02.04.2010 12:20 # −5

показать все, что скрытоЧто значит работает? Результат работы всегда будет null
Ответить
- lener 02.04.2010 12:50 # −3.6
  
  проверял? а если нет, так хоть посмотри внимательно )
  Ответить
  - Abductio 02.04.2010 13:14 # −5.2
    
    показать все, что скрытоДа в том то и дело, что проверял. null И все тут
    Ответить
    - lener 02.04.2010 13:22 # −3.4
      
      с cakePHP ? возможно там сработает
      Ответить
      - Abductio 02.04.2010 13:28 # −4.8
        
        А причем тут фреймворк?
        Ответить
        
        Abductio 02.04.2010 14:00 # −4
        
        Минус то за что?
        Ответить
    - nur 02.04.2010 14:34 # −3.8
      
      var_dump((true=='str'));
      
      var_dump((true==='str'));
      и подумайте почему так получается)
      Ответить
      - Abductio 02.04.2010 14:43 # −6.8
        
        показать все, что скрытоВау, и почему же? Неужели по тому, что == сравнивает не смотря на тип данных, а === сравнивает учитывая тип данных?
        Ответить
- danilissimus 02.04.2010 13:49 # −5.6
  
  показать все, что скрытоlener прав. результат работы разный
  Ответить
  - danilissimus 02.04.2010 13:52 # −5.6
    
    показать все, что скрытотолько вот null вернет даже если $this->params['pass'][1] вовсе не равно news
    Ответить
  - Abductio 02.04.2010 13:59 # −4.6
    
    Результат работы разный в каких случаях? Такое ощущение, что друг друга мы не понимаем.
    Ответить
    - lener 02.04.2010 14:30 # −5.2
      
      показать все, что скрытоиспользование функции isset .. вот где скрыт истинный ответ )
      Ответить
      - Abductio 02.04.2010 14:40 # −4.6
        
        Эм, хорошо я понимаю что все дело в isset, что подразумевается под работает? Конструкция isset($this->params['pass'][1])?null:5; будет работать точно также.
        
        Интересно и чего мы тут обсуждаем...
        Ответить
        
        lener 02.04.2010 15:40 # −4.2
        
        просто кинуть код и рассказать в чем тут проблема ... неееет, попробуй заинтересовать людей ) теперь ты узнаешь из КОНКРЕТНОГО ответа где собака зарыта, ну или .. возможно ... это все таки поможет вспомнить тебе, чтото очень давно забытое ..
        Ответить
guest 02.04.2010 14:45 # +7

К.О. поясняет: если переменная установлена, isset() вернёт True. При сравнении True и "news" строка будет приведена к логическому типу, и значение будет True, так как строка не пустая и не "0". Естественно что True==True.
Аналогично для False при неустановленной переменной.
Ответить
- guest 02.04.2010 14:56 # −4.6
  
  если это не свойство класса в cakephp
  Ответить
- lener 02.04.2010 15:01 # −4.8
  
  Спасибо К.О.! Воть человек действительно шевелит мозгами .. нет ничего необъяснимого. В действительности все было завязано на типах )
  Эта глупая строка получилася в середине ночи, когда хотелось сделать все быстрее чем голова уже осмыслевала!
  +1 респект и уважуха )
  Ответить
- nur 02.04.2010 15:20 # −3.8
  
  Кэп, почему так долго??
  Ответить
- comnimh 02.04.2010 16:48 # −4.2
  
  Опередил.
  Ответить
pee 02.04.2010 18:24 # −4.4

Топик зла!
Ответить
- guest 03.04.2010 14:03 # −3.4
  
  А ТФ
  минуснул все комментарии подряд?!
  Ответить
  - pee 03.04.2010 14:08 # −3.4
    
    >А ТФ
    Что?
    Ответить
    - guest 03.04.2010 16:39 # −4
      
      нее это был КФ
      Ответить
      - guest 03.04.2010 18:50 # −4.2
        
        да нее.. на самом деле это был ТК
        Ответить
        
        pee 04.04.2010 16:08 # +3.6
        
        Первый раз в интернетах, не могу понять о чем речь Оо
        Ответить
      - guest 14.04.2010 21:03 # 0
        
        это был КО
        Ответить
    - guest 14.04.2010 17:13 # −0.2
      
      Аденозинтрифосфат, глупышка!
      Ответить
- lener 05.04.2010 05:33 # −0.6
  
  ну как сказать ) твая кабинка просто одуванчик по сравнению с этой ...
  Ответить
guest 14.04.2010 01:28 # −0.2

$limit = (isset($this->params['pass'][1]) == 'news')?null:5;
$this->params['pass'][1] --- покуй..
isset(покуй) --- покуй, но boolean..
(boolean == 'news') --- всегда возвращает false
(false)?null:5; --- всегда возвращает 5
отсюда --- $limit = 5;
Ответить
guest 14.04.2010 21:04 # 0

isset(по*уй) --- по*уй, но boolean..
Ответить

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

PHP / Говнокод #2917

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

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