Pomoc - Szukaj - Użytkownicy - Kalendarz
Pełna wersja: Usuwanie pozycji z koszyka - $_GET ?
Forum PHP.pl > Forum > PHP
shpaque
12.07.2011, 16:50:42
Witam, dziś mam jeszcze zagwostkę - jeśli mój koszyk pobiera zmienne za pomocą $_GET
[PHP] pobierz, plaintext 
  1. function AddToCart()
  2. {
  3.    $cart = isset($_SESSION['cart']) ? $_SESSION['cart'] : '';
  4.    $itemcount = isset($_SESSION['itemcount']) ? $_SESSION['itemcount'] : 0;
  5.  
  6.    $cart[PRODUCTIMAGE][$itemcount] = $_GET['productimage'];
  7.    $cart[CATEGORY][$itemcount] = $_GET['category'];
  8.    $cart[PRODUCTCODE][$itemcount] = $_GET['productcode'];
  9.    $cart[PRODUCTNAME][$itemcount] = $_GET['productname'];
  10.    $cart[FIRMA][$itemcount] = $_GET['firma'];
  11.    $cart[PRICE][$itemcount] = $_GET['price'];
  12.    $itemcount = $itemcount + 1;
  13.  
  14.    $_SESSION['cart'] = $cart;
  15.    $_SESSION['itemcount'] = $itemcount;
  16. }
[PHP] pobierz, plaintext 


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


to czy mógłbym zrobić coś takiego?
[PHP] pobierz, plaintext 
  1. $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
12.07.2011, 16:58:42
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
13.07.2011, 21:57:12
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 
  1. <?php
  2. session_start();
  3.  
  4. define("PRODUCTIMAGE", 0);
  5. define("CATEGORY", 1);
  6. define("PRODUCTCODE", 2);
  7. define("PRODUCTNAME", 3);
  8. define("FIRMA", 4);
  9. define("PRICE", 5);
  10.  
  11. if ($_SERVER['REQUEST_METHOD'] == 'GET')
  12. {
  13.    if (!empty($_GET['productimage']))
  14.    {
  15.       AddToCart();
  16.    }
  17.    else
  18.    {
  19.       $action = isset($_POST['action']) ? $_POST['action'] : '';
  20.       $value = strtoupper(substr($action, 0, 6));
  21.       switch ($value)
  22.       {
  23.  
  24.       // Opróżnij torbę
  25.       case "Opróżnij torbę":
  26.          unset($_SESSION['cart']);
  27.          header("Location: "."torba.php");
  28.          break;
  29.  
  30.       // Dalej
  31.       case "Dalej":
  32.          header("Location: "."klient.php");
  33.          break;
  34.  
  35.       }
  36.  
  37.       $action = isset($_GET['action']) ? $_GET['action'] : '';
  38.       $value = strtoupper(substr($action, 0, 6));
  39.       switch ($value)
  40.       {
  41.  
  42.       // Delete
  43. 	  case "delete":
  44. 	     echo "fakju";
  45. 		 break;
  46. 	  }
  47.  
  48.    }
  49. } 
  50.  
  51. function AddToCart()
  52. {
  53.    $cart = isset($_SESSION['cart']) ? $_SESSION['cart'] : '';
  54.    $itemcount = isset($_SESSION['itemcount']) ? $_SESSION['itemcount'] : 0;
  55.  
  56.    $cart[PRODUCTIMAGE][$itemcount] = $_GET['productimage'];
  57.    $cart[CATEGORY][$itemcount] = $_GET['category'];
  58.    $cart[PRODUCTCODE][$itemcount] = $_GET['productcode'];
  59.    $cart[PRODUCTNAME][$itemcount] = $_GET['productname'];
  60.    $cart[FIRMA][$itemcount] = $_GET['firma'];
  61.    $cart[PRICE][$itemcount] = $_GET['price'];
  62.    $itemcount = $itemcount + 1;
  63.  
  64.    $_SESSION['cart'] = $cart;
  65.    $_SESSION['itemcount'] = $itemcount;
  66. }
  67.  
  68. ?>
[PHP] pobierz, plaintext


nikt mi nie pomoze..?
muk4
13.07.2011, 22:33:18 
[PHP] pobierz, plaintext 
  1. case 'delete':
  2.     if(isset($_SESSION['itemcount'])) $_SESSION['itemcount'] -= 1;
  3.     break;
[PHP] pobierz, plaintext


Nie wiem czy o to chodzi. Dziwny masz ten koszyk...
shpaque
14.07.2011, 07:37:36
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
14.07.2011, 08:54:30
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
14.07.2011, 10:05:05
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)
Crozin
14.07.2011, 10:26:16
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
25.07.2011, 12:04:43
to jak mam kod zmodyfikowac zeby dzialalo?

[PHP] pobierz, plaintext 
  1. $total = 0;
  2.    for ($i=0; $i<$itemcount; $i++)
  3.    {
  4.       $strHTML .= "<tr>"."\n";
  5.       $strHTML .= "<td align=center><img src='./db/images/tn".$cart[PRODUCTIMAGE][$i]."'></td>"."\n";
  6.       $strHTML .= "<td align=center><h2>".$cart[CATEGORY][$i]."</td>"."\n";
  7.       $strHTML .= "<td align=center><h2>".$cart[PRODUCTCODE][$i]."</td>"."\n";
  8.       $strHTML .= "<td align=center><h2>".$cart[PRODUCTNAME][$i]."</td>"."\n";
  9.       $strHTML .= "<td align=center><h2>".$cart[FIRMA][$i]."</td>"."\n";
  10.       $strHTML .= "<td align=center><h2>"."".number_format($cart[PRICE][$i],2)."</td>"."\n";
  11.       $strHTML .= "<td align=center><a href='./torba.php?act=delete&id=$i'><img src='./images/del.png' title='Usuń pozycję'></a></td>"."\n";
  12.       $strHTML .= "<tr>"."\n";
  13.  
  14. 	  $act=$_GET['act'];
  15. 	  $id=$_GET['id'];
  16.  
  17. 	  if ($act == 'delete')
  18. 	  {
  19. 		 unset($_SESSION['itemcount']);
  20. 		 unset($_SESSION['cart']['id']);
  21. 		 break;
  22. 	  }	
[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
26.07.2011, 14:44:32
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.
shpaque
21.04.2013, 18:24:11
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 
  1.      ... $strHTML .= "<td align=center><a href='./torba.php?action=delete&id=$i'><img src='./images/del.png' title='Usuń pozycję'></a></td>"."\n";
  2.       $strHTML .= "<tr>"."\n";
  3.  
  4. 	  $act = 'action';
  5. 	  $id = 'id';  
  6. 	  if ($act == 'delete')
  7. 	  {
  8. 		  if(isset($_SESSION['itemcount'])) $_SESSION['itemcount'] -= 1;
  9.           break;
  10. 	  }
[PHP] pobierz, plaintext

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

a może tak?:

[PHP] pobierz, plaintext 
  1.    // Odejmij
  2.       case "ODEJM":
  3.          if(isset($_SESSION['itemcount']))
  4. 		 {
  5. 			 $_SESSION['cart'] -= $_SESSION['cart']['id'];
  6. 		 }
  7. 		 break;
[PHP] pobierz, plaintext
To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.
Invision Power Board © 2001-2025 Invision Power Services, Inc.