[MySQL][PHP]Usuwanie notatek administracyjnych - metoda $_GET Opcje

[Kerth]

Witam,
zrobiłem system notatek administracyjnych w moim panelu administracyjnym. Z dodawaniem notatek wszystko jest w porządku. Gorzej jest z ich usuwaniem. Mam taki kod:

[PHP] pobierz, plaintext 
$pobierzNotatka = $db->prepare("SELECT `id`, `nick`, `notatka`, `data` FROM `notatki_admin`");
 
if(!$pobierzNotatka->execute()){
	throw new Exception("Błąd zapytania z NOTATKAMI ADMINISTRATORÓW.");
}
 
foreach($pobierzNotatka as $row_not_admin)
{
	echo "</br>Nick: \t";
	echo('<span class = "red">');
	echo ''.$row_not_admin['nick'].'</br>' ;
	echo('</span>');
 
	echo "Data: \t";
	echo('<span class = "red">');
	echo ''.$row_not_admin['data'].'</br>';
	echo('</span>');
 
	echo "Notatka: \t";
	echo('<span class = "red">');
	echo ''.$row_not_admin['notatka'].'</br>';
	echo('</span>');
 
	echo '- - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -<a href="notatki.php?id='.$row_not_admin['id'].'">[X]</a></br>';
	if(isset($_GET["id"]))
	{
		$id = str_replace("\'","\\\'",$_GET["id"]);
		$usunNotatka = $db->prepare("DELETE FROM `notatki_admin` WHERE `id` = :id");
		$usunNotatka->bindValue(":id", $id, PDO::PARAM_INT);
	}
}
[PHP] pobierz, plaintext 

Gdy kliknę na [X] w linku poprawnie pobiera id notatki. Niestety na tym się kończy. Notatka się nie usuwa. Co zrobić? Jakieś pomysły/rady?

[Damonsson]

[PHP] pobierz, plaintext 
	if(isset($_GET["id"]))
	{
		$usunNotatka = $db->prepare("DELETE FROM `notatki_admin` WHERE `id` = :id");
		$usunNotatka->bindValue(":id", $id, PDO::PARAM_INT);
                $usunNotatka->execute();
	}
[PHP] pobierz, plaintext 

Ten post edytował Damonsson 14.07.2014, 14:08:23

[Kerth]

Niestety nie działa. Po próbowałem i mam coś takiego:

[PHP] pobierz, plaintext 
$pobierzNotatka = $db->prepare("SELECT `id`, `nick`, `notatka`, `data` FROM `notatki_admin`");
 
if(!$pobierzNotatka->execute()){
	throw new Exception("Błąd zapytania z NOTATKAMI ADMINISTRATORÓW.");
}
 
foreach($pobierzNotatka as $row_not_admin)
{
	echo "</br>Nick: \t";
	echo('<span class = "red">');
	echo ''.$row_not_admin['nick'].'</br>' ;
	echo('</span>');
 
	echo "Data: \t";
	echo('<span class = "red">');
	echo ''.$row_not_admin['data'].'</br>';
	echo('</span>');
 
	echo "Notatka: \t";
	echo('<span class = "red">');
	echo ''.$row_not_admin['notatka'].'</br>';
	echo('</span>');
 
	echo '- - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -<a href="notatki.php?id='.$row_not_admin['id'].'">[X]</a></br>';
	if(isset($_GET["id"]))
	{
		$usunNotatka = $db->prepare("DELETE FROM `notatki_admin` WHERE `id` = :id");
		$usunNotatka->bindValue(":id", $row_not_admin['id'], PDO::PARAM_INT);
		$usunNotatka->execute();
	}
}
[PHP] pobierz, plaintext 

lecz ten kod usuwa wszystkie notatki. I jakby tego było mało link nie reaguje na 1 kliknięcie zadziała dopiero po drugim(i wszystko usuwa). Nie wiem czemu. Co jest nie tak?

[Damonsson]

Zgubiłem jedną linijkę

Poza tym włącz sobie wyświetlanie błędów, powinno Ci od razu wyskoczyć, że nie masz $id zdefiniowanego.

[PHP] pobierz, plaintext 
if(isset($_GET["id"]))
	{
                $id = $_GET["id"]; // str_replace jest niepotrzebne jak bindujesz dane
		$usunNotatka = $db->prepare("DELETE FROM `notatki_admin` WHERE `id` = :id");
		$usunNotatka->bindValue(":id", $id, PDO::PARAM_INT); // tu też był błąd. Po co to dodałeś? "$row_not_admin['id']" ?
		$usunNotatka->execute();
	}
[PHP] pobierz, plaintext 

Ten post edytował Damonsson 14.07.2014, 14:22:06

[Kerth]

Ale czemu trzeba kliknąć 2 razy na [X] żeby usunęło?

[Turson]

