どう直せば良いのか分からなくなってしまいました。

Question

###前提・実現したいこと
初心者なので、まだ理解していないところもあり説明が分かりづらい部分もあるかと思いますがご了承ください。
「スッキリわかるJava入門」を読み、Dokojavaを使いながら書いています。
あるクラスが書かれているページがあり、それを実現させるためにどうすればよいのかを考えて自分で他のクラスも書いて正常に動作できるように書いていましたが、どうしても
エラーの原因が分かりません。
ご指導のほうよろしくお願いします。
以下は元のコードです
（Hero.java）

public class Hero { 
  private String name = "ミナト";
  private int hp = 100;

  private void attack(Matango m){
    System.out.println(this.name + "の攻撃！");
    m.hp -= 5;
    System.out.println("5ポイントのダメージ！");
  }

  public void run(){
    System.out.println(this.name + "は逃げ出した！");
  }
}

###発生している問題・エラーメッセージ
上のコードに対してメインクラスと敵クラスを付けたし、private設定されているのでゲッター、セッターを使いながら正常に動作するのをめざし書いていたのですが、以下のエラーがずっと解決できません。

互換性のない型 期待値: java.lang.String 検出値: void
Hero.java(17行目)

###該当のソースコード
<Main.java>

public class Main { 
  public static void main(String[] args){
    Hero h = new Hero();
    Matango m = new Matango();
    System.out.println(m.getName() + "が現れた！");
    System.out.println("HP:" + m.getHp());
    System.out.println(h.getName() + "は戦うことにした！");
    System.out.println("HP:" + h.getHp());
    h.getAttack(m);
    h.run();
  }
}

<Hero.java>

public class Hero { 
  private String name = "ミナト";
  public String getName(){return this.name;}
  public void setName(String name){this.name = name;}

  private int hp = 100;
  public int getHp(){return this.hp;}
  public void setHp(int hp){this.hp = hp;}

  private void attack(Matango m){
    System.out.println(this.name + "の攻撃！");
    System.out.println("切りつける！");
    m.getHp();
    System.out.println(m.getName() + "に5ポイントのダメージ！");
    System.out.println("HP:" + m.getHp());
  }
  public String getAttack(Matango m){return this.attack(m);}

  public void run(){
    System.out.println(this.name + "は逃げ出した！");
    System.out.println("HP:" + this.hp); 
  }
}

<Matango.java>

public class Matango { 
  private String name = "お化けキノコ";
  public String getName(){return this.name;}
  public void setName(String name){this.name = name;}

  private int hp = 30;
  public int getHp(){return this.hp;}
  public void setHp(int hp){this.hp = hp;}
}

###試したこと
Heroクラスのどこかを直さないといけないのかなんなのか分からなくなってしまいました。
###補足情報(言語/FW/ツール等のバージョンなど)
より詳細な情報

Answer 1

attackの戻り値がないのに、getAttackでそれを返そうとしているからです。

private void attack(Matango m){
    System.out.println(this.name + "の攻撃！");
    System.out.println("切りつける！");
    m.getHp();  // ← これも不要
    System.out.println(m.getName() + "に5ポイントのダメージ！");
    System.out.println("HP:" + m.getHp());
}

...

public String getAttack(Matango m){return this.attack(m);}

---

ゲッターを積極的に使おうとすることは良いことですが、少し履き違えている気がします。
ゲッターでゲットするのは変数であって、単にメソッドを呼び出すときにはそのままでよいのです。

getAttackというメソッドは不要です。外部からattackを直接呼び出して良いでしょう。
その場合、attackのアクセス修飾子をprivateでなくす必要はありますが。

ついでに

ちょっと気になったことを。

• setterは出来るだけ作らない

アクセッサはアクセスを必要最低限に制限するためにあるので、出来るだけ絞った方が良いです。

	ゲッター	セッター
外部から参照されない	x	x
外部から参照される	o	x
外部から変更される必要がある	o	o

• 必要に応じて抽象的なクラスを導入する

まだクラスの継承について学習していないのかもしれませんが。
現状勇者がマタンゴ以外と戦えないので、Monsterクラスがあるといいですね。

質問の仕方について

teratailには、上記のようにコードを見やすく表示する機能があります。
質問編集画面を開き、コードを選択した状態で<code>ボタンを押してください。

Comments

[shuuji]

回答ありがとうございます！とても勉強になります。
無事に作動させることが出来ました^^
<code>ボタンもさっそく使って修正しました。

習ったことを手当たり次第試そうとしてしまいますね汗。

[LouiS0616]

質問編集の対応感謝します。
いろいろな機能やノウハウを試してみるのは良いことだと思いますよ。
なぜそのような書き方が必要なのか？というところまで理解していたら◎です。

[shuuji]

もう一つよろしいでしょうか？

無事に作動したと思ったらお化けキノコのHPが減っていませんでした。attackはpublicにしてありますが、Matangoはprivateなのでこれはゲッターを利用して呼び出そうとし　m.getHp() -= 5;　と書いてみましたがエラーメッセージで　”クラス Matangoのメソッド getHpは指定された型に適用できません。 期待値: 引数がありません 検出値: int 理由: 実引数リストと仮引数リストの長さが異なります”　とでてしまいました。変数を呼出し、その値を変更するにはどうすれば良いでしょうか？そもそも可能でしょうか？

[LouiS0616]

getHp()によって返ってくるのはただの数値です。
ですので、HPが30の場合、m.getHp() -= 5;は30 -= 5;と同じなのでコンパイルエラーになります。
m.setHp(m.getHp() - 5); と書く必要がありますね。

[LouiS0616]

ただし、やはり外部からセッターを使うのは出来るだけ避けたいです。
マタンゴにdamagedメソッドを作った方が綺麗かと思いますよ。
public void damaged(int damage) {
this.setHp(this.getHp() - damage);
}

[LouiS0616]

セッターはよほど必要性がない限り使わないと思っていいです。

[shuuji]

丁寧な解説ありがとうございます！
セッターについては参考にしている書籍でもゲッターと比べて説明がサラッとしていてあまり理解できてませんでした。damagedメソッドのバージョンでも試してみたいと思います！
ありがとうございました！