JAVAコンストラクタ

Question

<Sword.java>
public class Sword{
String name;
int damage;
}

<Hero.java>
public class Hero{
String name;
int hp;
Sword sword;

public void attack(){
    System.out.println(this.name + "は" + s.name + "で攻撃した！");
    System.out.println("敵に5ポイントのダメージをあたえた！");
}

}

<Main.java>
public class Main{
public static void main(String[] args){
Sword s = new Sword();
s.name = "炎の剣";
s.damage = 10;
Hero h = new Hero();
h.name = "ミナト";
h.hp = 100;
h.sword = s;
System.out.println("現在の武器は" + s.name);
h.attack();
}
}

実行結果として、

"ミナトは炎の剣で攻撃した！　敵に5ポイントのダメージを与えた！"

と出力したいのですが、コンパイル時にHero.javaの、System.out.println(this.name + "は" + s.name + "で攻撃した！");の箇所で、シンボルが見つけられないとのエラーが出てきてしまいます。

間違いを探してみましたが、どこが間違っているかわかりません。

どの箇所が違っているのでしょうか？

Comments

[amiya]

とりあえず、タイトルのif文は何の関係もないので、直したほうが良いかと思います。

[m.ts10806]

コードやエラーはマークダウンのcode機能を利用してご提示ください。
https://teratail.com/questions/238564

Answer 1

既出ですが変数名を修正すればよいです。
Mainを見ずに、Heroだけで考えれば、sという変数がおかしいと感じると思います。

<Hero.java>
public class Hero{
    String name;
    int hp;
    Sword sword; // ★ sでなく、sword

    public void attack(){
        //System.out.println(this.name + "は" + s.name + "で攻撃した！");
        System.out.println(this.name + "は" + sword.name + "で攻撃した！");
        System.out.println("敵に5ポイントのダメージをあたえた！");
    }
}

Answer 2

Javaの変数には、スコープと言うものがあり、
基本的にメソッドの中で定義した変数はそのメソッドの中でしか使えません。
(さらに、ブロック内で宣言した場合はそのブロックの中だけになります)

Sword s = new Sword();
と、sが宣言されているのはmainメソッドの中なので、
sはmainメソッドの中だけで使えるローカル変数です。

System.out.println(this.name + "は" + s.name + "で攻撃した！");
の存在するattackメソッドの中ではsという変数は存在していません。
その為、s.nameと書かれていても、
『sなんて見つからないよ』
というエラーになっています。

そのメソッドの内部で有効な変数を使ってください。

Answer 3

その場所では s という変数が定義されてません