Java ゲーム作成 勝敗判定、引き分け判定について教えていただきたいです。

Question

前提・実現したいこと

Javaを勉強し始めて1ヶ月半になり、簡単なゲームを作成する課題があります。
ゲームの内容は以下になります。
・プレイヤー3人
・それぞれサイコロを3回ずつ振り、各々その合計値を求める
・合計値が大きい人の勝ち

勝敗判定や引き分け判定の処理を入れたいです。
sumは毎回初期化されてしまうため、各プレイヤーのスコアを他に保存しないといけないのですが、その書き方がわかりません。
アドバイスをいただけないでしょうか。
よろしくお願い致します。
過去の質問を編集したのですが、解決済みのステータスになってしまっていたため、新規で質問させていただきました。

発生している問題・エラーメッセージ

[1]:4 5 2
合計値:11

[2]:2 4 5
合計値:11

[3]:2 3 1
合計値:6

勝者は1回目の人で、合計値は6でした。←最後にsumに入った値が出てしまいます。
↑毎回"1回目"と出てしまいます。

該当のソースコード

package test;

public class SaikoroGames {

public static void main(String[] args) {

// プレイヤー人数を決める
int r[] = new int[3];

// 値を代入
r[0] = 1;
r[1] = 2;
r[2] = 3;

int dice = 0;
// サイコロを振る回数を代入
int count = 3;
int score = 0;

// 人数分ループする
for(int j = 0;j < r.length;j++){
System.out.print("[" + r[j] + "]" + ":");
int sum = 0;

for(int i = 0;i < count;i++){
dice = (int)(Math.random()*6)+1;
int hokan = dice;
sum += hokan;

System.out.print(hokan + " ");
}
score = sum;
System.out.println();
System.out.println("合計値:" + sum);
System.out.println();

}
// 勝敗判定　　　　　　　　　　←ここからの処理の書き方がわからないです。
int minCnt = 100;
int minCntPlayer = 0;
for(int i = 0; i < r.length; i++) {
if(minCnt > r.length + score) {
minCnt = r.length + score;
minCntPlayer = r[i];
}
}
System.out.println("勝者は" + minCntPlayer + "回目の人で、合計値は" + score + "でした。");
}

}

試したこと

sumを代入する変数を複数作りましたが、最後に入った値が出てしまいました。

補足情報（FW/ツールのバージョンなど）

Comments

[dodox86]

Javaのコードの書き方云々というよりは、まず具体的な手順（ステップ）が頭の中でできあがっているかどうかが重要です。コードを拝見するに、かなり混乱しているように思います。各ステップを言葉で説明できるでしょうか。

> sumは毎回初期化されてしまうため、各プレイヤーのスコアを他に保存しないといけないのですが、その書き方がわかりません。
> 勝者は1回目の人で、合計値は6でした。←最後にsumに入った値が出てしまいます。
> ↑毎回"1回目"と出てしまいます。

毎回初期化されてしまう、とか、出てしまうとか、そのようにコードを書いているからそうなっているにすぎません。実際に3人がサイコロをふるとき、それぞれの合計値を記録しておく場所は1つだけでしょうか。違いますよね。3つ必要です。

[dodox86]

> それぞれの合計値を記録しておく場所は1つだけでしょうか。違いますよね。3つ必要です。

ただし、プログラムの作りによっては必ずしも3つ必要とはならないです。一人の番が終わるたびに比較していけば良いだけです。あくまで、プログラムの作り（設計）によります。

[guriko]

申し訳ございません。
ありがとうございます。
正確なロジックを作成して本作成をします。

Answer 1

プログラムでは3人分の合計値を出した後に勝敗の判定を行っているので、合計値は配列で作成しておいた方が分かりやすいでしょう。それと、判定処理で合計値が高い人を勝ちにさせたいときは、初期値は最低値の「0」にしておきます。
※判定処理の例を載せておきます。引き分けを考慮させたい場合には、アレンジしてみてください。

int winner = 0;
int winsum = 0;
for ( int i = 0 ; i < count ; i++) {
  if ( sum[i] > winsum ) {
    winner = r[i];
    winsum = sum[i];
  }
}

Comments

[guriko]

ありがとうございます。
いただいた例をもとに書いてみます。

[guriko]

すいません。
＞合計値は配列で作成しておいた方が分かりやすいでしょう。
上記は、もうひとつ合計値用に配列を入れる、という認識でよろしかったでしょうか。

[退会済みユーザー]

もうひとつというか、合計値を表す変数を「配列に変更する」だけで良いと思いますよ。

[guriko]

プレイヤーの人数を配列に入れていたのですが、合計値を配列に入れることにしました。（プレイヤーの人数を配列に入れる必要がないと気づきました。）
ありがとうございます。