Puste obiekty i sesja, czyli skad skrypt ma wiedziec kiedy wykonac unset()? Opcje

[Edpl]

Witam,
pisze koszyk internetowy na podstawie klasy podpatrzonej gdzies w necie (nie pamietam juz mozliwe cos zmienialem). Jednak mam problem z usuwaniem ostatnich elementow. Jesli np w koszyku zostaly 2 sztuki jakiegos produktu i chce sie je usunac... w tym momencie poprawnie wypisuje ze usunieto, koszyk nie wyswietla zawartosci, ale po przeladowaniu znowu te 2 sztuki sie pojawiaja (tak jakby zmiany nie byly wprowadzone do zmiennej sesyjnej). Probowalem wiec uzyc unset gdy obiekt jest pusty... ale cos niebardzo chce dzialac. Poradzcie cos. Zamieszczam skrypt:

[PHP] pobierz, plaintext 
// fragment index.php:
<? if (isset($_SESSION[cart])){
	$take=unserialize($_SESSION[cart]); }
	else
	{
		$take=new cart; 
 }
 if ($_GET[page]=='cart') { include('cart.php');}
?>
//a tu cart.php
<?
function showall(cart $take){
	while ( list($ky,$vl) = each($take->goods)){
		$dbase=new database;
		$wynik=$dbase->search('id',$klucz); //od tego momentu $wynik jest obiektem. wlasciwosci wyciagniete z bazy danych na p
dstawie id3
		$klucz=$wynik->name;
		echo "nazwa:$ky - ilosc:$vl<br>";
		unset($wynik);
		$_SESSION[cart]=serialize($take);
		if (empty($take)) {
			unset($_SESSION[cart]);
			unset($take); 
			}
		}
	}
 
switch ($_GET[action]) {
	case 'add':
		if(isset($_GET[id]) && isset($_GET[amount])){
			$take->add($_GET[id],$_GET[amount]);
 
			echo "added<br><br>";
			showall($take);
			}
		else
		{
			echo "Error";
			}
	break;
	case 'del':
		if(isset($_GET[id]) && isset($_GET[amount])){
			$take->del($_GET[id],$_GET[amount]);
 
			echo "deleted";
			showall($take);
			}
		else
		{
			echo "Error";
			}
		break;
	default:
		showall($take);
		break;
	}
 
?>
//klasa cart:
<?
class cart {
 
		public $goods = array();
 
		function add($id, $amount) {
			$this->goods[$id] += $amount;
			}
		function del($id, $amount) {
			 if ($this->goods[$id] > $amount) {
				$this->goods[$id] -= $amount;
				return true;
				} 
			elseif ($this->goods[$id] == $amount) {
				unset($this->goods[$id]);
				return true;
				} 
			else {
				return false;
			}
		}
	}
?>
[PHP] pobierz, plaintext 

Ten post edytował Edpl 6.08.2008, 08:03:04

[Edpl]

Mowie ze nie jest calkiem poprawny ;p ale jakby nie bylo... dziala ;p Poprawial tego typu bledy bede pozniej ;]

[mike]

Mowie ze nie jest calkiem poprawny ;p ale jakby nie bylo... dziala ;p Poprawial tego typu bledy bede pozniej ;]

To nie jest "niecałkiem poprawne". To jest całkiem źle.

Bez apostrofów:
1. przypadkowe, "poprawne" działanie
2. Generowanie błędów;
3. Opóźnianie skryptu, nawet do 30% przy częstym uzywaniu takiejgo czegoś.

Z apostrofami:
1. Działa dobrze

A tak w ogóle to po co robisz wcięcia w kodzie? Napisz skrypt i później dorób (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif)
Ja tu mówię o dobrych i popranwych nawykach a nie o "poprawianiu później".