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

Question

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

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

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

/**
 * 力と速さを比較し、キャラクターの属性を判定します。
 *
 * @power 力
 * @speed 速さ
 * @return 属性
 */
private String hogehoge(int power, int speed){

    if(power < speed){
        return "SpeedType";
    } else if(power > speed) {
        return "PowerType";
    } else if (power == speed){
        return "BalanceType";
    }
}

###試したこと

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

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

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

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

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

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

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

以下追記

• comparePowerWithSpeed()

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

Answer 1

こんにちは。

原則として、名前はあまり中身に拘らない方が良いと思います。
実装を外部から隠すことは、好ましい場合が多いですので。
とは言え、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"などとタイプミスしてもコンパイラが検出してくれますし。

Comments

[usukuti_hanako]

せっかく回答いただいたのにすみません。すこし、質問文を修正させていただきました。
よろしければ、修正後の内容で改めてご意見いただけたらと思います。

[usukuti_hanako]

ありがとうございます。
getType()とすることにしました。
enum型を返却するよにするご意見もありがとうございます。
確かに、文字てるリテラルを手打ちして返却・判定などしていると、打ち間違いをよく起こします。
ご意見ありがとうございました。勉強になりました！

Answer 2

[compare](https://docs.oracle.com/javase/jp/6/api/java/util/Comparator.html#compare(T, T))

Javaではcompareメソッドで-1,0,1を返します。

int compare(T o1, T o2)

compareXXXでよいのではと思います。

Comments

[usukuti_hanako]

せっかく回答いただいたのにすみません。すこし、質問文を修正させていただきました。
よろしければ、修正後の内容で改めてご意見いただけたらと思います。