1

我的 PHP MySQL Select 有一个小问题。该函数位于 PHP 类中。这是我得到的错误:

警告:mysql_fetch_array() 期望参数 1 是资源,整数在 C:\xampp\htdocs\include\database.php 第 59 行中给出警告:extract() 期望参数 1 是数组,在 C:\xampp\ 中给出 null htdocs\include\database.php 第 59 行

该功能只是简单地更新数据库以显示他们访问该站点的浏览器和操作系统。该函数是从另一个由 AJAX 调用调用的文件调用的,该调用使用 POST 发送从 Javascript 文件收集的有关操作系统和浏览器的数据。仅当数据库中已有 IP 地址条目时才会失败。如果数据库中没有 IP 地址条目,它将成功创建一个。

这是我的代码:

function addStat($browser, $os){
    $IE = 0; $Firefox = 0; $Safari = 0; $Opera = 0; $Chrome = 0; $otherb = 0;
    $Windows = 0; $Linux = 0; $Mac = 0; $Android = 0; $iOS = 0; $otheros = 0;
    $ql = 0; $totalVisits = 0;
    $ip = ip2long($_SERVER['REMOTE_ADDR']);
    $q1 = mysql_query("SELECT * FROM " . DB_STATS . " WHERE ip='$ip'", $this->connection);
    if (mysql_num_rows($q1)==0){
        $browser = mysql_real_escape_string($browser);
        $os = mysql_real_escape_string($os);
        switch($browser){
            case "Internet Explorer":
                $IE += 1;
            break;
            case "Firefox":
                $Firefox += 1;
            break;
            case "Safari":
                $Safari += 1;
            break;
            case "Opera":
                $Opera += 1;
            break;
            case "Chrome":
                $Chrome += 1;
            break;
            default:
                $otherb += 1;
            break;
        }
        switch($os){
            case "Windows":
                $Windows += 1;
            break;
            case "Mac OS X":
                $Mac += 1;
            break;
            case "Linux":
                $Linux += 1;
            break;
            case "Android":
                $Android += 1;
            break;
            case "iOS":
                $iOS += 1;
            break;
            default:
                $otheros += 1;
            break;
        }
        $q = $this->query("INSERT INTO " . DB_STATS . " VALUES (null, '$ip', '$Chrome', '$IE', '$Firefox', '$Opera', '$Safari', '$otherb', '$Windows', '$Mac', '$Linux', '$Android' , '$iOS' , '$otheros', 1)");
        if ($q == true){
           return(1);
        }
        else{
           return(0);
        }
    }
    else if (mysql_num_rows($q1)==1){
        extract(mysql_fetch_array($ql));
        switch($browser){
            case "Internet Explorer":
                $IE += 1;
            break;
            case "Firefox":
                $Firefox += 1;
            break;
            case "Safari":
                $Safari += 1;
            break;
            case "Opera":
                $Opera += 1;
            break;
            case "Chrome":
                $Chrome += 1;
            break;
            default:
                $otherb += 1;
            break;
        }
        switch($os){
            case "Windows":
                $Windows += 1;
            break;
            case "Mac OS X":
                $Mac += 1;
            break;
            case "Linux":
                $Linux += 1;
            break;
            case "Android":
                $Android += 1;
            break;
            case "iOS":
                $iOS += 1;
            break;
            default:
                $otheros += 1;
            break;
        }
        $totalVisits += 1;
        $q = $this->query("UPDATE " . DB_STATS . " set Chrome='$Chrome', IE='$IE', Firefox='$Firefox', Opera='$Opera', Safari='$Safari', otherb='$otherb', Windows='$Windows', Mac='$Mac', Linux='$Linux', Android='$Android' , iOS='$iOS' , otheros='$otheros', totalVisits='$totalVisits'");
        if ($q == true){
           return(1);
        }
        else{
           return(0);
        }
    }
    else{
        return(-1);
    }
}

我希望一切都有意义,并且有人会提供帮助。

4

1 回答 1

3

我现在明白了——$ql当你打算使用$q1. 让这成为反对使用非常短的变量名称或非常相似的名称的教训。

// $ql was initialized to 0
$ql = 0; $totalVisits = 0;

// $q1 holds the result resource
extract(mysql_fetch_array($q1));

除非您还将第二个参数指定为获取类型,否则不建议调用extract()的输出。默认情况下,它返回每列数字和关联索引。mysql_fetch_array()MYSQL_ASSOC

extract(mysql_fetch_array($q1, MYSQL_ASSOC));
// Or better
extract(mysql_fetch_assoc($q1));

一般来说,我可能会建议不要extract()在大多数情况下使用,因为它会导致大量变量转储到全局命名空间中,特别是当您在SELECT *没有具体选择哪些列的情况下完成操作时。最好通过他们的数组访问它们:

$row = mysql_fetch_assoc($q1);
echo $row['browser'];
于 2012-05-17T02:35:02.440 回答