-1

我的注销脚本有问题。它工作正常，如果用户按下注销它会终止会话并转到 logout.php，用户被告知他们已被注销。

但是当浏览器缓存被清空或者网站不应该连接到互联网并且如果用户单击注销按钮时，它会出现以下错误消息：

You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '' at line 1

它失败了，因为它无法将注销设置为“1”，所以我想知道如何将 else 语句放在某个地方说重定向到 logout.php，这样我就不会收到那个可怕的语法错误消息。

这是我的代码：

<?php   
ob_start();
    require('includes/_config/connection.php');
    require('includes/functions.php');  
?>

<?php
session_start();

$result = mysql_query("UPDATE ptb_users SET user_online='Offline' WHERE id=".$_SESSION['user_id']."") 
or die(mysql_error()); 
?>

<?php
    // Four steps to closing a session
    // (i.e. logging out)

    // 1. Find the session

    // 2. Unset all the session variables
    $_SESSION = array();

    // 3. Destroy the session cookie
    if(isset($_COOKIE[session_name()])) {
        setcookie(session_name(), '', time()-42000, '/');
    }

    // 4. Destroy the session
    session_destroy();

    redirect_to("login.php?logout=1");

    ob_end_flush()
?>
4

3 回答 3

2

你有双引号，你应该使用单引号

改变这个：

$result = mysql_query("UPDATE ptb_users SET user_online='Offline' WHERE id=".$_SESSION['user_id']."")

至：

$result = mysql_query("UPDATE ptb_users SET user_online='Offline' WHERE id='" . $_SESSION['user_id'] . "'")

请注意您应该替换所有mysql_*功能。由于PHP 5.5.0它们已被弃用。使用PDO或MySQLi 之类的东西

于 2012-12-22T08:32:36.350 回答
0

是的，您的引号有问题：只要 user_id 是整数值，您就可以这样做：

$result = mysql_query("UPDATE ptb_users SET user_online='Offline' WHERE id=".$_SESSION['user_id']);

如果是字符串，请切换到单引号：

$result = mysql_query('UPDATE ptb_users SET user_online="Offline" WHERE id="'.$_SESSION['user_id'].'"');
于 2012-12-22T08:37:18.137 回答
0

发送输出后您不能调用 session_start() - 您需要输出任何内容吗？

还要确保 $_SESSION['user_id'] 实际上有一个值；print_r($_SESSION)

于 2012-12-22T08:39:21.067 回答