[PHP][MySQL] złe przekazywanie danych z form do koszyka, Pomoc w naprawie błędu. nie dodaje mi ceny oraz źle wstawia ilość :( Opcje

[FliSs_tCv]

Witam nie owijając w bawełne (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) pragne przedstawić mój problem otóż polega on na tym ,że nie przekazuję danych tak jak powinien tzn. chodzi o nie dokładne przesyłanie ilości towaru (w form ustawiam 1 a w koszyku wyświetla 2) i tutaj nie o tyle błąd co moja nieumiejętność w zrozumieniu może i nawet w wykonaniu aby cena danego towaru która w form jest wybierana za pomocą option select byla przesłana taka jak została wybrana. w pliku "koszyk.pizza.php" będzie widoczne pole cena lecz nie wiem jak wykonać aby przyjmowała wartość wysłanego z wcześniejszego formularza

oto kod tego formularza który przesyła dane do koszyka nosi nazwę "zam.pizza.php"

[PHP] pobierz, plaintext 
<div style="margin : auto; width : 450px; text-align : left;">
	<div style="width : 420px; margin-top : 20px; margin-bottom : 30px;">
		<?php
			session_start();
				$sql_serwer = "localhost";
				$sql_login  = "";
				$sql_haslo  = "";
				$sql_baza   = "caprii";
				$sql_tabela = "pizza";
			mysql_connect($sql_serwer, $sql_login, $sql_haslo);
			mysql_select_db($sql_baza);
			$_id = $_GET['id'];
			$query = "SELECT nazwa, cena_m, cena_d FROM pizza WHERE id='$_id'";
			mysql_query("SET CHARSET latin2");
			$result = mysql_query($query) or die(mysql_error());
			$row = mysql_fetch_array($result);
			echo "<table style=\"border : 1px solid #363636;\">
					<tr>
							<td style=\"width : 400px; height : 50px; text-align : center; border : 1px solid white; background : white; color : #363636; f
ont-size : 14px;\">
						Pizza, którą wybrałeś, to :<b>$row[nazwa]
					</td>
				</tr>";
		?></b>
		<tr>
			<form action="index.php?page=koszyk.pizza&id=<?php echo $_id; ?>" method="post">
			<input type="hidden" name="pizza" value="<?php echo $row['nazwa']; ?>">
						<td style="width : 400px; height : 50px; text-align : center; border : 1px solid white; background : white; color : #363636; f
ont-size : 12px;">
				<b>Wybierz wielkość</b>: <select style="border : 1px solid #363636;" name="wielkosc">
							<option value="m" selected>Mała (<?php echo $row['cena_m']; ?> zł)</option>
							<option value="d">Duża (<?php echo $row['cena_d']; ?> zł)</option>
						  </select>
 
				<b>Wybierz ilość</b>: <input style="border : 1px solid #363636;" type="text" name="ilosc" value=1 size=5 maxlength=2>
			</td>
		 </tr>
		<tr>
						<td style="width : 400px; height : 330px; text-align : center; border : 1px solid white; background : white;">
								 
								<span style="font-size : 13px;">
					<b style="color : #363636;">Dodatek do pizzy Mała(1.00zł) Duża(2.00zł),<br />
					Podwójny ser Mała(1.50zł) Duża(3.00zl)
					</b>
					<br />
					<br />
				</span>
				<?php
					$query = "SELECT * FROM dodatki";
					$result = mysql_query($query);
					echo '<table>'; $i=1;
					while ($row = mysql_fetch_array($result)){
					if($i==1) echo '<tr>';
					  echo '<td style="border : 1px solid #363636; color : #363636; font-size : 11px;">
							<input type="checkbox" name="dodatek['.$row['id'].']" value=true>
						  <b>'.$row['nazwa'].'</b>
						   </td>';
					if($i==3) {
					  echo '<tr>';
					$i=0;
					}
					$i++;
					}
					echo '</table>';
				?>
			</td>
		</tr>
		<tr>
			  <td style="width : 400px; height : 250px; text-align : center; border : 1px solid white; background : white; color : #363636;">
				<b style="font-size : 13px;">Uwagi dotyczące wykonania pizzy.</b><br /><br />
				<textarea style="border : 1px solid #363636; width : 390px; height : 200px;" name="uwagi"></textarea>
			</td>
		</tr>
		</tr>
		  </table>
		  <p style="width : 400px; text-align : center;"><input type="submit" value="Dodaj do koszyka"></p>
			</form>
	</div>
</div>
[PHP] pobierz, plaintext 

a oto plik koszyk.pizza.php

[PHP] pobierz, plaintext 
<?php
Treść tego pliku uległa zmianie. na dole podam jak wygląda aktualnie koszyk :)
?>
[PHP] pobierz, plaintext 

