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

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

新規登録して質問してみよう
ただいま回答率
85.46%
Java

Javaは、1995年にサン・マイクロシステムズが開発したプログラミング言語です。表記法はC言語に似ていますが、既存のプログラミング言語の短所を踏まえていちから設計されており、最初からオブジェクト指向性を備えてデザインされています。セキュリティ面が強力であることや、ネットワーク環境での利用に向いていることが特徴です。Javaで作られたソフトウェアは基本的にいかなるプラットフォームでも作動します。

Q&A

解決済

2回答

766閲覧

java method void

junnnnchan

総合スコア26

Java

Javaは、1995年にサン・マイクロシステムズが開発したプログラミング言語です。表記法はC言語に似ていますが、既存のプログラミング言語の短所を踏まえていちから設計されており、最初からオブジェクト指向性を備えてデザインされています。セキュリティ面が強力であることや、ネットワーク環境での利用に向いていることが特徴です。Javaで作られたソフトウェアは基本的にいかなるプラットフォームでも作動します。

0グッド

0クリップ

投稿2020/06/04 07:42

編集2020/06/04 08:09
public class Country { String name; int gold = 0; int silver = 0; int bronze = 0; int sum = 0; public Country (String countryName) { this.name = countryName; } public String getName() { return name; } public void add(Color color) { if(color == Color.Gold) gold++; if(color == Color.Silver) silver++; if(color == Color.Bronze) bronze++; sum = gold + silver + bronze; } public String toString() { return (this.name + "(" + gold + "," + silver + "," + bronze + ")" + "[" + sum + "]"); } }
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(c,countries[j])) { countries[j+1] = countries[j]; //(cとcountries[j]がメダル数比較で同位 && j--; //cの国名は辞書順でcountries[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; } // 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(); } }

country classでしたいこと
<add method>
Color型の値を引数として持ち、引数がColor.GoldまたはColor.SilverまたはColor.Bronzeであるとき、指定された色のメダル獲得数を1増やす。
<toString method>
引数なし、インスタンスの持つデータを 国名(金メダル数,銀メダル数,銅メダル数)[合計メダル数] の形式の文字列にして返す。例えば、国名が JPN で金,銀,銅のメダル数がそれぞれ4,5,3であるとき、返す文字列は JPN(4,5,3)[12] となる。
です。
よろしくお願いします。

エラー文
Testsuite: medals3.AllTests
Tests run: 4, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: 0.026 sec

Testcase: testAdd took 0.008 sec
Testcase: testToString took 0.003 sec
Testcase: testGetName took 0 sec
Testcase: testCountry took 0.002 sec
FAILED
medals3.Countryクラスに不必要なメンバ変数: [sum] expected:<0> but was:<1>
junit.framework.AssertionFailedError: medals3.Countryクラスに不必要なメンバ変数: [sum] expected:<0> but was:<1>
at medals3.CountryTest.testCountry(CountryTest.java:84)
何が違うのかお願いします。

気になる質問をクリップする

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

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

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答2

0

エラーの原因をご教示お願いします。

toString()Objectにもあるメソッドで、返り値はString型です(Oracle)。

Javaで返り値の型によるオーバーロードはできませんので、toString()メソッドはStringを返り値にしなければなりません。

投稿2020/06/04 07:47

編集2020/06/04 07:48
maisumakun

総合スコア145208

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

maisumakun

2020/06/04 07:50

文字列を返さずにSystem.out.printlnを行うメソッドを作るのであれば、別な名前にしましょう。
junnnnchan

2020/06/04 07:55

修正しました! ですが Testsuite: medals3.AllTests Tests run: 4, Failures: 2, Errors: 0, Skipped: 0, Time elapsed: 0.025 sec Testcase: testAdd took 0.007 sec Testcase: testToString took 0.002 sec Testcase: testGetName took 0.001 sec FAILED medals3.CountryクラスのgetNameメソッドがString型でない expected:<class java.lang.String> but was:<void> junit.framework.AssertionFailedError: medals3.CountryクラスのgetNameメソッドがString型でない expected:<class java.lang.String> but was:<void> at medals3.CountryTest.invokeStringMethod(CountryTest.java:156) at medals3.CountryTest.testGetName(CountryTest.java:175) Testcase: testCountry took 0 sec FAILED medals3.Countryクラスに不必要なメンバ変数: [sum] expected:<0> but was:<1> junit.framework.AssertionFailedError: medals3.Countryクラスに不必要なメンバ変数: [sum] expected:<0> but was:<1> at medals3.CountryTest.testCountry(CountryTest.java: というのが出てしまいます。。。
maisumakun

2020/06/04 07:57

「medals3.CountryクラスのgetNameメソッドがString型でない」のが問題なのであれば、Stringを返すメソッドに作り変えましょう。
junnnnchan

2020/06/04 08:10

修正しました。 Testsuite: medals3.AllTests Tests run: 4, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: 0.026 sec Testcase: testAdd took 0.008 sec Testcase: testToString took 0.003 sec Testcase: testGetName took 0 sec Testcase: testCountry took 0.002 sec FAILED medals3.Countryクラスに不必要なメンバ変数: [sum] expected:<0> but was:<1> junit.framework.AssertionFailedError: medals3.Countryクラスに不必要なメンバ変数: [sum] expected:<0> but was:<1> at medals3.CountryTest.testCountry(CountryTest.java:84) このようなエラーがでてしまいました
maisumakun

2020/06/04 08:12

何が問題なのか、エラーメッセージに日本語で書いてあるかと思うのですが。
退会済みユーザー

退会済みユーザー

2020/06/04 08:32

いまの java なら 返り値の型は継承関係にあればオーバーロード(引数が同じだからオーバーライドか)できますよ。 ただし敬称関係で final なクラスは無理だがw
guest

0

ベストアンサー

引数なしのtoStringはObjectクラスで定義されています。同名で引数も同じメソッドを、返り値の違うメソッドとして定義することは出来ません。

投稿2020/06/04 07:47

swordone

総合スコア20651

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.46%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問