Question

我是OOP的新手，我正在将所有网站代码切换到它！我目前正在写一个抓住用户信息的课程，最终会更新它。

我正在使用的代码如下：

<?php

    require("C:\wamp\www\postin'\db_connection.php");
    session_start();

    class user {

        public function __construct($userid, $connection, $information) {
            $this->userid = $userid;
            $this->connection = $connection;
            $this->information = $information;
        }   

        public function user_information($userid, $connection, $information) {
            $query = "SELECT * FROM users WHERE id = :id";
            $params = array(':id' => $userid);
            try{
                $stmt = $connection->prepare($query);
                $result = $stmt->execute($params);
            }                               
            catch(PDOException $ex){
                echo ("Failed to run query: " . $ex->getMessage());
            }                       
            $columns = $stmt->fetch();
            return $columns["$information"];
        }

    }

    $username = new user($_SESSION["logged_in"], $connection, "username");
    echo $username->user_information($_SESSION["logged_in"], $connection, "username");
?>

现在你可以看到最后两行代码（最后一行），我必须使用两次参数。基本上第一个参数表示ID是什么，第二个参数表示$connection是什么，第三个参数是我想要从数据库中获取的内容。那么我做错了什么？我定义了一些我不需要的东西吗？

修改

以下内容是否也有效？

<?php

require("C:\wamp\www\postin'\db_connection.php");
session_start();

class user {

    public function user_information($userid, $connection, $information) {
        $query = "SELECT * FROM users WHERE id = :id";
        $params = array(':id' => $userid);
        try{
            $stmt = $connection->prepare($query);
            $result = $stmt->execute($params);
        }                               
        catch(PDOException $ex){
            echo ("Failed to run query: " . $ex->getMessage());
        }                       
        $columns = $stmt->fetch();
        return $columns["$information"];
    }

}

$username = new user();
echo $username->user_information($_SESSION["logged_in"], $connection, "username");
?>

就像这个推进器，或者错误......？

Answer 1

如果user类具有作为数据成员所需的所有信息，那么user_information不需要采用任何参数：

public function user_information() {
    $query = "SELECT * FROM users WHERE id = :id";
    $params = array(':id' => $this->userid);
    try{
        $stmt = $this->connection->prepare($query);
        $result = $stmt->execute($params);
    }                               
    catch(PDOException $ex){
        echo ("Failed to run query: " . $ex->getMessage());
    }                       
    $columns = $stmt->fetch();
    return $columns[$this->information];
}

Answer 2

由于你对课程的运作方式和OOP有很多疑问，我会尽量给你一点方向。没有标准的方法来建立你的班级。你是决定什么属于类和需要注入什么的人。这只是告诉你，你不能把自己压低。您需要了解它并构建逻辑。我带着你的课，并通过添加评论重建它。希望能帮助你一些。祝你好运！

    <?php

require ("C:\wamp\www\postin'\db_connection.php");
session_start();

class user {

    public $dbconnection;

    public function __construct($connection) {
        /**
         * Your user class interacts with the database.
         * Inject the connection here and set your
         * global class variable.
         */
        $this -> dbconnection = $connection;
    }

    public function user_information($userid, $column) {
        /**
         * The userid and column are specific for this
         * method action. No need to set these variables
         * in the global scope of the class.
         */

        $query = "SELECT" . $column . " FROM users WHERE id = :id";

        $params = array(':id' => $userid);
        try {
            $stmt = $this -> dbconnection -> prepare($query);
            $stmt -> execute($params);
        } catch(PDOException $ex) {
            echo("Failed to run query: " . $ex -> getMessage());
        }
        $result = $stmt -> fetch();

        return $result;
    }

}

$username = new user($connection);
echo $username -> user_information($_SESSION["logged_in"], $information);
?>

PHP类参数使用两次

2 个答案: