出力が重複するバグの原因がわかりません。 java超入門

Question

おはようございます。

現在youtubeのjava超入門という動画をみて勉強しているのですが、出力が重複するバグが起こり、何度見直しても解消されません。

import java.util.*;
import java.io.*;

public class part18 {
	
		static String name = "ほげさん";
		static int lv = 30;
		static int hp = 30;
		static int gold = 10000;
		
		public static void main( String[] args ) throws IOException {
			
			joshou();	 
			for( ; hp > 0; ){
				command();
			}			
		}
		
		static void joshou(){					//序章を出力
						
			System.out.println( "魔王が世界を滅ぼそうとしています。" );
		}
		
		static void status(){					//ステータスを出力
			
			System.out.println("--------------------------------------");
			System.out.println( name + "  Lv :" + lv + " HP :" + hp + "  Gold :"  + gold + "G" );
			System.out.println("--------------------------------------");
		}
		 
		static void command() throws IOException {			//コマンドを表示
		
			status();
			
			System.out.println( " 1, 魔王を倒しに行く。" );
			System.out.println( " 2, 修行する。" );
			System.out.println( " 3, 宿屋に泊まる" );
			System.out.println("--------------------------------------");
			
			cmd();
		}
		
		static void cmd() throws IOException {				//コマンド処理設定
		
			int c = System.in.read();
			
			switch( c ) {

				case ( '1' ) :
					
					if( lv < 40 ){
							System.out.println( "魔王に敗れた。" );
							gameover();
						}else{
							gameclear();
						}
						
						break;
						
				case ( '2' ) : 
				
					Random rnd = new Random();
					int monster = rnd.nextInt( 5 )+ 1;
					int damage = rnd.nextInt( 5 );
					int ex = rnd.nextInt( 5 );
					
					System.out.println( "敵が " + monster + "匹 現れた。" );
					
					hp -= damage;
					
					if( hp < 0 ){
						hp = 0;
					}
					
					if( hp > 0 ){
						System.out.println( name + "は " + damage + " のダメージを受けた。" );
						System.out.println( "敵を倒した。" + name + "は経験値を " + ex + " 獲得した。" );
						lv += ex;
					}else{
						System.out.println( name + "は" + damage + "のダメージを受けた。" );
						System.out.println( "HPが０になった。" );
						gameover();	
					}
					
					System.out.println();  		//改行
					break;
			
				case ( '3' ) :
					
					System.out.println( "HPが回復した。" );
					hp = hp;
					
					break;
			}
		}				
		
		static void gameover(){			//GAME OVER画面を出力
			
			System.out.println( "GAME OVER" );
		}
		
		static void gameclear(){			//GAME CLEAR画面を出力
			
			System.out.println( "魔王を倒した！" );
			System.out.println( "GAME CLEAR" );
		}
}

こちらのソースなのですが、出力が下記のように2度目の入力の時点でステータスとコマンド選択の出力が重複して表示されてしまいます。

魔王が世界を滅ぼそうとしています。
--------------------------------------
ほげさん  Lv :30 HP :30  Gold :10000G
--------------------------------------
 1, 魔王を倒しに行く。
 2, 修行する。
 3, 宿屋に泊まる
--------------------------------------
2
敵が 2匹現れた。
ほげさんは0のダメージを受けた。
敵を倒した。ほげさんは、0の経験値を手に入れた。

--------------------------------------
ほげさん  Lv :30 HP :30  Gold :10000G
--------------------------------------
 1, 魔王を倒しに行く。
 2, 修行する。
 3, 宿屋に泊まる
--------------------------------------
--------------------------------------
ほげさん  Lv :30 HP :30  Gold :10000G
--------------------------------------
 1, 魔王を倒しに行く。
 2, 修行する。
 3, 宿屋に泊まる
--------------------------------------

HPが０以上の場合、
ステータス表示
コマンド選択表示
入力受け取り
という流れをループさせたいのですが、分かる方がいましたら教えていただけると幸いでございます。

よろしくお願い致します。

Comments

[m.ts10806]

バグ、とは言いますが、プログラムは指示した通りにしか動かないのでご自身がそのように組んでしまっていることは認識しておいた方が今後のためになると思います。

[Nerd_run.]

指示をミスしている箇所がわからないので質問しています。

[dit.]

「希望通りに動かない」ことを「バグ」と表現するべきではない　という指摘かと思います。

[Nerd_run.]

ではなんと表現するべきなのでしょう？

[m.ts10806]

デバッグ実行で確かめるくらいはできるようになりましょう。質問して回答待つより早いこともよくあります。

[m.ts10806]

今回の場合は「出力が重複してしまう」だけで良いかと。いずれにしてもそのバグをおこしてしまってるのは自身に他ならないわけです。

[Nerd_run.]

