true / false をセットするための関数名の付け方

Question

現在、ある機能を有効にするために、「enableFilter()」「disableFilter()」のようにオン・オフで別々の関数を用意しているのですが、引数で true / false を渡して１つの関数で「setFilterEnability(true)」のような形にしたいと考えています。

あまりそういう名前の関数名を見かけたことがないので他にもっと良い関数名の付け方があるとは思うのですが、このような boolean を渡して機能の有効・無効を切り替える関数はどのような名前にするのが定番でしょうか？

Comments

[tanishi_a]

.NET でも setVisibility() といったのがあるので、特に変なことでもないように思います。（これはtrue/falseでないので良い例ではないか）

Answer 1

set○○で良いのではと。「切り替える」ならswitchもありです。
なんなら定義の方に引数の型宣言を入れてあげると、区別もできます。

function setFilterEnability(bool $flg){

こうなると「Enability」という単語もなくていいかもしれませんね。
要件次第ではありますが。

codicを使うのも手です。

一応、PSR-2のようなガイドラインはありますので
目を通しておいても良いかもしれません。

Comments

[退会済みユーザー]

正しく表現しようとして文字数が増えてタイプミスを誘うよりも、
そこそこで妥協してわかりやすさも担保できればいいんじゃないかと思ったけど、
開発上の規約次第ですな。

[m.ts10806]

そうですね。
なかには「下手に難しい英語で表現しようとするより日本語ローマ字表記の方がいい」という考え方も一定ありますね。
今関わっている人、今後関わるであろう人でちゃんと認識が統一できればいいと思います。

[m.ts10806]

もし自分だけだったとしても「数ヵ月後の自分が理解できる」がひとつの指標です。