Deleting a row from a Mysql table does not work. What is the problem?

L

Loligan2015-11-05 10:23:49

PHP

Loligan, 2015-11-05 10:23:49

<?

/* Соединяемся с базой данных */
$hostname = "localhost"; // название/путь сервера, с MySQL
$username = "root"; // имя пользователя (в Denwer`е по умолчанию "root")
$password = ""; // пароль пользователя (в Denwer`е по умолчанию пароль отсутствует, этот параметр можно оставить пустым)
$dbName = "loligan"; // название базы данных

$table = "auditoru";
/* Таблица MySQL, в которо
/* Создаем соединение */
mysql_connect($hostname, $username, $password) or die ("Не могу создать соединение");

/* Выбираем базу данных. Если произойдет ошибка - вывести ее */
mysql_select_db($dbName) or die (mysql_error());

/* Если была нажата ссылка удаления, удаляем запись */
$delete_ = $query = "DELETE FROM `auditoru` WHERE ID_audit='$delete_'";

/* Выполняем запрос. Если произойдет ошибка - вывести ее. */
mysql_query($query) or die(mysql_error());

/* Заносим в переменную $res всю базу данных */
$query = "SELECT * FROM $table";
/* Выполняем запрос. Если произойдет ошибка - вывести ее. */
$res = mysql_query($query) or die(mysql_error());
/* Узнаем количество записей в базе данных */
$row = mysql_num_rows($res);

/* Выводим данные из таблицы */
echo ("
<!DOCTYPE html PUBLIC \"-//W3C//DTD XHTML 1.0 Transitional//EN\" \"http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd\">
<html xmlns=\"http://www.w3.org/1999/xhtml\">

<head>

    <meta http-equiv=\"Content-Type\" content=\"text/html; charset=windows-1251\" />

    <title>Вывод и удаление данных из MySQL</title>

<style type=\"text/css\">
<!--
body { font: 12px Georgia; color: #666666; }
h3 { font-size: 16px; text-align: center; }
table { width: 700px; border-collapse: collapse; margin: 0px auto; background: #E6E6E6; }
td { padding: 3px; text-align: center; vertical-align: middle; }
.buttons { width: auto; border: double 1px #666666; background: #D6D6D6; }
-->
</style>

</head>

<body>

<h3>Вывод и удаление ранее сохраненных данных из таблицы MySQL</h3>


<table border=\"1\" cellpadding=\"0\" cellspacing=\"0\">
 <tr style=\"border: solid 1px #000\">
  <td><b>ID</b></td>
  <td align=\"center\"><b>Этаж</b></td>
  <td align=\"center\"><b>Площадь</b></td>
  <td align=\"center\"><b>Двери</b></td>
  <td align=\"center\"><b>Стены</b></td>
  <td align=\"center\"><b>Другое</b></td>
    <td align=\"center\"><b>Удаление</b></td>
 </tr>
");

/* Цикл вывода данных из базы конкретных полей */
while ($row = mysql_fetch_array($res)) {
    echo "<tr>\n";
    echo "<td>".$row['ID_audit']."</td>\n";
    echo "<td>".$row['Etash']."</td>\n";
    echo "<td>".$row['Plosthad']."</td>\n";
    echo "<td>".$row['Door']."</td>\n";
    echo "<td>".$row['Steni']."</td>\n";
    echo "<td>".$row['Other']."</td>\n";
    echo "<td><a name=\"del\" href=\"del.php?delete_=".$row["ID_audit"]."\">Удалить</a></td>\n";
    echo "</tr>\n";
}


echo ("</table>\n");

/* Закрываем соединение */
mysql_close();

/* Выводим ссылку возврата */
echo ("<div style=\"text-align: center; margin-top: 10px;\"><a href=\"index.html\">Вернуться назад</a></div>");

?>

Reply

Answer the question

In order to leave comments, you need to log in

3 answer(s)

D

Dmitry, 2015-11-05
@Loligan

And where do you get the $delete variable?
This is
$delete_ = $query = "DELETE FROM `auditoru` WHERE ID_audit='$delete_'";
Replace with this:
$delete = $GET["delete_"];
$query = "DELETE FROM `auditoru` WHERE ID_audit='$delete'";

M

MetaDone, 2015-11-05
@MetaDone

$delete_ = $query = "DELETE FROM `auditoru` WHERE ID_audit='$delete_'";

your $delete_ is not defined. what are you going to delete?
mysql_query has been deprecated a long time ago, switch to pdo

echo ("
"-//W3C//DTD XHTML 1.0 Transitional//EN\

for such an output html you need to beat hands

K

Kirill Nesmeyanov, 2015-11-05
@SerafimArts

Just in case, I'll add:
1) Mysql extensions are not supported in php for a long time (removed, use mysqli or pdo)
2) Specifying the $delete variable as in the first comment will allow the user to execute any sql code, just replace get `delete_=42` with ` delete_=some; drop ...`.