javaのことで聞きたい

Question

package model;
import dao.FollowDAO;

public class FollowLogic {



	public void execute(int userId, int followId){
		FollowDAO dao = new FollowDAO();
		dao.follow(userId, followId);
	}

	public int execute(int userId){ //エラーになる
			FollowDAO dao = new FollowDAO();
			int followCount = dao.getFollowCount(userId);
			return followCount;
		}

	public int execute(int userId){  //エラーになる
		FollowDAO dao = new FollowDAO();
		int followerCount = dao.getFollowerCount(userId);
		return followerCount;
	}

}

このエラーは名前を変える他に、何か対策はありますか？？
classを作ればよいのでしょうか？

Comments

[m.ts10806]

タイトルがざっくり大幅すぎて不適切に思います。また、エラーはどのようなエラーが出ているか記載してください。

[iwamoto_takaaki]

エラーメッセージを張り付けましょう。どういう意図をもって書いたのかも書きましょう。

Answer 1

同じ名前を付けていることが問題というよりは、名前の付け方が問題だと思います。
処理内容が分かるような名前を付けるべきです。

上から順に、

• フォローする関数
• フォロー数を取得する関数
• フォロワー数を取得する関数

だと思うので、それぞれ、

• void follow(int userId, int followId)
• int followCount(int userId)
• int followerCount(int userId)

とかでいいんじゃないでしょうか。

Comments

[m.ts10806]

僕も色々書きましたがこのやり方が最も正しいと思います。

Answer 2

回答としては「名前を変えなければならない」です。

問題の本質(大げさな表現ですが)は、
異なるものに同じ名前 execute を付けていることです。
同じものには同じ名前を、異なるものには異なる名前を付けましょう。

違う仕事をするのに、同じ名前では、コンパイラもプログラマも困ってしまいます。

Answer 3

どのようなエラーが出るのかは記載いただくとして、ご提示のソースから分かるのは

public int execute(int userId)

全く同じものが２つあるからでしょう。
引数の型や引数などが全く同じであれば重複メソッドとして扱われます。

追記

classを作ればよいのでしょうか？

要件はそれぞれ「フォローしているユーザー（フォロー）の情報」「フォローされているユーザー（ふぉ露和ー）の情報」で「ユーザー情報を取得」という要件は同じなので、メソッドを1つにして「フォローフラグ」を渡して判断すればよいと思います。

サンプル

    public int execute(int userId ,boolean followflg){
            FollowDAO dao = new FollowDAO();
            //フォローフラグtrueの場合「getFollowCount」を実行
            if(followflg){
               return dao.getFollowCount(userId);
            }else{
　　　　　　　  //フォローフラグfalseの場合「getFollowerCount」を実行
              return dao.getFollowerCount(userId);
            }
        }

  //呼び出し
  //フォローユーザーが欲しい場合
  FollowLogic.execute(userId,true);
  //フォロワーユーザーが欲しい場合
  FollowLogic.execute(userId,false);

※動作確認できてないのであくまでイメージとして捉えてください

#注記

あくまでサンプルソースは現状のコードだけに焦点をあてたその場限りの打開策とご理解ください。
既に　public void execute(int userId, int followId)　というメソッドがあります。
全体の要件によってはやはり同名メソッドがあること自体に問題が発生することもあります（nob.さんのおっしゃっているように）
その場合はおとなしく処理内容にあわせたメソッド名に変更することを推奨します。

Comments

[Yoshi--]

抽象的ですいませんでした。
エラーというのは、コンパイル前の話で名前が重複してしまうので
名前を変える以外に対策はないか？という意味でした
一つのメソッドでやる場合は例えばどのような手順でやればよろしいでしょうか？「フォローフラグ」を渡して判断が分かっていません。
理解度が低くて
すいませんでした。

[m.ts10806]

サンプルソース書きますので少々お待ちください。

[Yoshi--]

ありがとうございます。参考にやってみます。

[m.ts10806]

注記も熟読してください。あくまでサンプルコードは現状回避のみです。