Questa implementazione di esempio del pattern Observer è ben scritta?

notifyObservers è in effetti qualcosa di interno all'implementazione di Subject e non dovrebbe essere necessariamente pubblico.

È utile per una classe helper come ObserverSet . A cui inoltrare le chiamate registerObserver e removeObserver e chiamare internamente notifyObservers su.

Una classe osservabile, che ha i metodi di aggiunta / rimozione:

private List<MyObserver> observers;

public void addObserver(MyObserver o) {
    observers.add(o);
}

public void removeObserver(MyObserver o) {
    observers.remove(o);
}

I metodi di notifica, che non devono essere pubblici. Possono essere chiamati quando succede qualcosa, anche nella classe osservabile.

protected void notifySomething() {
    for (MyObserver o : observers) {
        o.onSomethingHappened();
    }
}

In addizione, l'interfaccia dell'osservatore. Puoi solo aggiungere oggetti che implementano questa interfaccia:

public interface MyObserver {
    public void onSomethingHappened();
}

Infine, avrei un'interfaccia generica osservabile in modo che qualsiasi classe osservabile la implementa:

public interface Observable<T> {
    public void addObserver(T o);
    public void removeObserver(T o);
}