[PHP] Błąd przy dodawaniu do shooping card Opcje

[-Gość-]

wyskakuje mi błąd: Notice: Undefined index: action in ...
Może mi ktoś pomóc, nie wiem co robić, dodanie isset generuje inny błąd , help

[PHP] pobierz, plaintext 
session_start();
$cart = $_SESSION['cart'];
$action =$_GET['action'];
switch ($action) {
	case 'add':
		if ($cart) {
			$cart .= ','.$_GET['id'];
		} else {
			$cart = $_GET['id'];
		}
		break;
	case 'delete':
		if ($cart) {
			$items = explode(',',$cart);
			$newcart = '';
			foreach ($items as $item) {
				if ($_GET['id'] != $item) {
					if ($newcart != '') {
						$newcart .= ','.$item;
					} else {
						$newcart = $item;
					}
				}
			}
			$cart = $newcart;
		}
		break;
	case 'update':
	if ($cart) {
		$newcart = '';
		foreach ($_POST as $key=>$value) {
			if (stristr($key,'qty')) {
				$id = str_replace('qty','',$key);
				$items = ($newcart != '') ? explode(',',$newcart) : explode(',',$cart);
				$newcart = '';
				foreach ($items as $item) {
					if ($id != $item) {
						if ($newcart != '') {
							$newcart .= ','.$item;
						} else {
							$newcart = $item;
						}
					}
				}
				for ($i=1;$i<=$value;$i++) {
					if ($newcart != '') {
						$newcart .= ','.$id;
					} else {
						$newcart = $id;
					}
				}
			}
		}
	}
	$cart = $newcart;
	break;
}
[PHP] pobierz, plaintext 

[toaspzoo]

wyskakuje mi błąd: Notice: Undefined index: action in ...

I co dalej, w której linii ?

[-Gość-]

sorki nie wrzuciłm

[PHP] pobierz, plaintext 
$action =$_GET['action'];
[PHP] pobierz, plaintext 

Notice: Undefined index: action in C:\xampp\htdocs\Myonlinestore\card.php on line 14

[toaspzoo]

Czyli już wszystko w porządku ?

[-Gość-]

nie, dalej to samo

[wNogachSpisz]

Musisz sprawdzić czy zmienna GET['action'] istnieje (czy została zadeklarowana).
Użyj w tym celu funkcj isset(), to chyba najszybsza i najczęsciej wykorzystywana funkcja w całym PHP.

Jeśli w parametrze do funkcji isset() przekażesz niezadeklarowaną zmienną, nie spowoduje to wyświetlenia otrzeżenia. Na tym polega jej wyjątkowość.

Ten post edytował wNogachSpisz 12.08.2012, 15:36:05

[-Gość-]

zmieniłem na

[PHP] pobierz, plaintext 
$action =isset($_GET['action']);
[PHP] pobierz, plaintext 

i wywala mi błąd: Notice: Undefined index: id in C:\xampp\htdocs\Myonlinestore\card.php on line 17
i po odswieżeniu dodaje mi jeden w

[PHP] pobierz, plaintext 
$cart = $_SESSION['cart'];
	if (!$cart) {
		return '<p>You have no items in your shopping cart</p>';
	} else {
		// Parse the cart session variable
		$items = explode(',',$cart);
		$s = (count($items) > 1) ? 's':'';
		return '<p>You have'.count($items).' item'.$s.' in your shopping cart</p>';
	}
[PHP] pobierz, plaintext 

tu jest cały kod
card.php

