Casten oder vielleicht was anderes?

[bluepeople12]

Hallo Leute,

ich hab hier eine Bankaufgabe zu machen, d.h. ich habe eine Bank mit verschiedenen Sorten von Konten (normales, Sparkonto, Girokonto etc.). Und nun schreibe ich in der Bank eine Funktion die prüfen soll, ob ich einen bestimmten Transfer machen darf. Hier meine Funktion:

	//checkt ob ein Geldtransfer möglich ist oder nicht
	public boolean checkTransfer(Account accountFrom, Account accountTo, float amount) {
		if (accountFrom instanceof CheckingsAccount) {
			if(((CheckingsAccount)accountFrom.checkWithdraw(amount) == false)) {
				return false;
			}
		}
	}

Nur unterstreicht mir eclipse die innerste if-Verschachtelung. Kann ich das so schreiben wie das hier steht?

Edit:
Was noch wichtig ist zu wissen. Es gibt eine Oberklasse Account, zwei Klassen erben von ihr: SavingsAccount und CheckingsAccount. CheckingsAccount hat einen kreditlimit, d.h. es darf nicht mehr als ein bestimmter Betrag abgehoben werden. Die check-Funktion dazu hier:

	//checkt, ob man überhaupt etwas abheben kann
	public boolean checkWithdraw(float amount) {
		if (this.getBalance() - amount < this.creditLimit) {
			return false;
		} else {
			return true;
		}
	}

 

[SlaterB]

im Moment hast du in etwa kodiert

if (accountFrom instanceof CheckingsAccount) {
     CheckingsAccount ca = (CheckingsAccount)    [[[ accountFrom.checkWithdraw(amount) ]]];
            if(ca == false) {
                return false;
            }
}

was du aber brauchst ist

if (accountFrom instanceof CheckingsAccount) {
     CheckingsAccount ca = (CheckingsAccount)accountFrom;
     boolean b = ca.checkWithdraw(amount);
            if(b == false) {
                return false;
            }
}

bzw.

if (accountFrom instanceof CheckingsAccount) {
     CheckingsAccount ca = (CheckingsAccount)accountFrom;
            if(ca.checkWithdraw(amount) == false) {
                return false;
            }
}

so würde ich es dir empfehlen, wenn es sein muss lautet die Kurzvariante

        if (accountFrom instanceof CheckingsAccount) {
            if(((CheckingsAccount)accountFrom).checkWithdraw(amount) == false) {
                return false;
            }
        }

 

[Andi_CH]

Na ja, ist logischerweise Geschmackssache ;-)

if (((CheckingsAccount)accountFrom).checkWithdraw(amount)== false) // igitt ;-)
    return false;

if (!((CheckingsAccount)accountFrom).checkWithdraw(amount)) // sieht doch eleganter aus
    return false;

 

[VfL_Freak]

Moin,

Na ja, ist logischerweise Geschmackssache ;-)

if (((CheckingsAccount)accountFrom).checkWithdraw(amount)== false) // igitt ;-)
    return false;

if (!((CheckingsAccount)accountFrom).checkWithdraw(amount)) // sieht doch eleganter aus
    return false;

ja, das ist es in der Tat
Ich würde nämlich Deine Kommentare (zumindest im konkreten Fall) genau andersherum setzen ...
Stichwort: Wartbarkeit/Lesbarkeit des Codes
Eleganz ist hier wohl obsolet resp. sehr relativ ...

Gruß
Klaus

 

[muemmel_0811]

zum Thema Geschmack:

if (accountFrom instanceof CheckingsAccount) {
     CheckingsAccount ca = (CheckingsAccount)accountFrom;
            if(ca.checkWithdraw(amount) == false) {
                return false;
            }
}

würd ich das in der Anfängerecke bevorzugen

 

[timbeau]

Also ich finde innerhalb von if-Schleifen macht sich der "!" Operator schon sehr viel besser als auf true oder false zu prüfen. Wir können ja abstimmen

 

[Andi_CH]

	boolean error = !((!true)!=(!false));
	 if(!((!error)!=(!false)))
		 System.out.println("Ich bin dafür dass wir dagegen sind");
	 else
		 System.out.println("Ich bin dagegen dasss wir dafür sind");

oder gehört das in die Rätselecke?

 

[VfL_Freak]

	boolean error = !((!true)!=(!false));
	 if(!((!error)!=(!false)))
		 System.out.println("Ich bin dafür dass wir dagegen sind");
	 else
		 System.out.println("Ich bin dagegen dasss wir dafür sind");

oder gehört das in die Rätselecke?

:wuerg:

 

[Gastredner]

	boolean error = !((!true)!=(!false));
	 if(!((!error)!=(!false)))
		 System.out.println("Ich bin dafür dass wir dagegen sind");
	 else
		 System.out.println("Ich bin dagegen dasss wir dafür sind");

oder gehört das in die Rätselecke?

[c]Ich bin dafür dass wir dagegen sind[/c]
Lustig, aber wenn ich sowas in einem Projekt sehen würde...:autsch:

 

[Andi_CH]

[c]Ich bin dafür dass wir dagegen sind[/c]
Lustig, aber wenn ich sowas in einem Projekt sehen würde...:autsch:

Ih musste heute diesen Thread hervorsuchen um den jemandem zu zeigen, denn ich habe heute morgen folgendes Statement gefunden:

 if (success != true)

Ist noch harmlos aber einfach nicht elegant