Java定番レビューgetという名前のついたメソッドで、検索処理を書いているのをよく見かけます。昔からできるだけ直してもらうように提案してきました。深く考えずにメソッド名を付けていただけとかであれば、ちょっと違う名前にして欲しいなと。内部で持っているインスタンス変数をただ戻すだけのメソッドというイメージあります。紛れて、ぱっと見でループ内検索していることがちょっと気付きづらいんですよね。いわゆる「ぐるぐる系」です。それを発見しづらくなるという話です。読めばそりゃわかりますが、みんなでやるレビュー会とかって、そんなじっくり読めなかったりします。(他のテーマで時間たくさん取るから)もし、パッと見で気付ければ、「それ直しておいて」ですぐに終了。個人レビューでも、見るものはたくさんあるので、サラッと追うだけで気付けるコードってのは、すごく心強いのです。それができずに放置されると...後々パフォーマンス劣化トラブルとして、ボディブローのように効いてくるのです。...いろいろな処理}レビューの時とか、パッと見で「おおっと、なんかおかしいよ!?」ってのがすごく気付きやすい。なんでもいいですが、そこで...通信して別のサーバーのハードディスクにアクセスしているというのが直感的に分かれば。書いてる人も書いた後で「いやぁ、これはないなぁ」とか、自分で気付いてリファクタリングするきっかけになりやすいかなと。仮に、スタートアップの開発で実装スピード優先のため、ぐるぐる検索はいったん置いておくと判断をしたとしても、後でリファクタリングする人がすぐに発見しやすいようになっていればとまだ救い。(まあ、そもそもtodoコメント残しておいて欲しい)コードの解析ツールとかでも、発見しやすいかもしれませんしね。別のところから取っているのか?それをわざと隠蔽したいとかであれば別ですよ。要は、getってわりとボカした名前なわけです。いう場面はそんなに多くはない。あと、アーキテクチャ次第でもありますね。今の時代、IDEのリファクタリング機能があるので、メソッド名を直すのは簡単です。「ユーザの期待に合わせる」という欄に書いてありました。他の言語でも同じでしょうかね。レビューしてもらいやすいコードぼくはよく、レビューしてもらいやすいコードを書こうね、と若いプログラマーに教えています。一発で正確に動くコードを書けるようになる、ってのは当然みんな頑張ることではありますが、なかなか限度もあります。成長段階の若者であればなおさら。仮に間違ってたとしても、レビューしやすいコードだったら、誰か別の人が間違いに気付いてくれるかも。プルリクでコメントもらってそれで直るんなら、業務としては万々歳なわけです。メソッド名で意図を表現するというのは、レビューで間違いを見つけてもらいやすくするテクニックの中の一つなんじゃないかなって思ってます。JPOUGに登壇させていただきました。再開したディズニーシーにはてなブログをはじめよう!jfluteさんは、はてなブログを使っています。ブログを報告するスターをつけました引用をストックしましたストック一覧を見る閉じる引用するにはまずログインしてくださいログイン閉じる引用をストックできませんでした。再度お試しください閉じる限定公開記事のため引用できません。