私は観察者のパターンで学習して遊ぶプログラムを書いたが、正しく印刷されているデータに問題がある。ペーストの7-9行目のデータ入力を出力するはずですが、代わりに0,0,0を出力します。正しいデータが受信されないのはなぜですか?
メインクラス
public class ObserverPattern {
public static void main(String[] args) {
Subject weatherData = new Subject();
weatherData.setTemp(81);
weatherData.setHumidity(14);
weatherData.setWindSpeed(8);
Observer johnsIpad = new Observer(weatherData);
weatherData.notifyObserver();
System.out.println(johnsIpad.toString());
}
}
Subjectクラス
import java.util.ArrayList;
public class Subject {
// fields
ArrayList<Observer> observers;
public double temp;
public double humidity;
public double windSpeed;
// Constructor
public Subject() {
observers = new ArrayList<Observer>();
}
// Observer pattern methods
public void register(Observer o) {
observers.add(o);
}
public void unregister(Observer o) {
observers.remove(observers.indexOf(o));
}
public void notifyObserver() {
for (Observer observer : observers) {
observer.update(temp, humidity, windSpeed);
}
}
// set weather data
public void setTemp(double temp) {
this.temp = temp;
}
public void setHumidity(double humidity) {
this.humidity = humidity;
}
public void setWindSpeed(double windSpeed) {
this.windSpeed = windSpeed;
}
}
オブザーバークラス
public class Observer {
double temp;
double humidity;
double windSpeed;
Subject weatherData;
Observer(Subject weatherDataSource) {
weatherData = weatherDataSource;
}
public void update(double temp, double humidity, double windSpeed) {
this.temp = temp;
this.humidity = humidity;
this.windSpeed = windSpeed;
}
public String toString() {
return temp + "\n" + humidity + "\n" + windSpeed;
}
}
あなたは正しいですか?私はそれをコンストラクタに追加しました。 sidenoteとして、あなたがやったやり方やコンストラクタの中で行うことは、異なる結果をもたらします。コンストラクタでは、それは自動的に毎回発生することになり、望ましくない可能性があります。しかし、それがコンストラクタになく、毎回それを望んでいなければ、それを登録するのを忘れるのは簡単でしょう。それはおそらくそれが最初に登録される頻度に依存します。 – Supetorus
いいえ、コンストラクタに追加しないでください。私はこれが悪い考えであるいくつかの理由で上記にコメントしました。 –
(私が「上記」と言うとき、私は[OPの回答](http://stackoverflow.com/a/38316898/3788176)を参照しています)。 –