[PHP][MYSQL] Usuwanie kilku rekordów, czyli problem z checkbox'ami Opcje

[ergoline]

Witam!

Chciałbym, mieć możliwośc usuwania kilku rekordów "za jednym zamachem" przy pomocy pól checkbox. Ogónie strona składa się z dwóch części:
1. usuwanie pojedynczego rekorku
2. usuwanie kilku rekordów

Pierwsza część działa dobrze, natomiast z drugą mam problem. Najpierw użytkownik wyszukuje rekordy (wybiera kierunek i semestr -> wyszukiwany jest numer grupy -> nr grupy zapisywany jest do tablicy -> pobrany z tablicy nr grupy jest wykorzystywany do wyszukiwania rekordów), wyświetla się tableka z wynikiem szukania, gdzie przy każdym rekordzie jest checkbox. Użytkownik zaznacza wybrane, klika "Usuń wszystkie" i ... nie działa

[PHP] pobierz, plaintext 
<?php
 
require ("check.php");
 
$polaczenie = mysql_connect ("localhost", "tergoline", "123456")
or die ("Nie można połączyć z serwerem baz danych");
$wybor_bazy = mysql_select_db ("db_ergoline")
or die ("Nie można wybrać wymaganej bazy danych"); 
 
echo '1. Proszę podać imię i nazwisko studenta:
</br>
<form action = "usun.php" method = "post">
    <table>
        <tr>
            <td>Imię:</td>
            <td><input type = "text" name = "imie"></td>
        </tr>
        <tr>
            <td>Nazwisko:</td>
            <td><input type = "text" name = "nazwisko"></td>
        </tr>
    </table>
    </br>
    <input type = "submit" value = "Skreśl z listy studentów">
    <input type = "reset" value = "Wyczyść forumlarz">
</form>';
 
$imie = $_POST['imie'];
$nazwisko = $_POST['nazwisko'];
 
$usun = mysql_query ("DELETE FROM osoba WHERE (imie = '$imie' and nazwisko = '$nazwisko')")
or die ("Bład przy usuwaniu rekordu");
 
 
echo ' 2. Znajdź studenta:';
echo '</br>';
 
echo '<form action = "usun.php" method = "post">';
 
$sql = mysql_query ("SELECT nazwa FROM kierunek")
or die ("Bład przy pobieraniu kierunków");
echo 'Kierunek: <select name = "kierunek">';
while ($row = mysql_fetch_assoc($sql))
{
    echo '<option value = " '.$row['nazwa'].' "> '.$row['nazwa'].' </option>';
}
echo '</select>';
echo '</br>';
 
$sql1 = mysql_query ("SELECT ktory FROM semestr")
or die ("Błąd przy pobieraniu semestrów");
echo ' Semestr: <select name = "semestr">';
while ($row1 = mysql_fetch_assoc($sql1))
{
    echo '<option value = " '.$row1['ktory'].' "> '.$row1['ktory'].' </option>';
}
echo '</select>';
echo '</br>';
 
$kierunek = $_POST['kierunek'];
$semestr = $_POST['semestr'];
 
echo '
</br>
<input type = "submit" value = "Znajdź">
</form>';
 
echo '<form action = "usun.php" method = "post">';
$grupa = mysql_query ("SELECT grupa_id FROM grupa where (kierunek = '$kierunek' and semestr = '$semestr')")
or die ("Nie można pobrać ID grupy");
$tabela = mysql_fetch_array($grupa)
or die ("Nie można zapisać wyniku w tabeli lub nie znaleziono żadnych studentów");
$znajdz  = mysql_query ("SELECT * FROM osoba where (grupa_id = ".$tabela['grupa_id'].")")
or die ("Nie można znaleźć studentów o podanym ID grupy");
echo '<table cellpadding = \"11\" border = 3>';
echo '     <tr>
        <td>ID</td>
        <td>Grupa ID</td>
        <td>Imię</td>
        <td>Nazwisko</td>
        <td>Usuń</td>
    </tr>';
    
while ($row = mysql_fetch_array($znajdz))
{
    echo '<tr>';    
    echo '<td>'.$row[0].'</td>';
    echo '<td>'.$row[1].'</td>';
    echo '<td>'.$row[2].'</td>';
    echo '<td>'.$row[3].'</td>';
    echo '<td> <input type = "checkbox" name = "delete['.$row['0'].']" value = "on"></td>';
    echo '</tr>';
}
echo '</table>';
echo '</br>';
echo '<input type = "submit" name = "usun" value = "Usuń zaznaczone">';
echo '</form>';
 
