javaでランキング更新システムを作りたい

処理にかかった時間のランキングを更新するプログラムを作りたいです。
Practiceクラスにmainメソッドを置き、Timeクラス、Rankingクラス、Retryクラスを作成するものとします。
イメージとしては
ーーーーーーー
処理プログラム
ーーーーーーー
かかった処理時間：6000[ms]
1: 6000[ms]
2: 999999999999[ms]
3: 999999999999[ms]
4: 999999999999[ms]
5: 999999999999[ms]

もう一度処理を実行しますか？（1、はい 2、しない）
1を選択した場合

ーーーーーーー
処理プログラム
ーーーーーーー
かかった処理時間：5000[ms]
1: 5000[ms]
2: 6000[ms]
3: 999999999999[ms]
4: 999999999999[ms]
5: 999999999999[ms]

といった風にランキングを更新していきたく、上位5位まで表示したいです。
もともと999999999999の値が入っているリスト型の可変長配列に、値を更新していくものとします。

java
1public class Practice{
2    public static long totalTime;
3    public static void main(String[] args) {
4        Time time = new Time();
5        Ranking ranking = new Ranking();
6        Retry retry = new Retry();
7 
8        //計測開始
9        time.setStartTime(System.currentTimeMillis());
10        //処理プログラム
11        --------------
12　　　　 処理プログラム
13　　　　 --------------
14        //計測時間を表示
15        time.printTotalTime();
16        totalTime = time.getTotalTime();
17　　　　//ランキングの表示
18        ranking.makeDefaultTime();
19        ranking.printTime();
20        //もう一度実行するか
21        retry.retry();
22    }
23}

java
1public class Time {
2    long startTime;
3    long endTime;
4    long totalTime;
5
6    //開始時間を保持しておくメソッド
7    void setStartTime(long startTime) {
8        this.startTime = startTime;
9    }
10    //計測時間を返すメソッド
11    long getTotalTime( ) {
12        endTime = System.currentTimeMillis();
13        totalTime = endTime - startTime;
14        return totalTime;
15    }
16    //計測時間を表示するメソッド
17    void printTotalTime() {
18        System.out.println("time=" + getTotalTime() + "[ms]");
19    }
20}

java
1public class Ranking {
2    Time time = new Time();
3
4　　//元からあるランキングリストの作成
5    ArrayList<Long> times = new ArrayList<>();
6    void makeDefaultTime () {
7        for (int i = 0; i < 5; i++) {
8            times.add(999999999999L);
9        }
10    }
11    //処理時間をもとあったリストに更新していく
12    void printTime() {
13        times.add(Practice.totalTime);
14        Collections.sort(times);
15        for (int i = 0; i < 5; i++) {
16            System.out.println( i + 1 + ":　" + times.get(i) + "[ms]");
17        }
18    }
19}

java
1public class Retry {
2    //リトライするかしないかの処理
3    Time time = new Time();
4    Ranking ranking = new Ranking();
5    void retry() {
6        System.out.println("もう一度処理を実行しますか？（1、はい 2、しない");
7        int retry = StandardInputReader.getInputInt(">");
8        if(retry == 1) {
9            time.setStartTime(System.currentTimeMillis());
10            --------------
11　　　　　　　処理プログラム
12　　　　　　 --------------
13            time.printTotalTime();
14            Practice.totalTime = time.getTotalTime();
15            ranking.printTime();
16            retry();
17        } else {
18            System.out.println("プログラムを終了します");
19            return;
20        }
21    }
22}

今のところ一回目の表示は成功するのですが、二回目以降の処理でうまくいってくれません。
具体的には一回目で作成された処理時間の値も含まれているTimesリストが二回目では無くなってしまい、
リストの要素が一つしかできていない状態になり、IndexOutOfBoundsExceptionになってしまいます。

どなたかお力添えをいただけたら幸いです。

m.ts10806

2020/05/16 22:42

コードブロックはファイル毎にわけてください。動作確認の際に逐一範囲選択しなければなりません。

行動規範の内容に同意します

回答1件

ベストアンサー

retryメソッドで使用しているrankingには、何も入ってないと思いますよ。
最後の方の　Ranking ranking = new Ranking();　で新しく作った変数ですから・・・

初回で作成したrankingを、引数としてretryメソッドに渡すべきだと思います。
あと、未定義のcharacterという変数が何なのか、よく分かりません。

投稿2020/05/16 18:16

mattrick

総合スコア135

sosowatabo

2020/05/17 16:30

ご回答ありがとうございます。未定義の変数caharacterはretryの間違いでした。修正いたしました。 mainメソッド内で作成したrankingをretryメソッドに引数として渡し、それを使用するにはどのようにすればよいでしょうか？作成したランキングのリストのgetterを作成してmainメソッドでクラス変数に持たせて渡そうとしてもうまくいきませんでした、、。