Pomoc - Szukaj - Użytkownicy - Kalendarz
Pełna wersja: [PHP][MYSQL]Usuwanie rekordu z bazy danych
Forum PHP.pl > Forum > Przedszkole
Nitz
8.06.2008, 14:10:43
Jestem nowicjuszem w PHP oraz MySQL dlatego przepraszam za poziom trudności mojego błędu oraz prosiłbym o wyrozumiałość i pomoc w jego rozwiązaniu.

Wszystko fajnie chodzi, tylko nie usuwa rekordu z bazy danych.

Mógłby ktoś mi wyjaśnić dlaczego?

Z góry dziękuję za odpowiedzi.

[PHP] pobierz, plaintext 
  1. <?php
  2.  
  3. mysql_connect ("localhost","root","haslo");
  4. mysql_select_db (nitz);
  5. $zapytanie = "SELECT * FROM dane ORDER BY id";
  6. $wykonaj = mysql_query ($zapytanie);
  7.  
  8. while($wiersz=mysql_fetch_array ($wykonaj)) {
  9. echo "
  10.  
  11.  
  12. ID:".$wiersz['id']." Imię: ".$wiersz['imie']." Nazwisko:".$wiersz['nazwisko']." <a href=\"index.php?delid=".$wiersz['id']."\">Kasuj</a><br>
  13.  
  14.  
  15.  
  16. ";
  17. }
  18. $zapytanie2 = "DELETE FROM dane WHERE id='$delid'";
  19. $wykonaj2 = mysql_query ($zapytanie2);
  20. mysql_close;
  21. ?>
[PHP] pobierz, plaintext
Puzi
8.06.2008, 14:17:02 
[PHP] pobierz, plaintext 
  1. <?php
  2.  
  3. mysql_connect ("localhost","root","haslo");
  4. mysql_select_db (nitz);
  5. $zapytanie = "SELECT * FROM dane ORDER BY id";
  6. $wykonaj = mysql_query ($zapytanie);
  7.  
  8. while($wiersz=mysql_fetch_array ($wykonaj)) {
  9. echo "
  10.  
  11.  
  12. ID:".$wiersz['id']." Imię: ".$wiersz['imie']." Nazwisko:".$wiersz['nazwisko']." <a href=\"index.php?delid=".$wiersz['id']."\">Kasuj</a><br>";
  13. }
  14. $zapytanie2 = "DELETE FROM `dane` WHERE `id`='".$delid."'";
  15. $wykonaj2 = mysql_query ($zapytanie2);
  16. mysql_close;
  17. ?>
[PHP] pobierz, plaintext



Tak raczej powinno być
Nitz
8.06.2008, 14:21:19
Po wprowadzeniu poprawek (dokładniej zasugerowanej przez Puczi) wywalił błąd:

Cytat
Parse error: syntax error, unexpected T_ENCAPSED_AND_WHITESPACE, expecting T_STRING or T_VARIABLE or T_NUM_STRING in C:\xampp\htdocs\strona\txt\kontakt.txt on line 14
marcio
8.06.2008, 14:24:08 
[PHP] pobierz, plaintext 
  1. <?php
  2. $zapytanie2 = ('DELETE FROM dane WHERE id="'.$delid.'"');
  3. $wykonaj2 = mysql_query($zapytanie2);
  4. mysql_close();
  5. ?>
[PHP] pobierz, plaintext
kallosz
8.06.2008, 14:24:43
zamiast 
[PHP] pobierz, plaintext 
  1. <?php
  2. $wykonaj2 = mysql_query ($zapytanie2);
  3. ?>
[PHP] pobierz, plaintext

zastosuj 
[PHP] pobierz, plaintext 
  1. <?php
  2. mysql_query ($zapytanie2);
  3. ?>
[PHP] pobierz, plaintext
mike
8.06.2008, 14:27:20
~kallosz zmiana, którą proponujesz jest kompletnie bez sensu i nic nie wnosi. Utworzenie zmiennej lub nie nic nie znaczy.
Bład powoduje to że autor zgubił nawiasy i zamiast napisać mysql_close(); napisał mysql_close;
Nitz
8.06.2008, 14:29:43
Dalej nic.


Żadnych błędów, ale rekordu dalej nie usuwa.
mike
8.06.2008, 14:37:55
A co to jest id? Jeśli ciąg znaków (string) to jest dobrze.
Jeśli liczba (najprawdopodobniej) to zapytanie jest złe. Powinno być:
[PHP] pobierz, plaintext 
  1. <?php
  2.  
  3. // ...
  4. mysql_query('DELETE FROM dane WHERE id=' . $delid);
  5. // ...
  6.  
  7. ?>
[PHP] pobierz, plaintext
kallosz
8.06.2008, 14:41:52
to zrob 
[PHP] pobierz, plaintext 
  1. <?php
  2. mysql_query('DELETE FROM dane WHERE id='.$delid.) or die (mysql_error());
  3. ?>