if ($_POST['usun'] == 'Usuń zaznaczone' and isset($_POST['delete']) and is_array($_POST['delete']))
{
    foreach ($_POST['delete'] as $key=>$value)
    {
        $q = "DELETE FROM osoba WHERE id = '{$key}';";
        mysql_query($q);
    }
}        
?>
[PHP] pobierz, plaintext 

Gdzie popełniam błąd?

Pozdrawiam...

P.S.
Dlaczego przy pierwszym wejściu na stronę dostaję błąd "Nie można zapisać wyniku w tabeli lub nie znaleziono żadnych studentów"?
Taki sam błąd otrzymuję, gdy chcę usunąć kilka rekordów...

[ziqzaq]

Tak, kolumna to osoba_id, jest pierwszą kolumną w tabeli.

No to masz odpowiedź w moim pierwszym poście. Powtórzę na wszelki wypadek tworzysz tablicę w której kluczami są liczby (mysql_fetch_array()) a odwołujesz się do ciągu znaków ('0') dlatego to nie będzie działać (na końcu mojego pierwszego postu masz poprawny kod):

[PHP] pobierz, plaintext 
<?php
echo '<td> <input type = "checkbox" name = "delete['.$row['0'].']" value = "on"></td>';
?>
[PHP] pobierz, plaintext 

Teraz ten kod:

[PHP] pobierz, plaintext 
<?php
// ...
$kierunek = $_POST['kierunek'];
$semestr = $_POST['semestr'];
// ... Jaki tu jest blad (polacz kod powyzej z ponizszym)? I nie chodzi tu o SQL.
$grupa = mysql_query ("SELECT grupa_id FROM grupa where (kierunek = '$kierunek' and semestr = '$semestr')")
or die ("Nie można pobrać ID grupy");
$tabela = mysql_fetch_array($grupa)
or die ("Nie można zapisać wyniku w tabeli lub nie znaleziono żadnych studentów");
// ...
?>
[PHP] pobierz, plaintext 

Podstawowy banalny błąd. Uruchamiasz skrypt za pierwszym razem to przecież zmienne $_POST['kierunek'] i $_POST['semestr'] nie są ustawione bo jeszcze nie wysłałeś formularza. Jak zapytanie ma zadziałać skoro budujesz je na tych właśnie zmiennych? Na końcu przy usuwaniu rekordów najpierw sprawdzasz czy zmienne istnieją - i dobrze. Tutaj zastosuj podobne podejście.
Ustaw w php.ini error_repoting = E_ALL, będziesz widział wszystkie błędy i ostrzeżenia parsera (a ten kod który napisałeś da właśnie ostrzeżenie). Położenie php.ini szukaj na forum, lub google.

Czyli rozumiem, że najpierw zaznaczone checkbox'y mają zostać przesłane do tablicy, później mam stworzyć string z tablicy, w którym będą wymienione id (nazwy checkbox'ów) po przecinku, a później usunąć?

Tak, ja bym starał się tak to zrobić żeby ograniczyć niepotrzebne zapytania, ale jeśli na razie nie chcesz kobinować to zostań przy starym sposobie.
I zapamiętaj tą strukturę. Ten kod prześle (po zatwierdzeniu formularza) do skryptu tablicę numeryczną (klucze są liczbami od 0 wzyż):

[PHP] pobierz, plaintext 
<?php
echo '<td> <input type = "checkbox" name = "delete[]" value = "'.$row[0].'"></td>';
?>
[PHP] pobierz, plaintext 

natomiast to stworzy tablicę asocjacyjną w której klucze bedą nazwami:

[PHP] pobierz, plaintext 
<?php
$klucz = 'mojklucz'.$row[0]; // tutaj tworze ciag znakow 'mojklucz0', 'mojklucz1' itd.
echo '<td> <input type = "checkbox" name = "delete['.$klucz.']" value = "'.$row[0].'"></td>';
?>
[PHP] pobierz, plaintext 

Ad. 1) Tak. Odwołujesz się do atrybutu "name" elementu formularza i w tym przypadku dostajesz tablicę w której klucze są liczbami (od 0 wzwyż) a elementy wartościami wpisanymi w value="wartość".
Ad. 2) Tak. O coś takiego mi chodziło. Przetestuj sobie najpierw (echo $zapytanie) czy dostajesz dobrą listę elementów zanim zaczniesz robić zapytania.

Ok. To chyba już wszystko.

PS Nie przejmuj się tak, chyba chcesz żeby się czegoś nauczyć, nie? (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)
I pamiętaj o szukaniu (google, forum, manual php, manual mysql), naprawdę dużo samemu mozna znaleźć.
No, a teraz kombinuj żeby to działało (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

Ten post edytował ziqzaq 29.12.2008, 09:18:14