javaの戻り値が反映されていない？

Question

前提・実現したいこと

初めて質問させていただきます。
javaでじゃんけんをするシステムを作っています。
3回勝負にしようとして、numという変数に勝ちなら＋１、負けなら−１をすることで、三回勝負が終わった後に最終結果を表示したいのですが、何度試しても引き分けと出ます。
最初に代入した０からmethod内の処理がなされていないのでしょうか？
どのようにしたら反映されるようになるでしょうか？

該当のソースコード

public class task{
  public static void main(String[] args){
    String[] a ={"グー","チョキ","パー"};
    System.out.println("じゃんけん、3回勝負です");
    int count = 0;
    while(count < 3){
      System.out.println("1.グー, 2.チョキ, 3.パー");
      int i = new java.util.Scanner(System.in).nextInt();
      int Y = i - 1;
      System.out.println("あなたは" + a[Y]);
      int P = new java.util.Random().nextInt(3);
      System.out.println("PCは" + a[P]);
      int num += janken(Y,P);
      count++;
      if(count >= 3){
        System.out.println("３回勝負の結果");
        if(num == 0){
          System.out.println("引き分け");
        }else if(num < 0){
          System.out.println("PCの勝ち");
        }else if(num > 0){
          System.out.println("あなたの勝ち");
        }
      }
    }
  }
  public static int janken(int Y,int P){
    if(Y == P){
      System.out.println("あいこ");
    }else if((Y == 0 && P == 1)||(Y == 1 && P == 2)||(Y == 2 && P == 0)){
      System.out.println("あなたの勝ち");
      return 1;
    }else if((P == 0 && Y == 1)||(P == 1 && Y == 2)||(P == 2 && Y == 0)){
      System.out.println("PCの勝ち");
      return -1;
    }
  }
}

試したこと

java:13: エラー: ';'がありません
int num += janken(Y,P);
^
エラー1個

と出てしまいます。

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

ここにより詳細な情報を記載してください。

Answer 1

int mun = janken(Y,P,num);
は
int num = janken(Y,P,num);
の誤りではないですか。munはどこも参照していません。
また、その場合、int num = 0; が不要になります。

Comments

[LouiS0616]

同名の変数の最宣言はできないように思います。
やるなら num =janken(Y, P, num); です。

Answer 2

int mun = janken(Y,P,num);

返り値をせっかく受け取っているんですから、munを判定に用いるようにして下さい。

---

そもそも引数numなんて要らないのです。仮引数への再代入は実引数には一切影響しません。

static int janken(int y, int p) {
    if(yの勝ち) return  1;
    if(pの勝ち) return -1;

    return 0;
}

返り値を元のnumに足していけば良いです。
単一責任の原則を踏まえ、jankenメソッドは単にその場の勝敗だけ判定するようにすべきです。

Comments

[dice142]

> そもそもnumなんて要らないのです。
質問文を読むと 3回連続で勝ったら+3、3回連続で負けたら-3 のようにしていきたいのかなという風に見えました。
なので、numは必要だけど宣言位置がおかしいって感じでしょうか。

[LouiS0616]

引数numが不要という意味で書きました。語弊があるので修正しておきます。

[LouiS0616]

同時に私の問題誤読にも気付けました。
コメントありがとうございます。

[Shiho.]

戻り値を元のnumに足すにはどうしたら良いのでしょうか？
メインメソッドでの宣言がわからないです...

[LouiS0616]

num += janken(y, p); で充分です。

[Shiho.]

度々申し訳ないです。
エラー: ';'がありません
と出るのですがなぜでしょうか

[LouiS0616]

どこかセミコロンが抜けています。
エラー行も表示されている筈なので、その前後を確認してみて下さい。

[Shiho.]

エラー行には；をつけています。
全ての行を確認しましたが；の抜けはないように思うのですが...

[LouiS0616]

該当のコードが無いと何とも言えません。
行末のセミコロンをやはり見落としていたり、for文のセパレータが抜けていたり、見ているコードとコンパイルしているコードが違ったり、何かしらミスがある筈です。

[Shiho.]

質問を編集しました。
書き直した部分のどこかがおかしいのでしょうか？

[LouiS0616]

複合代入文(+=など)は宣言文と一緒には書けません。
whileループに入るより前に int num = 0; とでもしておいて、ジャンケンをするごとに num += janken(Y, P); として下さい。

[LouiS0616]

ここで『そういうルールなのか』で終わってしまうと勿体ないので、
なぜ int num += ほにゃらら; と書くことができないのか考えてみても勉強になるでしょう。

[Shiho.]

解決しました！
丁寧なご指導本当に助かりました。
ありがとうございました！
これから一層勉強しようと思います。