メソッド名と、メソッドの戻り値を入れる変数名 が同じ

Question

例えば、以下の様なコードです

boolean xxxExists = false;
xxxExists = xxxModel.xxxExists(hoge, hogehoge);

Javaの仕様では何も問題はないと思いますが、

変数名、メソッド名のいずれか変更し、異なる名称とすべきではないかと思うのですが、いかがでしょうか？
むしろ同じ名称の方がわかりやすい？

みなさまのご意見お待ちしております

Answer 1

xxxExistence = xxxExists();

どんな命名規約を採用するかにもよりますが、
私は原則的に「異なる名称とすべき」派です。
その上で、メソッド名は動詞、変数名は名詞が命名の基本と考えます。

なぜ使い分けるのか、かんたんに言いますと、
区別できる情報は多い方が単純に良いから。
デバッグ時にカッコの付け忘れかどうかも、ひと目で判別できます。

またたとえば完全に同名だと、検索のときに両方引っかかります。
もし全体で100個もあれば、けっこう分かりにくいだろうと。

まあRubyだと、カッコを省略し、変数名とメソッド名を曖昧化して、
トリッキーな呼び出しを実現したりもしますが、これは例外です。

Answer 2

私は戻り値がboolであるなら、直接if判定させます。

if (xxxModel.xxxExists(hoge, hogehoge)){
    ...
}

boolじゃなくとも直接渡すようにしてコードを短くしています。
メソッドの中身が十分に短けれは、意図が判りやすいので変数名はある程度いい加減でも読み難くならないと思います。そういうわけで、String sやint iなどもよく使いますし、上記の例であれば変数の定義をbool existsあたりにすると思います。

逆にルーブが複雑だと、int xxxIndexなどの説明的な変数名を使います。そういう場合はメソッドやクラスの分割を検討するようにしていますので、あまり機会は多くありません。

いずれにせよメソッド名と変数名はあまりかぶらないようにしています。こうということも無いのですが、私もやはり変数名とメソッド名が被るのは好きではありません。