PHP - 带有 SQL 查询的构造函数 [关闭]

Question

方案一：先查询数据，再将数据传给构造函数

方案二：使用构造函数查询数据，然后填充属性

---

选项 1 示例

$val1 = 1;
$query = mysql_query("SELECT val2, val3, val4 FROM table WHERE val1 = '".$val1."'");
$row = mysql_fetch_assoc($query);
$o = new Class($row['val1'], $row['val2'], $row['val3'], $row['val4']);

选项 2 示例

$val1 = 1;
$o = new Class($val1);

// in Class constructor
public function __construct($val1) {
    $query = mysql_query("SELECT val2, val3, val4 FROM table WHERE val1 = '".$val1."'");
    $row = mysql_fetch_assoc($query);
    $this->val1 = $row['val1'];
    $this->val2 = $row['val2'];
    // etc ...
}

注意事项

我完全清楚 mysql_query 已被弃用。请克制住那种告诉我的压倒性冲动。这些是项目的要求。

我在问选项 2 是否是不好的做法，或者是否存在任何在面向对象领域众所周知的可预见的困境。对我来说，这似乎是更清洁的选择。

Answer 1

嗯...你有比在哪里放置查询更大的问题但是...

我会说选项 1 是最好的，因为您可以使用数据库中的数据非在其他地方实例化该类。

换句话说，该类不受数据库的限制。

Answer 2

把这个给负责的人看。

---

不过还是回到你的问题。

尽管我讨厌这样回答，但我认为选项 2 肯定会使代码更干净，但您也可以通过在类中创建一个返回您正在查找的内容的方法来使其更干净。

<?php

class SomeName extends DBClass
{
    // No need for the construct in this case

    public function fetchResults($val)
    {
        $notSoDirtyVal = mysql_real_escape_string($val);

        $query = "SELECT val2, val3, val4 FROM table WHERE val1 = '".$notSoDirtyVal."'"// Make sure you escape, sanitize, and clean this!!!!!

        $stmt = mysql_query($query); 

        $results = mysql_fetch_assoc($query);

        return $results;
    }
}
?>

然后您可以在其他文件上执行此操作

<?php
$value1 = 1;

$o = new SomeName;
$results = $o->fetchResults($value1);