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

    +160

    1. 1
    2. 2
    unset($this->session->data['order_id']);
    $this->data['pap4_orderid'] = $this->session->data['order_id'];

    Не разбирался зачем сохраняется order_id, но в любом слусае - где логика?
    opencart, /catalog/controller/checkout/success.php, 13-14 строки

    Запостил: antongorodezkiy, 09 Ноября 2011

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

    • интересно кто минусанул ?

      в первой строчке ключ из массива будет удален и во второй строке будет сохранен null
      Ответить
      • Да что вы вообще понимаете в PHP???
        Ответить
        • видимо не чего
          <?php
          $array = array('order_id'=>"someFoo");
          unset($array['order_id']);
          echo $array['order_id'];
          ?>


          == "" ну не null, хорошо … смысл был в том, что запись логически неверная
          Ответить
          • ...
            unset($array['order_id'])
            eггог_геpoгting($array['order_id']); // делов-то...
            :)
            Ответить
            • да я не об этом … логика то где в коде ТС ?
              $this->data['pap4_orderid'] всегда будет =="" … 
              Ответить
              • Вы тут походите, присмотритесь. #8461 покажется вам мелочью.
                Ответить
          • <?php
            $array = array('order_id'=>"someFoo");
            $element = &$array['order_id'];
            unset($array['order_id']);
            echo isset($array['order_id']);
            ?>
            Ответить
            • А нет, как-то иначе стало работать. Раньше при сохранении ссылки unset не удалял объект.
              Ответить
      • Нет, не интересует.
        Миссией Говнокод.ру считаю предупреждение разработчиков, что-то типа крика "Не делайте так никогда!". И вопросы собственной кармы в этом деле находятся в конце очереди приоритетов, извините.
        Ответить
    • $this->data['pap4_orderid'] = $this->session->data['order_id'];
      unset($this->session->data['order_id']);
      нет говна
      автор ошибся
      Ответить
    • показать все, что скрытоvanished
      Ответить

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