Проблема с сессиями

[impulsgraw]

Здравствуйте!
Пишу скрипт интернет-магазина.
Допустим, что $_SESSION состоит из
Array ( [basket] => Array ( [tovarnum] => 1 [199847] => Array ( [coll] => 1 ) [tovars] => 199847 ) ) .
На страничке корзины, при нажатии на кнопку Пересчитать выполняется следующий скрипт:

PHP код:

 if(($_GET['step'] == 'mybasket')&&(!empty($_POST)))
 {
 session_start();
 $ni = explode(',',$_SESSION[basket][tovars]);
 $g = $_SESSION[basket][tovarnum];
  
 while($g != 0){
     if((int)$_POST[$ni[$g-1]] > '0'){
     $pperror[count($pperror)+1] = 'Ошибка при изменении количества товара #'.$ni[$g-1];
     }elseif((int)$_POST[$ni[$g-1]] == '0'){
     $_SESSION[basket][tovarnum]--;
     unset($_SESSION[basket][$ni[$g-1]]);
     
     $ntemp = explode(',',$_SESSION[basket][tovars]);
     unset($ntemp[$g-1]);
     $_SESSION[basket][tovars] = implode(',',$ntemp);
     
     }else{
     $_SESSION[basket][$ni[$g-1]] = $_POST[$ni[$g-1]];
     }
     $g--;
 }
 } 


Почему после выполения данного скрипта массив $_SESSION преобретает данную конструкцию:
Array ( [basket] => Array ( [tovarnum] => 0 [tovars] => ) )

Весь смысл выполнения данного скрипта заключается в пересчёте товаров на страничке.
(На начальной странице корзины предоставлено меню со всеми товарами в корзине и по текстовому полю для каждого товара. После этого все данные отправляются (post) на страничку обработки с вышенаписанным скриптом.)
Раньше этот скрипт работал, но сейчас он неконтролируем. В чём проблема не могу понять.

[Johnatan]

$_SESSION[basket][tovars]
что это такое? какой это язык? Это уж точно не PHP. Где кавычки?

Что за китайский код? Где вы учились так программировать?

PHP код:

if(($_GET['step'] == 'mybasket')&&(!empty($_POST)))
{
    session_start();
    $_SESSION = array('basket'=>array('199847'=>array('amount'=>1))); // для примера
    foreach ($_SESSION['basket'] as $k=>$item) {
        if(!isset($_POST[$k]) || (int)$_POST[$k] == 0) {
            unset($_SESSION['basket'][$k]);
        } elseif((int)$_POST[$k] > '0') {
            $pperror[count($pperror)+1] = 'Ошибка при изменении количества товара #'.$k;
        } else {
            $_SESSION['basket'][$k]['amount'] = (int)$_POST[$k];
        }
    }
} 


Никаких explode/implode, никаких $ni[$g-1]. Если нужно узнать общее количество наименований в корзине, то можно сделать count($_SESSION['basket'])

[impulsgraw]

Вообщето так тоже будет правильно писать:
$_SESSION[basket][tovars]
$_SESSION['basket']['tovars']
это одно и тоже, хоть и не очень правильно, но php это варит и это экономит время.

$_SESSION['basket']['tovars'] содержит коды товаров через запятую (например 19923,13324) от того и explode и implode, тоесть, если указано товара #19923 0 штук, то тогда он удаляется из корзины. Если меньше нуля, то пишется ошибка.

$_SESSION['basket']['tovarnum'] содержит количество (например 2) товаров в корзине.

$_SESSION['basket']['13324']['coll'] содержит штучное количество товара.
Да, и вот исправленый код.

if(($_GET['step']=='mybasket') && (!empty($_POST['basket_post']))){

$mExpl = explode(",",$_SESSION['basket']['tovars'].",");

$tShet = $_SESSION['basket']['tovarnum'];

while($tShet != 0){
	
	$jmexpl = $mExpl[$tShet-1];

	if((int)$_POST[$jmexpl] < 0)
	{
	$error .= "<br>Неверно введено количество товара (#".$jmexpl.")";
	$tShet--;
	}
	elseif((int)$_POST[$jmexpl] == 0)
	{
	print (int)$_POST[$jmexpl];
	unset($_SESSION['basket'][$jmexpl]);
	$_SESSION['basket']['tovarnum']--;

	$tTempExpl = explode(',',$_SESSION['basket']['tovars']);
	unset($tTempExpl[$tShet-1]);
	$_SESSION['basket']['tovars'] = implode(',',$tTempExpl);

	$tShet--;
	}
	else
	{
	$_SESSION['basket'][$jmexpl]['coll'] = $_POST[$jmexpl];
	$tShet--;
	}
}
}

[impulsgraw]

В любом случае спасибо, проблему решил путём отделения корзины и каталога на отдельные страницы.

[Johnatan]

PHP много чего переваривает. Вы пишете быдлокод. С вашими соплями потом настоящим программистам приходится глаза и мозг ломать. То, что вы даже не удосужились попробовать разобраться в том, как работает приведённый мной пример кода, говорит о многом.

Что если я пошлю запрос формы и передам вашему скрипту через POST basket_post=1 и всё. Больше ничего постом не передавать. Что будет с вашим быдлоскриптом? Он сдуется. Где проверки? Вы пишете в сессию напрямую переменную из POST, без проверки типа. Что, если вместо количества передать abcde? Как вы потом это количество будете считать? Небось и в базу без проверки записывается.

После таких вот "всё-и-так-работает-хорошо" клиенты от PHP открещиваются и ищут программистов на ASP, Perl, Python и т.д., потому что считают, что PHP глючный и кривой.

[impulsgraw]

На ваше удивление, я скажу вам, что проверка есть. Она запихана в процедуре, которая запускается автоматически при открытии страницы. Но я всё это ненужное вырезал из моего БЫДЛОКОДА дабы сэкономить вам время, чтобы сразу было понятно, где ошибка.

Сообщение от Johnatan

PHP много чего переваривает.

Ну наверно не зря? Действительно, в данном случае, я не прав. Но рас php варит код такого вида, почему бы не воспользоваться? Данный код приватный и больше ни один программист не будет его просматривать.
P.S. Не обязательно так ЯРОСТНО РУГАТЬСЯ.