Question

我很难理解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>";

Answer 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>";

Answer 2

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

一切。

来自用于自身功能的驱动程序。

如果您希望数据库函数选择查询的所有结果，请创建一个函数以返回所有结果。但是，不要将它限制在你能想象的最愚蠢的查询中！

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

请注意，此代码使用的是PDO，您需要使用构造函数来设置$ this-＆gt; pdo变量

与OOP php很难相处

2 个答案:

一切。