validate関数throwExceptionIf ...という名前を付ける必要がありますか？

Question

私は、同じ命名規則validateSomething（）を使用するコード内に多くのプライベートな検証関数を持っています。 validateが失敗した場合、そのうちのいくつかは例外をスローします。 throwExceptionIfSomethingInvalid（）の名前を変更してコードを読みやすくします。しかし、誰かが新しい名前はあまりにも奇妙で、混乱させてしまうと主張する。あなたはそれについて私にいくつかの助言を与えるのを助けることができますか？

Answer 1

「正確な」回答はありません。条約のみ。あなたが選んだ人はそれほど重要ではありません。より重要な点は、あなたが選んだ名前付けポリシーは何ですか：

1. あなたはそのことに固執する必要があります。ここでは1つの種類の「命名方式」を使用し、もう1つを使用しないでください。
2. 他の人とチームを組んでいる場合：チームの皆様にと同じがあることをご確認ください。

他の言葉：あなたはPOLA（最小の驚きの原則）に従いたいと思っています。あなたのコードの読者は、メソッドの名前があなたの文脈では、メソッドが何をするかを正確に示していることを常に知るべきです。

のほかに、「私のポリシーは」checkSomethingかvalidateSomethingElseのように起動しのボイドメソッドは常にチェックが失敗した場合に例外をスローすることが一般的である（よく：何がボイドメソッドが例外をスローする以外に行うことができます） 。

ブールと対照的に、isSomethingValid()程度です。

Answer 2

しかし、誰かが新しい名前は、私は同意

あまりにも奇妙であると主張しています。メソッドがチェック例外をスローできるように指定する正しい方法は、それをメソッドシグネチャの一部として宣言することです。例：あなたがこれを行うと

1. validateName()throws InvalidInputException
2. validateAge()throws InvalidInputException

、呼び出し側は常に例外を処理以上のモジュールにそれを伝播するか強制されます。

これを見るもう1つの方法は、あなた自身にこのことを尋ねることです：値void doSomethingWithoutReturnを返さないメソッドを指定しますか？署名の内容がvoidであることを確認したときに、このメソッドが何も返さないことがわかっているからです。