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

    • ava striker 21.04.2012 04:39 # −1

      А что не так?
      Ответить

      • ava roman-kashitsyn 21.04.2012 08:23 # −1

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

        • ava psycho-coder 21.04.2012 15:25 # −1

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

    • ava guest 21.04.2012 10:30 # −1

      if ( !isset( $basket[$id] ) )

      вот это строка лишняя, тк при добавление нового товара, в массиве баскет ключа айди точно не будет ...
      Ответить

      • ava roman-kashitsyn 21.04.2012 10:38 # −1

        Скорее всего, эта строка тоже лишняя:
        $this->setBasket( $basket );
        если, конечно, в сеттере никаких дополнительных действий не осуществляется
        Ответить

      • ava psycho-coder 21.04.2012 15:26 # −1

        А вдруг?
        Ответить

    • ava Crabbe 25.08.2021 03:04 # 0

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

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