1

我有几个模型类处理数据库层，并且在如何以 OO 方式将数据库连接传递给查询时出现逻辑错误。Class1 是一个 db_connect，我在 class2 查询方法的构造函数中调用它：

class db {
    public $host = 'localhost';
    public $username = 'root';
    public $password = '';
    public $database = 'molecule';
    public $mysqli = '';

   function __construct() {
        $mysqli = new mysqli($this->host, $this->username, $this->password, $this->database);
            if (mysqli_connect_errno()) {
                printf("Database Connection failed: %s\n", mysqli_connect_error());
                exit();
            }
        return $mysqli;
    }   
}

class nodeModel {
    public $node;
    public $node_name;
    public $node_link;
    public $node_comment;
    public $mysqli;

    function __construct() {
        $mysqli = new db;
        return $mysqli;
    }

    public $insert;
    public $insert_id;
    function insertNode($node_name, $node_link, $node_comment) {
            $this->insert = $insert;
            $this->insert = $mysqli->prepare("INSERT INTO node(node_name, name_link, node_comment) VALUES (?, ?, ?)");
            $this->insert->bind_param("sss", $this->node_name, $this->node_link, $this->node_comment);
            if($this->insert->execute()) {
                $this->insert_id = $mysqli->insert_id;
            } 
            return $this->insert_id;
            print_r($this->insert_id);
            $this->insert->close();
        }}

但是我收到 $insert 的未定义变量错误。

我用以下内容调用该层：

include 'node_model.php';
$dbTest = new nodeModel;
$node_name = 'My Node Title';
$node_link = 'My Node Link';
$node_comment = 'My Node Comment. This one should be longer so I will write more stuff';
$dbTest->insertNode($node_name, $node_link, $node_comment);

我与查询类的构造函数连接的方式似乎正在工作，但不确定我需要在范围内将哪个变量附加到准备/绑定/执行语句？

而且我知道我应该使用 PDO 或其他 ORMy 类型的工具。我会的，但这更多的是关于一些基本的 OO 获取/设置/检索最佳实践......感谢您的帮助。

4

2 回答 2

1

您的代码存在一些主要问题...

1.将您的模型类属性设为私有，并创建 setter 和 getter 方法来设置或检索属性的值。你的方法 insertNode 也是错误的。

class nodeModel {
    private $node;
    private $nodeName;
    private $nodeLink;
    private $nodeComment;
    private $mysqli;

    function __construct() {
        $this->mysqli = new db;
    }

    public function getNodeName() {
        return $this->nodeName;
    }

    public function getNodeLink() {
        return $this->nodeLink;
    }

    public function getNodeComment() {
        return $this->nodeComment;
    }

    public function setNodeName($value) {
        $this->nodeName = $value;
    }

    public function setNodeLink($value) {
        $this->nodeLink = $value;
    }

    public function setNodeComment($value) {
        $this->nodeComment = $value;
    }

    function insertNode() {
        $this->insert = $this->mysqli->prepare("INSERT INTO node(node_name, node_link, node_comment) VALUES (?, ?, ?)");
        $this->insert->bind_param($this->node_name, $this->node_link, $this->node_comment);
        if($this->insert->execute()) {
            $this->insert_id = $this->mysqli->insert_id;
        }
        $this->insert->close();

        print_r($this->insert_id);

        return $this->insert_id;
    }}


}

2.在课堂db上，当您创建连接时，将其存储到$this->mysqli中，而不仅仅是$mysqli.

function __construct() {
    $this->mysqli = new mysqli($this->host, $this->username, $this->password, $this->database);
        if (mysqli_connect_errno()) {
            printf("Database Connection failed: %s\n", mysqli_connect_error());
            exit();
        }
    return $this->mysqli;
}

3.在您的第三个代码中，您只创建了一个传递给方法的变量，insertNode但在此方法中您对传递的变量不做任何事情 - 您希望设置类属性但它们没有......因此这样做：

include 'node_model.php';
$dbTest = new nodeModel;
$dbTest->setNodeName('My Node Title');
$dbTest->setNodeLink('My Node Link');
$dbTest->setNodeComment('My Node Comment. This one should be longer so I will write more stuff');
$dbTest->insertNode();

希望这可以帮助...

于 2012-06-13T08:51:21.887 回答
-1

但是我收到 $insert 的未定义变量错误。

那是因为你的函数的第一行是

$this->insert = $insert;
$this->insert = $mysqli->prepare("INSERT INTO node(node_name, name_link, node_comment) VALUES (?, ?, ?)");

$insert 不在您的参数列表中，因此第一行会导致错误。然后在您的第二行中，无论如何您都会覆盖该变量。

在 return 语句之后还有两行代码，它们当然永远不会执行。

也许是时候睡一觉了？:)

于 2012-06-13T08:40:35.217 回答