[MySQL][PHP]Wybór produktu <przycisk kup> Opcje

[-jowice-]

Zwracam się do was kolejny raz o poradę (jakąś małą podpowiedź choćby czy idę dobrym tropem). Otóż tworzę formularz, który po przyciśnięciu przycisku kup wysyła do bazy danych informacje o tym jaki produkt został kupiony i jaka jego ilość. Problem w tym, że produkty są pobierane z bazy danych przy pomocy pętli i nie za bardzo wiem, jak to zrobić żeby przy naciśnięciu przycisku kup wiadomo było który konkretnie produkt został kupiony.
Cóż.. jak na razie po naciśnięciu przycisku kup nie dzieje się nic, wiem, że to niezbyt dobrze świadczy o tym skrypcie, no ale [w sumie nie mam nic na swoje usprawiedliwienie].

[PHP] pobierz, plaintext 
<?php
  function form ($ilosc = ""){
?>
 
<form action="produkty.php" method="POST">
<div>
Ilość sztuk: <input name="ilosc" value="<?= $ilosc; ?>"/><br />
<input type="hidden" name="id" value="<?php echo $dane['P_ID'] ?>">
<input type="submit" name="submit" value="Kup" />
</div>
</form>
 
<?php
}
 $link = mysql_connect('','','');
 
  if(!$link)
    {
       die('Could not connect: ' .mysql_error());
    }
  else
  {
        mysql_select_db('nsuder');
 
        $result = mysql_query("SELECT * FROM PRODUKT");
        if (!$result) {
                echo 'invalid query';
                echo mysql_error();
        }
 
        while($dane=mysql_fetch_array($result)){
 
        echo "<h3>$dane[P_NAZWA]</h3>";
        echo "<p>$dane[P_OPIS]";
        echo "<br /><i>$dane[P_CENA]</i></p>";
        echo "$dane[P_OBRAZEK]";
        echo "<br /><br /><br /><br /><br />";
 
        if ($dane['P_ILOSC'] > 0){
             form();
        }
        else{
             echo 'Produkt chwilowo niedostępny:(';
             echo "<br/>";
        }
        echo ".........................................";
 
         if (isset($_GET["action"]) == 'kup' ) {
             $zapytanie = mysql_fetch_array(mysql_query("SELECT * FROM PRODUKTY WHERE P_ID = '$_POST['id']'"));
             mysql_query("INSERT INTO LINK (L_P_ID, L_ILOSC) VALUES($zapytanie[P_ID], $_POST['ilosc'])")
             or die("Nie można zakupić");
 
			 header('Location: formularz.php');
			 }
 
         }
 
  }
 
mysql_close($link);
?>
[PHP] pobierz, plaintext 

[darko]

Cóż.. jak na razie po naciśnięciu przycisku kup nie dzieje się nic, wiem, że to niezbyt dobrze świadczy o tym skrypcie, no ale [w sumie nie mam nic na swoje usprawiedliwienie].

Haha. To, co tutaj widzę to jakaś masakra. Błędy:
- mieszanie POST z GET (albo jedno albo drugie) jeśli wysyłasz formularz postem to do przesłanych danych dobierasz się tylko i wyłącznie za pomocą albo $_POST albo $_REQUEST
- przemieszanie widoku z logiką funkcji
- brak jakichkolwiek zabezpieczeń
- powinno być: $zapytanie = mysql_fetch_array(mysql_query("SELECT * FROM PRODUKTY WHERE P_ID = " . $_POST['id']));
- mieszanie tagów <?php z tagami krótkimi <?=, w ogóle używanie tagów <?=

Ten post edytował darko 1.05.2011, 12:18:13