koszyk na zakupy, na sesjach Opcje

[Olimpia_ona]

Witam,
na początek powiem, że nie umiem programować obiektowo, ale znalazłam bardzo czytelny kawałek skryptu, który bym chciała dołączyć do swojego strukturalnego kodu.
Na początek wkleje ten piękny skrypt.
plik koszyk.php

[PHP] pobierz, plaintext 
<?php
/* koszyk.php - klasa bazowa koszyka na zakupy 
*/ 
/* zawiera definicje podstawowych operacji na koszu 
*/  
/* Zaimplementowane funkcje: 
*/ 
/* koszyk	   - konstruktor tworzy koszyk z zawartoscia 
*/ 
/* dodaj		- dodaje $ilosc $elementow do koszyka 
*/ 
/* usun		 - usuwa $ilosc $elementow z koszyka 
*/ 
/* stan		 - zwraca zawartosc w postaci hasza 
*/ 
 
 
class koszyk 
{ 
	/* tablica asocjacyjna zawierajaca stan koszyka 
*/ 
	/* klucz, to wartosc id produktu, wartosc to ilosc elementow 
*/ 
	var $zawartosc; 
 
 
	function dodaj($element, $ilosc) 
	{ 
		$this->zawartosc[$element] += $ilosc; 
	} 
 
 
	function usun($element, $ilosc) 
	{ 
		if ($this->zawartosc[$element] > $ilosc) 
			$this->zawartosc[$element] -= $ilosc; 
		else 
			unset($this->zawartosc[$element]); 
	} 
 
 
	function stan() 
	{ 
		return $this->zawartosc; 
	} 
 
 
 
}
?>
[PHP] pobierz, plaintext 

plik session.php

[PHP] pobierz, plaintext 
<?php
/* klasa koszyka */ 
include('koszyk.php'); 
 
 
/* start sesji */ 
session_start(); 
 
 
if (!$koszyk) 
	$koszyk = new koszyk; 
 
 
if ($dodaj) 
	$koszyk->dodaj($dodaj, 10); 
 
 
if ($usun) 
	$koszyk->usun($usun, 10); 
 
 
session_register('koszyk'); 
 
 
$stan = $koszyk->stan(); 
if ($stan) 
	while (list($key, $value) = each($stan)) 
		echo "$key $value<br>";
?>
[PHP] pobierz, plaintext 

i teraz tak, to co jest pod słowem plik session.php wkleiłam sobie na stronę pomiędzy jakieś tam <td></td> no i wyskakuje mi błąd
Fatal error: Unknown(): The script tried to execute a method or access a property of an incomplete object. Please ensure that the class definition <b>koszyk</b> of the object you are trying to operate on was loaded _before_ the session was started

więc zrobiłam tak jak napisali i wkleiłam fragment include("koszyk.php"); na górze swojej strony index.php

[PHP] pobierz, plaintext 
<?php
include ('incl/funkcje.php');
  include("koszyk.php");
  session_start();
  $status=$_GET['status'];
..itd
?>
[PHP] pobierz, plaintext 

no ale w tym wypadku wyświetla mi się błąd:
Warning: session_start(): Cannot send session cache limiter - headers already sent (output started at c:\usr\krasnal\www\sexshop\koszyk.php:60) in c:\usr\krasnal\www\sexshop\index.php on line 4

Wiec jak ja mam to rozwiązać? Pomocy

[Cysiaczek]

session_start() na samiuteńkim początku skryptu, bo musi zostać zainicjowana zanim jakikolwiek znak zostanie wysłany do przeglądarki. Było na forum z 1000 razy.

Pozdrawiam.

[Babcia@Stefa]

Jak powiedział kolega z góry.

[PHP] pobierz, plaintext 
<?php /* start sesji */ session_start(); 
/* klasa koszyka */ 
include('koszyk.php');  
if (!$koszyk)	 
$koszyk = new koszyk; 
if ($dodaj)	 
$koszyk->dodaj($dodaj, 10); 
if ($usun)	 
$koszyk->usun($usun, 10); 
session_register('koszyk'); 
$stan = $koszyk->stan(); 
if ($stan)	 
while (list($key, $value) = each($stan))		 
echo "$key $value<br>";?>
[PHP] pobierz, plaintext 

To w razie jakiś nie jasności

Dziekuję, Babcia@Stefa

Ten post edytował Babcia@Stefa 10.05.2007, 20:44:25

[Sedziwoj]

Taki fajny cytacik:

