【設計】入力値チェック・バリデーション

オブジェクト指向を学んでいる者です。
基礎的な質問で申し訳ありませんが、設計について悩んでいて、相談したいです。

みなさんは、設定値のバリデーションは処理中のどこで行っていますか？
自分は、GUIアプリケーションなどでは、入力された時に入力値チェックを行います。
一方で、DataBaseや外部ファイルの値を取得して動作するアプリケーションの場合、どこでその値をチェックするか悩んでいます。

具体的には、
１　外部ファイルやDataBaseから設定値を取得：get～～()
２　設定値を使用するクラスへ値をセット（初期化）：init()
３　セットされた設定値を使用し、処理を実行：execute～～()
（get***()などはメソッド名の例です）

この流れの時、
②の値をセットする時にバリデーションをするか
③のその値を使用する時にバリデーションをするか
と考えていますが、みなさんはどちらのタイミングで行いますか？

悩んでいる理由としては、
値は基本的に「使用するクラス」がチェックすべき、ということを教わったことがあるのですが、
ではチェックするタイミングとしては、
セットされた時？使用する時？と悩んだ次第です。

自分としては、それぞれの対応のメリットとして以下のようなものを考えています。
②でチェックする場合は、エラーなら初期化の時点で終わるので以降の処理が無駄にならなくて済む。
③でチェックする場合は、使用する時にチェックしているので、確実にその値が妥当だと明示できる。

もしくは、両方のタイミングで行うべきなのでしょうか？
しかしそれは冗長な気もして…質問させていただきました。

よろしくお願いいたします。

m.ts10806

2018/06/02 23:16

具体的なコード例を挙げられた方が的確な回答を得られると思います。また、入力チェックの結果を「どのタイミングで」「どのように出したいか」によっても違ってくるように思います。そういった要件部分も具体的に記載してください。

行動規範の内容に同意します

回答3件

入力時のチェックは、範囲とか妥当性などきっちりしたチェックを行って、
使用するときにチェックするのはあくまでも実行に際して例外が出ないか、実行に破綻が出ないかという程度のチェックしかしないですね

そもそも、入力時以外のチェックって、それでダメでたときにどーすんの？という話になると思いますが、ダメなときにそのレコードを削除するのか、ダメな項目をテキトーな値に置き換えるのか、ダメな値のまま通すのか、そこらへんの仕様によってやり方がかわりますわな。

投稿2018/06/03 01:23

y_waiwai

総合スコア87774

退会済みユーザー

2018/06/17 11:56

ログインできず、返答が遅れて申し訳ありません。 > そもそも、入力時以外のチェックって、それでダメでたときにどーすんの？という話になると思いますが、ご指摘の通りですね。そのチェックで不正になる場合、以降の処理はまったくできない、ということであれば早々にチェックする必要がありそうです。一方で、デフォルト値などに変更可能ならば、直前でも構わないように思えます。回答ありがとうございました。

行動規範の内容に同意します

こんにちは。私もよく悩みます。

質問文の範囲で考えると、2 のタイミングが妥当かなと考えます。
異常値で初期化されたクラスが正常に動作するとは思えませんので、3のメソッドがコールされた度に値の確認してエラーを通知（または例外のスロー）を行うのは現実的でないと思います。

y_waiwaiさんの回答と一部重複します、申し訳ありません。
こういう話をさんざん議論して結局『で、ダメなときどうすんの？』の一言で、最初に戻るパターンはままあると思います。
なんのためにチェックするのか（"単にクラスがそのレンジで値を必要としている", "データソースの破損を確認したい", "異常値の場合は処理をパスしたい"とか）がもう少し明確になれば、具体的な回答を得られると思います。

極論ですが...
データソースに異常値が存在すること自体が異常なのだから、そのデータソースにデータを入れた側に問題がある！だから受け側ではチェックしません！てのも一つです。

投稿2018/06/05 06:10

編集2018/06/05 06:11