[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

[-jowice-]

No tak- pewnie nie najlepiej to wygląda. No fajnie by było gdyby wyglądało dobrze, ale cóż- prawda jest taka, że nigdy nie miałam do czynienia z jakimkolwiek programowaniem[jak się chce to znajdzie się jednak coś na usprawiedliwenie], no ale jako, że muszę coś takiego zrobić, więc pomyslałam- zaszaleje i spróbuję, może przy okazji czegoś się nauczę.
No i w sumie tak jest- nawet dzięki Tobie:). Dziękuję za odpowiedź w każdym razie:)
Ahaha- już się nie doczekam aż wyśmieje mnie mój informatyk na uczelniXDXD

[darko]

Spokojnie, nie jest tragicznie, widziałem znacznie gorszy kod pisany przez programistów z dość długim wydawałoby się stażem. Po prostu trzeba kilka rzeczy zmienić i będzie dobrze

[-jowice-]

Dzięki za pokrzepiające słowa:).
Poprawiłam już (chyba!) wszystkie błędy, które wymieniłeś (nie dodawałam zabezpieczeń, bo na poziomie na którym to robię nie jest to najważniejsze:). No i niestety podpowiedź, ze idę w dobrym kierunku okazała się niewystarczająca .
Myślałam, że to wina złego umiejscowienia tej instrukcji:

[PHP] pobierz, plaintext 
if (isset($_POST["action"]) == 'kup' ) {
}
[PHP] pobierz, plaintext 

bo cokolwiek bym nie dała do środka, nie daje to żadnego efektu. Ale nie wiem czy mój tok rozumowania jest dobry, bo próbowałam to dawać w różnych miejscach- bez efektu.
Tak więc gdybym mogła prosić jeszcze o jakieś wskazówki byłabym wdzięczna

[darko]

Tutaj chcesz jak rozumiem sprawdzić dwa warunki, pierwszy czy $_POST['action'] jest w ogóle ustawione oraz czy zawartość $_POST['action'] jest równa wartości 'kup', należałoby połączyć te warunki w taki sposób:

[PHP] pobierz, plaintext 
if (isset($_POST["action"])  && $_POST["action"] == 'kup' ) {
 
}
[PHP] pobierz, plaintext 

Poza tym zawsze w razie niepewności i/lub brak chęci do użycia debuggera możesz zobaczyć, czy i co znajduje się w $_POST['action']:

[PHP] pobierz, plaintext 
var_dump($_POST['action']);
[PHP] pobierz, plaintext