0

这是我在 StackOverflow 上的第一个问题，坦率地说，我对 PHP 还很陌生。只是给你一个简短的提醒;）

我正在构建一个基于 OOP 的网站，采用 3 层架构。在我的数据抽象层中，我有一个名为 DbAdapter 的对象，它包含与数据库通信所需的所有函数。这些函数之一是下面的函数：read($sql)，它接受一个 SQL 查询并将结果存储在一个二维数组中。

为此，它使用两个嵌套的 for 循环（一个用于行，一个用于每行的列）。虽然迭代器 $i 像往常一样递增，但不知何故，数组的最后一个元素被覆盖了。

我完全不知道这怎么可能，所以我犯的错误一定是非常愚蠢的。

有人愿意帮助新手吗？

在此先感谢，山姆

public $loadedRows;
public function read($sql)
{
    if ($this->connect())
    {
        $result = mysql_query($sql);
        if ($result)
        {
            $totalRows = mysql_num_rows($result);
            $totalFields = mysql_num_fields($result);

            for ($i = 0; $i < $totalRows; $i++)
            {
                for ($j = 0; $j < $totalFields; $j++)
                {
                    $fieldName = mysql_field_name($result, $j);
                    $loadedFields["$fieldName"] = mysql_result($result, $i, $fieldName);
                }

                $this->loadedRows[i] = $loadedFields;
            }

            $this->closeConnection();
            return $this->loadedRows;
        }
    }
}
4

4 回答 4

5

你只是在我之前忘记了$$this->loadedRows[$i]

并且这段代码应该更短：

public function read($sql)
{
    $a = array();
    $result = mysql_query($sql);
    if ($result)
    {
        while($row = mysql_fetch_assoc($res)) $a[]=$row;
    }
    return $a;
}

就这样。

并且要自己捕获这些错误，您应该将错误报告级别设置E_ALL
为可以添加此行

error_reporting(E_ALL);

在你的配置文件中。
它会告诉 PHP 观察这些错误（在这种情况下使用未定义的常量 i）并通知你

于 2011-02-11T14:39:18.943 回答
2

任何你不能这样做的理由：

while($row = mysql_fetch_assoc()) {
    $this->loadedRows[] = $row;
}

代替你的两个循环？与仅获取已包含字段名/值的关联行相比，像这样单独获取字段名/值非常慢。

就个人而言，我更喜欢吃一碗cheerios，而不是单独从盒子里拿出一个，放在碗里，倒一滴牛奶，吃完，然后回到盒子里再吃一个。

于 2011-02-11T14:37:47.700 回答
2

我的猜测是失踪$之前i这里是你的罪魁祸首：

           $this->loadedRows[i] = $loadedFields;
于 2011-02-11T14:39:36.643 回答
1

如果我是你，我会使用 PDO 而不是这些旧的 mysql 函数。 PDOStatement::fetchAll做你需要的。

于 2011-02-11T14:40:13.597 回答