PHPよくある実践 - あまりにも多くのパラメータを持つメソッド

Question

私は自分のウェブサイト上で検索するだけのクラスとメソッドを作成しました。あまりにも多くのパラメータ、検索パラメータがあります。私のコントローラはフォームからデータを取得し、モデルに渡します。

public function search($name, $age, $foo, ... $bar, $lorem) { 

この種の方法のヒントはありますか？あまりにも多くのパラメータを持つメソッドについては、おそらく良い習慣です。おかげさまで

EDIT：

パラメータは、検索のためのものです... $名はように$の年齢 の値を持つ人々を検索しなければならない$名 $時代の価値を持つ人々を検索する必要があります...のような 何かSQLのWhere句

もう一度おねがいします。

Answer 1

DarhazerとZanathelはすでに良い答えを与えた、と私はあなたに一つのことを示したい：流れるようなインターフェイスでセッターを。すべてのパラメータがオプションの場合のみ。

$finder-> 
setName($name)-> 
setAge($age)-> 
setFoo($foo)-> 
setBar($bar)-> 
setLorem($lorem)-> 
search(); 

または

$query = new SearchQuery($required_argument); 
$query->setAge($optional)->setLorem($optional); 

$finder->search($query); 

ちょうど私が/多くのパラメータを持っているかもしれない機能のための配列を使用したいとセッターの体内にreturn $this;

Answer 2

多くの名前と値のペアを持つパラメータとして配列を受け入れるようにします。次に、extract($paramsArray)を使用して、配列内のすべての名前を$変数として作成できます。

Answer 3

あなたがキーに物事を詰めることができ=>値に基づく配列

例：

$params = array("Name"=>"Bob", "Age"=32.....); 
Class->search($params); 
public function search($params) { 
    // access keys 
} 

この方法は非常に簡単に任意の配列を渡すことができますので、間違って使用される可能性があるため、少し疲れています配列の内容を検証するために、他のメソッド呼び出しで検証が必要な場合があります。

編集：コメントの中で、いくつかの議論があったので....ここで新しいクラスを作成し、この

を行うための別の方法があります！名前の年齢など、その他の人口統計を含むユーザー・クラス。とにかく他の場所にあるものを使うでしょう。

オブジェクトを渡すか、引数

$object = new Object(); 
SearchClass->search($object) 

public function search(Object $object){ 
    // Do junk here 
} 

Answer 4

Introduce Parameter Object

Answer 5

としてオブジェクト私はあなたがするプロパティなど、すべての必要なパラメータが含まれていますオブジェクトを渡すところ、value objectsに最も近い比喩を使用したいです関数。

<?php 
class FilterVO { 
    public $id; 
    public $name; 
    // etc ... 
} 

class SomeCollection { 
    public function FilterResults(FilterVO $prefs) { 
     // got all the goodies in here 
    } 
} 
?> 

Answer 6

を書き、fluent interfaceを作成します。この種のアプローチは、パラメータのほぼ無限の拡張を可能にし、func_get_args()のようなものを使用するよりも簡単でより良い方法です。

public function search(array $options = array()) 
{ 
    $defaults = array(
     'name' => null, 
     'age' => null, 
     'order' => null, 
     'limit' => null, 
     'offset' => null, 
    ); 
    $options = array_merge($defaults, $options); 

    extract($options); 

    $select = $this->select(); 

    if (!is_null($name)) { 
     $select->where('name = ?', $name); 
    } 
    if (!is_null($age)) { 
     $select->where('age = ?', $age, Zend_Db::INT_TYPE); 
    } 
    if (!is_null($order)) { 
     $select->order($order); 
    } 
    if (!is_null($limit) || !is_null($offset)) { 
     $select->limit($limit, $offset); 
    } 

    $results = $this->fetchAll($select); 

    return $results; 
} 

...またはあなたがオブジェクト指向のアプローチを使用することができます：配列の要素として、各パラメータを含む単一の配列を渡し

class SearchQuery 
{ 
    public function __construct(array $options = null) 
    { 
     if (!is_array($options)) { 
      return; 
     } 

     if (array_key_exists('name', $options)) { 
      $this->setName($options['name']); 
     } 
     if (array_key_exists('age', $options)) { 
      $this->setAge($options['age']); 
     } 
    } 

    public function setName($name) 
    { 
     if (!is_string($name)) { 
      throw InvalidArgumentException('$name must be a string'); 
     } 

     $this->_name = $name; 

     return $this; 
    } 

    public function setAge($age) 
    { 
     if (!is_numeric($age) || $age <= 0) { 
      throw new InvalidArgumentException('$age must be a positive integer'); 
     } 

     $this->_age = $age; 

     return $this; 
    } 
} 

// then you can use dependency injection in your main search class 

class SearchService 
{ 
    public function search(SearchQuery $query) 
    { 
     // search 
    } 
}