質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

90.51%

  • PHP

    20352questions

    PHPは、Webサイト構築に特化して開発されたプログラミング言語です。大きな特徴のひとつは、HTMLに直接プログラムを埋め込むことができるという点です。PHPを用いることで、HTMLを動的コンテンツとして出力できます。HTMLがそのままブラウザに表示されるのに対し、PHPプログラムはサーバ側で実行された結果がブラウザに表示されるため、PHPスクリプトは「サーバサイドスクリプト」と呼ばれています。

  • Laravel 5

    1885questions

    Laravel 5は、PHPフレームワークLaravelの最新バージョンで、2014年11月に発表予定です。ディレクトリ構造がが現行版より大幅に変更されるほか、メソッドインジェクションやFormRequestの利用が可能になります。

  • HTTP

    547questions

    HTTP(Hypertext Transfer Protocol)とはweb上でHTML等のコンテンツを交換するために使われるアプリケーション層の通信プロトコルです。

  • Model

    82questions

    MVCモデルの一部であるModelはアプリケーションで扱うデータとその動作を管理するために扱います。

requestに応じた検索をするmethodの引数の多さ、

受付中

回答 4

投稿

  • 評価
  • クリップ 0
  • VIEW 406

hotsplus

score 656

現在はlaravel5を使って開発をしているのですが、どのFWにもどのPHPのコードにも言えると思っています。

検索BOXなどで来た検索queryに対して結果を返すmethodをmodelなどで定義すると思うのですが、
検索条件が3こ程度なら良いと思いますが10ことかになるととてもmethodが見にくくなってしまいます。

その場合はどのようにしたらいいのでしょうか?現状に違和感を感じるのですが解決策が思いつきません。

article table

  • id
  • title
  • description
  • body
  • mail
  • created_at
  • updated_at
  • status
  • published_at
  • push_flag

適当な記事を扱うテーブルがあったとして
全てを検索できるような一覧画面があり、例としてtitleの検索やstatusの検索、published_atの期間指定などを行いたいと思っています。

public function index(Request $request){
    $ret = Article::search(
        $request->get("id", null),
        $request->get("title", null),
        $request->get("body", null),
        $request->get("mail", null),
                 ︙
        )
    dd($ret)
}
public static function search($id, $title, $body, $mail ....たくさん){
     return static::where("id", "=", $id)->where.....たくさん -> get();
}

沢山の引数を持ったmethodができるの普通でこんなものなのでしょうか?
よろしくお願いします。

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 4

+6

多数の条件等で引数が多い場合は、受け渡し専用のConfigクラスなどを作ってそれを渡してあげるという手もあります。
(好みによっては連想配列でも良いです)

class SearchConfig{
    public $id;
    public $title;
    public $body;
    public $mail;
}

whenメソッドを使って条件値がある場合に、条件追加をするとある程度すっきりするかと思います。

    public static function search(SearchConfig $config){
        return static::when(!empty($config->id),function($q) use($config){
            $q->where("id", "=", $config->id);
        })->when(!empty($config->$title),function($q) use($config){
            $q->where("title", "=", $config->$title);
        });
    }

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2017/09/13 21:03

    なんとなく連想配列よりこっちのほう僕はスッキリして好きです。
    やったことないのでわからないのですが引数にするClassの設計が入ってくるのでややこしくなるのかな~と

    ちょっと前に$this->fill()でどうにか出来ないかなと思ったんですがダメでした;c;
    回答ありがとうございます!

    キャンセル

  • 2017/09/13 21:10

    クラスで定義したほうがコード量が増えますがIDEの補完が効いたり、より堅牢な感じにはなるかと思います。

    キャンセル

+3

ふつう、そのような場合には、「1個の連想配列」でまとめて検索条件を受けるような引数を作ります。

名前で指定できるので、どれを何番目にするかという混乱もありませんし、好きな分だけ指定できます。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2017/09/13 20:55

    やっぱりそうですか!

    何が指定できるのかとかがわかりにくくなると思うんですが
    そこら辺はPHPDOCとかコメントでカバーなんでしょうか?

    キャンセル

  • 2017/09/13 20:56

    そうですね、ドキュメンテーションによるフォローは必須でしょうね。

    キャンセル

  • 2017/09/13 21:04

    回答ありがとうございました!

    そこら辺応用して書いてみようと思います。

    キャンセル

