新しい行のパラメータを持つメソッドを呼び出すことは悪い習慣と考えられますか？

Question

私はメソッドに複数のパラメータを渡すためのベストプラクティスが何であるかについての好奇心が強いです。具体的には、新しい行のコメントやパラメータを持つメソッドを呼び出す方が悪い習慣であると思われるかどうかは不思議です。

// get json from database 
$json = $this->mysql->get_json_from_entries($this->table, // data table 
              $foo,   // column 
              $bar,   // column value 
              $this->limit); // maximum rows to retrieve 

は個人的に

// get json from database 
$json = $this->mysql->get_json_from_entries($this->table, $foo, $bar, $this->limit); 

Answer 1

その味の問題ではなく、通常、私はそのすべてを1行上を見てきたものから、その後、あなたはそうのようなあなたのDOCのブロックを使用します。

/** 
    * Get json from database 
    * 
    * @param $this->table object data table 
    * @param $foo string column name 
    * ...etc 
    */ 
$json = $this->mysql->get_json_from_entries($this->table, $foo, $bar, $this->limit); 

Answer 2

とは反対に、私は本当に長い行が悪化していると思います。

あなたは、より一般的にZend Framework Coding Standard for PHPに興味があるかもしれません。

Answer 3

私は通常のベストプラクティスの方に傾くのが好き、時には（関数の引数が多すぎる場合）、私も最初のスタイルを優先します。しかし

2物事： 機能があまりにも多くのパラメータがある場合、あなたはそれをリファクタリング検討するかもしれない、多分新しいクラスにパラメータを置くか、クラス自体 第二の実装を変更、冗長な名前の変数を宣言しようとすると、その使用方法を説明しています。ヒント：私はそれが悪い習慣だとは思わないが、あなたは間違いなくあなたのインターフェイスをドキュメント化する必要があります:)