可以在PHP类构造函数中运行查询吗？

Question

这是我第一次使用PHP创建面向对象的站点，也是有关构造函数的问题。

我有一个用户登录后生成的角色类。角色对象将保存游戏的所有用户数据。

我的问题是关于构造函数/实例化类。 像我在所附代码中一样，可以在类构造函数中运行查询吗？

当前传递的参数只是登录期间获得的用户名，这是在查询中运行以获取字符信息的内容。

这是好的方法还是最佳实践，是在构造或类之外运行查询并将参数传递给类构造函数？

只是想确保我不会忽略当前代码的任何潜在问题-只要考虑到异常，就可以将查询保留在构造函数中，是吗？

<?php

include $_SERVER['DOCUMENT_ROOT']."/rx/includes/classes/connection.php";

class Character {
 private $charid;
 private $userid;
 private $username;
 private $class;
 private $exp;
 private $money;
 private $attack;
 private $defense;
 private $hp;
 private $turns;


function __construct($data){
 $pdo = Connection::getInstance();

try {
  $pdo->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);

  $char = $pdo->prepare("SELECT users.userid, users.username, characters.charid, characters.money, characters.attack, characters.defense, characters.class, characters.hp, characters.userid, characters.turns, characters.exp
                         FROM characters
                         JOIN users ON users.userid=characters.userid
                         WHERE username=?");
  $char->execute(array($data));
  $res = $char->fetchAll(PDO::FETCH_ASSOC);


  $this->charid = $res[0]['charid'];
  $this->userid = $res[0]['userid'];
  $this->username = $res[0]['username'];
  $this->class = $res[0]['class'];
  $this->exp = $res[0]['exp'];
  $this->money = $res[0]['money'];
  $this->attack = $res[0]['attack'];
  $this->defense = $res[0]['defense'];
  $this->hp = $res[0]['hp'];
  $this->turns = $res[0]['turns'];

}
catch (Exception $e){
  echo $e;
 }
}


public function __get($property) {
 if (property_exists($this, $property)) {
  return $this->$property;
 }
}

public function __set($property, $value) {
 if (property_exists($this, $property)) {
  $this->$property = $value;
 }
return $this;
 }
}

Answer 1

没有。

并且也建议避免使用魔术二传手和吸气剂。 单身人士也是如此。

构造函数实际上不应包含任何逻辑，因为它使代码很难测试。 相反，您的类构造函数应将连接视为参数，并在单独的方法中执行逻辑。 有点像这样：

class Character 
{
    private $connection;

    public function __construct(\PDO $pdo) 
    {
        $this->connection = $pdo;
    }

    public function populateByUsername($data) 
    {
        // come code
        $statement = $pdo->prepare("SELECT us ....");
        $statement->execute(array($data));
        $data = $char->fetchAll(PDO::FETCH_ASSOC);

        // more code
    }
}

为了进一步改进代码，我建议实际上将SQL代码与实体的逻辑分开（在您的示例中为-字符）。 最好的方法是使用数据映射器模式-这基本上意味着您有两个类-一个用于实体，一个用于sql部分。 最终看起来像这样：

$char = new Character;
$char->setUsername($username);

$mapper = new CharacterMapper($pdo);
$mapper->fetchByUsername($char);

在这种情况下，映射器类将您的实体用作对象，使用getter提取用户名，调用SQL，然后将数据放入character对象。 稍后，当您更加熟悉OOP时，您还将学习在必要时可以在同一实体上调用多个映射器（例如，可以对其进行设置，以便为缓存使用单独的映射器，对于数据库使用单独的映射器。 ..您首先调用缓存映射器，如果失败，则调用db映射器。

Answer 2

缺点是您的“ Character”类构造函数将与数据库紧密耦合。 这意味着除了从数据库中获取数据之外，没有其他方法可以初始化“字符”。 您将无法使用其他源（例如XML文件）对其进行初始化，也无法使用用于单元测试的测试数据进行初始化。

我建议将查询分成类似“ InitializationProvider”的类（这只是一个示例，因为我不确定您的应用程序的逻辑）。 然后，您可能具有不同类型的InitializationProvider，例如DBInitializationProvider，XMLInitializationProvider等，这些类型会动态注入到Character类中。

但是，将来是否需要这种灵活性取决于您。