+2

PHP7 で堅牢なコードを書く - 例外処理、表明プログラミング、契約による設計 / PHP Conference 2016

パラメータの渡し方に関して、非常にためになる概念が記述されているので、何度も読み返している資料です。
参考になれば。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

+1

お手軽オブジェクト指向というコンセプトで、「検索パラメーター」はArticleクラスのプロパティとして捉える場合は、下記のようにするとわかりやすいかもしれません。

検索パラメーター変数にstdClassを作成して、プロパティとするか、プロパティの初期化をどこでするのか、オブジェクトとしないで連想配列とするかはクラスの設計次第だと思います。

public function index(Request $request){
    //Articleクラスの検索パラメーターのプロパティとする(初期化はArticleクラスでやったほうがいいかもしれません)
    Article::$searchParams = new stdClass();

    Article::$searchParams->id    = $request->get("id", null);
    Article::$searchParams->title = $request->get("title", null);
    Article::$searchParams->body  = $request->get("body", null);
    Article::$searchParams->mail  = $request->get("mail", null);
    ...

    $ret = Article::search();
    dd($ret)
}
public static $searchParams;

public static function search(){
     return static::where("id", "=", static::$searchParams->id)->where.....たくさん -> get();
}

Articleクラスに検索パラメーター設定用メソッドでセットする方法でもよいかもしれません。個人的には下記のようにスタティックメソッドを何度も呼ぶ方法はあまり好きではないです。

public function index(Request $request){
    //Articleクラスに検索パラメーター設定用メソッドでセット
    Article::setSearchParams('id',    $request->get("id", null));
    Article::setSearchParams('title', $request->get("title", null));
    Article::setSearchParams('body',  $request->get("body", null));
    Article::setSearchParams('mail',  $request->get("mail", null));
    ...

    $ret = Article::search();
    dd($ret)
}
protected static $searchParams;

//検索パラメータープロパティ設定
public static function setSearchParams($key, $value){
    if (! is_object(static::$searchParams))
        static::$searchParams = new stdClass();
    }
    static::$searchParams->{$key} = $value;
}

public static function search(){
     return static::where("id", "=", static::$searchParams->id)->where.....たくさん -> get();
}

連想配列にまとめる考え方と同じコンセプトで、オブジェクトを使いたい場合「検索パラメーター」変数にstdClassのプロパティとして保存することも可能です。ループでセットしているのは単純な例として書きましたので、ひとつひとつセットしても良いと思います。

public function index(Request $request){
    $searchParams = new stdClass();

    foreach(['id','title','body',...] as $key) {
        $searchParams->{$key} = $request->get($key, null);
    }

    //検索パラメーター変数を渡す
    $ret = Article::search($searchParams);
    dd($ret)
}

どうしても、Article::search()メソッドの引数がよい場合はfunc_get_args()でメソッド引数の配列を受け取ることができます。オブジェクト指向という観点からだといまいちだと思いますが、可変長の引数を簡単に取得できます。

public static function search(){
     list($id,$title,...) = func_get_args();
     return static::where("id", "=", $id)->where.....たくさん -> get();
}

どうするかは、ある程度クラス設計上の意味を考えたうえで、パフォーマンスとメンテナンス性を重視して決めると良いと思います。

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 90.51%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る

  • PHP

    20352questions

    PHPは、Webサイト構築に特化して開発されたプログラミング言語です。大きな特徴のひとつは、HTMLに直接プログラムを埋め込むことができるという点です。PHPを用いることで、HTMLを動的コンテンツとして出力できます。HTMLがそのままブラウザに表示されるのに対し、PHPプログラムはサーバ側で実行された結果がブラウザに表示されるため、PHPスクリプトは「サーバサイドスクリプト」と呼ばれています。

  • Laravel 5

    1885questions

    Laravel 5は、PHPフレームワークLaravelの最新バージョンで、2014年11月に発表予定です。ディレクトリ構造がが現行版より大幅に変更されるほか、メソッドインジェクションやFormRequestの利用が可能になります。

  • HTTP

    547questions

    HTTP(Hypertext Transfer Protocol)とはweb上でHTML等のコンテンツを交換するために使われるアプリケーション層の通信プロトコルです。

  • Model

    82questions

    MVCモデルの一部であるModelはアプリケーションで扱うデータとその動作を管理するために扱います。