observer thread safe

[usrr]

Hallo,

wie könnte ich das Observer-Pattern threadsafe programmieren?

public interface Receiver {
    void ping(Sender sender);
}

import java.util.ArrayList;

public class Sender {

    private ArrayList<Receiver> receivers;

    public Sender() {
        receivers = new ArrayList<Receiver>();
    }

    public synchronized void addReceiver(Receiver receiver) {
        this.receivers.add(receiver);
    }

    public synchronized void removeReceiver(Receiver receiver) {
        this.receivers.remove(receiver);
    }

    public synchronized void ping() {
        for ( Receiver receiver : receivers ) {
            receiver.ping(this);
        }
    }
}

Das Problem ist, dass es meiner Meinung nach nicht ausreicht die drei Methoden mit synchronized auszustatten.

Wie kann ich es erreichen, dass wenn eine der drei Methoden ausgeführt wird, dann keine andere gestartet werden kann? Also praktisch ein synchronized-Block um alle drei Methoden.

 

[SlaterB]

> Also praktisch ein synchronized-Block um alle drei Methoden.
genau das hast du bereits erreicht, schau dir doch die Definition von synchronized an, es geht nicht um die Codestelle, sondern den Monitor,

edit:
also inhaltlich gehts schon um Codestellen ("Synchronisation über kritische Abschnitte") ,
aber technisch betrifft es Monitore, das gesamte Objekt wird gesperrt

Galileo Computing :: Java ist auch eine Insel - 12 Einführung in die nebenläufige Programmierung

 

[usrr]

Danke. Ich wußte nicht, dass sich der Monitor immer auf das ganze Objekt bezieht. Werde wohl auf die Möglichkeit die Lock-Objekte bieten zurückgreifen (steht auch in der genannten Quelle).

 

[SlaterB]

> Ich wußte nicht, dass sich der Monitor immer auf das ganze Objekt bezieht.

nur in diesem Fall mit synchronized an der Methode, kann man auch anders machen,
wobei es dem ursprünglichen Text nach ja eigentlich genau gewünscht ist?

 

[usrr]

Ja, aber die Klasse macht ja noch mehr und es ist nicht notwendig alles mit nur einem Lock zu versehen.

Manchmal sind es auch nur einige Code-Zeilen einer Methode die geschützt werden sollen.