Kleine Frage zu OOP

[1337Avdu]

@jono das wird mir hier zu doof @mihe7 viel Spaß dir noch ich gehe schlafen wünsche dir ne angenehme nacht

 

[mihe7]

@1337Avdu Danke, gleichfalls

 

[jono]

1.meldung.abmelden(this)
2.meldung.anmelden(this)
:D

 

[jono]

Wie mache ich den Unterschied zwischen alt und neu im Code manoman

 

[mihe7]

Meintest Du nicht eben, dass Du den Unterschied verstehst?

 

[mihe7]

Du hast ein Familien-Objekt. Dieses hat ein Attribut, das nennt sich meldung. In diesem Attribut ist das Haus gespeichert, bei dem die Familie aktuell gemeldet ist. Wie kannst Du auf dieses Attribut in jedem Fall (innerhalb einer Methode der Klasse Familie) zugreifen?

 

[jono]

Ja, so oder
public boolean ummelden(Haus meldung) {
		if(meldung.anmelden(this)&&this.meldung.abmelden(this)) {
			return true;

 

[jono]

???

 

[jono]

@mihe7 Bist du noch wach ?? ^^

 

[mihe7]

Ja, jetzt mach aus #257 erstmal einen Zweizeiler ohne if.

 

[jono]

public boolean ummelden(Haus meldung) {
		meldung.anmelden(this)&&this.meldung.abmelden(this) return true;

 

[mihe7]

Ach komm, das kann doch nicht so schwer sein:

    meldung.anmelden(this);
    this.meldung.abmelden(this);

Nächste Baustelle: die Abmeldung soll nur dann ausgeführt werden, wenn die Anmeldung geklappt hat. Bitte umschreiben.

 

[jono]

Sorry du sagtest soll genau aus dem einen Zweizeiler machen (Missverständnis)

if(meldung.anmelden(this==true){
this.meldung.abmelden(this) return true;
}

 

[mihe7]

Formatier den Code vernünftig. Vergleiche mit true kannst Du Dir sparen.

if(meldung.anmelden(this)){
    this.meldung.abmelden(this);
    return true;
}

Und was musst Du zurückgeben, wenn die Anmeldung nicht geklappt hat?

 

[jono]

*if(Meldung.anmelden(this)==true

 

[jono]

Die Abmeldung muss zurückgegeben werden.

 

[mihe7]

Sag mal, jetzt setze ich das ANmeldung extra fett und Du faselst was von Abmeldung. Was muss die Methode zurückgeben, wenn die ANNNNmeldung nicht geklappt hat?

 

[mihe7]

BTW: es geht um das return...

 

[jono]

meldung.anmelden(this)

?

 

[jono]

return false;

 

[jono]

if(meldung.anmelden(this)==true && this.meldung.abmelden(this)==false) {
	this.meldung.abmelden(this);
}
return false;

Das müsste unter das hier:

if(meldung.anmelden(this)){
    this.meldung.abmelden(this);
    return true;
}

oder?

 

[mihe7]

Jetzt lass endlich diese UND-Verknüpfüngen weg. Das return false hat halt gefehlt:

if(meldung.anmelden(this)){
    this.meldung.abmelden(this);
    return true;
}
return false;

D. h. wenn die Anmeldung nicht funktioniert, wird false zurückgegeben.

Wenn die Anmeldung funktionierte, wird eine Abmeldung beim alten Haus versucht. Nächste Aufgabe: prüfe, ob die Abmeldung funktioniert.

 

[jono]

if(this.meldung.abmelden(this)==true){
meldung.anmelden(this)==true);
return true;
}
return false;

 

[jono]

Vergleiche kann man sich sparen

 

[mihe7]

Wo ist das if? Und rück den Code endlich vernünftig ein.

 

[jono]

Das if ist doch da

 

[mihe7]

Dein Code (mit Einrückungen):

if(this.meldung.abmelden(this)==true){
    meldung.anmelden(this)==true);
    return true;
}
return false;

Die zweite Zeile ist kein gültiger Code. Da fehlt ein if.

 

[jono]

Warum klappt das denn gerade nicht

 

[jono]

if(this.meldung.abmelden(this)==true){
   if( meldung.anmelden(this)==true);
    return true;
}
return false;

 

[jono]

Da steht unrechable Code

 

[jono]

	public boolean ummelden(Haus meldung) {
		if (meldung.anmelden(this)) {
			this.meldung.abmelden(this);
			return true;
		}
		return false;
		
		if(this.meldung.abmelden(this)==true){  // -> rot angestrichen
		  if (meldung.anmelden(this)==true);
		    return true;
		}
		return false;
	
	}

 

[jono]

Und wenn ich die unterste Klammer entferne dann wird nur noch angezeigt complete Statement mit } aber sobald die klammer da ist wieder unrechable code

 

[jono]

Noch da ?

 

[mihe7]

Ja, ich überlege nur gerade, wie ich Dich nicht in der Luft zerreiße.

 

[mihe7]

Du solltest in #271 einfach nur ein if bzgl. der Abmeldung einfügen. Das kann doch nicht so schwer sein.

 

[687b46c0da97]

Nunja, du solltest nun folgenden Code haben:

public boolean ummelden(Haus meldung) {
    if(meldung.anmelden(this)){
        this.meldung.abmelden(this);
        return true;
    }
    return false;
}

Dieser, wie du hoffentlich verstanden hast, führt die Meldung an dem Haus an, welches als Parameter übergeben worden ist. Falls das Erfolgreich war, werden die Zeilen innerhalb der If-Abfrage ausgeführt. Falls die anmeldung nicht erfolgreich war, gibt deine Methode "false" zurück.

Innerhalb der If-abfrage, versucht du, eine abmeldung am aktuellen Haus auszuführen. Allerdings prüfst du nicht ob diese Erfolgreich war, sondern gibst immer true zurück.

Somit solltest du nun, wenn die anmeldung erfolgreich war, den Rückgabewert von .abmelden() prüfen.

public boolean ummelden(Haus meldung) {
    if(meldung.anmelden(this)){
        this.meldung.abmelden(this); //Hier den Rückgabewert prüfen
        return true;
    }
    return false;
}

(Sorry ich kanns mir nicht verkneifen ... )

 

[jono]

public boolean ummelden(Haus meldung) {
		if (meldung.anmelden(this)) {
			this.meldung.abmelden(this);
			if(this.meldung.abmelden(this)==true){
			    meldung.anmelden(this);
			    return true;
			}
			return false;
			
		}
		return false;
		
		
	}

 

[jono]

Ist echt nicht normal, aber bin seit 5 Uhr wach und bin total am Ar...

 

[mihe7]

Ist echt nicht normal,

Das stimmt.

Du sollst einfach nur den Rückgabewert prüfen, ein einfaches if einfügen, mehr nicht:

    public boolean ummelden(Haus meldung) {
        if (meldung.anmelden(this)) {
            if(this.meldung.abmelden(this)){
                return true;
            }
            /* hier */
        }
        return false;
    }

Nach dem inneren if (EDIT: im Code mit /* hier */ markiert) machst Du jetzt noch die Anmeldung rückgängig, dann bist Du fertig.

 

[687b46c0da97]

Nun, aktuell hast du folgende Logik implementiert

Wenn (Am neuen Haus Anmelden) erfolgreich war
    Melde mich am aktuellen Haus ab
    Wenn (Melde mich am aktuellen Haus ab) erfolgreich war
        Melde mich am neuen Haus an
        Gebe "true" zurück
    Gebe "false" zurück
Gebe "false" zurück

Wenn (Am neuen Haus Anmelden) erfolgreich war
    Melde mich am aktuellen Haus ab
    Wenn (Melde mich am aktuellen Haus ab) erfolgreich war //Du meldest dich ein zweites mal ab. Braucht's das? Der alte Vermieter wird froh sein.
        Melde mich am neuen Haus an //Wieso meldest du dich nochmal an? Das machst du bereits in Zeile 1. Der Vermieter schüttelt nur mit dem kopf.
        Gebe "true" zurück
    Gebe "false" zurück //Das kannst du einfach weglassen, nachdem in der nächsten Zeile bereits false zurückgegben wird
Gebe "false" zurück

//Zuviele Köche machen die Suppe fett - ich halte mich raus

 

[jono]

@mihe Um es rückgängig zu machen, muss es ja return false zurückgeben nh?

 

[mihe7]

ich halte mich raus

Bitte nicht, ich möchte jetzt dann gehen

@mihe Um es rückgängig zu machen, muss es ja return false zurückgeben nh?

Du meldest Dich beim Studium an und wenn Du die Anmeldung rückgängig machen willst, sagst Du "false"?

 

[jono]

ne dann true

 

[jono]

Bin nicht mehr in der Lage was zu checken verzeih es mir würde gerne schlafen gehen aber muss die Aufgabe noch korrekt hinbekommen

 

[jono]

Ich bekomme trotzdem noch Fehlermeldungen...

 

[mihe7]

An der Rückgabe (return) musst Du nichts ändern. Eine Anmeldung machst Du durch eine Abmeldung rückgängig. Unglaublich.

 

[jono]

Ja gut dann kommt das dahin

this.meldung.abmelden(this);

 

[687b46c0da97]

Damit meldest du dich am alten Haus ab. Du sollst aber die Anmeldung am neuen Haus Rückgängig machen.

 

[mihe7]

Wenn die Abmeldung beim alten Haus nicht klappt, dann melde Dich beim alten Haus ab?

 

[mihe7]

So, und ich gehe jetzt. Gute Nacht.