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

    +161

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    9. 9
    public function setAmbiguityIdentity($flag)
    {
        if (is_integer($flag)) {
            $this->_ambiguityIdentity = (1 === $flag ? true : false);
        } elseif (is_bool($flag)) {
            $this->_ambiguityIdentity = $flag;
        }
        return $this;
    }

    Zend Framework компонент Zend_Auth_Adapter_DbTable. (Слегка смутила строка 4)

    Запостил: sonic, 21 Апреля 2011

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

    • ava akaDElpher 21.04.2011 02:57 # 0

      bool result = (result)?true:false ^_^
      Ответить

    • ava hidespb 21.04.2011 08:43 # 0

      $this->_ambiguityIdentity = !!$flag
      Ответить

      • ava rO_ot 21.04.2011 08:47 # 0

        Если прийдет 2, должено быть по всей видимости false, а у тебя true
        Ответить

        • ava hidespb 21.04.2011 08:53 # 0

          я не в курсе этого компонента, но скорее всего там приходит 1/0 в первом случае и true/false во втором
          Ответить

      • ava GhOsTMZ 21.04.2011 09:37 # 0

        раз на то пошло, то должно быть так:
        $this->_ambiguityIdentity = (1 === $flag);
        А вместо этого:
        $this->_ambiguityIdentity = !!$flag
        Было бы даже красивее так:
        $this->_ambiguityIdentity = (bool)$flag;
        Ответить

    • ava Lure Of Chaos 21.04.2011 09:58 # +2

      вот почему зенд такой медленный )
      Ответить

    • ava scalar4eblo4no 21.04.2011 10:44 # 0

      Я так извиняюсь, но это из какой версии?
      Заглянул в сорцы Zend/Auth/Adapter/DbTable.php
      и не нашел там такого метода=)
      Ответить

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