メソッドの呼び出しについて

Question

前提・実現したいこと

生成したメソッドのフィールドをsetterメソッドを使って設定したいと思っています。

発生している問題・エラーメッセージ

"chを解決できません"
"oを解決できません"
"hを解決できません"

と表示される

該当のソースコード

public class Main {
	public static void main(String[] args) {
		Molecule o = new Molecule("O2", 5, 10);
		Molecule h = new Molecule("H2", 2, 8);
		Molecule ch = new Molecule("H2O");
	}
}

public class Molecule {
	private String name;
	private int weight;
	private int sec;

	public Molecule(String name) {
		this.name = name;
	}

	public  Molecule(String name, int weight, int sec) {
		this(name);
		this.weight = weight;
		this.sec = sec;
	}

	//getterメソッド
	public String getName() {
		return this.name;
	}
	public int getWeight() {
		return this.weight;
	}
	public int getSec() {
		return this.sec;
	}
	//setterメソッド
	public void setName(String name) {
		this.name = name;
	}
	public void setWeight(int weight) {
		this.weight = weight;
	}
	public void setSec(int sec) {
		this.sec = sec;
	}
	public void chemical(Molecule a1, Molecule a2) {
		System.out.println(a1 + "と" + a2 + "が化学反応を起こした！");
		ch.setWeight(o.getWeight() + h.getWeight());
	}
}

試したこと

ここに問題に対して試したことを記載してください。

補足情報（FW/ツールのバージョンなど）

Eclipseを使っています

Answer 1

半分勘ですけれど。

public class Molecule {
    ...

    public void chemical(Molecule a1, Molecule a2) {
        System.out.println(a1 + "と" + a2 + "が化学反応を起こした！");
-       ch.setWeight(o.getWeight() + h.getWeight());
+       setWeight(a1.getWeight() + a2.getWeight());
    }
}

chemicalメソッド内からは、mainメソッドの変数は参照できません。

メソッドの呼び出しについて

定義しているだけで呼び出していないですけれど、そのあたりは問題無く理解できているでしょうか。

---

また、各セッターの可視性はprivateにした方が良いでしょう。
あるいはそもそも必要無いのかもしれません。

このままでは外部からいくらでも質量を変更できる分子が爆誕します。

Comments

[nainai]

回答ありがとうございます。
たしかにそのとおりですね。
しかし、この場合chemicalメソッドを引数をoとhにして呼び出したらエラーで「メソッド chemical(Molecule, Molecule) は型 Molecule で未定義です」と出てしまいました。Moleculeクラスでchemicalメソッドを定義したはずなのですが、これはなぜなのでしょうか？

[LouiS0616]

chemicalメソッドはどこから呼び出しましたか？

[nainai]

public class Main {
public static void main(String[] args) {
Molecule o = new Molecule("O2", 5, 10);
Molecule h = new Molecule("H2", 2, 8);
Molecule ch = new Molecule("H2O");

ch.chemical(o,h);
}
}

このように呼びました

[LouiS0616]

再現できないです。https://wandbox.org/permlink/N2U2T986hMjP9R5I

Molecule.javaを上書き保存し忘れているとか、そもそも別のファイルをいじってしまっているとか、そういう凡ミスな気がします。再度ご確認ください。
ともすれば、キャッシュが悪さをしているのかもしれませんが。

[nainai]

Main.javaの保存し忘れでした。
すみません、ありがとうございます。

[LouiS0616]

解決されたようで何よりです。
よろしければ、BAをつけて質問をクローズしてください。