If you are using $_SESSION (or $HTTP_SESSION_VARS), do not use session_register(), session_is_registered(), and session_unregister().

Komentarze tyczące się metod powinny być przed tymi metodami.
Czy koniecznie musi być PHP4? obiekty w PHP5 są 'bardziej obiektowe'.
Nazwa 'usun' sugeruje, że robi coś innego niż robi w rzeczywistości.
Do tego najczęściej jest modyfikowana ilość elementów, a nie zmniejszana/powiększana.

Do tego nie do końca rozumiem, co tak na prawdę to ma robić.

EDIT lit.

Ten post edytował Sedziwoj 10.05.2007, 21:24:19

[Olimpia_ona]

Mam jeszcze jedno pytanie do tego kodu który wkleiłam na górze:
Jak to przerobić, by w koszu jeszcze przetrzymywać kolor i rozmiar produktu?
Jak by miało wyglądać ciało funkcji dodaj, gdybym zrobiła tak

[PHP] pobierz, plaintext 
<?php
function dodaj($element, $ilosc, $kolor, $rozmiar)
{
  //?
}
?>
[PHP] pobierz, plaintext 

wywoływałabym tak:
dodaj($dodaj, 1, $kolor, $rozmiar)

no i jeśli ktoś wie, to w takim razie jak wyświetlić taką tablicę wielowymiarową tym sposobem
while (list($key, $value) = each($stan))

[NuLL]

Musialaby zamiast ilosci w tej tablicy jako kolejne elementy trzymac tablice i tam w odpowiedni sposob zapisywac takie dane.

[Sedziwoj]

Mam pytanie to ma być obiektowo? czy bawimy się proceduralnie?
Przecież tan koszyk to nic innego niż kolekcja... (nie wiem czy to ta nazwa)
Ogólnie powinien być np.:

[PHP] pobierz, plaintext 
<?php
class Produkt{
   private $intId;
   private $strKolor;
   private $intRozmiar;
 
   public function getId(){
	   return $this->intId;
   }
   public function __construct( $intId ){
	  //chyba najlepiej gdyby tu pobierało (podkreślam chyba, bo można przekazywać przy tworzeniu)
 
	  //zamiast na sztywno, powinno skądś to pobrać
	  $this->strKolor = 'czarnr'.$intId;
	  $this->intRozmiar = 4+$intId;
   }
   
   public function getKolor(){
	  return $this->strKolor;
   }
   
   public function getRozmiar(){
	  return $this->intRozmiar;
   }
}
?>
[PHP] pobierz, plaintext 

no i wtedy ten koszyk by wyglądał jakoś tak:

[PHP] pobierz, plaintext 
<?php
class Koszyk{
   private $arrProdukt;
 
   public function getByProduktId( $intId ){
	  return $this->arrProdukt[$intId];
   }
   
   public function getWszystkie(){
	  return $this->arrProdukt;
   }
   
   public function dodaj( Produkt $objProdukt, $intIlosc ){
	  if( !isset(this->arrProdukt[$intId]) ){
		 this->arrProdukt[$objProdukt->getId] = array(
			'produkt' => $objProdukt,
			'ilosc' => $intIlosc
			);
	  }else{
		 $this->arrProdukt[$intId]['ilosc'] += $intIlosc;
	  }
   }
   
   public function usun( Produkt $objProdukt ){
	  $this->arrProdukt[$intId]['ilosc'] -= $intIlosc;
   }
}
?>
[PHP] pobierz, plaintext 

co prawda można to jeszcze ładniej... do tego nie dałem żadnej weryfikacji.
Ale teraz wystarczy wiedzieć, że obiekt Produkt ma metodę getId(), która zwraca unikalny identyfikator, a reszta nas nie obchodzi, no bo oprucz co to za element i ile go jest nie powinno nas nic obchodzić.
(sam kod na tej zasadzie można jeszcze lepiej zorganizować, choć się zastanawiam czy warto trzymać coś więcej niż id produktu i ilość, bo PHP i tak to wszystko traci między stronami, a nie ma sensu trzymać całych opisów w danych sesyjnych, więc iloć i id produktu wystarczyło by, a potem kiedy trzeba wyświetlić zawartość to już nie kłopot jedno zapytanie i mamy wszystkie produkty z koszyka)

[Olimpia_ona]

