Java 戦闘 RPGみたいな

Question

Javaで戦闘プログラムを書きたいと思っています。
敵に遭遇（sysout("遭遇")）
while(自分のHPが0になるか敵のHPが０になるかで終わり)
選択肢(switch)
数値入力(BufferedReader)
case1:
①戦う（ダメーじ１０）
case2:
②魔法（ダメージ２０）
case3:
③逃げる（戦闘終了）

敵の行動
2回に一回攻撃(for i = 0; i<2; i++)
攻撃（ダメージ２５）
なにもしない（０）

クラスを分ける
自分クラス
名前
HP 100
たたかう　attack 10
魔法　　　attack 20
逃げる

敵クラス
HP 100
名前
たたかう attack 25
なにもしない attack 0

といった感じのざっくりとしたフローはかけたのですが
プログラムがうまく書けません。
主にダメージ計算がわかりません。
現在はこのような感じになっています。
どなたか教えてくださる方よろしくお願いいたします。

public class Main {
	public static void main(String[] args) throws NumberFormatException, IOException {
		Braver b = new Braver();
		Enemy e = new Enemy();
		b.setName("Braver");
		e.setEnemyname("Enemy");

		System.out.println("敵に遭遇しました。");
		while(e.getEnemyHP() <= 0 || b.getHP() <= 0){
			try {
				for(int i = 0; i<2; i++){
					BufferedReader br = new BufferedReader(new InputStreamReader(System.in));
					int num = Integer.parseInt(br.readLine());
					System.out.println(b.name+"の行動1:たたかう 2:まほう 3:にげる");
					switch(num){
					case 1:
						System.out.println("たたかう");
						System.out.println(e.getEnemyname()+"に10のダメージを与えた");

						if(e.getEnemyHP() <= 0){
							break;
						}


					case 2:
						System.out.println(("魔法"));
						System.out.println(e.getEnemyname()+"に20のダメージを与えた");
						if(e.getEnemyHP() <= 0){
							break;
						}
					case 3:
						System.out.println(("にげる"));
						System.out.println(b.getName()+"は逃げ出した");
						break;
					}
					System.out.println(e.getEnemyname()+"の行動");
					if(i == 1){
						System.out.println(e.getEnemyname()+"の攻撃");

						System.out.println(b.getName()+"に25のダメージ");
						if(b.getHP() <= 0){
							break;
						}
					}
					System.out.println("何もしなかった");
				}
			} catch (Exception e1) {
				e1.printStackTrace();
			}
		}
	}
}



public class Braver {
	int HP = 100;
	String name = "Braver";
	public int getHP() {
		return HP;
	}
	public void setHP(int hP) {
		HP = hP;
	}
	public String getName() {
		return name;
	}
	public void setName(String name) {
		this.name = name;
	}


}



public class Enemy {
	int enemyHP = 100;
	String enemyname = "Enemy";
	public int getEnemyHP() {
		return enemyHP;
	}
	public void setEnemyHP(int enemyHP) {
		this.enemyHP = enemyHP;
	}
	public String getEnemyname() {
		return enemyname;
	}
	public void setEnemyname(String enemyname) {
		this.enemyname = enemyname;
	}

}

Comments

[m.ts10806]

そもそもプログラムを書かれていない（提示されていない）ので「何が」「どう」うまく書けないのか見ている人には伝わりません。まず書いてみること、書いてみた結果、起きた事を質問に追記すること、自身が分かっている範囲と分かっていない範囲を明確にしてください（というか質問テンプレート使って書いて下さい）https://teratail.com/help/question-tips

[m.ts10806]

現在の質問だけだと「こうやりたいから誰かあとは書いてくれ」と言っているのとほぼ同等です。「やってほしいことだけを記載した丸投げの質問」として非推奨とされるものです。

[euledge]

「うまくかけない」といわれてもどこが問題なのか正解は質問者にしかわからないので回答に困ります。どのように動くのが正解で、それに対して今こうなっているということを質問に加えてください。

[m.ts10806]

以前の質問が解決しているように思えません。 https://teratail.com/questions/131087 私が指摘している部分も含めて解決しているのでしたら、ここで「うまくいかない」と仰っていることはだいぶ解決できるはずです

[m.ts10806]

私が最初のコメントで貼った「質問するときのヒント」を見ていただければ分かるはずですが、「うまくいかない」はこういった質問サイトでは禁句です。その表現では他人には何をやって何がおきたか伝わらないからです。「主に」も同様です。これまでの質問者さんの質問の感じから「ほぼ理解しないまま回答を得たことで分かった気になった状態で突っ走っているだけ」なのは明白です。問題が山積みであるのはご自身でも分かっているはずです。「主に」などとはぐらかさずに”自身が分かっている範囲と分かっていない範囲を明確にして”、もう一度きちんと「質問するときのヒント」を読み直して、質問テンプレートを利用して、質問を再編してください。

