1. PHP / Говнокод #1658

    +156

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    /**
     * Set flag indicating whether or not request has been dispatched
     *
     * @param boolean $flag
     * @return Zend_Controller_Request_Abstract
     */
    public function setDispatched($flag = true)
    {
        $this->_dispatched = $flag ? true : false;
        return $this;
    }

    вот такую штуку сегодня увидел

    Запостил: getme, 22 Августа 2009

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

    • ava tz-lom 22.08.2009 10:15 # 0

      напомнило http://govnokod.ru/1641
      Ответить

    • ava guest 22.08.2009 14:56 # +1

      а может тут пытаются избежать случайного присвоения булевому полю стринги или объекта :-)
      Ответить

      • ava tinimi 22.08.2009 15:44 # +1

        кажется именно так ) $flag на вхлже не обязательно bool. может быть что угодно )
        Ответить

      • ava guest 23.08.2009 16:42 # 0

        Именно так. И ни какой это не говнокод
        Ответить

        • ava guest 27.08.2009 10:37 # 0

          'г'.'о'.'в'.'н'.'о'.'к'.'о'.'д';


          $this->_dispatched = (bool)$flag;
          Ответить

    • ava guest 22.08.2009 23:33 # 0

      Цель понятна — и это нормальная практика в OOP.
      Ответить

    • ava guest 23.08.2009 18:24 # −4

      говнокод и поддерживаю. и ваще язык отстойный, который позволяет такое делать. Сорри я не знаю языка этого. Улучшенный вариант кода (более дешевый и легче понимаемый):

      public function setDispatched($flag)
      {
      if($flag is empty)
      $this->dispatched = true;
      else
      $this->_dispatched = $flag;

      return $this;
      }
      Ответить

      • ava guest 25.08.2009 10:25 # −1

        лол, вы из нормального кода сделали гавно, поздравляю
        Ответить

      • ava grimskin 26.08.2009 22:36 # 0

        предлагаю это запостить отдельным говнокодом
        Ответить

      • ava guest 27.08.2009 10:39 # 0

        этот говнокод из комента переговнокодил говнокод из поста
        ржу нимагу

        тот хотябы в $this->_dispatched запишет булёво значение
        Ответить

    • ava Bobby 23.08.2009 19:44 # +5

      Не говнокод, конечно, а особенность языка с динамической типизацией. Хотя можно было и просто привести тип к булеву:
      $this->_dispatched = (boolean) $flag;
      Ответить

    • ava guest 23.08.2009 20:26 # 0

      Очевидно же это PHP. И это нормальная практика.
      Ответить

      • ava Bartelby 27.08.2009 12:51 # 0

        Для баранов повторю слова Bobby: "$this->_dispatched = (boolean) $flag;"

        Это PHP и это нормальная практика, а быдло считает, что такой прием недопустим и PHP это зло...
        Ответить

    • ava guest 02.09.2009 17:45 # 0

      Хрен с ним, с (bool), зачем сеттеру возвращать свой объект?
      Ответить

    • ava guest 17.09.2009 09:29 # 0

      Может они хотели что можно было делать
      $obj->setDispatcher()->setEcheShtoto();
      как то так ))
      Ответить

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