Assoziationen

[nub12]

Hallo Leute,
ich bin ganz neu in Java und besuche eine HTL Informatik in Österreich.
Hier mein Problem:
Wir müssen aktuell mit Assoziationen arbeiten und nutzen aktuell BlueJ als Entwicklungsplattform.
Konkret:
Unsere Aufgabe ist es dass der jeweilige "Besitzer" ein Haus mit einem anderem Besitzer tauscht, hier ein Beispiel wie wir es im Unterricht gemacht haben, der ganze Code (unter anderem mit Hauskauf usw).

Problemstelle ist "public void tausche (Besitzer wem)":

public class Besitzer
{
    private double kassa;
    private Haus haus;

    public Besitzer()
    {
        setKassa(1000);
        setHaus(null);
    }

    public Besitzer(double kassa, Haus haus)
    {
        setKassa(kassa);
        setHaus(haus);
    }

    public void setKassa(double kassa)
    {
        this.kassa = kassa;
    }

    public void setHaus(Haus haus)
    {
        this.haus = haus;
    }

    public double getKassa()
    {
        return kassa;
    }

    public Haus getHaus()
    {
        return haus;
    }

    public void printBesitzer()
    {
        System.out.println("Besitzer:\t" + kassa + "\t" + haus);
    }

  public void tausche(Besitzer wem)
    {
        if (wem != null)
        {
            if (haus != null)
            {
                if (wem.getHaus() != null)
                {
                   wem.getHaus().setBesitzer(null);
                   wem.getHaus().setBesitzer(this);
              
               
                }
           
            }
            else
            {
                System.out.println ("Hat kein Haus!");
            }
        }
        else
        {
            System.out.println ("Kein Tauschpartner!");
        }
    }

    public void kauft(Haus neu)
    {
        if (neu != null)
        {
            if (haus == null)
            {
                if (neu.getBesitzer() != null)
                {
                    if (kassa >= neu.getPreis())
                    {
                        setKassa(kassa-neu.getPreis());
                        neu.getBesitzer().setKassa(neu.getBesitzer().getKassa()+neu.getPreis());
                        neu.getBesitzer().setHaus(null);
                        setHaus(neu);
                        neu.setBesitzer(this);
                    }
                    else
                    {
                        System.out.println("Fehler: kein Geld!");
                    }
                }
                else
                {
                    System.out.println("Fehler: Haus hat keinen Vorbesitzer!");
                }
            }
            else
            {
                System.out.println("Fehler: hat schon ein Haus!");
            }
        }
        else
        {
            System.out.println("Fehler: kein Haus!");
        }
    }

    public void bekommt(Haus neu)
    {
        if (neu != null)
        {
            if (haus == null)
            {
                if (neu.getBesitzer() != null)
                {
                    neu.getBesitzer().setHaus(null);                 
                }
                setHaus(neu);
                haus.setBesitzer(this);
            }
            else
            {
                System.out.println("Fehler: hat schon ein Haus!");
            }
        }
        else
        {
            System.out.println("Fehler: kein Haus!");
        }
    }
}

Das sieht dementsprechend so aus:

http://www.pic-upload.de/view-32061442/bj.png.html

Bin langsam verzweifelt, vielleicht kann mir ja wer helfen.
Achja, Schleifen, Arrays und co. haben wir noch nicht gemacht.

Wenn ihr mehr Infos braucht lasst es mich bitte wissen!
LG

 

[nub12]

Hallo,
sorry für Doppelpost, aber kann mir da niemand helfen?
LG

 

[Robat]

Was Genau funktioniert denn nicht ? Gibt es eine Fehlermeldung ?

 

[nub12]

Was Genau funktioniert denn nicht ? Gibt es eine Fehlermeldung ?

Es funktioniert nichts.
Nein es gibt keine Fehlermeldung.
Ich gehe mal davon aus der der gesamte "public void tausche(Besitzer wem)" falsch ist.

 

[mrBrown]

Beschreib doch mal in Worten, was passieren muss, damit zwei Häuser getauscht sind.

 

[nub12]