[m.ts10806]

まさか、ですが、全部同じファイルに記述してないですよね？通常、クラスは独自のファイルに定義されるもののはずですが。

Answer 1

System.out.println("たたかう");
System.out.println(e.getEnemyname()+"に10のダメージを与えた");
if(e.getEnemyHP() <= 0){
break;
}

と、あるので、10ダメージ分を減らせばよいです。

System.out.println("たたかう");
System.out.println(e.getEnemyname()+"に10のダメージを与えた");
e.setEnemyHP( e.getEnemyHP() - 10 );
if(e.getEnemyHP() <= 0){
break;
}

ということでいいのかな・・・

Answer 2

public class Main {
	public static void main(String[] args) throws NumberFormatException, IOException {
		//Braverのインスタンス
		Braver b = new Braver();
		//Enemyのインスタンス
		Enemy e = new Enemy();
		//BraverのBraverameに"Braver"をset
		b.setName("Braver");
		//EnemyのEnemynameに"Enemy"をset
		e.setEnemyname("Enemy");
		//"Braver"のHPを100に設定
		b.setHP(100);
		//"Enemy"のHPを100に設定
		e.setEnemyHP(100);
		//取ってきた値を変数化
		int enemyhp = e.getEnemyHP();
		int braverhp = b.getHP();
		//取ってきた名前を変数化
		String bravername = b.getName();
		String enemyname = e.getEnemyname();

		System.out.println(enemyname+"に遭遇しました。");
		//どちらかのHPが0より大きい時ループ
		loop1:
			while(enemyhp > 0 && braverhp > 0){
				//"Enemy"の攻撃設定ループ
				for(int i=0; i<2; i++){
					try {
						System.out.println(bravername+"の行動1:たたかう 2:まほう 3:にげる");
						BufferedReader br = new BufferedReader(new InputStreamReader(System.in));
						//読み込んだ文字を数値に変換してint numに格納
						int num = Integer.parseInt(br.readLine());
						//switch文
						switch(num){
						//numが1の場合 "Enemy"のHPを10減らす
						case 1:
							System.out.println("たたかう");
							System.out.println(enemyname+"に10のダメージを与えた。");
							enemyhp = enemyhp - 10;
							System.out.println(enemyname+"のHP"+enemyhp);
							break;
							//numが2の場合"EnemyのHPを20減らす"
						case 2:
							System.out.println(("まほう"));
							System.out.println(enemyname+"に20のダメージを与えた。");
							enemyhp = enemyhp - 20;
							System.out.println(enemyname+"のHP"+enemyhp);
							break;
							//numが3の場合ループを抜ける
						case 3:
							System.out.println(("にげる"));
							System.out.println(bravername+"は逃げ出した。");
							break;
						}
						if (num == 3){
							break loop1;
						}
						//"Enemy"のHPが0以下の時ループを抜ける
						else if(enemyhp <= 0){
							System.out.println(bravername+"は戦闘に勝利した。");
							break;
						}
						//"敵の行動"
						System.out.println(enemyname+"の行動");
						//iが0の時何もしない
						if(i == 0){
							System.out.println("何もしなかった");
						}
						//iが1の時"Braver"に攻撃する
						if(i ==1){
							System.out.println(enemyname+"の攻撃。");
							System.out.println(bravername+"に25のダメージ。");
							braverhp = braverhp - 25;
							System.out.println(bravername+"のHP"+braverhp);
							if(braverhp <= 0){
								System.out.println(bravername+"は戦闘に敗北した。");
								break;
							}
							break;


						}

					}

					catch (Exception e1) {
						e1.printStackTrace();
					}
				}
			}
	}
}

public class Enemy {
	int enemyHP;
	String enemyname;
	public int getEnemyHP() {
		return enemyHP;
	}
	public void setEnemyHP(int enemyHP) {
		this.enemyHP = enemyHP;
	}
	public String getEnemyname() {
		return enemyname;
	}
	public void setEnemyname(String enemyname) {
		this.enemyname = enemyname;
	}

}
public class Braver {
	int HP;
	String name;
	public int getHP() {
		return HP;
	}
	public void setHP(int hP) {
		HP = hP;
	}
	public String getName() {
		return name;
	}
	public void setName(String name) {
		this.name = name;
	}


}

Comments

[m.ts10806]

