"PHP" / Говнокод #9996 Ссылка на оригинал

0

  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 psycho-coder, (Updated )

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

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

    вот это строка лишняя, тк при добавление нового товара, в массиве баскет ключа айди точно не будет ...
    Ответить
  • Дырочка то сокращалась, то расслаблялась. Пашка стонал и извивался, член его содрогался в моих руках. Так долго продолжаться не могло, возбуждение доходило до краев. Пашка повернул ко мне голову, глаза его, казалось вращались уже совсем независимо друг от друга, он прохрипел:
    Ответить

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

Из-за тебя ушел bormand, guest!

    А не использовать ли нам bbcode?


    8