[PHP][MYSQL]Usuwanie rekordu z bazy danych Opcje

[-Nitz-]

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 
<?php
 
mysql_connect ("localhost","root","haslo");
mysql_select_db (nitz);
$zapytanie = "SELECT * FROM dane ORDER BY id";
$wykonaj = mysql_query ($zapytanie);
 
while($wiersz=mysql_fetch_array ($wykonaj)) {
echo "
 
 
ID:".$wiersz['id']." Imię: ".$wiersz['imie']." Nazwisko:".$wiersz['nazwisko']." <a href=\"index.php?delid=".$wiersz['id']."\">Kasuj</a><br>
 
 
 
";
}
$zapytanie2 = "DELETE FROM dane WHERE id='$delid'";
$wykonaj2 = mysql_query ($zapytanie2);
mysql_close;
?>
[PHP] pobierz, plaintext 

[Puzi]

[PHP] pobierz, plaintext 
<?php
 
mysql_connect ("localhost","root","haslo");
mysql_select_db (nitz);
$zapytanie = "SELECT * FROM dane ORDER BY id";
$wykonaj = mysql_query ($zapytanie);
 
while($wiersz=mysql_fetch_array ($wykonaj)) {
echo "
 
 
ID:".$wiersz['id']." Imię: ".$wiersz['imie']." Nazwisko:".$wiersz['nazwisko']." <a href=\"index.php?delid=".$wiersz['id']."\">Kasuj</a><br>";
}
$zapytanie2 = "DELETE FROM `dane` WHERE `id`='".$delid."'";
$wykonaj2 = mysql_query ($zapytanie2);
mysql_close;
?>
[PHP] pobierz, plaintext 

Tak raczej powinno być

[-Nitz-]

Po wprowadzeniu poprawek (dokładniej zasugerowanej przez Puczi) wywalił błąd:

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]

[PHP] pobierz, plaintext 
<?php
$zapytanie2 = ('DELETE FROM dane WHERE id="'.$delid.'"');
$wykonaj2 = mysql_query($zapytanie2);
mysql_close();
?>
[PHP] pobierz, plaintext 

[kallosz]

zamiast

[PHP] pobierz, plaintext 
<?php
$wykonaj2 = mysql_query ($zapytanie2);
?>
[PHP] pobierz, plaintext 

zastosuj

[PHP] pobierz, plaintext 
<?php
mysql_query ($zapytanie2);
?>
[PHP] pobierz, plaintext 

[mike]

~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-]

Dalej nic.


Żadnych błędów, ale rekordu dalej nie usuwa.

[mike]

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 
<?php
 
// ...
mysql_query('DELETE FROM dane WHERE id=' . $delid);
// ...
 
?>
[PHP] pobierz, plaintext 

[kallosz]

to zrob

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

[-Nitz-]

Niestety dalej nie usuwa rekordów.

Wklejam ponownie kod (z poprawkami):

[PHP] pobierz, plaintext 
<?php
 
mysql_connect ("localhost","root","haslo");
mysql_select_db (nitz);
$zapytanie = "SELECT * FROM dane ORDER BY id";
$wykonaj = mysql_query ($zapytanie);
 
while($wiersz=mysql_fetch_array ($wykonaj)) {
echo "
 
 
ID:".$wiersz['id']." Imię: ".$wiersz['imie']." Nazwisko:".$wiersz['nazwisko']." <a href=\"index.php?delid=".$wiersz['id']."\">Kasuj</a><br>
 
 
 
";
}
$zapytanie2 = "DELETE FROM dane WHERE id='.$delid'";
mysql_query($zapytanie2);
mysql_close();
?>
[PHP] pobierz, plaintext 

[mike]

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 
<?php
 
error_reporting(E_ALL);
ini_set('display_errors', 1);
 
// ...
 
?>
[PHP] pobierz, plaintext 

[marcio]

Sorki ale co zawiera zmienna $delid??

[nithajasz]

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

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

powinno być tak :

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

@mike
Co racja to racja rozpędziłem się

Ten post edytował nithajasz 8.06.2008, 14:59:32

[-Nitz-]

$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 
<?php
 
error_reporting(E_ALL);
ini_set('display_errors', 1);
 
mysql_connect ("localhost","root","haslo");
mysql_select_db (nitz);
$zapytanie = "SELECT * FROM dane ORDER BY id";
$wykonaj = mysql_query ($zapytanie);
 
while($wiersz=mysql_fetch_array ($wykonaj)) {
echo "
 
 
ID:".$wiersz['id']." Imię: ".$wiersz['imie']." Nazwisko:".$wiersz['nazwisko']." <a href=\"index.php?delid=".$wiersz['id']."\">Kasuj</a><br>
 
 
 
";
}
 
$zapytanie2 = ("DELETE FROM dane WHERE id='.$delid'") or die (mysql_error());
mysql_query($zapytanie2);
mysql_close();
 
 
?>
[PHP] pobierz, plaintext 

Wywala coś takiego:

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]

powinno być tak :

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

A to mój ulubiony błąd. Traktowanie liczb jako stringi. Zapytanie nic nie znajdzie i nic nie usunie

[-Nitz-]

Z kropką również to samo.

[kallosz]

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

Ten post edytował kallosz 8.06.2008, 14:58:01

[mike]

$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-]

Czyli zamiast

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

Powinno być:

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

Tak?

Jeśli tak, to:

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]

[SQL] pobierz, plaintext 
WHERE id=".$_GET['delid']
[SQL] pobierz, plaintext 

[nithajasz]

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

Pomijam fakt, że powinieneś walidować dane z $_GET bo tak można by było sobie usunąć dowolny rekord

[mike]

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 
<?php
$zapytanie2 = 'DELETE FROM dane WHERE id=' . $_GET['delid']) or die (mysql_error());
?>
[PHP] pobierz, plaintext 

Ten post edytował mike 8.06.2008, 15:14:46

[kallosz]

[PHP] pobierz, plaintext 
<?php
$zapytanie2 = (&#092;"DELETE FROM dane WHERE id=\".$_GET['delid']) or die (mysql_error());
?>
[PHP] pobierz, plaintext 

Pomijam fakt, że powinieneś walidować dane z $_GET bo tak można by było sobie usunąć dowolny rekord

a juz nie mowie ze wszystko z bazy mozna bylo by usunac mySQL injection

[Mlodycompany]

[PHP] pobierz, plaintext 
<?php
$zapytanie = mysql_query("DELETE FROM `dane` WHERE `id`='".$_GET['delid']."'");
?>
[PHP] pobierz, plaintext 

ja mam cos takiego i zawsze działa

Ten post edytował Mlodycompany 8.06.2008, 15:37:35

[nithajasz]

[PHP] pobierz, plaintext 
<?php
$zapytanie = mysql_query(&#092;"DELETE FROM `dane` WHERE `id`='\".$_GET['delid'].\"'\");
?>
[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

Też właśnie spojrzałem do siebie i takie coś używałem...

[Mlodycompany]

to przeciez mozna zrobic przed zapytaniem

[PHP] pobierz, plaintext 
<?php
$_GET['delid'] = int($_GET['delid']);
?>
[PHP] pobierz, plaintext 

albo warunek,

[PHP] pobierz, plaintext 
<?php
if(int($_GET['delid'])){
//zapytanie
}
?>
[PHP] pobierz, plaintext 

Ten post edytował Mlodycompany 8.06.2008, 15:46:26