通用观察员 - 铸造问题

问题描述:

我试图做一个通用观察者模式,模仿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 { 
+0

这是一个非常糟糕的选择来命名具有相同名称从JDK一类一类,正是这种原因:你的IDE会倾向于自动导入JDK类,就像您的代码的任何读者的大脑一样。选择另一个名字! – Bohemian 2013-03-14 19:35:36

+0

谢谢。我的mystake。我想我现在应该休息一下。再次感谢你。 – 2013-03-14 19:38:01

+0

@Bohemian我不同意,有很多情况下框架使用更常见的名称(例如File)。当发生冲突时,Eclipse和Netbeans都会要求开发人员选择正确的类进行导入。 – 2013-03-14 19:40:54

您正在试图调用WeatherData.registerObserver(java.util.Observer)this作为参数。但作为be.intec.Meteo.Codemeteo.CurrentConditionsDisplay的一个实例,只实现了接口be.intec.Meteo.Interfaces.Observerbe.intec.Meteo.Interfaces.DisplayElement,导致编译错误。

一对夫妇的意见:

  • Java包应全部小写(使得它更容易,什么是类区分,什么是一个包)
  • 通常更容易,如果你的类名称不与其他类共享名称,以避免混淆,如本(be.intec.Meteo.Interfaces.Observer VS java.util.Observer
+0

-1,因为我认为第二个要点是可怕的建议。没有类命名冲突,你几乎不能使用任何框架。真正的问题是在选择进口品时经验不足/粗心。 – 2013-03-14 19:37:56

+0

我并不是说从不共享任何类名,但是我不会选择与JRE类共享的类名,因为它可能会导致混淆,并且在我看来会降低可维护性。特别是在代码中也使用'java.util.Observer'类。 – beny23 2013-03-14 19:57:31