-1

我很难理解 OOP,我不知道这段代码有什么问题。

基本上我只需要从数据库中选择所有的 os_id 。这是我想出的解决方案

public function select_all_users($fields, $table){
    $query = mysql_query("select {$fields} FROM {$table}");
    while($row = mysql_fetch_array($query)){
        $result[] = $row['os_id'] ;
    }
    return $result;

}

从技术上讲,这就是我展示它的方式

require_once 'global/db.php'; 
$db = new Database();
$results = $db->select_all_users("*", "os_inventory"); 
echo "<td>". $results . "</td>";
4

2 回答 2

1

变量$results是数组,你使用 echo!。
使用 foreach 像:

require_once 'global/db.php'; 
$db = new Database();
$results = $db->select_all_users("*", "os_inventory"); 
foreach($results as $temp)
    echo "<td>". $temp[0] . "</td>";
于 2013-08-08T12:37:46.280 回答
0

我不知道这段代码有什么问题。

一切。

来自驱动程序本身的功能。

如果您希望数据库函数选择查询的所有结果,请创建一个函数以返回所有结果。但不要将其限制为您能想象到的最愚蠢的查询!

public function select_all($query, $params = array())
{
    $stm = $this->pdo->prepare($query);
    $stm->execute($params);
    return $stm->fetchAll();
}

请注意,此代码使用PDO,您需要一个构造函数来设置 $this->pdo 变量

于 2013-08-08T12:42:42.590 回答