[PHP] pobierz, plaintext
Nitz
8.06.2008, 14:42:23
Niestety dalej nie usuwa rekordów.

Wklejam ponownie kod (z poprawkami):

[PHP] pobierz, plaintext 
  1. <?php
  2.  
  3. mysql_connect ("localhost","root","haslo");
  4. mysql_select_db (nitz);
  5. $zapytanie = "SELECT * FROM dane ORDER BY id";
  6. $wykonaj = mysql_query ($zapytanie);
  7.  
  8. while($wiersz=mysql_fetch_array ($wykonaj)) {
  9. echo "
  10.  
  11.  
  12. ID:".$wiersz['id']." Imię: ".$wiersz['imie']." Nazwisko:".$wiersz['nazwisko']." <a href=\"index.php?delid=".$wiersz['id']."\">Kasuj</a><br>
  13.  
  14.  
  15.  
  16. ";
  17. }
  18. $zapytanie2 = "DELETE FROM dane WHERE id='.$delid'";
  19. mysql_query($zapytanie2);
  20. mysql_close();
  21. ?>
[PHP] pobierz, plaintext
mike
8.06.2008, 14:45:42
Jak nie uwzględniasz w kodzie tego co Ci piszemy to jak ma działać?
Popraw zapytanie tworzone w linii 18 to pogadamy.

A na samym początku skryptu dodatkowo daj:
[PHP] pobierz, plaintext 
  1. <?php
  2.  
  3. error_reporting(E_ALL);
  4. ini_set('display_errors', 1);
  5.  
  6. // ...
  7.  
  8. ?>
[PHP] pobierz, plaintext
marcio
8.06.2008, 14:46:10
Sorki ale co zawiera zmienna $delid??
nithajasz
8.06.2008, 14:54:08 
[PHP] pobierz, plaintext 
  1. <?php
  2. $zapytanie2 = "DELETE FROM dane WHERE id='.$delid'";
  3. ?>
[PHP] pobierz, plaintext


Mój ulubiony bład, zjadłeś kropkę po zmiennej $delid smile.gif

powinno być tak :

[PHP] pobierz, plaintext 
  1. <?php
  2. $zapytanie2 = "DELETE FROM dane WHERE id=".$delid.;
  3. ?>
[PHP] pobierz, plaintext


@mike
Co racja to racja rozpędziłem się smile.gif
Nitz
8.06.2008, 14:55:34
$delid zawiera numer rekordu, który ma zostać usunięty.

"id" to klucz tabeli "dane", zawiera tylko liczby.

Przepraszam za problemy, jestem nowicjuszem w PHP.

KOD:

[PHP] pobierz, plaintext 
  1. <?php
  2.  
  3. error_reporting(E_ALL);
  4. ini_set('display_errors', 1);
  5.  
  6. mysql_connect ("localhost","root","haslo");
  7. mysql_select_db (nitz);
  8. $zapytanie = "SELECT * FROM dane ORDER BY id";
  9. $wykonaj = mysql_query ($zapytanie);
  10.  
  11. while($wiersz=mysql_fetch_array ($wykonaj)) {
  12. echo "
  13.  
  14.  
  15. ID:".$wiersz['id']." Imię: ".$wiersz['imie']." Nazwisko:".$wiersz['nazwisko']." <a href=\"index.php?delid=".$wiersz['id']."\">Kasuj</a><br>
  16.  
  17.  
  18.  
  19. ";
  20. }
  21.  
  22. $zapytanie2 = ("DELETE FROM dane WHERE id='.$delid'") or die (mysql_error());
  23. mysql_query($zapytanie2);
  24. mysql_close();
  25.  
  26.  
  27. ?>
[PHP] pobierz, plaintext


Wywala coś takiego:

Cytat
Notice: Use of undefined constant nitz - assumed 'nitz' in C:\xampp\htdocs\strona\txt\kontakt.txt on line 9
ID:1 Imię: Marek Nazwisko:Lipowski Kasuj
ID:2 Imię: aaa Nazwisko:bbb Kasuj
ID:3 Imię: bbb Nazwisko:bbb Kasuj

Notice: Undefined variable: delid in C:\xampp\htdocs\strona\txt\kontakt.txt on line 24
mike
8.06.2008, 14:56:17
Cytat(nithajasz @ 8.06.2008, 15:54:08 ) *
powinno być tak :
[PHP] pobierz, plaintext 
  1. <?php
  2. $zapytanie2 = "DELETE FROM dane WHERE id='".$delid."'";
  3. ?>
