0

我有一个查询页面，该查询会将表设置为 0 或 1。设置为 1 有效，但不会设置回 0。这是到目前为止的代码片段。任何人都可以看到有什么问题吗？

页：

<p><? if($_SESSION['user_level'] >= GUARDIAN_LEVEL) { if ($row_settings['profile_list'] == 0) {?> 
                    <a onclick='$.get("dos.php",{ cmd: "sprofile", setp:$("1").val(),id: "<?php echo $row_settings['id']; ?>" } ,function(data){ $("#msgp").html(data); });' href="javascript:void(0);">List</a>
                <?php } else if ($row_settings['profile_list'] == 1) {?>
                    <a onclick='$.get("dos.php",{ cmd: "sprofile", setp:$("0").val(),id: "<?php echo $row_settings['id']; ?>" } ,function(data){ $("#msgp").html(data); });' href="javascript:void(0);">Unlist</a>
                <?php } else {echo "N/A";}} else {echo "N/A";} ?></p>

更新代码：

if($get['cmd'] == 'sprofile')
{
mysql_query("update users set profile_list='$get[setp]' where id='$get[id]'");
echo "Profile Listing Changed";
//header("Location: $ret");  
// exit();
}

编辑

更新代码：

if($_GET['cmd'] == 'sprofile')
{
$set = mysql_real_escape_string($_GET['setp']);
$id = mysql_real_escape_string($_GET['id']);
mysql_query("update users set profile_list='" . $set . "' where id='" . $id . "'");
echo "Profile Listing Changed";
//header("Location: $ret");  
// exit();
}

这会将其设置为 0 并显示正确的代码，但由于某种原因不会设置回 1。

4

3 回答 3

2

它是$_GET和不是$get。

请至少使用

$param = mysql_real_escape_string($_GET['your_param_which_goes_to_building_mysql_query']);
$param2 = //similar..
$query = "update users set profile_list='$param1' where id='$param2'"

或者更好的是，使用准备好的语句。

于 2012-06-26T09:23:10.740 回答
1

更新代码：

  1. 使用$_GET代替$get（第 1 行和第 3 行）

  2. 将此行用作第一行：

    if ($_GET['cmd'] == 'sprofile')

  3. 将此行用于第 3 行中的 mysql_query：

    mysql_query("update users set profile_list='" . mysql_real_escape_string($_GET['setp']) . "' where id='" . mysql_real_escape_string($_GET['id']) . "';");
于 2012-06-26T09:27:45.493 回答
0

setp:$("1").val()需要更改为，setp: "1"因为它不应该从任何地方提取值

也换$get工作$_GET

于 2012-06-28T02:06:30.137 回答