把变量变成类'

Question

所以我学到了很多关于 PHP 的知识，并且我从基础开始，即使其中一些已被弃用。现在我已经掌握了知识，我正在重写我为使用 PDO 和类所做的一切。现在我从来没有使用过，但我正在慢慢地（慢慢地，慢慢地）理解并理解它。我想问的是，如果我这样做是对的，如果不是，我哪里错了？

我基本上有这个功能：

function user_exists($username) {
    $username = sanitize($username);
    $query = mysql_query("SELECT COUNT(`user_id`) FROM `users` WHERE `username` = '$username'");
    return (mysql_result($query, 0) == 1) ? true : false;
}

我想把它变成一个类：

class user {
    function exists($username) {
        $username = $username;
        $query = $db->query("SELECT COUNT(`user_id`) FROM `users` WHERE `username` = '$username'");
        $results = $query->fetchAll(PDO::FETCH_ASSOC);
    }
}

我是否走在正确的轨道上，我是否应该将该函数（方法）中的变量设为“exists”方法的私有变量？

我打算创建一个“用户”类，其中包含诸如存在、禁止、管理等方法。我认为这是我应该做的？

如果有人回答这个问题，谢谢。

Answer 1

class User {
    public $name;

    private $db;

    function __construct($name) {
        $this->name = $name;

        $this->db = get_me_a_db_connection();
    }

    public function exists() {
        $query = $this->db->query('SELECT COUNT(user_id) FROM users WHERE username = "' . $this->name . '"');
        $results = $query->fetchAll(PDO::FETCH_ASSOC);

        return some_code_that_extracts_the_count_from_$results;
    }
}

但是： 1) $db 不应该是类属性。它至少应该在工厂里。所以你extract it in a factory：

class Factory {
    public static function db() {
        return get_me_a_db_connection();
    }
}

public function exists() {
    $query = Factory::db()->query(...);
    ...
}

这样你就不会用基础设施代码污染你的domain models。

2) 上面的那么多是指您可以实现基础架构代码（数据库连接、查询）和域模型之间的完全解耦。有tons of libraries 正在这样做。我会从Laravel's Eloquent开始。

3) 如果您使用prepared statements，PDO 查询是安全的。我上面链接的所有库都使用准备好的语句。作为菜鸟，我建议您从学习其中一个开始学习。

Answer 2

也许你并没有太偏离轨道，这是一个好的开始。但是您需要了解如何进入面向对象编程的内容。我将从一个简单的类（例如 User ）开始，其中包含真实用户的信息，无论该用户对您尝试创建的所有方法所需的信息是什么。

您需要的私有变量始终是各种方法所需的信息。比如用户名，以及您需要了解的任何其他信息。

也许 $results 可能是带有最后返回信息的变量，或者只是像第一个示例一样返回它。

在写下你的课程之前，你需要先思考你真正需要什么。