Podam również zrzut z bazy danych (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) nie da się tego wszystkiego wyświetlić w jetnym topicu (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) jak ktoś będzie chętny proszę napisać w nastepnej wypowiedzi podam takowy zrzut (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

Prosiłbym was o pomoc w wykonaniu prawidłowgo skryptu nie mam pojęcia jak przekazać poprawnie te dane do koszyk a może bardziej jak je odebrac w pliku koszyk (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) pozdrawiam

Ten post edytował FliSs_tCv 25.09.2007, 12:29:58

[hank]

hej:)

piszesz bardzo nieczytelny kod:) o tej porze już ciężko się domyślać jakie dane pobierasz np tutaj "$wynik = mysql_query("SELECT * FROM $sql_tabela WHERE id='$id'");". Co do twojego pytania, to prześledź dokładnie, co się dzieje z twoją zmienną od momentu przesłania jej do skryptu-koszyka. Błędy 'o jedynkę' zdarzają się bardzo często, zwłaszcza przy operatorach dekrementacji/inkrementacji

Kod

if ($sz=="") $_SESSION["koszyk"]["$id"]["ile"]++;

Być może to ta linia powoduje błąd - wrzuć w kilku miejscach echo, które wypisze ci aktualną wartość zmiennej w trakcie wykonywania się skryptu, wtedy szybko wyłapiesz miejsce, gdzie następuje niepożądana zmiana. Kolejnym punktem, gdzie może się wszystko sypać jest zapis typu

Kod

$_SESSION["koszyk"]["$id"]["ile"]

Jeśli $id jest zmienną, wówczas nie obejmuj jej cudzysłowem. W cudzysłowach umieszczaj tylko stringi (literały). Następną rzeczą, która może ci kiedyś uprzykrzyć życie to nie trzymanie się wybranych przez ciebie standardów. Może pomyślisz że się czepiam, ale tak nie jest:) na podobnych błędach straciłem kiedyś sporo czasu:) piszesz np. <br /> co sugeruje xhtml'a, a wartości atrybutów tagów podajesz bez cudzysłowów, co było dopuszczalne tylko w 'zwykłym' html'u:

Kod

<b>Wybierz ilość</b>: <input (...) name="ilosc" value=1 size=5 maxlength=2>

Wartości liczbowe atrybutów również warto podawać otaczając je cudzysłowem lub apostrofami. Wątpię, aby to było przyczyną twojego problemu, ale to tak na zapas:) Poza tym, zamiast tablicy $_REQUEST polecam ci używanie konkretnie $_GET lub $_POST, gdyż masz wtedy większą kontrolę nad przekazywanymi danymi - wiesz skąd się spodziewasz danych i stamtąd je pobierasz.

Polecam ci jeszcze jedną rzecz. Zamiast grzęznąć w kodzie obok zapisów takich jak

Kod

     if (mysql_connect($sql_serwer, $sql_login, $sql_haslo) && mysql_select_db($sql_baza))
    {
        $wynik = mysql_query("SELECT * FROM $sql_tabela WHERE id='$id'");
        mysql_close();
    }

warto sobie nawiązywanie i kończenie połączenia z bazą wywalić do osobnego pliku, gdzie znajdą się dwie podstawowe funkcje:

Kod

// łączenie z bazą danych
function ConnectDB()
{
     $sql = @mysql_connect('localhost', 'login', 'haslo');
     if (!$sql)
     {
         exit('Wystąpił błąd nr ('.mysql_errno().'): <b>'.mysql_error().'</b>');
     }
     $db = mysql_select_db('caprii') or die("BŁĄD - nieudane wybranie bazy!");
}
// rozłączenie z bazą danych
function DisconnectDB()
{
     @mysql_close();
}