System.in.read();の機能をよくわからず使っていた為に起こり、気づけなかったので、そういうところを指摘しているのでしょうか？

[Nerd_run.]

意図しない動きをバグと言わずなんと呼ぶべきなのかはわかりませんが。

[dit.]

私の「表現するべきではない」という言葉がいけませんでした。開発者の意図しない動作は確かにバグと呼べますね。バグという言葉に敏感になりすぎていたようです。（いつからか言われている「プログラムは思った通りに動かない、書いた通りに動く」という言葉を聞いてから、「仕様通りではないが記述どおりの動作」をなるべくバグと呼びたくなかったのですが、その考えを押し付けるような形になってしまいました。申し訳ないです。）

[Nerd_run.]

とんでもないです。　元はと言えば私の勉強不足です。　私にとってはバグでも、開発者の力量によっては即座に解消でき、バグと認識しないのかもしれません。ご指摘ありがとうございます。　日本語は難しいですね。

Answer 1

こんにちは

まず、ソースコードを見て気になったところを言いたいと思います。

1.Main関数にてfor( ; hp > 0; )とありますがなんとなく見づらいのでwhile(hp > 0)にしました。

2.関数command()の中にstatus()がありますが、Main関数にて

while(hp > 0){
     status();
     command();
}

としたとした方がいい気がします。

3.関数cmdのcase '3'のところにhp = hpにしていますが、これではhpは回復しません。
.static final int FIRST_HP = 30;などを宣言してhp = FIRST_HPにすべきです。
また、switch文なので一応defaultを書きましょう。

4.gameoverまたはgameclearしてもプルグラムが終了しないので
.static boolean gameover;
.static boolean gameclear;で判定するようにしました。

これ以外で変更したところ

入力をScannerにしました。これで解決するはずです。

以上です。ソースコードを載せておきます。

import java.util.*;


public class part18 {

        static String name = "ほげさん";
        static int lv = 30;
        static final int FIRST_HP = 30;
        static int hp = 30;
        static int gold = 10000;
        static boolean gameover;
        static boolean gameclear;

        public static void main( String[] args ){

            joshou();  
            
            while(hp > 0){
                status();
                command();
                if(gameover || gameclear){
                    break;
                }
            }
        }

        static void joshou(){                    //序章を出力

            System.out.println( "魔王が世界を滅ぼそうとしています。" );
        }

        static void status(){                    //ステータスを出力

            System.out.println("--------------------------------------");
            System.out.println( name + "  Lv :" + lv + " HP :" + hp + "  Gold :"  + gold + "G" );
            System.out.println("--------------------------------------");
        }

        static void command(){            //コマンドを表示

            System.out.println( " 1, 魔王を倒しに行く。" );
            System.out.println( " 2, 修行する。" );
            System.out.println( " 3, 宿屋に泊まる" );
            System.out.println("--------------------------------------");

            if(hp < 0){
                hp = 0;
                System.out.println( "HPが"+ hp + "になった。" );
                gameover();
            }

            cmd();
        }

        static void cmd(){                //コマンド処理設定

            Scanner sc = new Scanner(System.in);
            int c = sc.nextInt();

            switch(c) {

                case 1 :

                    if( lv < 40 ){
                        System.out.println( "魔王に敗れた。" );
                        gameover();
                    }
                    else{
                        gameclear();
                    }

                        break;

                case 2 : 

                    Random rnd = new Random();
                    int monster = rnd.nextInt( 5 )  + 1;
                    int damage = rnd.nextInt( 5 );
                    int ex = rnd.nextInt( 5 );

                    System.out.println( "敵が " + monster + "匹現れた。" );

                    System.out.println( name + "は" + damage + "のダメージを受けた。" );
                    hp -= damage;

                    System.out.println( "敵を倒した。" + name + "は、" + ex + "の経験値を手に入れた。" );
                    lv += ex;

                    System.out.println();          //改行
                    break;

                case 3 :

                    System.out.println( "HPが回復した。" );
                    hp = hp;
                    break;
                default:
                    break;
              
            }
        }                

        static void gameover(){            //GAME OVER画面を出力

            System.out.println( "GAME OVER" );
            gameover = true;
        }

        static void gameclear(){            //GAME CLEAR画面を出力

            System.out.println( "魔王を倒した！" );
            System.out.println( "GAME CLEAR" );
            gameclear = true;
        }
}

Comments

[Nerd_run.]

回答ありがとうございます。
自分で見ていても現状つっこみどころの多い処理ばかりで、Starsさんのご指摘も参考に動画を見ながら直していきます。
ありがとうございます。

Answer 2

System.in.read() で一文字づつ読み出してますが、２を読んだあと改行が読まれるので、２回繰り返してるように見えるのでしょう。
改行が入力されたときは無視する、とかする必要があります

Comments

[Nerd_run.]

回答ありがとうございます。
解決できました。