通过数组定义类选项是一个坏习惯吗？

Question

当我们看一下 Dojo、Mootools、jQuery、JS Prototype 等 Javascript 框架时，我们发现选项通常是通过这样的数组定义的：

dosomething('mainsetting',{duration:3,allowothers:true,astring:'hello'});

在编写 PHP 类时实现相同的想法是一个坏习惯吗？

示例：

class Hello {

    private $message = '';
    private $person = '';


    public function __construct($options) {

        if(isset($options['message'])) $this->message = $message;
        if(isset($options['person'])) $this->person = $person;
    }


    public function talk() {

        echo $this->person . ' says: ' . $this->message;
    }
}

常规方法：

class Hello {

    private $message = '';
    private $person = '';


    public function __construct() {}


    public function setmessage($message) {

        $this->message = $message;
    }


    public function setperson($person) {

        $this->person = $person;
    }


    public function talk() {

        echo $this->person . ' says: ' . $this->message;
    }
}

第一个示例的优点是您可以传递任意数量的选项，并且该类只会提取它需要的选项。

例如，从 JSON 文件中提取选项时这可能会很方便：

$options = json_decode($options);
$hello = new Hello($options);

这就是我定期执行此操作的方式：

$options = json_decode($options);
$hello = new Hello();

if(isset($options['message'])) $hello->setmessage($options['message']);
if(isset($options['person'])) $hello->setperson($options['person']);

此模式有名称吗？您认为这是一个不好的做法吗？

为了简单起见，我在示例中保留了验证等内容。

Answer 1

有好的方面，也有坏的方面。

优点：

• 不需要多个方法签名（即重载，如果支持的话）
• 与上一点保持一致：可以使用任何顺序的参数调用方法
• 可以动态生成参数，无需指定每个参数将出现的一个（例如：根据用户输入动态创建参数数组并将其传递给函数）
• 不需要“样板”方法，例如 setName、setThis、setThat 等，尽管您可能仍想包含它们
• 默认值可以在函数体中定义，而不是签名（jQuery 经常使用这种模式。它们经常 $ .extend 传递给带有默认值数组的方法的选项在您的情况下，您将使用 array_merge())

坏处：

• 除非您正确。宣传每个选项，您的类可能会更难使用，因为很少有人知道支持哪些选项
• 当您提前知道需要传递哪些选项时，创建一组参数又是一个步骤
• 对于用户来说，默认值并不总是显而易见的值是存在的，除非提供文档或者他们可以访问源代码

在我看来，这是一项很棒的技术。我最喜欢的方面是，您不需要提供具有不同签名的重载方法，并且签名不是一成不变的。

Answer 2

这种方法没有任何问题，特别是如果您有很多参数需要传递给构造函数。这还允许您在构造函数中为它们设置默认值和array_merge()（有点像所有 jQuery 插件）

protected $default_params = array(
    'option1' => 'default_value'
);
public function __construct($params = array()) {
    $this->params = array_merge($this->default_params, $params);
}

如果您想要这种“模式”的实时示例，查看 symfony 框架，他们几乎在所有地方都使用它：这里是 sfValidatorBase构造函数的示例

Answer 3

当您给出参数名称时，它称为“命名表示法”与“位置表示法”，其中参数必须按特定顺序排列。

在 PHP 中，您可以传递“选项”参数来实现与其他语言（如 Python）相同的效果，您可以在其中使用真正的命名表示法。这不是一个坏习惯，但通常在有充分理由的情况下这样做（即在您的示例中或有很多参数且不需要按任何特定顺序设置的情况下）。

Answer 4

我不知道这个名字，但我真的怀疑这是一个不好的做法，因为当你不想声明一个小或快速的函数或类属性时通常会使用它

Answer 5

如果有强制选项，它们应该在构造函数的参数列表中。然后添加具有默认值的可选选项。

public function __construc($mandatory1, $mandatory2, $optional1="value", $optional2="value") { }

如果所有选项都是可选的，那么创建一个采用数组的构造函数会很有用。创建对象比使用“普通构造函数”更容易：您可以只提供您想要的选项，而使用“普通构造函数”如果您想提供 $Optional2，则必须提供 $Optional1 （甚至设置它）为默认值）。

Answer 6

我不会说这是不好的做法，至少如果你相信数据的来源的话。

另一种可能性是根据选项数组键动态调用设置器，如下所示：

public function __construct($options) {
    foreach($options as $option => $value) {
        $method = 'set'.$option;
        if(method_exists($this, $method)
            call_user_func(array($this, $method, $value);
    }
}

Answer 7

为什么不两者都做呢？使用静态工厂“命名构造函数”让你的构造函数鱼与熊掌兼得：

$newHello = Hello::createFromArray($options);

首先，您的构造函数按顺序包含选项。然后将这样的静态方法添加到同一个类中：

public static function createFromArray($options){

    $a = isset($options['a']) ? $options['a'] : NULL;
    $b = isset($options['b']) ? $options['b'] : NULL;
    $c = isset($options['c']) ? $options['c'] : NULL;

    return new Hello($a, $b, $c);
} 

这将使新开发人员和 IDE 感到高兴，因为他们仍然可以看到构建对象所需的内容。

我同意这里答案的总体态度，无论哪种方式都是可行的解决方案，具体取决于您的需求，并且哪种方式对您的应用程序更有利。