[php+mysql] kasowanie wszystkich rekordów mysql Opcje

[klapaucius]

Witam ponownie. Na mojej stronei stworzylem system prywatnych wiadomosci. Wszystko by bylo ok, gdyby nie to, ze dalem opcje przy kazdej wiadomosci "usun wiadomosc". I jesli na to klikne to usuwa mi wszystkie wiadomosci, ktore sie otrzymalo. To samo się dzieje jesli dam opcje "przeczytana", to wszystkim wiadomosciom daje wartosc "przeczytana". Niem ma bladego pojecia co tam moze byc zle. Tutaj daje kod:

[PHP] pobierz, plaintext 
<?php
include("baza.php");
include("gora.php");
$przeczytana = 1;
$q = mysql_query("SELECT login, od, temat, tresc, przeczytana, date, id FROM wiadomosci WHERE login='$user'");
?>
<b><center>Wiadomości:</center></b>
<br>
<a href="wiadomosci.php">:: Nowa wiadomość</a><br>
:: Kasuj wszystkie wiadomości<br><hr>
 
<?php
while($r = mysql_fetch_array($q)) {
?>
<table width=100%>
<tr bgcolor=#e6e6e6>
<td><?php echo $r[2] ?></td>
<td><?php echo $r[5] ?></td>
<td><?php echo "<a href=\"profil.php?user=" . $r[1] . "\">"  . $r[1] . "</a>" ?></td>
</tr>
</table>
<table width=100%>
<tr bgcolor=#6e6e6e>
<td>
<?php echo $r[3] ?><br><br>
</td>
</tr>
<tr bgcolor=#e6e6e6>
 
<form action="wiadomosci2.php" method="post">
       <td><button style="border: 0px; background-color: #e6e6e6;" type="submit" name="przeczytana">przeczytana</button><button style="border: 0px; background-color: #e6e6e6;" type="submit" name="kasuj">kasuj</button></td>
</form>
</tr>
</table>
<hr>
<?php
if(isSet($_POST['kasuj'])){ #//tutaj jest ta nieszczesliwa funkja "kasuj" ///
          mysql_fetch_array(mysql_query("DELETE FROM `wiadomosci` WHERE `id`='".$r[6]."'") or die (mysql_error()));
          header("Location: wiadomosci2.php");
}
if(isSet($_POST['przeczytana'])){
          mysql_fetch_array(mysql_query("UPDATE wiadomosci SET przeczytana='".$przeczytana."' WHERE id='".$r[6]."'") or die (mysql_error()));
}
}
include("dol.php");
?>
[PHP] pobierz, plaintext 

Bardzo prosze o pomoc! Pozdrawiam!

[TrevorGryffits]

Z tego co widzę to masz na dużą pętlę. Mianowicie sięga ona aż za ify, co powoduje, że próbują być wykonane dla każdej wiadomości. A resztę możesz juz sobie sam dopowiedzieć;) Musisz pętlę zakończyć przed ifami. Zostanie ci wtedy problem skąd brać właściwy id postu... Ale to już temat na inne opowiadanie;)

[kfc4]

[PHP] pobierz, plaintext 
<?php
// Pobierz wiadomości
while (...) {
  //Wyświetl
}
// I dopiero tu:
if ($_POST['usun']) {
  // Usuń $_POST['co']/$_GET['co']
}
// Przeczytana tak samo.
?>
[PHP] pobierz, plaintext 

PS. Musisz się nauczyć robić dobre wcięcia. Tzn. w pętli wcinaj tak samo jak ify wcinasz.

Ten post edytował kfc4 26.07.2009, 19:43:56

[klapaucius]

no wlasnie, nawet jesli zrobie costakiego :

[PHP] pobierz, plaintext 
<?php
include("baza.php");
include("gora.php");
$przeczytana = 1;
$q = mysql_query("SELECT login, od, temat, tresc, przeczytana, date, id FROM wiadomosci WHERE login='$user'");
?>
<b><center>Wiadomości:</center></b>
<br>
<a href="wiadomosci.php">:: Nowa wiadomość</a><br>
:: Kasuj wszystkie wiadomości<br><hr>
 