[PHP] pobierz, plaintext
A to mój ulubiony błąd. Traktowanie liczb jako stringi. Zapytanie nic nie znajdzie i nic nie usunie tongue.gif
Nitz
8.06.2008, 14:56:56
Z kropką również to samo.
kallosz
8.06.2008, 14:57:08
a jak pobierasz to delid? moze jakis $_GET by sie przydal?
bo on nie wie co kasowac...

To wygloda teraz tak
USUN Z tabeli GDZIE id=

czyli wszystko jest ok
mike
8.06.2008, 14:57:55
Cytat(Nitz @ 8.06.2008, 15:55:34 ) *
$delid zawiera numer rekordu, który ma zostać usunięty.
Nic nie zawiera. PHP nie zna takiej zmiennej, bo niby skąd ma znać?
Z adresu URL? Z adresu URL robi się tak: $_GET['delid']
Nitz
8.06.2008, 15:08:55
Czyli zamiast

[PHP] pobierz, plaintext 
  1. <?php
  2. $zapytanie2 = ("DELETE FROM dane WHERE id='.$delid.'") or die (mysql_error());
  3. ?>
[PHP] pobierz, plaintext

Powinno być:
[PHP] pobierz, plaintext 
  1. <?php
  2. $zapytanie2 = ("DELETE FROM dane WHERE id='.$_GET['delid'].'") or die (mysql_error());
  3. ?>
[PHP] pobierz, plaintext


Tak?

Jeśli tak, to:

Cytat
Parse error: syntax error, unexpected T_ENCAPSED_AND_WHITESPACE, expecting T_STRING or T_VARIABLE or T_NUM_STRING in C:\xampp\htdocs\strona\txt\kontakt.txt on line 23
kallosz
8.06.2008, 15:12:44 
[SQL] pobierz, plaintext 
  1. WHERE id=".$_GET['delid']
[SQL] pobierz, plaintext
nithajasz
8.06.2008, 15:12:50 
[PHP] pobierz, plaintext 
  1. <?php
  2. $zapytanie2 = ("DELETE FROM dane WHERE id=".$_GET['delid']) or die (mysql_error());
  3. ?>
[PHP] pobierz, plaintext


Pomijam fakt, że powinieneś walidować dane z $_GET bo tak można by było sobie usunąć dowolny rekord smile.gif
mike
8.06.2008, 15:14:38
Nie wiem jak bardzo trzeba być ... żeby dwa razy nie zrozumieć. Pisze trzeci raz.
Wartość do id w zapytaniu w klauzuli WHERE ma być bez apostrofów

[PHP] pobierz, plaintext 
  1. <?php
  2. $zapytanie2 = 'DELETE FROM dane WHERE id=' . $_GET['delid']) or die (mysql_error());
  3. ?>
[PHP] pobierz, plaintext
kallosz
8.06.2008, 15:15:37
Cytat(nithajasz @ 8.06.2008, 16:12:50 ) *
[PHP] pobierz, plaintext 
  1. <?php
  2. $zapytanie2 = (&#092;"DELETE FROM dane WHERE id=\".$_GET['delid']) or die (mysql_error());
  3. ?>
[PHP] pobierz, plaintext


Pomijam fakt, że powinieneś walidować dane z $_GET bo tak można by było sobie usunąć dowolny rekord smile.gif
a juz nie mowie ze wszystko z bazy mozna bylo by usunac smile.gif mySQL injection
Mlodycompany
8.06.2008, 15:37:18 
[PHP] pobierz, plaintext 
  1. <?php
  2. $zapytanie = mysql_query("DELETE FROM `dane` WHERE `id`='".$_GET['delid']."'");
  3. ?>
[PHP] pobierz, plaintext

ja mam cos takiego i zawsze działa
nithajasz
8.06.2008, 15:39:51
Cytat(Mlodycompany @ 8.06.2008, 16:37:18 ) *
[PHP] pobierz, plaintext 
  1. <?php
  2. $zapytanie = mysql_query(&#092;"DELETE FROM `dane` WHERE `id`='\".$_GET['delid'].\"'\");
  3. ?>
[PHP] pobierz, plaintext

ja mam cos takiego i zawsze działa


Działać będzie ale fakt faktem id zapisujesz jako przeważnie int a dodając apostrofy porównujesz id ze stringiem smile.gif

Też właśnie spojrzałem do siebie i takie coś używałem...
Mlodycompany
8.06.2008, 15:46:09
to przeciez mozna zrobic przed zapytaniem 
[PHP] pobierz, plaintext 
  1. <?php
  2. $_GET['delid'] = int($_GET['delid']);
  3. ?>
[PHP] pobierz, plaintext
albo warunek, 
[PHP] pobierz, plaintext 
  1. <?php
  2. if(int($_GET['delid'])){
  3. //zapytanie
  4. }
  5. ?>
[PHP] pobierz, plaintext
To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.
Invision Power Board © 2001-2025 Invision Power Services, Inc.