В этой теме нет ответов

[badisoft]

1. Функция CodeItemInClient используется в одном-единственном месте кода - в функции cartGetCartContent().
Код выглядит как

$session_items[]=

	CodeItemInClient($_SESSION["configurations"][$j],

		$_SESSION["gids"][$j]);

Один минус - массив $session_items далее не используется нигде и никогда. Нахрена этот код?

2. Та же функция cartGetCartContent(). Ваще трындец. Как такое мог написать программист?

$costUC = GetPriceProductWithOption( $variants,

	$shopping_cart_item["productID"] );

$tmp =

	array(

		"productID" =>  $product["productID"],

	   "id"				=>		$cart_item["itemID"],

	   "name"				=>		$product["name"],

	   "quantity"		=>		$cart_item["Quantity"],

	   "free_shipping"		=>		$product["free_shipping"],

	   "costUC"		=>		$costUC,

	   "cost"				=>		show_price($cart_item["Quantity"]*

	   GetPriceProductWithOption($variants,

		   $shopping_cart_item["productID"])),

	   "product_code"				=>		 $product["product_code"] );

	   

Сначала идет

$costUC = GetPriceProductWithOption( $variants, $shopping_cart_item["productID"] );

а потом в элементах массива снова

"cost" => show_price($cart_item["Quantity"]*GetPriceProductWithOption($variants,$shopping_cart_item["productID"])),

вместо

"cost" => show_price($cart_item["Quantity"]*$costUC),

Напомню, функция GetPriceProductWithOption содержит один SQL-запрос обязательно (по товару) и еще по запросу на каждый вариант характеристик. Т.е. сходу имеем в минимуме один, а в максимуме десяток-два лишних запросов. Вот зачем так писать? Понятно, что это корзина и что запросов - в общем! - не много по любому. Но зачем?

И - вау! - в конце цикла перл. Снова

$total_price += $cart_item["Quantity"]*

	GetPriceProductWithOption($variants,

		$shopping_cart_item["productID"]);

вместо все того же логичного

$total_price += $cart_item["Quantity"]* $costUC;