<?php
while($r = mysql_fetch_array($q)) {
?>
<table width=100%>
<tr bgcolor=#e6e6e6>
<td><?php echo $r[2] ?></td>
<td><?php echo $r[5] ?></td>
<td><?php echo "<a href=\"profil.php?user=" . $r[1] . "\">"  . $r[1] . "</a>" ?></td>
</tr>
</table>
<table width=100%>
<tr bgcolor=#6e6e6e>
<td>
<?php echo $r[3] ?><br><br>
</td>
</tr>
<tr bgcolor=#e6e6e6>
 
<form action="wiadomosci2.php" method="post">
       <td><button style="border: 0px; background-color: #e6e6e6;" type="submit" name="przeczytana">przeczytana</button><button style="border: 0px; background-color: #e6e6e6;" type="submit" name="kasuj">kasuj</button></td>
</form>
</tr>
</table>
<hr>
<?php
}
if(isSet($_POST['kasuj'])){
          mysql_fetch_array(mysql_query("DELETE FROM `wiadomosci` WHERE `id`='".$r[6]."'") or die (mysql_error()));
          header("Location: wiadomosci2.php");
}
if(isSet($_POST['przeczytana'])){
          mysql_fetch_array(mysql_query("UPDATE wiadomosci SET przeczytana='".$przeczytana."' WHERE id='".$r[6]."'") or die (mysql_error()));
}
include("dol.php");
?>
[PHP] pobierz, plaintext 

to i tak nic nie usunie bo faktycznie nie wie skad brac id wiadomosci... Nie mam zielonego pojecia jak to zrobic, siedze nad tym prae godzin. Prosze pomozcie

[dotangelo]

Sproboj uruchomic to zapytanie poprzez PhpMyAdmin i zobacz czy stanie się to samo.
Pamiętaj aby do bazy dodawać dwie kopie wiadomości, tak, jeśli jeden user usunie otrzymaną wiadomość, to drugiemu nie usunie się wysłana

[klapaucius]

w phpmyadmin dalem takie zapytanie:

[SQL] pobierz, plaintext 
DELETE FROM `wiadomosci` WHERE `id`='21'
[SQL] pobierz, plaintext 

Nie wiem czy chodzilo o to? W kazdym razie poprawnie usunelo wiadomosc o tym id. A co do tego, zeby robic 2 kopie wiadomosci to faktycznie nie pomyslalem Dzieki za pomysl.
Jednak dalej prosze o pomoc z moim problemem. pozdro

[kfc4]

Nie wiesz jak przesłać, którą wiadomość ma usunąć? Masz poważne braki widzę, ale już wyjaśniam:
- Podczas wyświetlania, dodaj sobie gdzieś pod wiadomością taki link:

[PHP] pobierz, plaintext 
<?php
// coś tam
while (...) {
  // Wyświetl tą wiadomość
  echo '<a href="skrypt.php?usun='.$r[6].'">Usuń</a>'; // Gdzie skrypt.php jest tym Twoim skryptem.
  // I coś tam dalej
}
?>
[PHP] pobierz, plaintext 

Dalej już poza pętlą dajesz(lub przed pętlą, tak będzie nawet lepiej):

