通用观察员 - 铸造问题
我试图做一个通用观察者模式,模仿headFirst设计模式示例。在标有行的错误处出现???下面。通用观察员 - 铸造问题
错误消息指出:在类型主题的方法registerObserver(观察者)是不适用的参数(CurrentConditionsDisplay)
package be.intec.Meteo.Codemeteo;
import be.intec.Meteo.Interfaces.DisplayElement;
import be.intec.Meteo.Interfaces.Observer;
import be.intec.Meteo.Interfaces.Subject;
//import javax.servlet.annotation.WebServlet;
//@supressWarning("unchecked")
public class CurrentConditionsDisplay implements Observer, DisplayElement {
private float temperature;
private float humidity;
private Subject weatherData;
public CurrentConditionsDisplay(Subject weatherData) {
this.weatherData = weatherData;
weatherData.registerObserver(this); // ??? Error: The method registerObserver(Observer) in the type Subject is not applicable for the arguments (CurrentConditionsDisplay)
}
public void update(float temperature, float humidity, float pressure) {
this.temperature = temperature;
this.humidity = humidity;
display();
}
public void display() {
System.out.println("Current conditions: " + temperature
+ "F degrees and " + humidity + "% humidity");
}
}
接口1
package be.intec.Meteo.Interfaces;
import java.util.Observer;
public interface Subject {
public void registerObserver(Observer o);
public void removeObserver(Observer o);
public void notifyObservers();
}
接口2
package be.intec.Meteo.Interfaces;
public interface Observer {
public void update(float temp, float humidity, float pressure);
}
接口3
package be.intec.Meteo.Interfaces;
public interface DisplayElement {
public void display();
}
Weatherdata类
package be.intec.Meteo.Codemeteo;
import java.util.ArrayList;
import be.intec.Meteo.Interfaces.Observer;
import be.intec.Meteo.Interfaces.Subject;
public class WeatherData implements Subject {
private ArrayList observers;
private float temperature;
private float humidity;
private float pressure;
public WeatherData() {
observers = new ArrayList();
}
@Override
public void registerObserver(java.util.Observer o) {
observers.add(o);
}
@Override
public void removeObserver(java.util.Observer o) {
int i = observers.indexOf(o);
if (i >= 0) {
observers.remove(i);
}
}
@Override
public void notifyObservers() {
for (int i = 0; i < observers.size(); i++) {
Observer observer = (Observer) observers.get(i);
observer.update(temperature, humidity, pressure);
}
}
public void mesurementChanged() {
notifyObservers();
}
public void setMeasurements(float temperature, float humidity,
float pressure) {
this.temperature = temperature;
this.humidity = humidity;
this.pressure = pressure;
}
// other weather data methos here
}
Tester类
package be.intec.Meteo.Codemeteo;
import java.util.*;
public class WeatherStation {
public static void main(String[] args) {
WeatherData weatherData = new WeatherData();
CurrentConditionsDisplay currentDisplay = new CurrentConditionsDisplay(
weatherData);
// StatisticsDisplay statisticsDisplay = new
// StatisticsDisplay(weatherData);
// ForecastDisplay forecastDisplay = new ForecastDisplay(weatherData);
weatherData.setMeasurements(80, 65, 30.4f);
weatherData.setMeasurements(82, 70, 29.2f);
weatherData.setMeasurements(78, 90, 29.2f);
}
}
你Subject
接口导入错误Observer
类。
您有:
import java.util.Observer;
public interface Subject {
您需要:
import be.intec.Meteo.Interfaces.Observer;
public interface Subject {
您正在试图调用WeatherData.registerObserver(java.util.Observer)
与this
作为参数。但作为be.intec.Meteo.Codemeteo.CurrentConditionsDisplay
的一个实例,只实现了接口be.intec.Meteo.Interfaces.Observer
和be.intec.Meteo.Interfaces.DisplayElement
,导致编译错误。
一对夫妇的意见:
- Java包应全部小写(使得它更容易,什么是类区分,什么是一个包)
- 通常更容易,如果你的类名称不与其他类共享名称,以避免混淆,如本(
be.intec.Meteo.Interfaces.Observer
VSjava.util.Observer
-1,因为我认为第二个要点是可怕的建议。没有类命名冲突,你几乎不能使用任何框架。真正的问题是在选择进口品时经验不足/粗心。 – 2013-03-14 19:37:56
我并不是说从不共享任何类名,但是我不会选择与JRE类共享的类名,因为它可能会导致混淆,并且在我看来会降低可维护性。特别是在代码中也使用'java.util.Observer'类。 – beny23 2013-03-14 19:57:31
这是一个非常糟糕的选择来命名具有相同名称从JDK一类一类,正是这种原因:你的IDE会倾向于自动导入JDK类,就像您的代码的任何读者的大脑一样。选择另一个名字! – Bohemian 2013-03-14 19:35:36
谢谢。我的mystake。我想我现在应该休息一下。再次感谢你。 – 2013-03-14 19:38:01
@Bohemian我不同意,有很多情况下框架使用更常见的名称(例如File)。当发生冲突时,Eclipse和Netbeans都会要求开发人员选择正确的类进行导入。 – 2013-03-14 19:40:54