たった20分そこらで理解できたのでしょうか（ごめんなさい。信じられません）
結果がこのコードなら私の回答がベストアンサーに選ばれる意味が分かりません。
私のコードも自分が把握しているだけで1箇所は挙動がおかしいところがありますし。

下記、読まれましたか？
> ざっと作ってざっと確認しただけなのでどこかおかしいところあるかもしれませんが、冒頭に書いたとおり解説はしませんので自身で修正してください。それも勉強になるでしょう。
これで動いたからって分かった気にならないように気をつけてください。
本当に全て理解して自分でヒント無しで組めるようになってから「解決済み」としていただきたいのが本音です。

[m.ts10806]

何のためにクラス作ってget,setしているのかこれだと全く意味がないです。
オブジェクト指向必要ないですよ。ローカル変数に持ってしまうんだったら。
理解できてないなら解決済みにしないでください。

Answer 3

※あくまで興味本位で作ったものです。

なのでコードに対して解説はしません。（もともと丸投げ質問なので）

コードを見て気になったこと。

• HPとNameしか持ってないなら個別クラスは用意しなくて良い。大きくなったら別途親クラスもって継承させるとかになるんじゃないかな。
• 「ダメージを与えた」という文言だけで実際にHP減らしてないから延々と終わらない
• breakはswitch抜けるだけでは？
• whileの終了条件はどっちかのHPが0以下になったときだけ？「逃げ出した」じゃ終わらないの？
• 要件としてはBraverは先行で且つ2回連続行動できるということでしょうか。1回行動だけだと毎回魔法使ってもEnemyには勝てない。
• 残HP表示ないと次の行動考えられないのでは。

下記は興味本位で作っただけのコードです。
ざっと作ってざっと確認しただけなのでどこかおかしいところあるかもしれませんが、冒頭に書いたとおり解説はしませんので自身で修正してください。それも勉強になるでしょう。
これで動いたからって分かった気にならないように気をつけてください。
本当に全て理解して自分でヒント無しで組めるようになってから「解決済み」としていただきたいのが本音です。

Main.java

import java.io.BufferedReader;
import java.io.IOException;
import java.io.InputStreamReader;

public class Main {
	public static void main(String[] args) throws IOException {
		Character Braver = new Character(100, "Braver");
		Character Enemy = new Character(100, "Enemy");

		System.out.println("敵に遭遇しました。");

		boolean fin = false; //終了フラグ
		while (!fin) {
			for (int i = 0; i < 2; i++) {
				System.out.println(Braver.getName() + "の行動1:たたかう 2:まほう 3:にげる");
				BufferedReader br = new BufferedReader(new InputStreamReader(System.in));
				int num = Integer.parseInt(br.readLine());
				switch (num) {
				case 1:
					System.out.println("たたかう");
					System.out.println(Enemy.getName() + "に10のダメージを与えた");
					Enemy.setHP((Enemy.getHP() - 10));
					if (Enemy.getHP() <= 0) {
						fin = true;
					}
					break;
				case 2:
					System.out.println("魔法");
					System.out.println(Enemy.getName() + "に20のダメージを与えた");
					Enemy.setHP((Enemy.getHP() - 20));
					if (Enemy.getHP() <= 0) {
						fin = true;
					}
					break;
				case 3:
					System.out.println("にげる");
					System.out.println(Braver.getName() + "は逃げ出した");
					return;
				default:
					System.out.println(Braver.getName() + "は何もしなかった");
				}
				nowHp(Braver,Enemy);
			}

			if (!fin) {
				System.out.println(Enemy.getName() + "の行動");
				System.out.println(Enemy.getName() + "の攻撃");

				System.out.println(Braver.getName() + "に25のダメージ");
				Braver.setHP((Braver.getHP() - 25));
				nowHp(Braver,Enemy);
			}
			if (Braver.getHP() <= 0) {
				fin = true;
			}
		}
		if (Enemy.getHP() <= 0) {
			System.out.println(Enemy.getName() + "をやっつけた");
		}
		if (Braver.getHP() <= 0) {
			System.out.println(Braver.getName() + "は倒された");
		}
	}

	static void nowHp(Character Braver,Character Enemy) {

		System.out.println(Braver.getName() +" 残HP："+Braver.getHP());
		System.out.println(Enemy.getName() + "残HP："+Enemy.getHP());

	}
}

Character.java

public class Character {
	Character(int hp,String name){
		this.setHP(hp);
		this.setName(name);
	}

	private int HP = 0;
	private String Name = "";
	public int getHP() {
		return HP;
	}
	public void setHP(int hp) {
		HP = hp;
	}
	public String getName() {
		return Name;
	}
	public void setName(String name) {
		Name = name;
	}
}