名前がアンダースコアで始まるメソッドについて

Question

作った人がもういないシステム（Laravelを使用）の改修を命じられてコードを呼んでいるのですが、よくわからないところがあります。
モデルクラスのメソッドの中に「名前がアンダースコアで始まる」メソッドがあります。全部ではなく一部です。
名前がアンダースコアで始まるメソッドは、例えば "_getAll()" のようにすべて "_get〇〇()" です。
何か意図があって一部だけアンダースコアで始まる名前にしたのかと思うのですが、その意図がわかりません。
ググると
・JavaScriptだとプライベートな関数、変数はアンダースコアで始めるという慣習があるよう
・CakePHPの古いバージョンではアンダースコアで始まるメソッドにするとプライベートメソッドになる。
という情報はありましたが、JavaScriptじゃないしCakePHPでもないし。それに件のメソッドは全部publicだし。
作った本人に聞ければいいのですがもう会社にいません。連絡先も知りません。
「PHPやLaravelでこういう場合にメソッド名をアンダースコアで始める」みたいなコーディングルールとか慣習とか、そういうのをご存じでしたら教えてください。

Answer 1

ごく一般論で言うと、プライベートメソッドでしょうけど、強制された保証のない規約を想定しても無意味です。

エディタやレビュアーがチェックしない場合、規約は機能しません。
たとえ機能していても、一人開発のオレオレ規約なんて、どこに例外が潜むかわかりません。

見切りをつけて、通常のメソッドと同じく扱い、可能であればリファクタリングの際に適切に修正してあげると良いです。

Comments

[m.ts10806]

ですね。
いない人のオレオレ仕様に悩む時間がもったいない

[intron]

> ごく一般論で言うと、プライベートメソッドでしょうけど、強制された保証のない規約を想定しても無意味です。

これがパブリックメソッドなんです。
「保証のない規約を想定しても無意味」と言われれば、そうですね。想像することしかできないし、その想像が合っている保証もないし。
悩むのはやめて改修のついでにリファクタリングしてメソッド名を変更します。
回答ありがとうございました。

Answer 2

それに件のメソッドは全部publicだし。

そもそも、private なメソッドはありますか？

その人がpraivateなメソッドも作り、publicなアンダースコアで始まるメソッドも作っているのであれば謎ですね。

ちなみに、アンダースコアで始まるメソッドはクラス外から呼ばれていますか？
そのへんから探るしかなさそうな。。

php4まではアクセス修飾子が無かったので、
「アンダースコアで始めたメソッドはprivateってことな！」っていう慣習がメジャーでした。
その名残りかと思いますが。。

Answer 3

実際のコード提示されないとなんとも言えませんが
ハンガリアン記法の類なのではないかと。
その作った方のルールでアンダースコアつけてただけかなと。

あくまで推測しかできないので、どうしても知りたければ作った人にどんな手段でもいいので連絡を取って聞くしかありません。
引継ぎ資料が全くないとは思えませんが、ないのなら聞くしかありません。
会社が全く連絡先知らないわけではないと思いますよ。