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

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

新規登録して質問してみよう
ただいま回答率
85.48%
PHP

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

Q&A

解決済

2回答

557閲覧

クラスのメソッドにおいて、どこで引数チェックをするべきなのか。

退会済みユーザー

退会済みユーザー

総合スコア0

PHP

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

0グッド

1クリップ

投稿2020/02/11 14:04

編集2020/02/11 14:33

質問したいこと

クラスのメソッドにおいて、どこで引数チェックをするべきなのか。

詳細

例えば

  • ユーザーからの入力を受け取り、保存するようなメソッド(下記のsaveメソッドのようなもの)

php

1class Hoge_Controller() 2{ 3 public function save($inputs) 4 { 5 $errors = // バリデーション 6    7 if (!empty($erros)) { 8 return $errors; 9 } 10 11 // DBに保存 12 } 13}

であれば、ユーザーからの入力は基本的に信用するべきでは無いと思うので、バリデーションで引数をチェックするのが一般的かなとは思っているのですが、

  • モデルにあるprivateメソッドのようなアプリケーションの内部で呼ばれるメソッド

php

1class Hoge_Controller 2{ 3 public function save($inputs) 4 { 5 $hoge = new Hoge_Model($inputs); 6 7 $errors = $hoge->validate(); 8 if (!empty($errors)) { 9 return $errors; 10 } 11 12 // 適当に考えた処理です。 13 $number = rand(); 14 $hoge->set((string)$number . (string)$hoge->calc($number)); 15 16 $hoge->save(); 17 } 18} 19 20class Hoge_Model 21{ 22 // 2乗して2倍にして返す 23 public function calc($number) 24 { 25 $number = $this->square($number); 26 $number = $this->double($number); 27 28 return $number; 29 } 30 31 protected function square($number) 32 { 33 return $number * $number; 34 } 35 36 protected function double($number) 37 { 38 return $number * 2; 39 } 40} 41

の引数に対して、どの段階でチェックを行うべきなのかがイマイチよく分かっていません。

$numberに対する引数チェックを行うべきパターンとしては、

  1. calc だけで行う (呼び出し側で行う)
  2. square, double の2箇所で行う (呼び出される側で行う)
  3. calc, square, double の3箇所で行う (どっちも行う)

の3パターンがあるかと思います。

1にした場合、squareやdoubleにnullとか配列を渡せばバグってしまったり、間違った使い方が出来てしまう問題があります。が、アプリケーション的に見れば問題ない気もします。

2にした場合は、これが自分的には一番しっくりきているし、あまり問題を感じません。

3にした場合は、やりすぎかなと感じるのですが、それぞれ単体のメソッドとしてみると、正しい値だけに処理を実行する設計になるので、理論的には正しい気がしています。

という感じで、メソッドを新しく作るときに結構悩んでいます。
みなさんはどうしているのか、意見をお聞きしたいです。
また自分の考えにおかしなところがあればご指摘頂けると幸いです。

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

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

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

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答2

0

ベストアンサー

個々のメソッドの設計によります。
本質的にメソッドとは(public だろうが private だろうが)ブラックボックスなのであって、in から out が定まり、かつそれが公表されているべきです。
つまり、「想定外のデータが来た場合にどのような動作をするか」は、メソッドごとに定義するべきものでしょう。

そう考えると、2. が普通かなとは思います。
ただし、想定外データを入力したことによるエラーハンドリングは呼び出し元で行わねばならないでしょう。
それが例えば戻り値チェックなのか、あるいは例外キャッチなのかは好みもあるでしょうが。

投稿2020/02/12 05:08

編集2020/02/12 05:14
tacsheaven

総合スコア13703

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

0

save()メソッドの中で$inputsをどうしたいのかによります。
無条件に保存するだけならバリデーションは不要です

投稿2020/02/12 03:46

yambejp

総合スコア114814

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.48%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問