Nun, in BlueJ erstellen wir 2 Besitzer und 2 Häuser.
Besitzer 1 erhält Haus 1, Besitzer 2 erhält Haus 2.
Beim Tausch soll Besitzer 1 Haus 2 bekommen und Besitzer 2 Haus 1.

 

[mrBrown]

Beim Tausch soll Besitzer 1 Haus 2 bekommen und Besitzer 2 Haus 1.

Dann schreib doch mal diesen Teil in Code um.
Du hast zwei Häuser, jeweils eins von beiden Besitzern, und weist das jeweils dem anderem zu.

Den Code drum herum lass erstmal weg, den kann man danach noch ergänzen

 

[Jardcore]

Das hört sich nach einem einfachen Ringtausch an, bei dem du ein zusätzliches temporäres Objekt brauchst.

public void tausche(Besitzer wem) {
    Besitzer temp = new Besitzer(wem.getKassa(), wem.getHaus());

    wem.setHaus(this.getHaus());
    wem.getHaus().setBesitzer(this);

    this.setHaus(temp.getHaus());
    this.getHaus().setBesitzer(this);
}

Wobei man hier im setter von Haus (setHaus()) auch einfach mal den Besitzer setzen könnte.

Dann würde das so aussehen...

public void setHaus(Haus haus) {
    this.haus = haus;
    getHaus().setBesitzer(this);
}

public void tausche(Besitzer wem) {
    Besitzer temp = new Besitzer(wem.getKassa(), wem.getHaus());
    wem.setHaus(this.getHaus());
    this.setHaus(temp.getHaus());
}

Oder du machst das mit einem temoprären Haus.

public void tausche(Besitzer wem) {
    Haus tempHaus = new Haus(wem); // in Annahme, das Haus diesen Konstruktor hat.
    wem.setHaus(this.getHaus());
    this.setHaus(tempHaus);
}

 

[mrBrown]

Das hört sich nach einem einfachen Ringtausch an, bei dem du ein zusätzliches temporäres Objekt brauchst.

Aber doch keinen temporären Besitzer?

 

[Jardcore]

Aber doch keinen temporären Besitzer?

Haha Hatte meinen Post noch erweiter
Da Haus hier nicht gepostet wurde, hatte ich mich erstmal auf den Besitzer bezogen.

 

[nub12]

Hallo Leute,
so, im Unterricht hat der Professor das so gemacht:

public void tausche(Besitzer wem)
    {
        if (wem != null)
        {
            if (haus != null)
            {
                if (wem.getHaus() != null)
                {
                    Haus h;
                    double differenz;
                    differenz = wem.getHaus().getPreis() - haus.getPreis();
                    if (uberweisen(wem, differenz) == true)
                    {
                        haus.setBesitzer(wem);
                        wem.getHaus().setBesitzer(this);
                        h = haus;
                        setHaus(wem.getHaus());
                        wem.setHaus(h);
                    }
                }
                else
                {
                    System.out.println("Fehler: andere besitzt kein Haus!");
                }
            }
            else
            {
                System.out.println("Fehler: ich besitzte kein Haus!");
            }
        }
        else
        {
            System.out.println("Fehler: kein Besitzer!");
        }
    }

Inkl. Geldüberweisung.

Ich kann den Code zwar einigermaßen verstehen, aber von selber drauf kommen würde ich wohl kaum.
Gibt es zu solchen Assoziationen irgendwelche gute Lernmittel?

 

[mrBrown]

Ich kann den Code zwar einigermaßen verstehen, aber von selber drauf kommen würde ich wohl kaum.

Das ist nicht wirklich mehr als das, was du selbst oben schon gesagt hast: Das Haus von Besitzer 1 geht an 2, dann das Haus von 2 an 1.
Was da mehr ist, ist das setzen der Besitzer für die Häuser, sowas muss man einfach im Hinterkopf behalten (und wissen das es nötig ist, oftmals ist es das nicht) (oder schreibt die setHaus-Funkionen entsprechend, dann muss man das an der Stelle nicht beachten und wird die zig Doppelungen los).
(Und das Geld, aber dazu hätte man erstmal wissen müssen das es nötig ist)