PHP Class 参数使用了两次

Question

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

我使用的代码如下：

<?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 class 具有作为数据成员所需的所有信息，那么 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

由于您对 class 的工作方式和 OOP 有很多疑问，我将尝试为您提供一些指导。没有构建 class 的标准方法。您是决定什么属于 class 以及什么需要注入的人。这只是告诉你，你不能把自己固定下来。您需要感受一下并建立逻辑。我拿走了你的 class 并用添加的评论重建了它。希望对你有所帮助。祝你好运！

    <?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 Class 参数使用了两次

PHP Class Parameters used twice

php

oop