[PHP] pobierz, plaintext 
<?php
error_reporting(E_ALL);
ini_set('display_errors','1');
?>
<?php
// Include database connection
require_once('storesript/conecttomysql.php');
// Include functions
require_once('functions.inc.php');
// Start the session
session_start();
$cart = $_SESSION['cart'];
$action =isset($_GET['action']);
switch ($action) {
	case 'add':
		if ($cart) {
			$cart .= ','.$_GET['id'];
		} else {
			$cart = $_GET['id'];
		}
		break;
	case 'delete':
		if ($cart) {
			$items = explode(',',$cart);
			$newcart = '';
			foreach ($items as $item) {
				if ($_GET['id'] != $item) {
					if ($newcart != '') {
						$newcart .= ','.$item;
					} else {
						$newcart = $item;
					}
				}
			}
			$cart = $newcart;
		}
		break;
	case 'update':
	if ($cart) {
		$newcart = '';
		foreach ($_POST as $key=>$value) {
			if (stristr($key,'qty')) {
				$id = str_replace('qty','',$key);
				$items = ($newcart != '') ? explode(',',$newcart) : explode(',',$cart);
				$newcart = '';
				foreach ($items as $item) {
					if ($id != $item) {
						if ($newcart != '') {
							$newcart .= ','.$item;
						} else {
							$newcart = $item;
						}
					}
				}
				for ($i=1;$i<=$value;$i++) {
					if ($newcart != '') {
						$newcart .= ','.$id;
					} else {
						$newcart = $id;
					}
				}
			}
		}
	}
	$cart = $newcart;
	break;
}
$_SESSION['cart'] = $cart;
?><!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">
 
<html xmlns="http://www.w3.org/1999/xhtml" xml:lang="en" lang="en">
<head>
	<title>PHP Shopping Cart Demo ˇ Cart</title>
	<link rel="stylesheet" href="style/styles.css" />
</head>
 
<body>
 
<div id="shoppingcart">
 
<h1>Your Shopping Cart</h1>
 
<?php
echo writeShoppingCart();
?>
 
</div>
 
<div id="contents">
 
<h1>Please check quantities...</h1>
 
<?php
echo showCart();
?>
 
<p><a href="products.php">Back to bookshop...</a></p>
 
</div>
 
</body>
</html>
[PHP] pobierz, plaintext 

functions.inc.php

[PHP] pobierz, plaintext 
<?php
function writeShoppingCart() {
	$cart = $_SESSION['cart'];
	if (!$cart) {
		return '<p>You have no items in your shopping cart</p>';
	} else {
		// Parse the cart session variable
		$items = explode(',',$cart);
		$s = (count($items) > 1) ? 's':'';
		return '<p>You have'.count($items).' item'.$s.' in your shopping cart</p>';
	}
}
 
function showCart() {
	global $db;
	$cart = $_SESSION['cart'];
	if ($cart) {
		$items = explode(',',$cart);
		$contents = array();
		foreach ($items as $item) {
			$contents[$item] = (isset($contents[$item])) ? $contents[$item] + 1 : 1;
		}
		$output[] = '<form action="card.php?action=update" method="post" id="cart">';
		$output[] = '<table>';
		foreach ($contents as $id=>$qty) {
			$sql = mysql_query("SELECT * FROM products WHERE id='$id' LIMIT 1");
		while ($row = mysql_fetch_array($sql)) {
			$productname= $row ["productname"];
			$producent= $row ["producent"];
			$cena= $row ["cena"];
			$output[] = '<tr>';
			$output[] = '<td><a href="card.php?action=delete&id='.$id.'" class="r">Remove</a></td>';
			$output[] = '<td>'.$productname.' by '.$producent.'</td>';
			$output[] = '<td>&pound;'.$cena.'</td>';
			$output[] = '<td><input type="text" name="qty'.$id.'" value="'.$qty.'" size="3" maxlength="3" /></td>';
			$output[] = '<td>&pound;'.($cena * $qty).'</td>';
			$total += $cena * $qty;
			$output[] = '</tr>';
		}}
		$output[] = '</table>';
		$output[] = '<p>Grand total: <strong>&pound;'.$total.'</strong></p>';
		$output[] = '<div><button type="submit">Update cart</button></div>';
		$output[] = '</form>';
	} else {
		$output[] = '<p>You shopping cart is empty.</p>';
	}
	return join('',$output);
}
?>
[PHP] pobierz, plaintext 

