Q&A
質問は何でしょう。
提示のコードでなんか不具合があるんでしょうか
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 Ranking { // メンバ変数宣言部分 // final int NUM_OF_COUNTRIES = 1000; // String[] country; // int[] gold, silver, bronze; int next; Country[] countries = new Country[1000]; // Country[] gold , silver , bronze; // コンストラクタ部分 public Ranking() { //country = new String[NUM_OF_COUNTRIES]; // gold = new Country[countries.length]; // silver = new Country[countries.length]; // bronze = new Country[countries.length]; // // 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; } countries[i] = new Country(countryName); } 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].name = 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; String c = countries[i].name; int g = countries[i].gold; int s = countries[i].silver; int b = countries[i].bronze; while (j >= 0 && c.compareTo(countries[j].name) < 0) { countries[j+1].name = countries[j].name; countries[j+1].gold = countries[j].gold; countries[j+1].silver = countries[j].silver; countries[j+1].bronze = countries[j].bronze; j--; } j++; countries[j].name = c; countries[j].gold = g; countries[j].silver = s; countries[j].bronze = b; } } // 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 class Country { String name; int gold, silver, bronze; public Country (String countryName) { this.name = countryName; } }
public enum Color { Gold, Silver, Bronze }
ーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーー
エラー文
Exception in thread "main" java.lang.NullPointerException
at medals1.Ranking.addMedal(Ranking.java:57)
at medals1.Competition.main(Competition.java:15)
Rankingクラスで国名やメダル情報を格納していたメンバ変数を廃止して、代わりにCountryクラスの配列変数countriesを宣言して、これを使用するように書き換えたいです。
アバウトな説明ですが、アドバイスよろしくお願いします。
どんな説明を求めてるのか知りませんが、せめて自分で組んだコードでどこがどう問題になっているのかぐらいは質問文に書きましょう。さもなければただの丸投げです。
Exception in thread "main" java.lang.NullPointerException
at medals1.Ranking.addMedal(Ranking.java:57)
at medals1.Competition.main(Competition.java:15)
というエラー文が表示されてしまいます。
自己解決
