このコードをチェックしてください

Question

私のコードでエラーを見つけて修正するための助けが必要です。

import java.util.*; 

public class Main 
{ 
     public static void main(String[] args) { 
      Random r = new Random(); 
      Scanner s = new Scanner(System.in); 
      Fighter f1 = new Fighter(); 
      Fighter f2 = new Fighter(); 
      System.out.println("Enter the name of the first fighter:"); 
      String str=s.nextLine(); 
      f1.name=str.substring(0,1).toUpperCase()+str.substring(1).toLowerCase(); 
      System.out.println("Choose the style:\n1/ Boxing\n2/ Kick Boxing\n3/ MMA"); 
      int style1 = s.nextInt(); 
      switch (style1){ 
       case 1 :f1 =new Boxer(); 
        break; 
       case 2 :f1 = new Kickeboxer(); 
        break; 
       case 3 :f1 = new MMA_Fighter(); 
       break;} 
      System.out.println("Enter the name of the seconde fighter:"); 
      String str1=s.nextLine(); 
      f2.name=str1; 
      System.out.println("Choose the style:\n1/ Boxing\n2/ Kick Boxing\n3/ MMA"); 
      int style2 = s.nextInt(); 
      switch (style2){ 
       case 1 :f2 = new Boxer(); 
        break; 
       case 2 :f2 = new Kickeboxer(); 
        break; 
       case 3 :f2 = new MMA_Fighter(); 
       break;} 
      for (int rnd=1;rnd <=5;rnd++){ 
       f1.punch=r.nextInt(f1.power); 
       f2.punch=r.nextInt(f2.power); 
       f1.hp-=f2.punch; 
       f2.hp-=f1.punch; 
       if (f1.hp <= 0){ 
        System.out.println("Round "+rnd+" result:\n K.O !"); 
        break; 
       }else if (f2.hp <= 0){ 
        System.out.println("Round "+rnd+" result:\n K.O !"); 
        break; 
       }else{ 
        System.out.println("Round "+rnd+" result:\n"+f1.name+"'s health : "+f1.getHp()+"\n"+f2.name+"'s health : "+f2.getHp()+"\n **********");} 
      if (f1.hp>f2.hp){ 
       System.out.println(f1.name+" wins !"); 
      }else if(f1.hp<f2.hp){ 
       System.out.println(f2.name+" wins !"); 
      }else {System.out.println("It's a DRAW !");} 
     }//end for 
    }} 
class Fighter { 
     String name; 
     int hp; 
     int power; 
     int punch; 
     public int getHp() { 
      return hp; 
     } 
} 
class Boxer extends Fighter{ 
    int hp = 100; 
    int power = 20; 
} 
class Kickeboxer extends Fighter{ 
    int hp =80; 
    int power = 40; 
} 
class MMA_Fighter extends Fighter{ 
    int hp = 70; 
    int power = 50; 
} 

あなたの助けのためにそれを実行する場合、それは私にエラーを与える

Enter the name of the first fighter: 
Choose the style: 1/ Boxing 2/ Kick Boxing 3/ MMA 
Enter the name of the seconde fighter: 
Choose the style: 1/ Boxing 2/ Kick Boxing 3/ MMA 

    Exception in thread "main" java.util.InputMismatchException  at 
java.util.Scanner.throwFor(Unknown Source)  at 
java.util.Scanner.next(Unknown Source)  at 
java.util.Scanner.nextInt(Unknown Source) at 
java.util.Scanner.nextInt(Unknown Source) at Main.main(Main.java:26) 

感謝:)

Answer 1

あなたが呼び出すので、あなたがInputMismatchExceptionを得る：ここで コードですs.nextInt()ですが、入力をintとして解析できませんでした。ユーザーは数字以外の文字を入力している可能性があります。この例外をキャッチすることで、インタフェースをより使いやすくすることができます。あなたはスキャナオブジェクトは、ラインだけの整数部分を読み、それがキャリッジリターンを考慮していないs.nextInt();を呼び出す

boolean choseStyle = false; 
while (!choseStyle) { 
    try { 
     System.out.println("Choose the style:\n1/ Boxing\n2/ Kick Boxing\n3/ MMA"); 
     int style2 = s.nextInt(); 
     choseStyle = true; 
    } catch (InputMismatchException e) { 
     System.out.println("Please enter a number between 1 and 3."); 
    } 
} 

Answer 2

。だから次のs.nextLine()を呼び出すと、それは次の行を読み上げていない、実際には前に読んだ整数のリマインダ（キャリッジリターン）を読み込んでいる。 だからs.nextLine();

nextInt();各後

int style2 = s.nextInt(); 
s.nextLine(); 

を追加あなたはので、私はあなたの中に見ることができ相続に関連したものではIllegalArgumentException

Exception in thread "main" java.lang.IllegalArgumentException: bound must be positive 
    at java.util.Random.nextInt(Unknown Source) 
    at com.stackoverflow.Main.main(Main.java:49) 

があるでしょう、ラインの「ゴミ」をきれいにしますクラス（Boxer、Kickboxer、MMA_Fighter）。

class Fighter { 
    String name; 
    int hp; 
    int power; 
    int punch; 

    public int getHp() { 
     return hp; 
    } 

    public int getPower() { 
     return power; 
    } 

    public void setPower(int power) { 
     this.power = power; 
    } 

} 

class Boxer extends Fighter { 
    int hp = 100; 
    int power = 20; 

    public int getHp() { 
     return hp; 
    } 

    public int getPower() { 
     return power; 
    } 
} 

class Kickeboxer extends Fighter { 
    int hp = 80; 
    int power = 40; 

    public int getHp() { 
     return hp; 
    } 

    public int getPower() { 
     return power; 
    } 
} 

class MMA_Fighter extends Fighter { 
    int hp = 70; 
    int power = 50; 

    public int getHp() { 
     return hp; 
    } 

    public int getPower() { 
     return power; 
    } 
} 

はこれを解決するための別の方法はありますが、今の私はそれがうまくいくと思う：

それを修正するために私はあなたがこれを行うに推薦しました。 継承の詳細については、https://docs.oracle.com/javase/tutorial/java/IandI/subclasses.html