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

    +156

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    9. 9
    // добавление элемента в корзину
    public function basketAdd( $id, $product ) {
    	$basket = $this->getBasket( );
    
    	if ( !isset( $basket[$id] ) )
    		$basket[$id] = array( 'id' => $product['id'], 'price' => $product['price'], 'count' => $product['count'] );
    
    	$this->setBasket( $basket );
    }

    Было обнаружено в предрелизе)
    Надо больше спать...

    Запостил: psycho-coder, 20 Апреля 2012

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

    • А что не так?
      Ответить
      • могу лишь предположить, что нужно было писать так:
        $basket[$id] = $product;
        Но этот код не иквивалентен тому, что в топике (может, нужна проекция того, что в product...).
        Ответить
        • > могу лишь предположить, что нужно было писать так:
          Верно. Там ничего не меняется и подразумавается, что передается уже сформированный массив.
          Ответить
    • if ( !isset( $basket[$id] ) )

      вот это строка лишняя, тк при добавление нового товара, в массиве баскет ключа айди точно не будет ...
      Ответить
      • Скорее всего, эта строка тоже лишняя:
        $this->setBasket( $basket );
        если, конечно, в сеттере никаких дополнительных действий не осуществляется
        Ответить
      • А вдруг?
        Ответить
    • Дырочка то сокращалась, то расслаблялась. Пашка стонал и извивался, член его содрогался в моих руках. Так долго продолжаться не могло, возбуждение доходило до краев. Пашка повернул ко мне голову, глаза его, казалось вращались уже совсем независимо друг от друга, он прохрипел:
      Ответить

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