例外処理

import java.io.*;

class janken
{
public static void main(String[] args) throws IOException{
BufferedReader br = new
BufferedReader(new InputStreamReader(System.in));
int k=0,m=0,h=0,z=0;
while(true){

System.out.print("あなたの手を入力して下さい。（1:グー、2:チョキ、3:パー、0:終了）？\n");

String str = br.readLine();
int a = Integer.parseInt(str);

if(a == 0){
break;
}
z++;
switch(a){
case 1:
System.out.println("あなたの手はグーです。");
break;
case 2:
System.out.println("あなたの手はチョキです。");
break;
case 3:
System.out.println("あなたの手はパーです。");
break;
default:
System.out.println("1～3を入力してください。");
break;
}

{
int ran = ((int)(Math.random()*3)+1);
int b=ran;
switch(b){
case 1:
System.out.println("コンピュータの手はグーです。");
break;
case 2:
System.out.println("コンピュータの手はチョキです。");
break;
case 3:
System.out.println("コンピュータの手はパーです。");
break;

}

int c;
c=a-b;

if(c == 2){
System.out.println("あなたの勝ちです。");
k++;
}
else if(c== -1){
System.out.println("あなたの勝ちです。");
k++;
}
else if(c==0){
System.out.println("あいこです。");
h++;
}
else if(c==1){
System.out.println("コンピュータの勝ちです。");
m++;
}
else {
System.out.println("コンピュータの勝ちです。");
m++;
}
}
}
System.out.println("じゃんけん終了。");
System.out.println("あなたは"+z+"試合中、"+k+"勝"+m+"敗"+h+"分けです");
}
}
このようにじゃんけんのプログラムを作りました。
しかし0～3以外の数字が入力されても、じゃんけんの勝敗を勝手に判定してしまいます・・・
try文でやってみたのですがいまいちうまくいきませんでした。
0～3を入力したら入力エラーと表示して、ふたたびユーザーの手を聞くようにしたいです。
どうすればよいかアドバイスお願いします。

No.4 ベストアンサー 回答者： syouji_08 回答日時： 2007/07/14 04:36

とりあえず、これで質問者さんの望むとおりの動作はすると思います。

********************ソースファイル******************************
import java.io.*;

class janken
{
public static void main(String[] args) throws IOException{
BufferedReader br = new
BufferedReader(new InputStreamReader(System.in));
int k=0,m=0,h=0,z=0;
/***********************修正部分*******************************/
String str;
int ec;
while(true){
ec = 0;
do{
if(ec > 0 ){
System.out.println("入力値が不正です。再度入力し直して下さい");
}
System.out.print("あなたの手を入力して下さい。（1:グー、2:チョキ、3:パー、0:終了）？\n");
str = br.readLine();
ec++;

}while(!(str.equals("1") || str.equals("2") || str.equals("3") || str.equals("0")));
/*************************************************************/
int a = Integer.parseInt(str);

if(a == 0){
break;
}
z++;
switch(a){
case 1:
System.out.println("あなたの手はグーです。");
break;
case 2:
System.out.println("あなたの手はチョキです。");
break;
case 3:
System.out.println("あなたの手はパーです。");
break;
default:
System.out.println("1～3を入力してください。");
break;
}

{
int ran = ((int)(Math.random()*3)+1);
int b=ran;
switch(b){
case 1:
System.out.println("コンピュータの手はグーです。");
break;
case 2:
System.out.println("コンピュータの手はチョキです。");
break;
case 3:
System.out.println("コンピュータの手はパーです。");
break;

}

int c;
c=a-b;

if(c == 2){
System.out.println("あなたの勝ちです。");
k++;
}
else if(c== -1){
System.out.println("あなたの勝ちです。");
k++;
}
else if(c==0){
System.out.println("あいこです。");
h++;
}
else if(c==1){
System.out.println("コンピュータの勝ちです。");
m++;
}
else {
System.out.println("コンピュータの勝ちです。");
m++;
}
}
}
System.out.println("じゃんけん終了。");
System.out.println("あなたは"+z+"試合中、"+k+"勝"+m+"敗"+h+"分けです");
}
}

この回答へのお礼

ご回答ありがとうございます。
コンパイルしたところ私の思っていたとおりに動いてくれて、とても感動しました。
こんな方法があったなんて・・・まだまだ勉強不足ですね＾＾；
本当にありがとうございました。

お礼日時：2007/07/14 23:27

No.3 回答者： choconamacream 回答日時： 2007/07/14 03:41

横から失礼。

基本的に、１文字だけの変数が悪いんじゃなくて、使い方だと思われます。同じ変数でもループカウンターなどの場合でしたら、あくまでそのブロック内だけで処理が完結する事を条件として、iでもjでもいいと思います。

ただ、質問者さんの場合は、インクリメントなどのロジックの部分だけではなくて、標準出力の所でも同じように使用されていたので、ちょっと見づらいんじゃないかな～、というわけです。（嫌味ったらしく聞こえるようでしたら、ごめんなさい。）

この回答へのお礼

いえいえ、おっしゃるとおりです。
プログラムを作る以上誰から見てもわかるような変数がいいにきまってますし、これからは意識して変数をつけたいとおもいます。

お礼日時：2007/07/14 23:21

No.2 回答者： koko_u_ 回答日時： 2007/07/14 02:01

私も所詮素人なのです。

メソッドの分け方も「長くなりそうなら分ける」という程度。
変数名も最初に「これは試合数を格納する変数にしよう → gameCount」という程度。

変数名については、巷の入門書でも 1文字のいい加減な変数名が使用されていたりするので、その悪影響か？

No.1 回答者： koko_u_ 回答日時： 2007/07/14 01:05

0 ～ 3 以外の場合には continue で while ループの先頭に戻ればよいだけ？

そして、コンピュータの出し手を決める箇所や、勝敗を判定する箇所をメソッドにして切り出すのが正常なプログラマの作法だと思う。

また、k l m c などの「意味の取れない」変数名も止めましょう。

この回答への補足

やっぱり、自分の手、コンピュータの手、勝敗の判定をメソッドで分けたほうがいいのですか・・・
まだ習いたてでメソッドをどのようにしたらわからないんですが、調べてみます。

変数なんですがまだ試作段階だったので適当にいれてました。
ややこしくてもうしわけないです。

補足日時：2007/07/14 01:45