java private method 引数

Question

public class Ranking {

	// メンバ変数宣言部分
	final int NUM_OF_COUNTRIES = 1000;
	int next;
	Country[] countries = new Country[ NUM_OF_COUNTRIES ];



	// コンストラクタ部分
	public Ranking() {
		next = 0;
	}

	/**
	 * 指定された国が配列の何番に登録されているか探し、その添え字を返す
	 * @param countryName 国(3文字のIOCコード)
	 * @return countryNameがcountryの何番に格納されているか、なければ-1
	 */
	// indexOfCountryメソッド部分
	public int indexOfCountry(String countryName) {
		for (int i = next - 1; i >= 0; i--) {
			if (countries[i].name.equals(countryName)) {
				return i;
			}
		}
		return -1;
	}

	/**
	 * IOCコードで指定された国に指定された色のメダルを1つ追加
	 * @param countryName 国(3文字のIOCコード)
	 * @param color メダルの色
	 */
	public void addMedal(String countryName, Color color) {
		// addMedal前半部分
		int index = indexOfCountry(countryName);
		if (index < 0) {
			countries[next] = new Country( countryName );
			index = next;
			next ++;
		}
		// addMedal後半部分
		if (color == Color.Gold) {
			countries[index].gold++;
		} else if (color == Color.Silver) {
			countries[index].silver++;
		} else if (color == Color.Bronze) {
			countries[index].bronze++;
		}
	}

	// sortResultsメソッド部分
	public void sortResults() {

		for (int i = 1; i < countries.length && countries[i] != null; i++) {
			int j = i - 1;
			Country c = countries[i];
			while (j >= 0 && compare_Medal(countries) == true ) {       //(cの方がcountries[j]よりメダル数比較で上位 ||
				countries[j+1] = countries[j];                            //(cとcountries[j]がメダル数比較で同位 &&
				j--;                                                     //cの国名は辞書順でcountries[j]の国名よりも前)
			}
			j++;
			countries[j] = c;
		}
	}

	/**
	 * @param country
	 * @return
	 */
	//cの国名は辞書順でcountries[j]の国名よりも前//compareTo => 左　-　右
	//(cとcountries[j]がメダル数比較で同位 &&
	//(cの方がcountries[j]よりメダル数比較で上位 ||
	private boolean compare_Medal(Country countries[]) {

		for (int i = 1; i < countries.length && countries[i] != null; i++) {
			int j = i - 1;
			Country c = countries[i];
			if (( ( (c.gold > countries[j].gold ) ||( c.gold == countries[j].gold  &&  c.silver > countries[j].silver  )|| 
			 ( c.gold == countries[j].gold  &&  c.silver == countries[j].silver  &&  c.bronze > countries[j].bronze )  || 
			(  c.gold == countries[j].gold  &&  c.silver == countries[j].silver  &&  c.bronze == countries[j].bronze  &&  c.name.compareTo(countries[j].name  ) < 0 ) ))) {                    //compareTo => 左 - 右                                
				return true;
			} 
		}
		return false;
	}

	// printResultsメソッド部分
	public void printResults() {
		// 集計結果を表示
		for (int i = 0; i < countries.length && countries[i] != null; i++) {
			System.out.print(countries[i].name);
			int sum = countries[i].gold;
			System.out.print("(" + countries[i].gold);
			sum += countries[i].silver;
			System.out.print("," + countries[i].silver);
			sum += countries[i].bronze;
			System.out.print("," + countries[i].bronze);
			System.out.println(")[" + sum + "]");
		}

	}

}

