質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

88.09%

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

解決済

回答 2

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 5,280

score 29

質問が抽象的すぎたので具体的な例に返させていただきました。

既に回答を頂いた方々、すみません。

前提・実現したいこと

例えば、ゲームのキャラクターの属性を判定するメソッドとして、
力と早さの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()
    の用になり、比較する値が多くくなるに連れ、めちゃめちゃ長いメソッド名になってしまうので...。
  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 2

checkベストアンサー

0

こんにちは。

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

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

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

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


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

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2017/02/25 13:22

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

    キャンセル

  • 2017/02/25 18:37

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

    キャンセル

0

compare

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

int compare(T o1, T o2)

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

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2017/02/25 13:22

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

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 88.09%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る