potem sobie ten plik inkludujesz na początku każdego skryptu wymagającego dostępu do bazy, na początku skryptu wywołujesz ConnectDB() a na końcu DisconnectDB(). W środku zostaje ci już tylko miła zabawa z zapytaniami.

To tyle , mam nadzieję, że trochę ci pomogłem swoim gadaniem.

Pozdro:) Aha, pisz więcej komentarzy:)

Ten post edytował hank 25.09.2007, 01:55:36

[FliSs_tCv]

Witaj Hank wielkie dzięki za opr (IMG:http://forum.php.pl/style_emoticons/default/biggrin.gif) nic nie szkodzi (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) skrypt koszyka uległ zmianie. podam go niżej, lecz w nim mam również problem ale myślę że to proscizna (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) ja już nie mam pomysłów (IMG:http://forum.php.pl/style_emoticons/default/sad.gif)

[PHP] pobierz, plaintext 
<?php
session_start();
$sql_serwer = "localhost";
$sql_login  = "";
$sql_haslo  = "";
$sql_baza   = "caprii";
$sql_tabela = "pizza";
mysql_connect($sql_serwer, $sql_login, $sql_haslo);
mysql_select_db($sql_baza);
 
$_SESSION['koszyk']['pizza'][$_POST['pizza']]['nazwa'] = $_POST['pizza'];
$_SESSION['koszyk']['pizza'][$_POST['pizza']]['ilosc'] = $_POST['ilosc'];
$_SESSION['koszyk']['pizza'][$_POST['pizza']]['wielkosc'] = $_POST['wielkosc'];
$_SESSION['koszyk']['pizza'][$_POST['pizza']]['uwagi'] = $_POST['uwagi'];
$_SESSION['koszyk']['pizza'][$_POST['pizza']]['dodatki'] = $_POST['dodatek'];
 
$przetworzona_wielkosc = ($_POST['wielkosc'] == 'm') ? 'mała' : 'duża';
echo '<p>Wybrałeś, '.$przetworzona_wielkosc.' pizzę o nazwie '.$_POST['pizza'].'.</p> Ilość:'.$_POST['ilosc'].' <p>Dodatki od pizzy:</p>';
	 
$array = array_keys($_POST['dodatek']);
$query = "SELECT nazwa FROM dodatki WHERE id=";
foreach ($array as $dodatek)
{
	$query .= $dodatek.' OR id=';
}
$query = substr($query, 0, -7);
$result = mysql_query($query);
while ($row = mysql_fetch_array($result))
{
	echo $row['nazwa'].'<br>';
}
?>
<a href="add.php">Zamów</a>
[PHP] pobierz, plaintext 

Tutaj jest taki problemem że jak nie dodam żadnego dodatku wyświetla mi taki o to błąd patrzałem w manualu ale dla mnie mało jeszcze znającego się na php to nic nie powiedziało ehh. \

BŁĄD = "Warning: array_keys() [function.array-keys]: The first argument should be an array in D:\WebServ\httpd-users\test\koszyk.php on line 22

Warning: Invalid argument supplied for foreach() in D:\WebServ\httpd-users\test\koszyk.php on line 24
ser
podwójny ser
potrójny ser
papryka
szynka"

jak widać pokazuję mi wszystkie dostępne składniki poniżej (IMG:http://forum.php.pl/style_emoticons/default/sad.gif) . Zaznaczę że jak tylko nie dodam tego dodatku tak sie pokazuję a jak dodam chodżby jeden już nie ma tego błędu (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) please help me (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

-----------EDIT -------------------

zauważyłem również że nie da się za jednym zamówieniem zamówić 2 inych pizz kurde a to ważne (IMG:http://forum.php.pl/style_emoticons/default/sad.gif) bez kitu nie wiem już jak to zrobić:(

to jest kod starego koszyka może mi pomożecie z niego jakoś udoskonalić ten wyżej abo na odwrót (IMG:http://forum.php.pl/style_emoticons/default/sad.gif) ?

stary koszyk.php

[PHP] pobierz, plaintext 
<?php
session_start();
?>
<?php
$sql_serwer = "localhost";
$sql_login  = "";
$sql_haslo  = "";
$sql_baza   = "caprii";
$sql_tabela = "pizza";
 
function odbierz($txt)
{
	if (get_magic_quotes_gpc()) $txt = stripslashes($txt);
	return str_replace(array('',"'"), array('',"&#092;'"), trim($txt));
}
 
 
$id = odbierz($_REQUEST["id"]);
$sz = isset($_REQUEST["sz"]) ? number_format($_REQUEST["sz"],0,".","") : "";
 
if ($id<>"")
{
	if (mysql_connect($sql_serwer, $sql_login, $sql_haslo) && mysql_select_db($sql_baza))
	{
		$wynik = mysql_query("SELECT * FROM $sql_tabela WHERE id='$id'");
		mysql_close();
	}
	if ($wynik && mysql_num_rows($wynik)==1)
	{
		$dane = mysql_fetch_array($wynik);
		$_SESSION["koszyk"]["$id"]["nazwa"]=$dane["nazwa"];
		$_SESSION["koszyk"]["$id"]["cena"]=$dane["cena"];
		$_SESSION["koszyk"]["$id"]["wielkosc"]=$_REQUEST["wielkosc"];
		foreach ($_REQUEST['dodatek'] as $dodatek => $true)
		{
			$_SESSION["koszyk"]["$id"]["dodatki"][]=$dodatek;
		}
		$_SESSION["koszyk"]["$id"]["ile"]=$_REQUEST['ilosc'];
		if ($sz=="") $_SESSION["koszyk"]["$id"]["ile"]++;
		else if ($sz>=1) $_SESSION["koszyk"]["$id"]["ile"]=$sz;
		else if ($sz<=0) unset($_SESSION["koszyk"]["$id"]);
	}
}
if (count($_SESSION["koszyk"])>0)
{
	echo "<table border=\"1\">
	<colgroup>
	<col />
	<col align=\"right\" />
	<col align=\"center\" />
	<col align=\"right\" />
	</colgroup>
	<tr><th>towar</th><th>cena</th><th>sztuk</th><th>łącznie</th></tr>";
	foreach($_SESSION["koszyk"] as $indeks=>$kosz)
	{
		echo "<tr>";
		echo "<td>{$kosz["nazwa"]}</td>";
		echo "<td>{$kosz["cena"]} zł</td>";
		echo "<td>
		<form action=\"index.php?page=zam.dania\" method=\"post\" style=\"display:inline;\">
		<div style=\"display:inline;\">
	  <input type=\"hidden\" name=\"id\" value=\"$indeks\" />
	  <input type=\"text\" name=\"sz\" value=\"{$kosz["ile"]}\"
		style=\"width:25px;\" />
	  <input type=\"submit\" value=\"zmień\" style=\"width:45px;\" />
	</div></form>
	<form action=\"index.php?page=zam.dania\" method=\"post\" style=\"display:inline;\">
	  <div style=\"display:inline;\">
	  <input type=\"hidden\" name=\"id\" value=\"$indeks\" />
	  <input type=\"hidden\" name=\"sz\" value=\"0\"
		style=\"width:25px;\" />
	  <input type=\"submit\" value=\"usuń\" style=\"width:45px;\" />
	</div></form>
	</td>";
	echo "<td>".number_format($kosz["cena"]*$kosz["ile"],2,".","");
	echo " zł</td>";
	echo "</tr>";
	$lacznie +=$kosz["cena"]*$kosz["ile"];
  }
  echo "<tr><td colspan=\"3\" align=\"right\">
  w sumie: </td><td>";
  echo number_format($lacznie,2,".","")." zł</td></tr>";
  echo "</table>";
 
 
  echo "<table border=\"1\" style=\"font-size : 12px;\">";
  echo "<tr>
		  Aby zamówienie zostało zrealizowane, należy podać swoje dane osobowe<br /> oraz adres zamieszkania i nr. telefonu.
				Kliknij <a href=\"index.php?page=add\">tutaj</a> aby kontynuować!
	</tr>";
  echo "</table>";
} else echo "<p style=\"width : 730px; text-align : center; margin-top : 100px; margin-bottom : 100px;\">Twój koszyk jest pusty...</p>";
?>
[PHP] pobierz, plaintext 

Ten post edytował FliSs_tCv 25.09.2007, 13:59:35