[PHP] pobierz, plaintext 
<?php
if (isset($_GET['usun'] && is_numeric($_GET['usun']) {
  zapytanie("delete from wiadomosci where id =".$_GET['usun']); // Wykonujesz zapytanie usuwające wiadomość o ID = $_GET['usun']
}
?>
[PHP] pobierz, plaintext 

Oczywiście jeżeli przesyłasz już coś zmienną GET to musisz napisać nie ?usun= tylko &usun=.
Zrozumiałeś? I pamiętaj o wcięciach w kodzie! Bo taki pisany jak leci, w notatniku źle się czyta (najlepsze wcięcia są IMHO ta 2 spacje albo tabulator).

[klapaucius]

wiem ze wyczerpalem Wasz limit odpowiedzi... Ale zrobilem cos takiego:

[PHP] pobierz, plaintext 
<?php
include("baza.php");
include("gora.php");
$przeczytana = 1;
$q = mysql_query("SELECT login, od, temat, tresc, przeczytana, date, id FROM wiadomosci WHERE login='$user'");
?>
<b><center>Wiadomości:</center></b>
<br>
<a href="wiadomosci.php">:: Nowa wiadomość</a><br>
:: Kasuj wszystkie wiadomości<br><hr>
 
<?php
if (isset($_GET['usun'] && is_numeric($_GET['usun']) {
       mysql_fetch_array(mysql_query("DELETE FROM `wiadomosci` WHERE `id`=".$_GET['usun']."") or die (mysql_error()));
}
while($r = mysql_fetch_array($q)) {
?>
<table width=100%>
<tr bgcolor=#e6e6e6>
<td><?php echo $r[2] ?></td>
<td><?php echo $r[5] ?></td>
<td><?php echo "<a href=\"profil.php?user=" . $r[1] . "\">"  . $r[1] . "</a>" ?></td>
</tr>
</table>
<table width=100%>
<tr bgcolor=#6e6e6e>
<td>
<?php echo $r[3] ?><br><br>
</td>
</tr>
<tr bgcolor=#e6e6e6>
<td><?php echo '<a href="skrypt.php?usun='.$r[6].'">Usuń</a>'; ?></td>
</tr>
</table>
<hr>
<?php
}
include("dol.php");
?>
[PHP] pobierz, plaintext 

i wywala mi taki błąd:
Parse error: syntax error, unexpected T_BOOLEAN_AND, expecting ',' or ')' in /var/www/virtual/rekrutacjadogildii.cba.pl/wiadomosci2.php on line 13
Czy zamiast

[PHP] pobierz, plaintext 
<?php
<form action="wiadomosci2.php" method="post">
       <td><button style="border: 0px; background-color: #e6e6e6;" type="submit" name="przeczytana">przeczytana</button><button style="border: 0px; background-color: #e6e6e6;" type="submit" name="kasuj">kasuj</button></td>
</form>
?>
[PHP] pobierz, plaintext 

nie powinno byc porostu:

[PHP] pobierz, plaintext 
<?php echo '<a href="skrypt.php?usun='.$r[6].'">Usuń</a>' ?>
[PHP] pobierz, plaintext 

Bardzo Was jeszcze prosze o wyrozumialosc

[kfc4]

[PHP] pobierz, plaintext 
<?php
if (isset($_GET['usun'] && is_numeric($_GET['usun']) {
?>
[PHP] pobierz, plaintext 

na

[PHP] pobierz, plaintext 
<?php
if (isset($_GET['usun']) && is_numeric($_GET['usun'])) {
?>
[PHP] pobierz, plaintext 

Skopiowałeś co napisałem bez sprawdzania czy dobrze, a ja zapomniałem domknąć nawiasów. Ach te przyzwyczajenia, zawsze domyka za mnie edytor...

[klapaucius]

[PHP] pobierz, plaintext 
<?php
include("baza.php");
include("gora.php");
$przeczytana = 1;
$q = mysql_query("SELECT login, od, temat, tresc, przeczytana, date, id FROM wiadomosci WHERE login='$user'");
?>
<b><center>Wiadomości:</center></b>
<br>
<a href="wiadomosci.php">:: Nowa wiadomość</a><br>
:: Kasuj wszystkie wiadomości<br><hr>
 
<?php
if (isset($_GET['usun']) && is_numeric($_GET['usun'])) {
       mysql_fetch_array(mysql_query("DELETE FROM `wiadomosci` WHERE `id`=".$_GET['usun']."") or die (mysql_error()));
}
while($r = mysql_fetch_array($q)) {
?>
<table width=100%>
<tr bgcolor=#e6e6e6>
<td><?php echo $r[2] ?></td>
<td><?php echo $r[5] ?></td>
<td><?php echo "<a href=\"profil.php?user=" . $r[1] . "\">"  . $r[1] . "</a>" ?></td>
</tr>
</table>
<table width=100%>
<tr bgcolor=#6e6e6e>
<td>
<?php echo $r[3] ?><br><br>
</td>
</tr>
<tr bgcolor=#e6e6e6>
<form action="wiadomosci2.php" method="post">
<?php echo '<a href="skrypt.php?usun='.$r[6].'">'; ?>
      <td><button style="border: 0px; background-color: #e6e6e6;" type="submit" name="usun">
      Usuń
      </button></td>
<?php echo '</a>'; ?>
</form>
</tr>
</table>
<hr>
<?php
}
include("dol.php");
?>
[PHP] pobierz, plaintext 

Zrobilem co takiego i teraz wogole nie kasujewiadomosci... Nie wyskakuje zaden blad ani nic... Cos mi sie zdaje ze chyba sobie to cale php odpuszcze bo to nie dla mnie ;/

@edit

Haha dziala! Nie wiem jak to mozliwe ale dziala... Poprostu pobralem najpierw edytor php, a potem zaczolem robic ten skrypt od nowa i... Dziala!! Faktycznie wczesniej mailem zagmatwane jak cholera... Kurde ale tyle nad tym siedzialem i sie glowilem i jeszcze przede wszystkim Was dreczylem. Dzieki piekne szczegolnie dla kfc4 Nie wiem jak sie moge odwdzieczyc ... dam ci chociaz buziaka :* pozdro

Ten post edytował klapaucius 28.07.2009, 13:01:53