1

Как сделать удаление каждого сообщения. Что сдесь не так ?

<?php
$result = mysql_query("SELECT * FROM mail ");
$r = array();
while($row = mysql_fetch_array($result))
{
 $r[] = $row;
}

if (isset($_GET["delete"])) { 
$id = $_GET["delete"];
if(mysql_query("DELETE FROM `mail` WHERE id='$id' ")){
echo "<b>Сообщение удалено !</b>";
}
else
  echo "<b>Сообщение не может быть удалено!</b>";
}
?>

<table>
 <tr>
    <td>Имя</td>
    <td>Email</td>
    <td>Сообщение</td>
 </tr>
 <?php foreach($r as $row) : ?>
    <tr>
      <td><?php echo $row['name']; ?></td>
      <td><?php echo $row['email']; ?></td>
      <td><?php echo $row['messages']; ?><br>
 <a href="messages.php?delete=$id">Удалить</a></td>
    </tr>
 <?php endforeach; ?>
</table>
1
  • Все просто супер ОГРОМНОЕ СПАСИБО :)))
    – webkostya
    27 окт 2011 в 10:41

4 ответа 4

3

надо вот так делать

<?php
$result = mysql_query("SELECT * FROM mail ");
$r = array();
while($row = mysql_fetch_array($result))
{
 $r[] = $row;
}

if (isset($_GET["delete"])) { 
$id = (int)$_GET["delete"];
if(mysql_query("DELETE FROM `mail` WHERE id='$id' ")){
echo "<b>Сообщение удалено !</b>";
}
else
  echo "<b>Сообщение не может быть удалено!</b>";
}
?>

<table>
 <tr>
    <td>Имя</td>
    <td>Email</td>
    <td>Сообщение</td>
 </tr>
 <?php foreach($r as $row) : ?>
    <tr>
      <td><?php echo $row['name']; ?></td>
      <td><?php echo $row['email']; ?></td>
      <td><?php echo $row['messages']; ?><br>
 <a href="messages.php?delete=<?php echo $row['id'];?>">Удалить</a></td>
    </tr>
 <?php endforeach; ?>
</table>
2
  • "Фишка" данного ресурса в том, чтобы помочь человеку разобраться в проблеме, а не тупо решить проблему за него.
    – Dex
    27 окт 2011 в 10:49
  • Ой ну я думаю наглядно все видно ... Вы вносили id в ссылку не из базы а из гет параметра(который поидее у вас был пустой) 27 окт 2011 в 12:14
3

Откуда у вас значения берутся в переменной $id?

<a href="messages.php?delete=$id">Удалить</a></td>

Видимо, вы забыли где-то:

$id = $row['id'];

P.S.

В вопросы надо больше информативности вливать.

7
  • Если в место id написать цифру то и удаление будет по номеру id. А сдесь у каждого сообщения есть кнопка 'удалить' так вот как сделать что бы по нажатию этой кнопки удалялось то сообщение которому пренадлежит эта кнопка.
    – webkostya
    27 окт 2011 в 9:58
  • Ну так сделай то, что я тебе написал в первом сообщении.
    – Dex
    27 окт 2011 в 10:01
  • Да вот что то не работает этот $id = $row['id'];
    – webkostya
    27 окт 2011 в 10:05
  • Код покажите.
    – Dex
    27 окт 2011 в 10:17
  • 1
    Зачем вы каждый раз отвечаете на свой же вопрос? Покажите код, где вы сделали то, что я вам писал.
    – Dex
    27 окт 2011 в 10:35
0

Должно быть ?delete=1 где 1 это id

Запрос не передает значение, т.е. еденичку!

1
  • Так как сделать что бы не надо было писать 1, а можно было написать id.
    – webkostya
    27 окт 2011 в 10:01
0

После удаления Вы формируете ответ в который вставлена строка с уже удаленным сообщением.

Видимо при формировании таблицы (ответа) надо учитывать ID уже удаленной строки.

1
  • А как это реализуется собственно ???
    – webkostya
    27 окт 2011 в 10:24

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.