Usuwanie pozycji z koszyka - $_GET ?, czy można to wykonać w takowy sposób? Jak? Opcje

[shpaque]

Witam, dziś mam jeszcze zagwostkę - jeśli mój koszyk pobiera zmienne za pomocą $_GET

[PHP] pobierz, plaintext 
function AddToCart()
{
   $cart = isset($_SESSION['cart']) ? $_SESSION['cart'] : '';
   $itemcount = isset($_SESSION['itemcount']) ? $_SESSION['itemcount'] : 0;
 
   $cart[PRODUCTIMAGE][$itemcount] = $_GET['productimage'];
   $cart[CATEGORY][$itemcount] = $_GET['category'];
   $cart[PRODUCTCODE][$itemcount] = $_GET['productcode'];
   $cart[PRODUCTNAME][$itemcount] = $_GET['productname'];
   $cart[FIRMA][$itemcount] = $_GET['firma'];
   $cart[PRICE][$itemcount] = $_GET['price'];
   $itemcount = $itemcount + 1;
 
   $_SESSION['cart'] = $cart;
   $_SESSION['itemcount'] = $itemcount;
}
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
$total = 0;
   for ($i=0; $i<$itemcount; $i++)
   {
      $strHTML .= "<tr>"."\n";
      $strHTML .= "<td align=center><img src='./db/images/tn".$cart[PRODUCTIMAGE][$i]."'></td>"."\n";
      $strHTML .= "<td align=center><h2>".$cart[CATEGORY][$i]."</td>"."\n";
      $strHTML .= "<td align=center><h2>".$cart[PRODUCTCODE][$i]."</td>"."\n";
      $strHTML .= "<td align=center><h2>".$cart[PRODUCTNAME][$i]."</td>"."\n";
      $strHTML .= "<td align=center><h2>".$cart[FIRMA][$i]."</td>"."\n";
      $strHTML .= "<td align=center><h2>"."".number_format($cart[PRICE][$i],2)."</td>"."\n";
      $strHTML .= "<td align=center><img src='./images/del.png' title='Usuń pozycję'></td>"."\n";
      $strHTML .= "<tr>"."\n";
[PHP] pobierz, plaintext 

to czy mógłbym zrobić coś takiego?

[PHP] pobierz, plaintext 
$strHTML .= "<td align=center><a href='torba.php?action=delete'><img src='./images/del.png' title='Usuń pozycję'></a></td>"."\n";
[PHP] pobierz, plaintext 

Jeśli tak jak to powinno poprawnie brzmieć i jak zdefiniować akcję zeby usunąć pozycję (i jak usunąć wszystko z koszyka)?

dzięki za pomoc - z góry

[Fifi209]

Dodaj po action=delete&id=id_pozycji

Potem w skrypcie sprawdzaj czy action == delete, sprawdzaj czy id_pozycji występuje - jeżeli tak to usuń (ew. zmień ilość o jeden [sprawdzając czy będzie > 0])

A co do wszystkich to:

action=delete_all

Najprościej i wtedy usuwaj koszyk.

[shpaque]

ale czy akcja zostanie spełniona? czy mam podawac w request method w akcji $_GET?

podam cały początek z definicjami akcji - tu na pewno jest błąd...

[PHP] pobierz, plaintext 
<?php
session_start();
 
define("PRODUCTIMAGE", 0);
define("CATEGORY", 1);
define("PRODUCTCODE", 2);
define("PRODUCTNAME", 3);
define("FIRMA", 4);
define("PRICE", 5);
 
if ($_SERVER['REQUEST_METHOD'] == 'GET')
{
   if (!empty($_GET['productimage']))
   {
      AddToCart();
   }
   else
   {
      $action = isset($_POST['action']) ? $_POST['action'] : '';
      $value = strtoupper(substr($action, 0, 6));
      switch ($value)
      {
 
      // Opróżnij torbę
      case "Opróżnij torbę":
         unset($_SESSION['cart']);
         header("Location: "."torba.php");
         break;
 
      // Dalej
      case "Dalej":
         header("Location: "."klient.php");
         break;
 
      }
 
      $action = isset($_GET['action']) ? $_GET['action'] : '';
      $value = strtoupper(substr($action, 0, 6));
      switch ($value)
      {
 
      // Delete
	  case "delete":
	     echo "fakju";
		 break;
	  }
 
   }
} 
 
function AddToCart()
{
   $cart = isset($_SESSION['cart']) ? $_SESSION['cart'] : '';
   $itemcount = isset($_SESSION['itemcount']) ? $_SESSION['itemcount'] : 0;
 
   $cart[PRODUCTIMAGE][$itemcount] = $_GET['productimage'];
   $cart[CATEGORY][$itemcount] = $_GET['category'];
   $cart[PRODUCTCODE][$itemcount] = $_GET['productcode'];
   $cart[PRODUCTNAME][$itemcount] = $_GET['productname'];
   $cart[FIRMA][$itemcount] = $_GET['firma'];
   $cart[PRICE][$itemcount] = $_GET['price'];
   $itemcount = $itemcount + 1;
 
   $_SESSION['cart'] = $cart;
   $_SESSION['itemcount'] = $itemcount;
}
 
?>
[PHP] pobierz, plaintext 

nikt mi nie pomoze..?

[muk4]

[PHP] pobierz, plaintext 
case 'delete':
    if(isset($_SESSION['itemcount'])) $_SESSION['itemcount'] -= 1;
    break;
[PHP] pobierz, plaintext 

Nie wiem czy o to chodzi. Dziwny masz ten koszyk...

[shpaque]

w zasadzie nie jest zle, ale przy takim kodzie usuwa wszystko co jest dodane do tej konkretnej pozycji - to znaczy jesli mam rzecz A B C i D i kliknej usun pozycje na "B" ususnie mi B C i D

[Crozin]

1. Wszystkie dane modyfikujące coś (dodanie, usunięcie, modyfikacja koszyka) muszą być wysyłane w żądaniu POST, a nie jako część URL-a.
2. Skąd taki dziwny pomysł na przechowywanie danych jako:

Kod

Array (
  [PRODUCTNAME] => Array (
    [0] => 'Produkt #1',
    [1] => 'Produkt #2'
  ),
  [PRICE] => Array (
    [0] => 59.99,
    [1] => 9.90
  )
)

Zamiast w normalnej formie:

Kod

Array (
  [0] => Array (
    [PRODUCTNAME] => 'Produkt #1',
    [PRICE] => 59.99
  ),
  [1] => Array (
    [PRODUCTNAME] => 'Produkt #2',
    [PRICE] => 9.90
  )
)

Sam sobie rzucasz kłody pod nogi.

[shpaque]

Wlasnie problem w tym, ze to nie moj pomysl tylko programu, ktory wygenerowal taki kod, ja usiluje jedynie go pod siebie podpasowac, nie chcac wiele zmieniac bo nie mam kompletnie prawie wiedzy na ten temat... Mniej wiecej rozumiem o co chodzi tylko teraz znow musialbym tydzien siedziec zeby dojsc do tego sposobu i pozmieniac wszystko...

Ad1. Dlaczego MUSZĄ być wysyłane za pomocą POST? Umnie w GET wszystko chodzi pięknie pozostaje tylko usuniecie konkretnej pozycji i calego koszyka (bo program wygenerowal sobie usuniecie koszyka kasowaniem calej sesji - wowczas pada mi logowanie, obsluga baz danych, i extrasy uzytkownika)

Ten post edytował shpaque 14.07.2011, 10:15:44

[Crozin]

Ponieważ żądania typu GET mogą być cache'owne (na dowolnym poziomie), znacznie łatwiej jest wymusić na kimś nieświadome ich wywołanie (jeżeli takie rzeczy jak realizacja zamówienia również lecą GET-em to mogę właściwie zrealizować zamówienie z czyjegoś konta bez jego wiedzy) - w dużym skrócie: z ogólnych powodów bezpieczeństwa.

[shpaque]

to jak mam kod zmodyfikowac zeby dzialalo?

[PHP] pobierz, plaintext 
$total = 0;
   for ($i=0; $i<$itemcount; $i++)
   {
      $strHTML .= "<tr>"."\n";
      $strHTML .= "<td align=center><img src='./db/images/tn".$cart[PRODUCTIMAGE][$i]."'></td>"."\n";
      $strHTML .= "<td align=center><h2>".$cart[CATEGORY][$i]."</td>"."\n";
      $strHTML .= "<td align=center><h2>".$cart[PRODUCTCODE][$i]."</td>"."\n";
      $strHTML .= "<td align=center><h2>".$cart[PRODUCTNAME][$i]."</td>"."\n";
      $strHTML .= "<td align=center><h2>".$cart[FIRMA][$i]."</td>"."\n";
      $strHTML .= "<td align=center><h2>"."".number_format($cart[PRICE][$i],2)."</td>"."\n";
      $strHTML .= "<td align=center><a href='./torba.php?act=delete&id=$i'><img src='./images/del.png' title='Usuń pozycję'></a></td>"."\n";
      $strHTML .= "<tr>"."\n";
 
	  $act=$_GET['act'];
	  $id=$_GET['id'];
 
	  if ($act == 'delete')
	  {
		 unset($_SESSION['itemcount']);
		 unset($_SESSION['cart']['id']);
		 break;
	  }	
[PHP] pobierz, plaintext 

a coś w ten deseń się nie da wykombinować? Jestem pewien że dla kogoś kto zna się na php to jest sekunda odnaleźć kod który usuwa konkretną pozycję... ;(

[adbacz]

Odnaleźć? Albo nie napisałeś kodu "usuwania" aby go "odnaleźć" albo go po prostu nie ma i trzeba napisać.

Ja bym zrobił tak jak jeden użytkownik wyżej napsiał. W zmiennej $_GET['id'] dać ID danego produktu (ale tu nasuwa się pytanie - skąd wziąć te ID?) a w $_GET['action'] dać 'delete' i później sprawdzać za pomocą instrukcji warunkowej, czy istnieją te zmienne i jeśli tak, to za pomocą pętli, która przeleci po wszystkich rzeczach w schowku sprawdzając przy okazji warunek, czy któryś z tych rzeczy ma takie samo ID jak to w $_GET['id']. Jesli tak (faktycznie dziwnie jets ta tablica z rzeczami) to usunąć wszystkie dane głównych indeksów, gdzie podrzędne maja taki sam index jak ten ID.

Trosza namieszałem ale to dość proste jest...

PS.
Z tego co widać, to podczas dodawania czegoś do koszyka, nie ma gdzie się zapisać jakieś unikalny ID produktu więc musiałbyś dopisać jeszcze kilka linijek i dopiero to co napisałem będzie grało.

Ten post edytował adbacz 26.07.2011, 14:46:32

[shpaque]

Dziekuję za odzew w końcku ktoś z nowym pomysłem napisał - tyle że mi to nic nie mówi 'kilka linijek dopisac' ok gdybym sie na tym znal dopisalbym sobie kilka linijek - tyle ze ja nie wiem CO i GDZIE dopisac...

i dalej stoję z tematem - nie wiem co zrobic... ;/

[PHP] pobierz, plaintext 
     ... $strHTML .= "<td align=center><a href='./torba.php?action=delete&id=$i'><img src='./images/del.png' title='Usuń pozycję'></a></td>"."\n";
      $strHTML .= "<tr>"."\n";
 
	  $act = 'action';
	  $id = 'id';  
	  if ($act == 'delete')
	  {
		  if(isset($_SESSION['itemcount'])) $_SESSION['itemcount'] -= 1;
          break;
	  }
[PHP] pobierz, plaintext 

to nie działa i nie mam pojęcia dlaczego...

a może tak?:

[PHP] pobierz, plaintext 
   // Odejmij
      case "ODEJM":
         if(isset($_SESSION['itemcount']))
		 {
			 $_SESSION['cart'] -= $_SESSION['cart']['id'];
		 }
		 break;
[PHP] pobierz, plaintext