Problem beim Returnwert einer Methode

[Thesi]

Hi, ich bin blutiger Anfänger wenn es um das Programmieren geht und nun hab ich ein Problem mit einer Metode.
Ich bekomme ständig den Fehler, dass diese Methode einen Wert des Types Human zurückgeben muss.
Ich dachte eigentlich, dass sie das auch tut, aber anscheinend nicht.
Nur ich finde meinen Fehler nicht.
Könnt ihr mir vielleicht helfen.
Es liegt wohl an den Returnanweisungen: return girl/boy. Die werden anscheinend nicht anerkannt, aber warum?

public Human giveBirth(Male father, String nameGirl, String nameBoy) {
		if (father == null) {
			return null;
		} else {
			boolean GeschlechtKind = randomBool();
			if (GeschlechtKind == false) {
				this.father = father;
				Human girl = new Female(nameGirl, age = 0);
				return girl;
			}
			if (GeschlechtKind == true) {
				this.father = father;
				Human boy = new Male(nameBoy, age = 0);
				return boy;
			}
		}
	}

Wie kann ich mir den Jungen/das Mädchen zurückgeben lassen?


Gruß
Thesi

 

[maki]

Wenn keine der if Abfragen zutrifft, machst du auch kein return, schon mal aufgefallen?
Du musst sicherstellen dass immer etwas zurückgegeben wird.

 

[Thesi]

Oh man bin ich blind
So ein kleiner dummer Fehler und ich such da ne halbe Ewigkeit dran rum.
Vielen Dank!
So geht es jetzt

public Human giveBirth(Male father, String nameGirl, String nameBoy) {
        if (father == null) {
            return null;
        } else {
            boolean GeschlechtKind = randomBool();
            if (GeschlechtKind == false) {
                this.father = father;
                Human girl = new Female(nameGirl, age = 0);
                return girl;
            }
            if (GeschlechtKind == true) {
                this.father = father;
                Human boy = new Male(nameBoy, age = 0);
                return boy;
            }
            else {
                return null;
           }
        }
    }

Gruß
Thesi

 

[ARadauer]

alternativ:

if (GeschlechtKind == false) {
                this.father = father;
                Human girl = new Female(nameGirl, age = 0);
                return girl;
            }else{
                this.father = father;
                Human boy = new Male(nameBoy, age = 0);
                return boy;
            }

//wenns nicht false ist, kanns nur true sein ;-)

 

[SlaterB]

Denkanstoß:

    public Human giveBirth(Male father, String nameGirl, String nameBoy)
    {
        if (father == null) {
            return null;
        }

        this.father = father;
        age = 0; // welches age wird denn da auf 0 gesetzt?
        boolean weiblich = randomBool();
        if (weiblich) {
            return new Female(nameGirl, age);
        }
        return new Male(nameBoy, age);
    }

zumindest
> boolean weiblich
solltest du übernehmen, und dann ein if + vielleicht ein else,
if (true) {
}
if (false) {

} else {

}
ist ganz schlimm bei einem boolean der nur true oder false sein kann

 

[Thesi]

@SlaterB:

Es soll das age von dem Baby sein.
Aber wenn du so frägst wird es wohl ein anderes age sein ??

Kann man das
if (GeschlechtKind == true)
nicht einfach stehen lassen und wieso ist das bei einem boolean ganz schlimm?

 

[maki]

if (GeschlechtKind == true)

Ganz schlimm

Einerseits widerspricht es de Java CodeKonventionen (if (geschlechtKind == true) wäre besser), andererseits macht es doch gar keinen ersichtlichen Sinn eine boolean Variable Geschlecht zu nennen, woher sollte der Leser wissen das der Wert true für weiblich steht???
Da ist SlaterBs Vorschlag schon um längen besser

 

[Thesi]

@maki:

Ah ok, jetzt verstehe ich.
Man soll so schreiben, dass es auch andere leicht nachvollziehen können und da ist es dann z.b. leichter nachzuvollziehen, wenn boolean weiblich ist, weil das kann ja nur richtig sein oder falsch und meine Variable könnte ja auch einen String enthalten.

Aber auf sowas achte ich zur Zeit einfach noch nicht. Ich bin froh, wenn das Programm tut was es soll ^^

 

[maki]

Aber auf sowas achte ich zur Zeit einfach noch nicht. Ich bin froh, wenn das Programm tut was es soll ^^

Bald wirst du froh sein wenn du verstehst was der Code machen soll den du geschrieben hast

Je früher du auf so etwas achtest umso früher schreibst du sauberen Code

Es ist sehr einfach Code zu schreiben den ein Compiler versteht.. schwierig ist es Code zu schreiben, den Menschen verstehen.

 

[SlaterB]

Es soll das age von dem Baby sein.
Aber wenn du so frägst wird es wohl ein anderes age sein ??

das Baby bekommt sicherlich 0 als Parameter und speichert das dann vielleicht in seinen Variablen (wie im echten Leben )

aber wo steht die Methode, in der Klasse Human? ist das aktuelle Objekt die Mutter?
dann wird deren Age auch auf 0 gesetzt und der Vater der Mutter geändert,
das Baby erhält keine Information, wer sein/ ihr Vater oder Mutter ist

 

[Manuh]

Alternativ könntest du das

if(GeschlechtKind==true)

auch einfach als

if(GeschlechtKind)

schreiben, denn bei einer boolean prüft er dann einfach ob sie dann wahr ist.

Anders bei

if(!GeschlechtKind)

dann wird die Bedingung nur bei false ausgeführt

Gruß
Manuh

 

[Thesi]

Momentan hab ich noch genug Probleme einen Code zu schreiben, der das tut was er soll...
Besonders jetzt wo wir Klassen haben und nicht mehr alles in main ist. Da blick ich noch nicht so durch, wie ich dann auf die manche Attribute der verschiedenen Variablen zugreifen kann....

 

[Thesi]

das Baby bekommt sicherlich 0 als Parameter und speichert das dann vielleicht in seinen Variablen (wie im echten Leben )

aber wo steht die Methode, in der Klasse Human? ist das aktuelle Objekt die Mutter?
dann wird deren Age auch auf 0 gesetzt und der Vater der Mutter geändert,
das Baby erhält keine Information, wer sein/ ihr Vater oder Mutter ist

Die Methode steht in der Klasse Female, neben der es noch Male und Human gibt.
Sowohl Male als auch Female sind Unterklassen von Human.

Edit:

Jetzt weis ich auch was du mit dem age meintest. Ich hab das Alter der Mutter auch wunderschön auf 0 gesetzt.
Habs aber jetzt ausgebessert.
Danke für den Tipp SlaterB!!