PHP删除脚本，返回“viewsubjects.php?classroom_id=NO VALUE”

Question

正如标题所述...我正在从“教室”中删除“主题”，我查看教室，然后可以单击教室来查看该教室的主题。因此，我查看主题的链接如下所示：

viewsubjects.php?classroom=23

当用户选择删除按钮（连续）从班级中删除主题时，我只是希望将用户重定向回教室的主题列表（确切地说是他们之前在哪里！！）

所以我认为这只是在我的删除脚本中调用教室 ID 的情况。这是我所拥有的：

编辑：更正了代码中的拼写错误（这不是问题）

    $subject_id = $_GET['subject_id'];
    $classroom_id = $_GET['classroom_id'];

$sql = "DELETE FROM subjects WHERE subject_id=".$subject_id;
$result = mysql_query($sql, $connection)
    or die("MySQL Error: ".mysql_error());


header("Location: viewsubjects.php?classroom_id=".$classroom_id);
exit();

主题正在从数据库中删除，但是当我重定向回来时，URI 显示为空教室我想要：

viewsubjects.php?classroom_id=

有没有办法通过删除脚本成功携带教室ID，以便之后显示，从而允许用户重定向回页面？感谢您的帮助！

Answer 1

您的代码中有拼写错误吗？

将第 2 行更改为：
$classroom_id = $_GET['classroom'];

Answer 2

只是要注意，如果这是一个管理功能：那就太好了。

如果这是在前端，您需要考虑确保 subject_id 是干净的，因为它很容易入侵您的网站。

Answer 3

它应该是 $classroom_id = $_GET['classroom'];

不是：$classroom_id = $_GET['classrom_id'];

编辑 您已编辑您的代码，但 $_GET 变量中的字符串是否与 URL 中的字符串匹配？

Answer 4

为什么不将classroom_id添加到隐藏字段中的删除表单中？

Answer 5

这可能有点冗长……但让我们看看会发生什么

if ( !isset($_GET['subject_id']) ) {
  echo 'DEBUG: missing GET parameter subject_id';
  var_dump($_GET);
  die;
}
if ( !isset($_GET['classrom_id']) ) {
  echo 'DEBUG: missing GET parameter classroom_id';
  var_dump($_GET);
  die;
}
else if ( 0===strlen(trim($_GET['subject_id'])) ) {
  echo 'DEBUG: empty GET parameter subject_id';
  var_dump($_GET);
  die;
}
else if ( 0===strlen(trim($_GET['classrom_id'])) ) {
  echo 'DEBUG: empty GET parameter classroom_id';
  var_dump($_GET);
  die;
}

$subject_id = mysql_real_escape_string($_GET['subject_id'], $connection);

$sql = "DELETE FROM subjects WHERE subject_id='$subject_id'";
$result = mysql_query($sql, $connection) or die("MySQL Error: ".mysql_error());
if ( 0===mysql_affected_rows($connection) ) {
  echo 'no such subject_id found. no records have been deleted';
  die;
}

header("Location: viewsubjects.php?classroom_id=".urlencode($_GET['classrom_id']));
exit();

（它还修复了 sql 注入漏洞）

Answer 6

如果响应上面的内容，那就更容易做到：

if (!isset($_GET['subject_id']) || empty($_GET['subject_id']) || !is_numeric($_GET['subject_id'])) {
 throw new exception('Subject Id is not set');
}