處理在PHP

Question

額外的參數考慮以下情況：

class Factory { 
    private $x = 1; 
    private $y = 2; 
    private $z = 3; 

    public function create(string $instance) { 
     return new $instance($this->x, $this->y, $this->z); 
    } 
} 

class A { 
    private $x; 

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

    public function display() { 
     echo "<pre>$this->x</pre>"; 
    } 
} 

class B { 
    private $x; 
    private $y; 
    private $z; 

    public function __construct ($x, $y, $z) { 
     $this->x = $x; 
     $this->y = $y; 
     $this->z = $z; 
    } 

    public function display() { 
     echo "<pre>$this->x</pre>"; 
     echo "<pre>$this->y</pre>"; 
     echo "<pre>$this->z</pre>"; 
    } 
} 

$factory = new Factory; 

$a = $factory->create("A"); 
$a->display(); 

$b = $factory->create("B"); 
$b->display(); 

正如你所看到的，工廠將始終提供3 arguments時，它會創建一個新的實例。但是在class A中，構造函數只需要1 argument。由於php沒有通常的方法重載，所以這不會導致問題。但爲了安全起見，我要補充的參數列表參數爲class A的contructor是這樣的：

public function __construct ($x, ... $ignore) { ... } 

趕上那些無謂的爭論，因爲我知道它每次會得到這些額外的參數。還是現有的代碼是足夠的？

Answer 1

如果您傳遞的變量超過函數定義中所需的變量，那些額外變量將被忽略（示例：https://3v4l.org/fNfAQ）。

但是，這可能是一個跡象，你試圖對這個特定的工廠做太多的事情，並且使用DI容器可能會有更好的結果。

或者您可能需要擁有單獨的工廠，以創建具有不同依賴關係的實例，而不是製造單一的「make everything」工廠。

Answer 2

而不是傳遞單獨的參數，您可以爲所有的構造函數使用關聯數組。

class A { 
    private $x; 

    public function __construct ($params) { 
     $this->x = $params['x']; 
    } 

    public function display() { 
     echo "<pre>$this->x</pre>"; 
    } 
} 

class B { 
    private $x; 
    private $y; 
    private $z; 

    public function __construct ($params) { 
     $this->x = $params['x']; 
     $this->y = $params['y']; 
     $this->z = $params['z']; 
    } 

    public function display() { 
     echo "<pre>$this->x</pre>"; 
     echo "<pre>$this->y</pre>"; 
     echo "<pre>$this->z</pre>"; 
    } 
} 

則工廠可以這樣做：

public function create($instance) { 
    return new $instance(array('x' => $this->x, 'y' => $this->y, 'z' => $this->z)); 
}