java

Question

javaでじゃんけんゲームを作っています。

コード

 int i = 0;
        
        if(judge_main.losernum == 1){
        	System.out.println("勝った人でじゃんけん");
        	i = 1;
        }
        else if(judge_main.winnernum == 1){
            	System.out.println("負けた人でじゃんけん");
        	i = 2;
        }
        else{
        	System.out.println("勝った人でじゃんけんして、そのあと負けた人でじゃんけん");
        	i = 1;
        	i = 2;
        }

        
        
        if (i == 1){
    		judge_main.playernum = judge_main.winnernum;
		    players_main = judge_main.winner_list;
    	}
    	else if(i == 2){
    		judge_main.playernum = judge_main.losernum;
    	    players_main = judge_main.loser_list;    
    	}

そのじゃんけんゲームの一部分のプログラムです。
上のプログラムで、

else{
System.out.println("勝った人でじゃんけんして、そのあと負けた人でじゃんけん");

この時に
i == 1、i == 2の両方の処理をしたくて、

else{
System.out.println("勝った人でじゃんけんして、そのあと負けた人でじゃんけん");

のところにi =1, i = 2を両方書いたのですが、i = 2を書いているので上書きされてi == 2の処理しかできないようになっています。

i == 1, i == 2両方の処理を行えるようにするにはどうすればいいのでしょうか？
説明がわかりにくいと思いますがよろしくお願いいたします。

Answer 1

swordoneさんと似たような解答です。
こういうバリエーションもあるよって感じです：
ローカル変数しか使っていないので、
気にしているメンバの上書きの心配は一切ありません。
(というかstaticメソッドだからメンバも何も無い。)

//playersをジャンケンさせて順位でソートする
public static List<Player> playJanken(List<Player> players){

    if(players.size() < 2){
        return players;
    }
    List<Player> winners;
    List<Player> losers ;

    /*
    この辺りで勝敗処理
    winnersとlosersに振り分け
    */

    //勝者グループのplayJankenの後ろに敗者グループのplayJankenを結合
    List<Player> result = playJanken(winners);
    result.addAll(playJanken(losers));
    return result;
}

Comments

[kisaragizinzin7]

回答ありがとうございます＾＾
コードもありがとうです！
ローカル変数でするとできました！

Answer 2

なんかまた面倒なことしてますね・・・
前の質問からの引き続きとして回答しますが，「再帰」で考えるとこの手の問題は簡単になります．
「勝ったPlayer組」と「負けたPlayer組」に分けたあと，
それぞれの組でもう一度じゃんけんし，勝敗を決め，
また2組に分け，繰り返します．
一人だけの組ではじゃんけんせずに終了します．
「じゃんけんする」という手続きは人数が変わってるだけで基本的に同じはずなので，
こう考えればコードを組みやすくなるのではないでしょうか．

イメージが湧きにくいかもしれないので，概要をコードで書いてみました．

//list:じゃんけんするPlayerのリスト，firstRank:この組の最上位が，全体で何位か
public static void playJanken(List<Player> list, int firstRank){

    //listにいるPlayerでじゃんけんを行い，勝ち負けを決定する
    //(一人の場合には，そのPlayerの順位がfirstRankで確定して終了)

    List<Player> winnerList = new ArrayList<Player>();
    List<Player> loserList = new ArrayList<Player>();

    //勝ったPlayerをwinnerListに，負けたPlayerをloserListに入れる

    //再帰でじゃんけん続行
    playJanken(winnerList, firstRank);  //勝った人でじゃんけん
    playJanken(loserList, firstRank + winnerList.size());  //負けた人でじゃんけん．勝った人の人数分，順位が下がる
}

これで何が起こるかというと，
まず全員でじゃんけんし，勝ち負けで分けます(1)．
その後，勝ち組でじゃんけんします．この時，新たにplayJankenメソッドが呼ばれます．
そしてまた勝ち負けで分かれます(2)が，
この(1)のloserListと(2)のloserListは別に管理されていて，互いに干渉しません(上書きされません)．
メソッドから別のメソッドのローカル変数には干渉できませんよね？それと同じです．
これを繰り返して行くといずれ最初の勝ち組の処理が終わるので，負け組の処理に入れます．

Comments

[kisaragizinzin7]

この方法しか思いつかなくて・・・
すみませんです＞＜

勝った組と負けた組に分けたのですが、それぞれの組でもう一度じゃんけんさせるところがわかりません。

例えばプレイヤーが５人として、まずじゃんけんをして勝った人３人負けた人２人として勝った組と負けた組に分ける。
勝ち[プレイヤー１　プレイヤー４　プレイヤー５]
負け[プレイヤー２　プレイヤー３]

ここで勝った組でもう一度じゃんけんをすると、
勝ち[プレイヤー４]
負け[プレイヤー１　プレイヤー５]
その時点で結果が上書きされて負けた組でじゃんけんをしても[プレイヤー２　プレイヤー３]でのじゃんけんではなく、[プレイヤー１　プレイヤー５]でのじゃんけんになってしまいます。
[プレイヤー２　プレイヤー３]でじゃんけんをするために勝った組と負けた組で同時進行で処理するような機能みたいなのがあればいいのですが、何かいい方法はないでしょうか？

[swordone]

同時進行する意図はなんでしょうか？
じゃんけんを進めて最終的に順位が決定すればいいと思っていましたが・・・

[swordone]

ちなみに再帰で考えた時，「負けた組」が「上書きされる」ということはありません．
一回「クイックソート」について調べてどこかでソースコードを見てみてください．
再帰を上手く使ったアルゴリズムなので参考になると思います．
リストの上書きとは違う話になりますが，境界の値がどのように取られるかをよく見てください．

[kisaragizinzin7]

勝ち負けの結果を上書きされないようにするためです。

私の考えでは
players_mainという配列があります（プレイヤー５人なら５個格納されている）。
「じゃんけんをして判定して勝ち組負け組に分ける処理」を行う。
勝ち組winnerlist　負け組loserlistの配列を作ってそこに分ける。
勝ち組で再度じゃんけんを行う場合winnerlistをplayers_mainに代入して、同じように「じゃんけんをして判定して勝ち組負け組に分ける処理」を行う。
このときに勝ち組winnerlist　負け組loserlistの配列が上書きされてしまい、１回目のじゃんけんの結果の負け組loserlist配列が消えてしまうので、二組の配列を同時進行で「じゃんけんをして判定して勝ち組負け組に分ける処理」を行えるようにしたいと思いました。
この考え方は間違っているのでしょうか？

じゃんけんを進めて最終的に順位が決定すればいいと思っていましたが・・・ ＞
そのように考えています！
プレイヤー５人なら、１位から５位を決めようと考えています。

[swordone]

winnerList,loserListがクラスのメンバであれば上書きされてしまいますが，
再帰にしてローカル変数にしてしまえば上書きは起きません．
最初にじゃんけんした結果
勝ち[プレイヤー１　プレイヤー４　プレイヤー５]
負け[プレイヤー２　プレイヤー３]・・・１
で，この勝ち組で再度じゃんけんした時
勝ち[プレイヤー４]
負け[プレイヤー１　プレイヤー５]・・・２
で，この１と２は再帰で生まれた「別のメソッド」の中のローカル変数なので
互いに影響しません．
別のメソッドと言うと語弊があるかもしれませんが，要は「１回目のじゃんけん」と「２回目のじゃんけん」で勝ち，負けリストは別に管理するので，上書きはされません．

[kisaragizinzin7]

コードまでありがとうございます＾＾
ローカル変数にしたらできました。

ありがとうございました。

Answer 3

コードやゲームの内容がさっぱりわかりませんが、それには目をつぶります。

単純にi=3などして、後半の判定処理を１つ増やすわけにはいかないのでしょうか？

Comments

[kisaragizinzin7]

回答ありがとうございます＾＾
内容まったく載せなくてすみません。
これだけで大丈夫だと思ってしまいました。

int i = 0;

if(judge_main.losernum == 1){
System.out.println("勝った人でじゃんけん");
i = 1;
}
else if(judge_main.winnernum == 1){
System.out.println("負けた人でじゃんけん");
i = 2;
}
else{
System.out.println("勝った人でじゃんけんして、そのあと負けた人でじゃんけん");
i = 3;
}

if (i == 1){
judge_main.playernum = judge_main.winnernum;
players_main = judge_main.winner_list;
}
else if(i == 2){
judge_main.playernum = judge_main.losernum;
players_main = judge_main.loser_list;
}
else{
judge_main.playernum = judge_main.winnernum;
players_main = judge_main.winner_list;
judge_main.playernum = judge_main.losernum;
players_main = judge_main.loser_list;
}

こういうことですかね？

[kisaragizinzin7]

これでは最後の
else{
judge_main.playernum = judge_main.winnernum;
players_main = judge_main.winner_list;
judge_main.playernum = judge_main.losernum;
players_main = judge_main.loser_list;
}
この部分でjudge_main.playernumとplayers_mainが上書きされてしまうので、質問のところに載せたプログラムと同じ結果になってしまいます＞＜

[mtempa]

そうです。
その処理は満足のいく動作ではないのでしょうか？

[kisaragizinzin7]

上に書いた通り、上書きされてしまうので満足のいく動作ではないです。

[mtempa]

では、
常に配列で結果を返すようにして
メインの処理の方で、配列数分じゃんけん処理を行うようにするとかですかね？

メイン側が一切書き換えられないのならちょっと私の手に余ります。