...no bo oprucz co to za element i ile go jest nie powinno nas nic obchodzić.
(sam kod na tej zasadzie można jeszcze lepiej zorganizować, choć się zastanawiam czy warto trzymać coś więcej niż id produktu i ilość, bo PHP i tak to wszystko traci między stronami, a nie ma sensu trzymać całych opisów w danych sesyjnych, więc iloć i id produktu wystarczyło by, a potem kiedy trzeba wyświetlić zawartość to już nie kłopot jedno zapytanie i mamy wszystkie produkty z koszyka)

No właśnie ja mam tak zrobione, że do tego koszyka jest tylko id produktu i ilość, ale niektóre produkty mają wiele kolorów i różne rozmiary i chcę zrobić tak, by klient zanim wciśnie dodaj do koszyka mógł zaznaczyć, jaki kolor i rozmiar danego produktu chce kupić i to też bym chciała trzymać w koszyku/sesji.
Dla tego zależałoby mi by to było jakoś tak na podstawie takiej funkcji

[PHP] pobierz, plaintext 
<?php
function dodaj($element, $ilosc, $kolor, $rozmiar) 
{  //? 
}
?>
[PHP] pobierz, plaintext 

>> Sedziwoj w tej Twojej funkcji nie rozumiem tego fragmentu:

[PHP] pobierz, plaintext 
<?php
public function __construct( $intId )
{	  $this->strKolor = 'czarnr'.$intId;	  $this->intRozmiar = 4+$intId;   }
?>
[PHP] pobierz, plaintext 

czumu jest 'czarnr'.$intId z kropką, a tu z plusem 4+$intId

[Sedziwoj]

Jeśli o tamten fragment chodzi, to po prostu się nie zrozumieliśmy. Bo sądziłem, że kolor i rozmiar są na stałe przypisane do produktu, a jeśli nie są to zmienia postać rzeczy.
Co do tego co przekazujesz do Koszyka, to zależy jak uniwersalny kod ma być. Bo jeśli w koszyku chcesz mieć więcej rodzajów produktów przechowywać to nie możesz na sztywno ich właściwości przekazywać, tylko obiekt z tymi właściwościami, wtedy wystarczy że konkretny rodzaj dziedziczy po produktach i jest już dobrze.
Tylko trzeba umożliwić sobie identyfikacje typu produktu wyjętego z koszyka, aby móc specyficzne właściwości pobrać, choć też nie do końca, bo przy wyświetlaniu zawartości wyświetla się tylko to co mają wszystkie produkty, a jak się wejdzie w opis specyficznego to wtedy na podstawie id można rozpoznać typ, choć to akurat zależy od konkretnej budowy relacji.

Co do koszyka w sesji to odpowiednie metody w obiekcie __sleep() i __wakeup(), wtedy też każda klasa produktów też by musiała mieć to wbudowane, aby swoje specyficzne wartości odtworzyć.
Choć może to że próbuję to upakować w obiekty nie jest łatwiejsze/najlepsze rozwiązanie.

P.S. Jest specjalne metody aby sprawdzić ilość i zawartość argumentów przekazywanych do funkcji/metody.

[Olimpia_ona]

Na jednym z forów Pan o nicku 123456 napisał fajną klasę, którą chciałabym użyć, niestety nie działa ona w php wersji 4 tylko pod wersję 5. Wyskakuje mi błąd:
Parse error: parse error, expecting `T_OLD_FUNCTION' or `T_FUNCTION' or `T_VAR' or `'}''

[PHP] pobierz, plaintext 
<?php
class koszyk {  
 
	$zawartosc = Array(); 
	 
// -------------   
	function dodaj($goods)  
	{  
		$this->zawartosc[] = $goods; 
	}   
// ------------- 
	// (...) 
// ------------- 
	function stan() 
	{ 
		return $this->zawartosc; 
	} 
// ------------- 
	// (...) 
// ------------- 
}
?>
[PHP] pobierz, plaintext 

Jak należało by ją zmienić?

[rodzyn]

Nalezaloby napisac do admina zeby zainstalowal PHP5 bo obciach
Po co sie meczyc z OOP w PHP4 skoro w pelni go nie obsluguje? o_O

[Olimpia_ona]

Na serwerze jest PHP5, ale zanim coś tam wrzuce, to robie lokalnie na krasnalu, który ma php4. Próbowałam zainstalować PHP5, ale zaczeło mi wywalać jakieś błędy i musiałam odinstalować tą nakładkę.

[rodzyn]

Proponuje: Wamp5

[Nightmare]

W kransalu klikasz prawym na ikonkę apache (zielone piórko) -> ustawienia -> php -> php5 i już masz php5 na krasnalu