私の宿題は、次のことを理解することです。
- このコードの機能
- どのような潜在的な問題があるか
コード
/// Enum
public enum Event {
TORNADO, THUNDERSTORM, TSUNAMI, RAIN, SNOW, WIND, COLD, HEAT;
public final static String CRITICAL = "Critical";
public final static String SEVERE = "Severe";
public final static String MEDIUM = "Medium";
public final static String LIGHT = "Light";
private String severity;
private String location;
public void setSeverity(String severity) {
this.severity = severity;
}
public void setLocation(String location) {
this.location = location;
}
public String toString() {
return super.toString() + ":" + severity + " at " + location;
}
}
///Interface
public interface IEventListener {
void eventReceived(Event event);
}
///Class
public class EventPublisher implements IEventListener {
Map<Event, Collection<IEventListener>> listeners = new HashMap<Event, Collection<IEventListener>>();
@Override
public synchronized void eventReceived(Event event) {
for (IEventListener listener : listeners.get(event)){
listener.eventReceived(event);
}
}
public synchronized void register(IEventListener listener, Event event){
Collection<IEventListener> list = listeners.get(event);
if (list == null) {
list = new LinkedList<IEventListener>();
listeners.put(event, list);
}
list.add(listener);
}
public synchronized void deregister(IEventListener listener, Event event) {
listeners.get(event).remove(listener);
}
public String toString() {
String ret = "";
Iterator<Event> i = listeners.keySet().iterator();
while (i.hasNext()) {
Iterator<IEventListener> j = listeners.get(i.next()).iterator();
while (j.hasNext()) {
ret = ret + i.next() + "::" + j.next() + "\n";
}
}
return ret;
}
}
eventReceived
コードを調べたところ、このコードは天気予報を作成し、 (Event
登録/登録解除可能) に基づいて気象条件を分類しようとしていることがわかりました。
ただし、コードに問題はありません。私の先生は問題があると主張します。誰でも私を助けることができますか?