ブールメソッドの命名の可読性

読みやすさの観点から、ブールメソッドにどのメソッド名を好むかという簡単な質問：

public boolean isUserExist(...)

または：

public boolean doesUserExist(...)

または：

public boolean userExists(...)

public boolean userExists(...)

私の好みでしょうか。それはあなたの条件チェックを自然な英語にはるかに似せるので：

if userExists ...

しかし、私は厳格で迅速なルールはないと思います-一貫しているだけです

私の言うところはuserExists、私の呼び出しコードは90％の確率で次のようになるためです。

if userExists(...) {
  ...
}

そしてそれは文字通り英語で読む。

if isUserExistそして、if doesUserExist冗長に思えます。

読みやすさを追求しながら、明快さを犠牲にすることに注意してください。

はif (user.ExistsInDatabase(db))読みやすくなっていますif (user.CheckExistsInDatabase(db))が、ビルダーパターンを持つクラス（または状態を設定できるクラス）の場合を考えてみます。

user.WithName("Mike").ExistsInDatabase(db).ExistsInDatabase(db2).Build();

それExistsInDatabaseが存在するかどうかを確認するのか、それとも存在するという事実を設定するのかは明確ではありません。あなたは書きませんでしょうif (user.Age())か、if (user.Name())いずれかの比較値なしで、なぜですif (user.Exists())そのプロパティ/関数はブール型であり、あなたがより自然な英語などを読み取るために機能/プロパティの名前を変更することができますので、純粋に良いアイデア？ブール値以外の他のタイプに使用するのと同じパターンに従うのはとても悪いですか？

他のタイプでは、ifステートメントは関数の戻り値をコード内の値と比較するため、コードは次のようになります。

if (user.GetAge() >= 18) ...

これは、「ユーザードットゲットエイジが18以上の場合...」と読みます。これは「自然な英語」ではありobject.verbませんが、自然な英語に似たことはないと私は主張しますが、これは単に現代のプログラミングの基本的な側面です（多くの主流言語）。プログラマは通常、上記のステートメントを理解するのに問題はありません。

if (user.CheckExists() == true)

これは通常短縮されます

if (user.CheckExists())

致命的なステップが続きます

if (user.Exists())

「コードは書き込まれるよりも10倍頻繁に読み取られる」と言われていますが、バグを見つけやすいことも非常に重要です。オブジェクトを存在させ、成功に基づいてtrue / falseを返すExists（）という関数があるとします。コードif (user.Exists())を簡単に確認でき、バグを特定できませんでしたif (user.SetExists())。たとえば、コードを読み取れば、バグはより明確になります。

さらに、user.Exists（）には、複雑なコードや非効率なコードを簡単に含めることができ、データベースにラウンドトリップして何かをチェックします。user.CheckExists（）は、関数が何かを行うことを明確にします。

ここですべての応答も参照してください：命名規則：ブール値を返すメソッドの名前は何ですか？

最後の注意として、「Tell Do n't Ask」に続いて、true / falseを返す多くの関数がいずれにしても消え、オブジェクトにその状態を要求する代わりに、何かを実行するように指示します。その状態に基づく方法。

読みやすさの目標は、常に自然言語に最も近いコードを書くことです。したがって、この場合userExistsは、最良の選択のようです。プレフィックス「is」を使用しても、たとえば、別の状況では正しい場合がありisProcessingCompleteます。

userExists（）を使用するのは、1）自然言語で意味があり、2）これまでに見たAPIの規則に従っているためです。

自然言語で理解できるかどうかを確認するには、読み上げてください。「ユーザーが存在する場合」や「ユーザーが存在する場合」よりも、有効な英語のフレーズのように聞こえます。「ユーザーが存在する場合」の方が適切ですが、「the」はおそらくメソッド名では不要です。

Java SE 6にファイルが存在するかどうかを確認するには、File.exists（）を使用します。これは、バージョン7でも同じように見えます。C＃が使用する同じ規則を、として行うのPythonとRubyのを。うまくいけば、これはこれを言語にとらわれない答えと呼ぶのに十分多様なコレクションです。一般的に、私はあなたの言語のAPIに合わせてメソッドに名前を付けることに賛成です。

ここで他のいくつかの回答に見落とされたと思うことを考慮する必要があります

1. これがC ++クラスメソッドかC関数かによって異なります。これがメソッドである場合、呼び出されるif (user.exists()) { ... }かどうかif (user.isExisting()) { ... }
   は不明if (user_exists(&user))です。これがコーディング標準の背後にある理由です。ステートboolメソッドは、オブジェクトがその前にあるときに文のように読み取られるため、動詞で始まる必要があります。

2. 残念ながら、多くの古いC関数は、成功すると0を返し、失敗すると0以外を返すため、すべてのbool関数を動詞で始めるか、常にtrueと比較しない限り、使用されているスタイルを特定するのは困難です。 if (true == user_exists(&user))

この質問に対する私の簡単なルールは次のとおりです。

booleanメソッドが動詞をすでに持っている場合は、動詞を追加しないでください。それ以外の場合は、それを考慮してください。いくつかの例：

$user->exists()
$user->loggedIn()
$user->isGuest() // "is" added

純粋に主観的。

userExists(...)このようなステートメントが読みやすくなるので、私は好む：

if ( userExists( ... ) )

または

while ( userExists( ... ) )

この特定のケースでは、最初の例はひどい英語です。

if文でそれを読んだときの音の理由から、おそらく3番目に行くでしょう。「ユーザーが存在する場合」よりも「ユーザーが存在する場合」の方が音が良い。

もちろん、ifステートメントテストで使用されることを想定しています...

私はこれらのどれかが好きです：

userExists(...)
isUserNameTaken(...)
User.exists(...)
User.lookup(...) != null

メソッド名は読みやすくするために役立ちます。コード全体に適合するものだけが最適です。ほとんどの場合、条件で始まるため、subjectPredicateは自然な文構造に従います。

では、プロパティの名前を変更してみませんか？

if (user.isPresent()) {