PHP OOP调用类中的数据库

Question

我真的很困惑从类内部与数据库接口的正确方法是什么，我知道我在下面做错了，但是我仍然希望它至少可以工作？

class User{


    public function user_email($user_id){

        require_once("config/db.php");

        $finduser = $db->query("SELECT email FROM `user` WHERE user_id = '".$user_id."' ");

        $user_data = $finduser->fetch_assoc();

        return $user_data["email"];

    }

}

然后，我使用下面的代码在另一页上显示结果，第一个实例工作正常，但第二个实例抛出错误。

<?php echo $user->user_email($user_id); ?>
<?php echo $user->user_email($user_id); ?>

Notice: Undefined variable: db on line 89
Fatal error: Call to a member function query() on a non-object on line 89

因此，我基本上要问的是，从与数据库的连接位于一个位置的类中与数据库接口的一种好方法是什么？

谢谢你的帮助！

Answer 1

您应该看看@leneya在他/她的回答中所具有的含义，这几乎是可以解决的方法。

但是，如果您不只是为了学习而这样做，我建议您使用Doctrine2 ORM 。

现在到您的特定问题...

---

您应该将连接注入到类中。 最简单的形式：

class User {
   protected $conn;

   public function __construct(mysqli $conn) {
     $this->conn = $conn;
  }

  public function getConnection() {
     return $this->conn;
  }

  public function user_email($user_id){


    $finduser = $this->getConnection()->query("SELECT email FROM `user` WHERE user_id = '".$user_id."' ");

    $user_data = $finduser->fetch_assoc();

    return $user_data["email"];

  }
}

然后用法如下所示：

require 'connection.php';
$user = new User($db);
$email = $user->user_email($userid);

我还将更改connection.php以返回mysqli对象，这样您就可以在单独的文件中声明var了：

// connection.php

// vars and stuff for the connection
// and then we return the instance
return new Mysqli($host, $user, $pass, $dbname); 

现在，当您使用此功能时，您可以

$db = require('connection.php');
$user = new User($db);
$email = $user->user_email($userid);

话虽如此，这是一个非常奇怪的设计。 通常，您将使用getter访问完整用户对象上的属性并运行针对特定事物的查询。

Answer 2

对于数据库连接，我建议将数据库访问从类中分离出来。 您的User类只能处理成为用户的情况，不应该知道数据的来源。

我建议对不同的职责使用不同的类。

例如，

库

应该处理从数据库中检索完整的对象。 他们将从数据库中返回一组（或仅一个）对象，并使用数据库中的持久数据填充其信息。

实体或模型

应该只处理业务逻辑。 这意味着您不必调用数据库来获取用户的电子邮件，而应从数据库加载整个用户对象数据，然后将其设置在此处显示的User对象上。 我这样做的方式（不使用像Doctrine或Propel这样的ORM或MVC框架附带的任何其他ORM）将是在您的对象上创建一个名为“ setByArray”的函数，该函数将采用可设置根据从数据库中找到的结果，对象上的所有数据。

因此，实际上，应该从实际的类中抽象出对数据库的访问。 通过在班级内使用数据库，您将违反SOLID OOP原则的单一责任原则。 您的User类仅应考虑成为用户。 与如何访问它的数据无关。

编辑

抱歉，我很想念您让它“正常工作”。 您确定您的“ config / db.php”实际上是其中包含名为$ db的变量的路径吗？ 您确定$ db实际上是具有查询功能的对象吗？ 从我所看到的，$ db不是一个对象。