Question

我正在关于Udemy关于构建登录表单的本教程。一切顺利。

现在我正在尝试重用代码。我遇到的问题是当我试图从数据库中查询所有条目以获得总数的网站时。你能告诉我吗？

这是我的计数功能（这是我在获取分页数据库总数时遇到问题的地方）：

function get_all_websites(){

        $all = array();

        $db = DB::getInstance();
        $all = $db->query("SELECT * FROM website");

        if(!$all->count()){
            echo 'No Websites available. Please check back later.';
        } else {
            foreach($all->results() as $all){
                $all->id = $all;
            }
        }

        return $all;

    }

    function get_websites_count(){

        return count(get_all_websites());

    }

如果我使用它，我会列出所有ID。

function get_all_websites(){

    $all = array();

    $db = DB::getInstance();
    $all = $db->query("SELECT * FROM website");

    if(!$all->count()){
        echo 'No Websites available. Please check back later.';
    } else {
        foreach($all->results() as $all){
            echo $all->id;
        }
    }

}

数据库类。

class DB{
        private static $_instance = null;
        private $_pdo,
                $_query,
                $_error = false,
                $_results,
                $_count = 0;


        private function __construct(){

            try {
                $this->_pdo = new PDO('mysql:host=' .
                        Config::get('mysql/host') . ';dbname=' .
                        Config::get('mysql/db'),
                        Config::get('mysql/username'),
                        Config::get('mysql/password'));

            } catch(PDOException $e){
                die($e -> getMessage());
            }
        }

        public static function getInstance(){
            if(!isset(self::$_instance)) {
                self::$_instance = new DB();
            }
            return self::$_instance;
        }

        public function query($sql, $params = array()){

            $this->_error = false;

            // Check if query has been prepared properly

            if($this->_query = $this->_pdo->prepare($sql)){

                $x = 1;
                if(count($params)){
                    foreach($params as $param){
                        $this->_query->bindValue($x, $param);
                        $x++;
                    }
                }

            // If the query has been prepared successfuly, store the result
                if($this->_query->execute()){
                    $this->_results = $this->_query->fetchAll(PDO::FETCH_OBJ);
                    $this->_count = $this->_query->rowCount();
                } else {
                    $this->_error = true;
                }
            }

            return $this;
        }

        public function action($action, $table, $where = array()){
            if(count($where) === 3){
                $operators = array('=', '>', '<', '>=', '<=');

                $field      = $where[0];
                $operator   = $where[1];
                $value      = $where[2];

                if(in_array($operator, $operators)){
                    $sql = "{$action} FROM {$table} WHERE {$field} {$operator} ?";

                    if(!$this->query($sql, array($value))->error()){
                        return $this;
                    }
                }
            }

            return false;
        }

        // QUERYING DATA FROM DATABASE
        public function get($table, $where){
            return $this->action('SELECT *', $table, $where);
        }

        // DELETING DATA FROM DATABASE
        public function delete($table, $where){
            return $this->action('DELETE', $table, $where);
        }

        // INSERTING DATA INTO DATABASE
        public function insert($table, $fields = array()){
                $keys = array_keys($fields);
                $values = '';
                $x = 1;

                foreach($fields as $field){
                    $values .= "?";

                    if($x < count($fields)){
                        $values .= ', ';
                    }
                    $x++;
                }

                $sql = "INSERT INTO {$table} (`" . implode('`, `', $keys) . "`) VALUES({$values})";

                if(!$this->query($sql, $fields)->error()){
                    return true;
                }

            return false;
        }

        public function results(){
            return $this->_results;
        }

        public function update($table, $id, $fields){
            $set = '';
            $x = 1;

            foreach($fields as $name => $value){
                $set .= "{$name} = ?";

                if($x < count($fields)){
                    $set .= ', ';
                }
                $x++;
            }


            $sql = "UPDATE {$table} SET {$set} WHERE id = {$id}";

            if(!$this->query($sql, $fields)->error()){
                return true;
            }

            return false;
        }

        public function first(){
            return $this->results()[0];
        }

        public function error(){
            return $this->_error;
        }

        public function count(){
            return $this->_count;
        }

    }

Answer 1

这部分看起来不对：

    } else {
        foreach($all->results() as $all){
            $all->id = $all;
        }
    }
    return $all;

但我不确定你想要什么。

我们可以将每个id附加到名为$ allWebsiteIds：

的数组中

    } else {
        foreach($all->results() as $all){
            $allWebsiteIds[] = $all->id;
        }
    }
    return $allWebsiteIds;

这可能会给你你想要的东西。

Answer 2

SQL为计算表中的行提供了内置支持。

SELECT COUNT(*) FROM ...

这是高度可优化的，并且避免将每一行加载到PHP的内存中。

另外，要密切注意循环的构造;特别是它意味着什么：

foreach($foo as $foo)

作为学习的一部分，你应该能够说出为什么这种表达几乎从来都不是预期的。

有一个更广泛的规则：永远不要改变你正在迭代的东西。

Answer 3

为什么不使用select来计算行？

SELECT COUNT（1）FROM website;

如何通过id查询数据库中的所有条目？

3 个答案: