java エクリプスでのソースコードの実行をした際にエラーが出てきてしまいます。

Question

javanの勉強中にエクリプスでソースコードを実行していた際にこのようなエラーメッセージが出てきてしまい困っています。

Exception in thread "main" java.lang.StackOverflowError
at auderworld.Enemy1k.<init>(Enemy1k.java:12)　//1
at auderworld.Hero.<init>(Hero.java:14) //2
//1と//2に関しては何行も連なって出てきていますのでここでは割愛させていただきます。

ソースコードが問題なのかとも思いましたが、私が見る限りソースコードに不備は見当たりません。しかしソースコードも一応載せておきます。

package名は underworld
でクラスは３つです。

1 Main.javaクラス

package auderworld;

public class Main {
public static void main(String[] args) {
Hero h1 = new Hero();
Enemy1k e1 = new Enemy1k();
h1.appaer();
e1.appaer();
}
} //他のクラスを呼び出すクラス

2 Hero.javaクラス

package auderworld;

public class Hero {
String name ;
int hp;
int level;

Hero(){
	this.name = "ヤス";
	this.hp =100;
	this.level =1;
}
Enemy1k e = new Enemy1k();
void status() {
	System.out.println("勇者"+this.name+"\nlevel"+this.level+"\nhp"+this.hp);
}
void appaer() {
	System.out.println(this.name+"が現れた！");
}
void sleep() {
	this.hp +=100;
	System.out.println(this.name+"は眠った");
	System.out.println("hpは100回復した");
}
void slip() {
	this.hp -=5;
	System.out.println(this.name+"は、転んだ");
	System.out.println("5ダメージを受けた");
}
void sit(int sec) {
	this.hp += sec;
	System.out.println(this.name+"は座った");
	System.out.println("hpは、"+sec+"回復した");
}
void atack() {
	e.hp1 -=40;
	System.out.println(e.name1+"は、"+e.name1+"から攻撃を受けた");
	System.out.println(e.name1+"に４０のダメージ！");
}

}
// RPGゲームの主人公クラス

3 Enemy1k.java クラス

package auderworld;

public class Enemy1k {
String name1;
int hp1;
final int level1 =5;

Enemy1k(){
	this.name1 = "毒キノコ";
	this.hp1 = 60;
}
Hero h1 =new Hero();
void status() {
	System.out.println("敵"+this.name1+"\nHP"+this.hp1+"\nlevel"+this.level1);
}
void appaer() {
	System.out.println(this.name1+"レベル"+this.level1+"現れた!");
}
void atack() {
	h1.hp=40;
	System.out.println(this.name1+"の攻撃");
	System.out.println("毒ガス攻撃により"+h1.name+"のhpは40減少した!");
}
void escape() {
	System.out.println(this.name1+"逃げて行った");
}

}

//RPGゲームの敵役クラス

といったソースコードになります。ネットで検索してみましたがメモリーの使用上限を超えた際に出るエラーとの情報は得られましたがはっきりとした解決策が出てこずこの場を借りてご質問させていただきます。お忙しい中とは思いますが、どなたかアドバイスをよろしくお願いいたします。

Comments

[m.ts10806]

コードはマークダウンのcode機能を利用してください

Answer 1

ヒーローを new すると、インスタンス変数として敵が new され、
敵を new すると、インスタンス変数としてヒーローが new され、
これが無限に繰り返されますね。
まさに憎しみの連鎖ですね。

Enemy1k のインスタンス変数 Hero h1 は外部から与える、とかすると、とりあえず動くと思います。

public class Enemy1k {
  String name1;
  int hp1;
  final int level1 =5;
  Hero h1;

  Enemy1k(Hero hero){
    this.name1 = "毒キノコ";
    this.hp1 = 60;
    this.h1 = hero;
  }

Hero h1 = new Hero();
Enemy1k e1 = new Enemy1k(h1);

本当は attack メソッドの引数に攻撃対象として Hero とか Enemy1k のインスタンスを渡すのが良いと思いますが、そのへんは頑張ってください。
（atack はスペルミスです。）

Comments

[LouiS0616]

憎しみの連鎖 +1

[A-pZ]

同じく＋１

[退会済みユーザー]

英雄と魔物が循環することがおかしいけどね（どこの、まおゆう状態）

Answer 2

実行クラス、HeroクラスとEnemy1kクラスの循環参照が原因です。

クラス図を出力するプラグイン「AmaterasUML」をインストールしてみてはいかがでしょうか？
ソースコードからクラス図を作成してくれます。
クラス図を見れば循環参照がビジュアル的に分かると思います。

https://qiita.com/meio/items/e71dbcf32dbfce59a987

PS.
ソースを投稿する場合はコードタグを付けるようにしましょう。

コードのネストがなくなると非常に見づらいです。
特に長いコードだと読み間違いが発生しやすいです。
当然、回答の頻度がへる事になります。