Komposition

[Shantox]

Hallo,
ich habe 2 Klassen gegeben: Verzeichnis und Ressource. Meine Aufgabenstellung ist: Bevor eine Ressource in ein neues Verzeichnis eingefügt werden darf, muss die aus ihrem bisherigen Verzeichnis, falls vorhanden, entfernt werden.
Meine Frage: Wie überprüfe ich, in diesem Fall, ob eine Ressource bereits in einem Verzeichnis ist?

public class Verzeichnis extends Ressource {
    protected Set<Ressource> inhalte;
    public Verzeichnis() {
        inhalte = new HashSet<Ressource>();
    }
    public boolean addInhalt(Ressource inhalt) {
        if (inhalte.add(inhalt)) {
            inhalt.setVerzeichnis(this);
            return true;
        }
        return false;
    }
    public boolean removeInhalt(Ressource inhalt) {
        if (inhalte.remove(inhalt)) {
            inhalt.setVerzeichnis(null);
            return true;
        }
        return false;
    }
    public Set<Ressource> getInhalte() {
        return Collections.unmodifiableSet(inhalte);
    }

public abstract class Ressource {
    protected Verzeichnis verzeichnis;
    public Verzeichnis getVerzeichnis() {
        return verzeichnis;
    }
    public void setVerzeichnis(Verzeichnis verzeichnis) {
        if (verzeichnis != null && !verzeichnis.getInhalte().contains(this)) {
            verzeichnis.addInhalt(this);
        }
        this.verzeichnis = verzeichnis;
    }
}

Mit freundlichen Grüßen

 

[Kababär]

Falls removeInhalt true zurückliefert? Wäre so mein Ansatz. Oder mit einer überschriebenen Equals-Methode prüfen.

 

[mrBrown]

Indem die Instanzvariable verzeichnis nutzt

 

[JStein52]

if (getVerzeichnis() != null) {
   ...
}

 

[Shantox]

Ich hab jetzt versucht in der addInhalt Methode die Komposition zu implementieren. Meine Idee ist es, falls die Ressource bereits in einem Verzeichnis ist, wird sie aus diesem gelöscht und in das neue Verzeichnis eingefügt.

public boolean addInhalt(Ressource inhalt) 
        if(getVerzeichnis() != null) {
            inhalt.setVerzeichnis(null);
            inhalt.setVerzeichnis(this);
        }
        else if(inhalte.add(inhalt)) {
            inhalt.setVerzeichnis(this);
            return true;
        }
        return false;
    }

inhalt.setVerzeichnis(null); ist nicht ganz korrekt oder? Damit tue ich ja das "aktuelle" Verzeichnis referenzieren und nicht das was gelöscht werden muss richtig?

 

[mrBrown]

inhalt.setVerzeichnis(null); ist einfach überflüssig, da du es direkt danach auf this setzt.
Allerdings ist dein if falsch - da musst du inhalt.getVerzeichnis() nutzen

 

[Shantox]

Aber irgendwie muss ich die Ressource doch aus dem aktuellen Verzeichnis löschen und in das neue packen. Mit inhalt.setVerzeichnis(this) stecke ich die Ressource ja ins neue Verzeichnis. Lediglich das löschen aus dem alten Verzeichnis fehlt richtig?

 

[mrBrown]

Ja, das fehlt noch

 

[Shantox]

verzeichnis.removeInhalt(Inhalt);

Lösche ich die Ressource damit aus dem alten Verzeichnis?

 

[mrBrown]

Ja, das sollte gehen

 

[Shantox]

Es klappt immer noch nicht wirklich. Die Ressource wird nicht aus dem alten Verzeichnis gelöscht. hmm

 

[mrBrown]

Wie versuchst du es denn?

 

[Shantox]

public boolean addInhalt(Ressource inhalt) {
        if(inhalte.add(inhalt) && inhalt.getVerzeichnis() == null) {
            inhalt.setVerzeichnis(this);
            return true;
        }
        else if(inhalt.getVerzeichnis() != null && inhalte.add(inhalt)) {
            verzeichnis.removeInhalt(inhalt);
            inhalt.setVerzeichnis(this);
            return true;
        }
        return false;
    }

So. Es funktioniert sonst alles bis aufs löschen

 

[mrBrown]

Mit verzeichnis.removeInhalt(inhalt); löscht du die Ressource aus dem Verzeichnis, in dem dein Verzeichnis, dem du die Resource hinzufügen willst, liegst. Du solltest die Ressource aber aus dem Verzeichnis, in dem sie liegt, löschen

 

[Shantox]

Kannst du mir nen kleinen Tipp geben? :/ Ich blick grad nicht durch.

 

[Harry Kane]

Das gehört an den Anfang deiner addInhalt-Methode:

Verzeichnis altesVerzeichnis = inhalt.getVerzeichnis();
if(altesVerzeichnis != null){
    altesVerzeichnis.removeInhalt(inhalt);
}