変数名の命名について

Question

初歩的な質問で申し訳ないですが、真剣に悩んでおります。
お力添えよろしくお願いいたします。

前提・実現したいこと

特定の処理(たとえば検索)が可能なクライアントが存在するかを判断するための変数として、以下のような変数を定義しました。

isSearchableClientExist

上記変数についてですが、先輩から良くないと指摘があり、何が良くないのかを考えておりますが明確に何が悪いのか自分では理解できませんでした。

変数名の命名規則に沿っていないから良くないのか、そもそも英語の文法的に変なのか、今の私では理解ができませんでした。

そのためアドバイス等いただけないでしょうか。
初心者のため、的外れな質問かもしれませんがよろしくお願いします。

Comments

[Zuishin]

先輩に理由を聞いて、それを追記してください。
動詞が二つあるので英語の文法的には変ですが、それ以外に職場の規約であったり、他のコードとの兼ね合いであったりなどの理由がある可能性があります。

[gentaro]

ここで質問すること自体が的外れで、指摘した本人に聞けばその場で解決じゃね。

Answer 1

先輩から良くないと指摘があり

指摘した人にしか意図は分からないので、その先輩とやらに聞いてください。
赤の他人に聞いたところで、その先輩の意図が分からない限りはほぼ意味がありません。
観点を確認したうえで考えないと自身でも答えは出ないと思います。

Answer 2

stackoverflow に類似の質問（これはメソッド名ですが。）があり、これのベストアンサーから考慮すると、

• searchableClientExists

がよいということになりますが、boolean の変数名には接頭辞にisを付けるというのがチームの慣習であるという場合もあったりするので、上記回答のコメントにも例示されていますが、exist　を現在分詞(…ing)にして

• isSearchableClientExisting

とするのもよいかもしれません。少なくとも is で始めるなら、動詞の原形のままのexistだと、英語としてちょっと不自然かなという気もします。

もしくは、これは私の思いつきですが

• hasSearchableClients

というのも、has(持っている) の主語となるものが明確であるコンテキストの中のでは、ありかもしれません。

Answer 3

検索の処理が他の場面でSearch以外の表し方をされていたり、
「クライアント」が他の場面で別の表し方をされていたりするのかもしれません。
また、SearchableでないClientが存在しないのであれば、isClientExistでもいいかもしれません。
そもそもis_searchable_client_existを期待しているかもしれませんし、isSchAblCliExtのような頭のおかしいトンチの効いた変数名を好んでいるのかもしれません。
文法的にはisSearchableClientExistsが正しい気がしますが、個人的にはプロジェクト内で整合性が取れていればどうでもいいと思いますし、主体によってExistとExistsを分けるべきではないと考える人もいるかもしれません。

先輩に直接聞くか、先輩が書いたソースコードの変数と比較してみてはいかがでしょうか。