[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...

[artur_dziocha]

w checkbox za value podstaw id a nie on
chyli name = 'checkbox[]' value ='".$row['0']."'

proboj wyslwietlic dane po wyslaniu za pomocą var_dump

Pozdrawiam

[ziqzaq]

Nie dam ci gotowca na wszystko, ale podpowiem:
1. Czy kolumna w której trzymasz 'id' ma nazwę/index "'0'" (w tablicy podajesz stringa, jako indeks a nie liczbę)?

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

2.

Dlaczego przy pierwszym wejściu na stronę dostaję błąd "Nie można zapisać wyniku w tabeli lub nie znaleziono żadnych studentów"?

Jak byk widać błąd. Masz kod, szukaj.

[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 

Dodatkowo przy usuwaniu wielu id możesz się zainteresować klauzulą "delete ... where `kolumna` in (id,id,id)" - jedno zapytanie na wszystkie przesłane id - oraz funkcją php o nazwie implode().
Na koniec zaproponuję ci taki oto kod (odwołując się do problemu nr. 1):

[PHP] pobierz, plaintext 
<?php
// ... Dzieki temu dosajemy do skryptu tablice z _id jako wartosc w tablicy_
echo '<td> <input type = "checkbox" name = "delete[]" value = "'.$row[0].'"></td>';
// ...
?>
[PHP] pobierz, plaintext 

Ten post edytował ziqzaq 28.12.2008, 17:07:35

[skowron-line]

Nie dam ci gotowca na wszystko, ale podpowiem:
1. Czy kolumna w której trzymasz 'id' ma nazwę/index "'0'" (w tablicy podajesz stringa, jako indeks a nie liczbę)?

Ale czy napewno
http://pl2.php.net/manual/pl/function.mysql-fetch-row.php

No ale zwracam honor koledze nie uważnie przeczytałem bo użyte jest mysql_fetch_array a tam indexami są nazwy kolumn.

Ten post edytował skowron-line 28.12.2008, 17:32:17

[ergoline]

1. Czy kolumna w której trzymasz 'id' ma nazwę/index "'0'" (w tablicy podajesz stringa, jako indeks a nie liczbę)?

[PHP] pobierz, plaintext 
<?php
// ...
echo ' ';
// ...
?>
[PHP] pobierz, plaintext 

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

2.Jak byk widać błąd. Masz kod, szukaj.

[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 (&#092;"SELECT grupa_id FROM grupa where (kierunek = '$kierunek' and semestr = '$semestr')\")
or die (&#092;"Nie można pobrać ID grupy\");
$tabela = mysql_fetch_array($grupa)
or die (&#092;"Nie można zapisać wyniku w tabeli lub nie znaleziono żadnych studentów\");
// ...
?>
[PHP] pobierz, plaintext 

Wybacz, ale za czorta dalej nie wiem gdzie jest błąd... Może wyłożysz mi go łapatologicznie?

Dodatkowo przy usuwaniu wielu id możesz się zainteresować klauzulą "delete ... where `kolumna` in (id,id,id)" - jedno zapytanie na wszystkie przesłane id - oraz funkcją php o nazwie implode().

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ąć?

1. Jak się odwołać do tej tablicy? Zaznaczę kilka checkbox'ów, a jak się do tego odwołać?

[PHP] pobierz, plaintext 
<?php
$_POST['delete']
?>
[PHP] pobierz, plaintext 

?
Rozumiem, że do tablicy zostaną od razu przesłane ID wszystkich zaznaczonych checkbox'ów?

2. Coś w stylu:

[PHP] pobierz, plaintext 
<?php
//...
$tablica = $_POST['delete'];
$string = implode (",", $tablica);
$usun = mysql_query ("DELETE FROM osoba WHERE id IN ($string)");
//...
?>
[PHP] pobierz, plaintext 

?

Proszę Panstwa, to moja pierwsza styczność z PHP i MySQL, dlatego proszę o wyrozumiałość i cierpliwość...
Pozdrawiam...

Ten post edytował ergoline 29.12.2008, 02:37:24

[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?
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

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

[ergoline]

Witam!

Poprawiłem lekko kod, a przynajmniej tak mi się wydaje. Teraz ma postać:

[PHP] pobierz, plaintext 
<?php
 
require ("check.php");
 
$polaczenie = mysql_connect ("localhost", "trautbergp", "123456")
or die ("Nie można połączyć z serwerem baz danych");
$wybor_bazy = mysql_select_db ("db_trautbergp")
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" name = "znajdz" value = "Znajdź">
</form>';
 
echo '<form action = "usun.php" method = "post">';
 
if ($_POST['znajdz'] == "Znajdź")
{ 
    $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>';
        $klucz = 'mojklucz'.$row[0];
        echo '<td> <input type = "checkbox" name = "delete['.$klucz.']" value = " '.$row[0].' "></td>';
        echo '</tr>';
    }
    echo '</table>';
        echo '</br>';
        echo '<input type = "submit" name = "usun" value = "Usuń zaznaczone">';
        echo '</form>';
    
    if ($_POST['usun'] == "Usuń zaznaczone")
    {
        $tablica = $_POST['delete']
        or die ("Nie można utworzyć tablicy");
        $string = implode (",", $tablica)
        or die ("Nie można utowrzyć $string");
        $usun = mysql_query ("DELETE FROM osoba WHERE id IN ($string)")
        or die ("Nie można usunąć rekordów");
    }
}
 
?>
[PHP] pobierz, plaintext 

Jednak ciągle nie usuwa zaznaczonych rekordów. Co znowu robię źle?

Pozdrawiam...

[ziqzaq]

OMG człowieku, proszę cię, zacznij czytać ze zrozumieniem a nie przeklejać kod bezmyślnie.

na końcu mojego pierwszego postu masz poprawny kod

Patrzyłeś co wklejasz, i co to robi czy poprostu walnąłeś to do skryptu i tyle. Nadal tworzysz tablicę asocjacyjną.
Te dwie struktury które ci pokazałem to były przykłady żebyś sobie przyswoił/zrozumiał o co chodzi.
Pisałem:

Ad. 2) Tak. O coś takiego mi chodziło. Przetestuj sobie najpierw (echo $zapytanie) czy dostajesz dobrą listę elementów zanim zaczniesz robić zapytania.

po to żebyś widział to swoje zapytanie (u ciebie $zapytanie to zmienna $string + sql, oczywiście), daje głowę że tego nie zrobiłeś. Masz przykład:

[PHP] pobierz, plaintext 
<?php
echo "DELETE FROM osoba WHERE id IN ($string)";
?>
[PHP] pobierz, plaintext 

Chcesz wiedzieć co dostajesz z checkboxów? Ten kod wyświetli ci to co dostajesz z formularza (całą tablicę $_POST['delete']):

[PHP] pobierz, plaintext 
<?php
if (!empty($_POST['delete']) {
    print_r($_POST['delete']);
}
?>
[PHP] pobierz, plaintext 

Zacznij myśleć co robisz i analizować (co skąd się bierze) swój kod bo czuję że wszystko co piszę idzie na marne
Natomiast jeśli chodzi o kod to widzę poprawę, ale musisz jeszcze popracować/przeanalizować jakieś przykłady jak postępować z formularzami. Masz na forum kursy więc skorzystaj z tego.
Przeczytaj może jakieś tutoriale do php, ściągnij jakiś darmowy skrypt, odpal go i przeanalizuj.

Nie obrażaj się, ale musisz duużo ćwiczyć a jak sam tego nie zrobisz to się nie nauczysz.
Powodzenia.

PS Nadal widzę kwiatki typu:

[PHP] pobierz, plaintext 
<?php
$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");
?>
[PHP] pobierz, plaintext 

Zero sprawdzania zmiennych. A zwracałem ci na to uwagę.
Jeszcze to. Już jest dobrze ale po co wyświetlać pusty formularz? Wrzuć to echo do if'a (oczywiście zamknięcie formularza też):

[PHP] pobierz, plaintext 
<?php
echo '<form action = "usun.php" method = "post">';
 
if ($_POST['znajdz'] == "Znajdź")
?>
[PHP] pobierz, plaintext 

Acha i daję głowę że nie zmieniłeś error_reporting = E_ALL - wierz mi, ułatwiłoby ci to życie

I jeszcze jedno. Zdajesz sobie sprawę z tego że to zapytanie wykonuje się za każdym wywołaniem skryptu?

[PHP] pobierz, plaintext 
<?php
$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");
?>
[PHP] pobierz, plaintext 

Czyli za każdym wywołaniem próbujesz usunąć studenta... przynajmniej na to wygląda.

Ten post edytował ziqzaq 29.12.2008, 17:16:37

[ergoline]

Znowu coś tam próbowałem poprawić:

[PHP] pobierz, plaintext 
<?php
 
require ("check.php");
 
$polaczenie = mysql_connect ("localhost", "trautbergp", "123456")
or die ("Nie można połączyć z serwerem baz danych");
$wybor_bazy = mysql_select_db ("db_trautbergp")
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" name = "skresl" value = "Skreśl z listy studentów">
    <input type = "reset" value = "Wyczyść forumlarz">
</form>';
 
$imie = $_POST['imie'];
$nazwisko = $_POST['nazwisko'];
 
if ($POST['skresl'] == "Skreśl z listy studentów")
{
    $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" name = "znajdz" value = "Znajdź">
</form>';
 
if ($_POST['znajdz'] == "Znajdź")
{ 
    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[]" value = " '.$row[0].' "></td>';
        echo '</tr>';
    }
    echo '</table>';
        echo '</br>';
        echo '<input type = "submit" name = "usun" value = "Usuń zaznaczone">';
        echo '</form>';
    
    if ($_POST['usun'] == "Usuń zaznaczone")
    {
        if (!empty($_POST['delete'])) {print_r ($tablica);}
        //or die ("Nie można utworzyć tablicy");
        //$string = implode (",", $tablica)
        //or die ("Nie można utworzyć $string");
        //$usun = mysql_query ("DELETE FROM osoba WHERE id IN ($string)")
        //or die ("Nie można usunąć rekordów");
        
    }
}
 
?>
[PHP] pobierz, plaintext 

1. Na razie chcę zobaczyć, czy w ogóle checkbox'y "zaznaczają się"... Jednak nie widzę, żadnego efektu. To samo jeśli wyciągnę

[PHP] pobierz, plaintext 
<?php
if (!empty($_POST['delete'])) {print_r ($tablica);}
?>
[PHP] pobierz, plaintext 

przed warunek

[PHP] pobierz, plaintext 
<?php
if ($_POST['usun'] == "Usuń zaznaczone")
?>
[PHP] pobierz, plaintext 

2. Ciągle nie umiem zrozumieć o co dokładnie chodzi z testowaniem zapytania. Mógłbyś jeszcze raz, powoli, spokojnie i dużymi literami to powiedzieć?

3. Co do raportowania błędów, to nie pcham się do tego - nie mój serwer...

Pozdrawiam...

[ziqzaq]

Dobijesz mnie
ad. 1. Wrzuć albo przed albo w tym waruku. Dzięki temu zobaczysz jakie checkboxy idą do skryptu (jeśli jakiekolwiek idą).
ad. 2. Nawet nie testowanie. Poprostu zobacz je sobie i sprawdź czy jest poprawne (takie jak zakładałeś).
ad. 3. Ok. Myślałem, że testujesz na localhost.

Wkurzyłem się i nabazgrałem ten kod - porównaj sobie ze swoim.
Pisane z palca na szybkiego, nie sprawdzone (to znaczy że mogą być jakieś literówki i błędy, chociaż mam nadzieję że nie walnąłem jakiejś gafy ):

[PHP] pobierz, plaintext 
<?php
// UWAGA, sprawdzam czy mamy wszystkie potrzebne dane zanim pomysle o tworzeniu formularza
// Poniewaz $_POSR['znajdz'] jest tylko w jednym formularzu 
// to tylko sprawdzam czy zostalo przeslane
// (jesli go nie ma to znaczy ze to nie ta akcja ma byc wykonana)
if (!empty($_POST['znajdz']) AND !empty($_POST['kierunek']) AND !empty($_POST['semestr']))
{ 
    // Teraz mozna przypisac kierunek i semestr
    $kierunek = mysql_real_escape_string((string) $_POST['kierunek']); // Chcemy zeby to byly ciagi znakow
    $semestr = mysql_real_escape_string((string) $_POST['semestr']);
    
    // Oto zapytanie
    $sql = "SELECT grupa_id FROM grupa where (kierunek='$kierunek' and semestr='$semestr')";
    $grupa = mysql_query($sql);
    if($grupa) {
        // Ok jest jakas grupa
        // UWAGA: Pomysl co bedze gdy zaptanie wyzej pobierze wiecej grup
        $grupa = mysql_fetch_array($grupa);
        $sql2 = "SELECT * FROM osoba where (grupa_id=".$tabela['grupa_id'].")";
        $studenci  = mysql_query($sql2);
        if($studenci) {
            // Zapytanie powiodlo sie mamy studentow, tworze formularz
            echo '<form action="usun.php" method="post">';
            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($studenci))
            {
                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[]" value="'.$row[0].'" /></td>';
                echo '</tr>';
            }
            echo '</table>';
            echo '</br>';
            echo '<input type="submit" name="usun" value="Usuń zaznaczone">';
            echo '</form>';
        }
        else echo "Wystąpił błąd podczas pobierania studentów z bazy<br/>".mysql_error();
   }
   else echo "Wystąpił błąd podczas pobierania grupy z bazy<br/>".mysql_error();
}
// Zauwaz ze od razu sprawdzam czy przeslano jakies checkboxy i czy to jest tablica
if ($_POST['usun'] == "Usuń zaznaczone" AND !empty($_POST['delete']) AND is_array($_POST['delete']))
{
    $tablica = $_POST['delete'];
    // Zobaczmy co tam mamy w tablicy
    print_r($tablica);
    // Tworze ciag znakow z id (wartosciami z checkboxow)
    // Mozna jeszcze sprawdzic czy w tablicy mamy tylko liczby a nie jakis kod sql injection
    // ja zrobie tylko escape string
    $numery_id = mysql_real_escape_string(implode(",", $tablica));
    $sql = "DELETE FROM osoba WHERE id IN ($numery_id)";
    // Zobaczmy jak wyglada nasze zapytanie
    echo "SQL: ".$sql;
    
    // Twoje zapytanie
    //$usun = mysql_query ("DELETE FROM osoba WHERE id IN ($string)")
    //or die ("Nie można usunąć rekordów");
    
    // Teraz moje, mozesz odznaczyc pozniej dla sprawdzenia
    /*
    if(mysql_query($sql)) {
        echo 'Usunięto z bazy wybrane rekordy';
    }
    else echo "Wystąpił błąd podczas usuwania rekordów z bazy<br/>".mysql_error();
    */
}
?>
[PHP] pobierz, plaintext 

Ten post edytował ziqzaq 29.12.2008, 19:36:46