```
```
public enum Color {
	Gold,
	Silver,
	Bronze
}
```
```
public class Competition {
	public static void main(String[] args) {
		// Rankingインスタンスを作成
		Ranking rank = new Ranking();
		// サンプルデータを登録
		rank.addMedal("ITA",Color.Bronze);
		rank.addMedal("NOR",Color.Bronze);
		rank.addMedal("CAN",Color.Silver);
		rank.addMedal("CZE",Color.Silver);
		rank.addMedal("RUS",Color.Silver);
		rank.addMedal("RUS",Color.Gold);
		rank.addMedal("AUT",Color.Silver);
		rank.addMedal("CAN",Color.Gold);
		rank.addMedal("KOR",Color.Silver);
		rank.addMedal("NED",Color.Bronze);
		rank.addMedal("NOR",Color.Bronze);
		rank.addMedal("NOR",Color.Gold);
		rank.addMedal("USA",Color.Gold);
		rank.addMedal("NED",Color.Bronze);
		rank.addMedal("KOR",Color.Gold);
		rank.addMedal("RUS",Color.Bronze);
		rank.addMedal("NOR",Color.Gold);
		rank.addMedal("NED",Color.Gold);
		rank.addMedal("RUS",Color.Silver);
		rank.addMedal("NOR",Color.Silver);
		rank.addMedal("USA",Color.Bronze);
		rank.addMedal("CAN",Color.Gold);
		rank.addMedal("GER",Color.Gold);
		rank.addMedal("CAN",Color.Bronze);
		rank.addMedal("USA",Color.Silver);
		rank.addMedal("CAN",Color.Silver);
		rank.addMedal("USA",Color.Bronze);
		rank.addMedal("GER",Color.Gold);
		rank.addMedal("SUI",Color.Silver);
		rank.addMedal("GER",Color.Silver);
		rank.addMedal("GER",Color.Bronze);
		rank.addMedal("AUT",Color.Gold);
		rank.addMedal("RUS",Color.Bronze);
		rank.addMedal("AUT",Color.Silver);
		rank.addMedal("AUT",Color.Bronze);
		rank.addMedal("FRA",Color.Gold);
		rank.addMedal("NED",Color.Silver);
		rank.addMedal("FRA",Color.Bronze);
		rank.addMedal("CHN",Color.Gold);
		rank.addMedal("CHN",Color.Silver);
		rank.addMedal("BLR",Color.Gold);
		rank.addMedal("FRA",Color.Silver);
		rank.addMedal("SUI",Color.Gold);
		rank.addMedal("USA",Color.Gold);
		rank.addMedal("RUS",Color.Gold);
		rank.addMedal("POL",Color.Gold);
		rank.addMedal("JPN",Color.Silver);
		rank.addMedal("JPN",Color.Bronze);
		rank.addMedal("SWE",Color.Silver);
		rank.addMedal("SWE",Color.Bronze);
		rank.addMedal("NED",Color.Gold);
		rank.addMedal("FIN",Color.Silver);
		rank.addMedal("RUS",Color.Gold);
		rank.addMedal("SLO",Color.Bronze);

		// ランキングで並び替えて
		rank.sortResults();

		// 結果出力
		rank.printResults();
	}
}
```
```
public class Country {
	String name;
    int gold, silver, bronze;
    public Country (String countryName) {
        this.name = countryName;
    }
}
```
Ranking class のsortResults()のcompare_Medal(countries)と、privateメソッドcompare_Medalがうまくリンクできていないのかもしれないです。
エラーは出ないのですが、プログラムで行いたい、メダル数順に並び替えるという動作が行われません。
自分ではどこが間違えているのかわからないので、有識者様、修正、アドバイス、よろしくお願いします。
やりたいこと
Aの金メダル数がBの金メダル数より多いならばAが上位、その逆ならばBが上位と判定、という感じです
compare_Medalがtrueを返したらＡが上位、falseを返したらＢが上位になるといった感じでかんがえました。

Comments

[y_waiwai]

提示のコードではどういう動作をするんでしょうか

[junnnnchan]

RUS(3,2,2)[7]
CAN(2,2,1)[5]
NED(2,1,2)[5]
NOR(2,1,2)[5]
USA(2,1,2)[5]
GER(2,1,1)[4]
AUT(1,2,1)[4]
FRA(1,1,1)[3]
CHN(1,1,0)[2]
KOR(1,1,0)[2]
SUI(1,1,0)[2]
BLR(1,0,0)[1]
POL(1,0,0)[1]
JPN(0,1,1)[2]
SWE(0,1,1)[2]
CZE(0,1,0)[1]
FIN(0,1,0)[1]
ITA(0,0,1)[1]
SLO(0,0,1)[1]
このように出力したいです

[junnnnchan]

SLO(0,0,1)[1]
FIN(0,1,0)[1]
SWE(0,1,1)[2]
JPN(0,1,1)[2]
POL(1,0,0)[1]
BLR(1,0,0)[1]
CHN(1,1,0)[2]
FRA(1,1,1)[3]
SUI(1,1,0)[2]
GER(2,1,1)[4]
USA(2,1,2)[5]
NED(2,1,2)[5]
KOR(1,1,0)[2]
AUT(1,2,1)[4]
RUS(3,2,2)[7]
CZE(0,1,0)[1]
CAN(2,2,1)[5]
NOR(2,1,2)[5]
ITA(0,0,1)[1]
今の状態だとこうです

Answer 1

次のように書けば分かりやすいでしょう。

 	private boolean compare_Medal(Country countries[]) {
        for (int i = 1; i < countries.length && countries[i] != null; i++) {
            int j = i - 1;
            Country c = countries[i];
            if (c.gold > countries[j].gold) return true;
            if (c.gold < countries[j].gold) return false;
			if (c.silver > countries[j].silver) return true;
			if (c.silver < countries[j].silver) return false;
			if (c.bronze > countries[j].bronze) return true;
			if (c.bronze < countries[j].bronze) return false;
            if (c.name.compareTo(countries[j].name) < 0) return true;
        }
        return false;
    }

追記
すみません。これでもソートができません。
もう少し調べてみます。

追記2
countries[i] が countries[j] で置き換わるのに、
countries[i] の値を使っていました。
保存した c を使って、countries[j] との比較をしましょう。

    public void sortResults() {

        for (int i = 1; i < countries.length && countries[i] != null; i++) {
            int j = i - 1;
            Country c = countries[i];
            while (j >= 0 && compare_Medal(c, countries[j])) { // ★
                countries[j+1] = countries[j];
                j--;
            }
            j++;
            countries[j] = c;
        }
    }

    private boolean compare_Medal(Country c, Country d) {  // ★
        if (c.gold > d.gold) return true;
        if (c.gold < d.gold) return false;
        if (c.silver > d.silver) return true;
        if (c.silver < d.silver) return false;
        if (c.bronze > d.bronze) return true;
        if (c.bronze < d.bronze) return false;
        return c.name.compareTo(d.name) < 0;
    }

Comments

[junnnnchan]

ありがとうございます。よろしくお願いします。

[junnnnchan]

ありがとうございます。解決しました。

Answer 2

compare_Medal()の条件式で、金メダルの個数を比較していません。