[wNogachSpisz]

Istnieją dwie możliwości:

1. Jesteś najbardziej bezmyślną osobą jaką spotkałem na tym forum.
2. Twój pierwszy kontakt z programowaniem w PHP jest właśnie dzisiaj i należy Cię skierowac na kurs dla bardzo początkujących.

[-Gość-]

a może Ci się pomyliły fora - jaka sama nazwa wskazuje "przedszkole" jest dla początkujących debilu, jeżeli nie masz ochoty pomagać to po co tu jesteś, nie masz przyjaciół w realnym życiu i szukasz na forach, to już wiem dlaczego!!!

[wNogachSpisz]

Przedszkole przedszkolem, ale kurs dla początkujących wypadałoby zaliczyć.

Zresztą Twoje postępowanie jest ewidentne. Od mojego postu gdzie zasugerowałem użycie isset, do Twojej odpowiedzi - w której uraczyłeś nas bezmyślnym jej użyciem - minęło 8 minut. W tak krótkim czasie to nawet opisu funkcji w manualu nie zdążyłbyś przeczytać.

Skoro ja jestem debilem, to kim ty jesteś?
Bezmyślny leń któremu trzeba "przeżuć mięsko" to najdelikatniejsze określenie jakie przychodzi mi do głowy.


Ten post edytował wNogachSpisz 12.08.2012, 16:04:30

[-Gość-]

Przedszkole przedszkolem, ale kurs dla początkujących wypadałoby zaliczyć.

Zresztą Twoje postępowanie jest ewidentne. Od mojego postu gdzie zasugerowałem użycie isset, do Twojej odpowiedzi - w której uraczyłeś nas bezmyślnym jej użyciem - minęło 8 minut. W tak krótkim czasie to nawet opisu funkcji w manualu nie zdążyłbyś przeczytać.

Skoro ja jestem debilem, to kim ty jesteś?
Bezmyślny leń to najdelikatniejsze określenie jakie przychodzi mi do głowy.

juz sie gadałeś geniuszu bez przyjaciół jeżeli tak to odbij ode mne jeżeli nie to mi pomóż

[wNogachSpisz]

W jednej chwili nazywasz mnie debilem, w następnej geniuszem. Zdecyduj się

[-Gość-]

W jednej chwili nazywasz mnie debilem, w następnej geniuszem. Zdecyduj się

w porównaniu do mojej wiedzy w php to jesteś geniuszem , ale patrząc na wcześniejszy komentarz tylko jedno pzychodzi do głowy ...

[wNogachSpisz]

w porównaniu do mojej wiedzy w php to jesteś geniuszem , ale patrząc na wcześniejszy komentarz tylko jedno pzychodzi do głowy ...

Kiedy ktoś pisze że tylko jedno przychodzi mu do głowy, to mi też tylko jedno przychodzi do głowy...
O ile nie jesteś dziewczyną, to możesz zapomnieć

Ten post edytował wNogachSpisz 12.08.2012, 16:13:58

[-Gość-]

Kiedy ktoś pisze że tylko jedno przychodzi mu do głowy, to mi też tylko jedno przychodzi do głowy...
O ile nie jesteś dziewczyną, to możesz zapomnieć

następna debilna odpowiedz, cytat: "najgłupsza osoba na forum" zaczynasz wygrywać

[Daiquiri]

Autorze tamatu - niestety brakuje Ci podstaw PHP. User wNogachSpisz dał Ci kompletne rozwiązanie, jednak źle je zaimplementowałeś. Powinieneś dodać warunek z isset, w stylu
if (isset ($TwojaZmienna) { zrób cośtam }. Ostrzeżenie, które Ci się wyświetla to komunikat o tym, iż sprawdzasz zawartość zmiennej z indeksem, który nie został ustawiony (innymi słowy, nie masz w adresie końcówki action=.

Zamykam, nie ma sensu dalej się "wyzywać".