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

    +150

    1. 1
    2. 2
    3. 3
    if (!empty ($src) && !array_key_exists($src, $this->loadedjscripts)) {
            if (isset ($this->loadedjscripts[$src]))
                return;

    Если ключа нет, то проверяем не null ли значение в массиве по этому индексу.

    Запостил: Agel_Nash, 22 Июня 2013

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

    • ava Agel_Nash 22.06.2013 18:22 # +2

      Минусующим привет. Если ключа нет, или он null, то все равно достаточно будет проверки isset
      Ответить

      • ava myaut 22.06.2013 21:18 # 0

        Зачем тогда вообще array_key_exists в языке?
        Ответить

        • ava deep 22.06.2013 21:48 # −1

          array_key_exists какраз таки корректен и свое дело делает.
          Ответить

        • ava Lure Of Chaos 24.06.2013 09:05 # +2

          потому что
          1. записи нет: isset - false, array_key_exists - false
          2. значение null: isset - false, array_key_exists - true

          хотя если есть гарантия "правильного" ключа, то можно не заморачиваться
          Ответить

      • ava deep 22.06.2013 21:47 # −1

        В описании указано одно. Тут в комментарии ты говоришь другое. В контексте кода не понятно есть ли какие-либо действия далее после второго условия, которое кстати вообще не сработает:

        $arr = array("b" => 1, "a" => null, "c" => 2);
        var_dump(array_key_exists("a", $arr)); // true
        var_dump(array_key_exists("b", $arr)); // true
        var_dump(isset($arr['a'])); // false
        var_dump(isset($arr['b'])); // true

        Говно только вот втором условии. Но, считай что его просто нет. Может его просто удалить забыли от старой временной проверки.

        Что там было дальше, какие действия?
        Ответить

        • ava Agel_Nash 23.06.2013 10:08 # −1

          https://github.com/modxcms/revolution/blob/develop/core/model/modx/modx.class.php#L1436
          public function regClientStartupScript($src, $plaintext= false) {
        if (!empty ($src) && !array_key_exists($src, $this->loadedjscripts)) {
            if (isset ($this->loadedjscripts[$src]))
                return;
            $this->loadedjscripts[$src]= true;
            if ($plaintext == true) {
                $this->sjscripts[count($this->sjscripts)]= $src;
            } elseif (strpos(strtolower($src), "<script") !== false) {
                $this->sjscripts[count($this->sjscripts)]= $src;
            } else {
                $this->sjscripts[count($this->sjscripts)]= '<script type="text/javascript" src="' . $src . '"></script>';
            }
        }
    }


          В контексте использования. Второе условие никогда не выполняется
          Ответить

      • ava eth0 23.06.2013 14:03 # +2

        Ты воспринимаешь минус как личное оскорбление?
        Ответить

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