判定するメソッドにつけるメソッド名

###質問が抽象的すぎたので具体的な例に返させていただきました。
既に回答を頂いた方々、すみません。

###前提・実現したいこと
例えば、ゲームのキャラクターの属性を判定するメソッドとして、
力と早さの2つのステータス値を比較してどちらが大きいか、又は同じ値かで、
それぞれに対応する属性を返すメソッドにつける名前。

###該当のソースコード

java
1/**
2 * 力と速さを比較し、キャラクターの属性を判定します。
3 *
4 * @power 力
5 * @speed 速さ
6 * @return 属性
7 */
8private String hogehoge(int power, int speed){
9
10    if(power < speed){
11        return "SpeedType";
12    } else if(power > speed) {
13        return "PowerType";
14    } else if (power == speed){
15        return "BalanceType";
16    }
17}

###試したこと

TAPEを取得するメソッドなので、getTyep()

※ getHogeHoge は有るフィールドから値を取ってくるイメージで、大小比較が入っているので違うのかな？

TYPEを決定するメソッドなので、decideType() or judgeType()

※ メソッドの接頭語として、decide, judge を見たことがないので自信なし。

TYPEが何になるかチェックするので、checkType()

※ checkHogeHoge は判定をするメソッドにつけたりしますが、booleanを返すような判定につけるイメージ。

とどれもイマイチ違うような気がしています。
軽く調べてもドンピシャの回答を得られなかったので皆様のご意見を伺いたいです。

以下追記

comparePowerWithSpeed()

※ 既に頂いた回答から、compareを使うと上記のようになるかなと思うのですが、
比較することよりも、属性を判定することを表したほうが良いかと思っております。
また、力・速さに加えて、技も判定対象になるばらば、comparePowerWithSpeedAndTechnic()
の用になり、比較する値が多くくなるに連れ、めちゃめちゃ長いメソッド名になってしまうので...。

行動規範の内容に同意します

回答2件

ベストアンサー

こんにちは。

原則として、名前はあまり中身に拘らない方が良いと思います。
実装を外部から隠すことは、好ましい場合が多いですので。
とは言え、max()やmin()とか、equal()的なものはそういう訳にもいかないので、ケースバイケースですが。

今回の場合、もし、a<b, b>a, a==bの３つのパターンに分類することが目的の関数なら、私ならcompare()を選びそうです。戻り値は-1, 1, 0とか、enum型で3つを判別等ですね。

そうではなくて、Type-1, Type-2, Type-3がたまたまその３つの状況の場合であり、今後増えていく可能性があるのであれば、getTypeName()のような感じにするだろうと思います。
文字列は比較する時にタイプミスしてもエラー・チェックが働かないので、enum型を返却したくなることが少なくないです。なので、その時困らないようにNameやStringを付けることも多いです。

checkType()がboolean返却というイメージには同意です。

【質問の編集を踏まえて】
私なら、getType()ですね。
そして、その戻り値をプログラムで判定して分岐するでしょうから、enum型を返却すると思います。
enum型なら分類の拡張にも対応できますし、その分岐時に例えば"Type-1"を"Tyep-1"などとタイプミスしてもコンパイラが検出してくれますし。

投稿2017/02/25 02:47

編集2017/02/25 04:32