foreach($pobierzNotatka as $row_not_admin)
Co to ma być Gdzie w ogóle podział się jakich fetch()

[Kerth]

Ale jak ja mam użyć do tego fetch()? Przecież zwracam kilka wartości(więc dałem pętlę). Jak mam tego użyć?

[Turson]

W PDO powinno to być tak

[PHP] pobierz, plaintext 
$pobierzNotatka = $db->prepare("SELECT `id`, `nick`, `notatka`, `data` FROM `notatki_admin`");
 
if(!$pobierzNotatka->execute()){
throw new Exception("Błąd zapytania z NOTATKAMI ADMINISTRATORÓW.");
}
 
$rows = $pobierzNotatka->fetchAll(PDO::FETCH_ASSOC);
foreach($rows as ....){
 
}
[PHP] pobierz, plaintext 

[Kerth]

W PDO powinno to być tak

[PHP] pobierz, plaintext 
$pobierzNotatka = $db->prepare("SELECT `id`, `nick`, `notatka`, `data` FROM `notatki_admin`");
 
if(!$pobierzNotatka->execute()){
throw new Exception("Błąd zapytania z NOTATKAMI ADMINISTRATORÓW.");
}
 
$rows = $pobierzNotatka->fetchAll(PDO::FETCH_ASSOC);
foreach($rows as ....){
 
}
[PHP] pobierz, plaintext 

No tak, ale powiedz co to zmienia? Działanie jest takie samo. A pomimo tego nadal trzeba 2 razy kliknąć, żeby usunąć.

[Damonsson]

Teoretycznie nie powinieneś mieć w ogóle tego [X] widocznego, praktycznie masz.

Daj to:

[PHP] pobierz, plaintext 
if(isset($_GET["id"]))
	{
                $id = $_GET["id"]; // str_replace jest niepotrzebne jak bindujesz dane
		$usunNotatka = $db->prepare("DELETE FROM `notatki_admin` WHERE `id` = :id");
		$usunNotatka->bindValue(":id", $id, PDO::PARAM_INT); // tu też był błąd. Po co to dodałeś? "$row_not_admin['id']" ?
		$usunNotatka->execute();
	}
[PHP] pobierz, plaintext 

na początek pliku. Może teraz nie trzeba będzie klikać 2 razy.

[Kerth]

Teoretycznie nie powinieneś mieć w ogóle tego [X] widocznego, praktycznie masz.

Daj to:

[PHP] pobierz, plaintext 
if(isset($_GET["id"]))
	{
                $id = $_GET["id"]; // str_replace jest niepotrzebne jak bindujesz dane
		$usunNotatka = $db->prepare("DELETE FROM `notatki_admin` WHERE `id` = :id");
		$usunNotatka->bindValue(":id", $id, PDO::PARAM_INT); // tu też był błąd. Po co to dodałeś? "$row_not_admin['id']" ?
		$usunNotatka->execute();
	}
[PHP] pobierz, plaintext 

na początek pliku. Może teraz nie trzeba będzie klikać 2 razy.

A to coś z tym [X] jest nie tak? Nadal gdy dam na początek trzeba 2 razy kliknąć. Pierwszy raz, żeby "wczytało link do przeglądarki", drugi raz żeby usunąć. A chodzi mi o to, żeby od razu usuwało.

[Turson]

[PHP] pobierz, plaintext 
	if(isset($_GET["id"]))
{
$id = str_replace("\'","\\\'",$_GET["id"]);
$usunNotatka = $db->prepare("DELETE FROM `notatki_admin` WHERE `id` = :id");
$usunNotatka->bindValue(":id", $id, PDO::PARAM_INT);
}
[PHP] pobierz, plaintext 

nie trzymaj tego w pętli...

[Kerth]

[PHP] pobierz, plaintext 
	if(isset($_GET["id"]))
{
$id = str_replace("\'","\\\'",$_GET["id"]);
$usunNotatka = $db->prepare("DELETE FROM `notatki_admin` WHERE `id` = :id");
$usunNotatka->bindValue(":id", $id, PDO::PARAM_INT);
}
[PHP] pobierz, plaintext 

nie trzymaj tego w pętli...

Nie ma to znaczenia gdzie jest, czy w pętli czy nie. I tak jest tak samo.

[johny_s]

a nie masz czasem tak że najpierw wyświetlasz/pobierasz dane a dopiero potem kasujesz?

[Damonsson]

Skoro kazałem mu dać to na początek pliku i odpisał, że nie ma to znaczenia to widocznie nie.

Ten post edytował Damonsson 14.07.2014, 19:37:38

[Kerth]

a nie masz czasem tak że najpierw wyświetlasz/pobierasz dane a dopiero potem kasujesz?

Tak dokładnie. Najpierw wyświetlam/pobieram dane a dopiero potem kasuję.

[johny_s]

No a sprawdzasz w bazie czy na stronie czy się notatka usuneła?