Помогите сделать пересчет товаров в корзине, данные собираются в сессию: 
(мне удалось только сделать пересчет по каждому товару в отдельности, но это ужасно выглядит)

вот код корзины:


обрабатывать пересчет товара должен 'update_zakaz.php', передаю массивы id товара и количество товара, а как правильно эти массивы обработать и вернуть?:

Писдец


Тоже писдец

что в этом такого ужасного? зачем сессию открывать если она уже существует?

Risyaalex, 


Почему не if(isset($_SESSION['zakaz']))?
И почему не order? Моск сломать можно  

Насчет проверки существования сессии.... по идее это не обязательно

В реальном скрипте вам придется ставить E_ALL&~E_NOTICE и это избавит от отображения предупреждения, что сессия стартовала)

ок

Логичнее так:



И после session_start(); добавить ob_start();



Почему проверка на символ   "0"?
А если там ничего не будет?  

Что тогда? Может опять же на isset/!empty смотреть?

echo делать - не очень кошерно, как по мне, но это на любителя. Я предпочитаю что нить такое:



Ну это было немножко критики  

А теперь - что у вас не получается?

У вас есть список товаров в массиве $_SESSION['zakaz']?
Пробегаетесь форычем и считаете
foreach($_SESSION['zakaz'] as $k=>$v) {
}

если структура массива $_SESSION[zakaz] более сложная - выложите ее)

1 скрытие ошибок не есть гуд...

Добавлено @ 20:55
вроде так.


пс код не защищен и подвержен sql-injection  $_POST["id_zakaz"]  проверить и перебрать через фореач также проверить число или сразу intval() + чтобы не было пустых значений в масве иначе запрос феил. 


Это сообщение отредактировал(а) Sanchezzz - 31.7.2011, 20:57

Sanchezzz, скрытие ошибок не есть гуд в процессе написания. на лайв однозначно E_NOTICE надо отключать  
ЗЫ: не догнал, при чем тут implode   
По идее это одно число   
Айдишка текущего заказа, и достаточно просто _POST['id_zakaz']

спасибо за конструктивную критику, буду стараться соответствовать высоким нормам php)

по теме:
данные (id заказа и количество) хранятся в SESSION[zakaz]

при пересчете количества - передаются массивы id_zakaz[] и col_zakaz[] в update_zakaz.php
мне их надо опять собрать в SESSION[zakaz]
как их foreach-ем перебрать ?

Мда, прочитал всю тему...

ТС, во-первых неплохо бы изучить документально поведение функции session_start() - она либо стартует новую сессию, либо цепляется к уже существующей.

Во-вторых вот такой код:

 всегда будет возвращать false, так как $_SESSION - суперглобалка, она всегда существует.

В-третьих, прошу вас, не надо смешивать html и php, это ужасно смотрится и, по сути, нечитабельно...

И в-четвёртых - пересчёт товара правильно осуществлять на ajax, по идее. Ну потому что вот юзер щёлкнул по кнопке "Пересчитать", ему динамически должно всё обновиться. Это будет логично. А если у вас этим занимается отдельный скрипт, то, получается, что сначала перенаправление идёт на update_zakaz.php, там что-то такое делается, а потом идёт уже не знаю куда перенаправление.... получается очень много действий. Логичнее это всё на ajax вынести, чтобы клиенту было легче ))

Я просто сейчас делал инет-магазин, у меня всё именно так реализовано.

ajax - дело хорошее.

но мне просто нужно из массивов $id_zakaz и $col_zakaz собрать массив $_SESSION['zakaz']
если данные не в массиве - собираю просто $_SESSION['zakaz'][$id_zakaz] = $col_zakaz
с массивами так не получится, вот и спрашиваю, как это можно реализовать


я учу php пока что, так что на идеальность реализации не претендую

Добавлено через 3 минуты и 18 секунд

я ходила на мини курсы, и преподаватель учил  писать
так что пока с этим не столкнулась - писала вот так

 Не надо. Надо ошибки не на экран выводить, а в логи.

А совет - учиться программировать на всяких "написать блог". Рано топикстартеру писать магазины.
Надеюсь, это не готовый продукт, а курсовая, иначе заказчик встрял на переписывание 

мдя...
кто нибудь по теме ответит, или дальше поливать будете? 
я же написала что мне нужно, свой уровень я знаю и если бы не хотела учиться писала бы не на форумах а на фриланс

Покажите что находится в массивах и какой в итоге массив вам нужно получить? 

передаются методом post массивы id_zakaz[]  и col_zakaz[] (id товара и количество) 
нужно собрать в $_SESSION['zakaz']