Return

[Luk10]

Ich vestehe nicht wieso das return "" hier noch gebraucht wird ... ist doch eigentlich für jeden fall ein return da

    public String reifen () {
        if (erntezahl == 0) {
            erntezahl++;
            return "Die Früchte sind noch unreif ... es dauert sicher noch lange!";
        }
        else {
            if (erntezahl == 1) {
            erntezahl++;
            return "Die Früchte reifen herran";
        }
        else {
            if (erntezahl == 2) {
            erntezahl++;
            return "Die Früchte sind reif, bitte ernten!";
        }
     }
  }
    return "";
}

Danke luk

 

[Marcel_Handball]

Das liegt daran, da du in deinem letzten else Block noch eine if-Anweisung hast.

Wenn erntezahl z.B. gleich 3 ist, würde keiner deiner Bedingungen zutreffen.

 

[nrg]

für den Fall, wenn erntezahl weder 0 noch 1 noch 2 ist...

 

[Landei]

Noch ein paar Anmerkungen:

Die Klammerung der ifs ist furchtbar. Als "Standard" in solchen Fällen hat sich diese Form durchgesetzt:

if(x == y) {
  //tu was
} else if (a <= b) {
  //tu was anderes 
} else if (u > v) {
  //mache Handstand 
} else {
  //weiss nicht mehr weiter, ruf Mutti an
}

In deinem Beispiel wäre sowieso ein switch viel besser geeignet. Außerdem erhöst du in jedem Zweig die erntezahl, das könntest du einmal vor dem if oder switch erledigen (wobei dann natürlich dort der um eins erhöhte Wert abgefragt werden müsste). Aber ein Profi würde wohl schreiben:

static final String[] antworten = {"bla","laber","sülz"};

public String reifen () {
   return antworten[erntezahl++];
}

 

[Tomate_Salat]

Noch ein paar Anmerkungen:

Die Klammerung der ifs ist furchtbar. Als "Standard" in solchen Fällen hat sich diese Form durchgesetzt:

if(x == y) {
  //tu was
} else if (a <= b) {
  //tu was anderes 
} else if (u > v) {
  //mache Handstand 
} else {
  //weiss nicht mehr weiter, ruf Mutti an
}

oder

if(x == y) 
{
  //tu was
} 
else if (a <= b)
{
  //tu was anderes 
} 
else if (u > v) 
{
  //mache Handstand 
} 
else 
{
  //weiss nicht mehr weiter, ruf Mutti an
}

wobei man bei einzeilern die {} weglässt^^. Ich wollte auch gerade schreiben. Ich hab echt lange gebraucht, bis ich den code entziffert hatte. Das erinnert mich an meine alte infolehrerin die hatte so einen ähnl. stil:

if(foo) {
    bar();
    } else {
            foo();
            }

 

[eRaaaa]

oder

if(x == y) 
{
  //tu was
} 
else if (a <= b)
{
  //tu was anderes 
} 
else if (u > v) 
{
  //mache Handstand 
} 
else 
{
  //weiss nicht mehr weiter, ruf Mutti an
}

wobei man bei einzeilern die {} weglässt^^.

Na na na, das steht in den code conventions aber anders *g*
(http://java.sun.com/docs/codeconv/html/CodeConventions.doc6.html#430)

7.4 if, if-else, if else-if else Statements

The if-else class of statements should have the following form: 
if (condition) {
    statements;
}

if (condition) {
    statements;
} else {
    statements;
}

if (condition) {
    statements;
} else if (condition) {
    statements;
} else {
    statements;
}
 

Note: if statements always use braces {}. Avoid the following error-prone form: 
if (condition) //AVOID! THIS OMITS THE BRACES {}!
    statement;

 

[Tomate_Salat]

Na na na, das steht in den code conventions aber anders *g*

hehe, ach was die Brackets in einer einzlene Reihe sind aber durchaus gängig und da wir im Betrieb diese Konvention Einsetzen, durfte ich mir diesen Stil angewöhnen .

 

[Tharsonius]

Noch ein paar Anmerkungen:

Die Klammerung der ifs ist furchtbar. Als "Standard" in solchen Fällen hat sich diese Form durchgesetzt:

if(x == y) {
  //tu was
} else if (a <= b) {
  //tu was anderes 
} else if (u > v) {
  //mache Handstand 
} else {
  //weiss nicht mehr weiter, ruf Mutti an
}

Das kommt sehr darauf an.
Die ganzen if sind in einer Spalte, dass das ganze verschachtelt ist sieht man nicht auf den ersten Blick. Da ists deutlich besser jede Ebene einzurücken.

Bei uns in der Firma ist es sogar Vorgabe, dass 1. jede Ebene separat eingerückt wird und die Klammer auch nicht hinter der Zeile steht sondern eine extra Zeile bekommt:

if()
{
  // irgendwas
}
else
{
  if()
  {
    // was anderes
  }
  else
  {
    // nochwas
  }
}

Und von der Lesbarkeit muss ich sagen sehe ich auf ersten Blick in meiner Variante wie es zusammen hängt und zu welchem if / else welcher Block gehört und wie die alle verschachtelt sind.

 

[Tomate_Salat]

würde sagen, gewöhnungssache. Wie auch schon gesagt: bei mir im Betrieb muss (soll) ich das auch so ähnl. machen wie bei Tharsonius (ich darf else if benutzen ), vorher hatte ich es gemacht wie Landei. Am Anfang war es total ungewohnt, aber jetzt hab ich mich dran gewöhnt und finde den anderen Stil ungewohnt. Aber ich akzeptiere beides, da beides doch gut lesbar ist. Man erkennt bei Landei's variante zwar nichit an den Klammern genau wo eine Schachtelung ist, dafür aber an dem if-else. Aber das vom TO gepostet wurde (oder was meine alte info-Lehrerin fabriziert) geht garnicht

 

[Landei]

Das kommt sehr darauf an.
Die ganzen if sind in einer Spalte, dass das ganze verschachtelt ist sieht man nicht auf den ersten Blick. Da ists deutlich besser jede Ebene einzurücken.

Bei uns in der Firma ist es sogar Vorgabe, dass 1. jede Ebene separat eingerückt wird und die Klammer auch nicht hinter der Zeile steht sondern eine extra Zeile bekommt:

if()
{
  // irgendwas
}
else
{
  if()
  {
    // was anderes
  }
  else
  {
    // nochwas
  }
}

Und von der Lesbarkeit muss ich sagen sehe ich auf ersten Blick in meiner Variante wie es zusammen hängt und zu welchem if / else welcher Block gehört und wie die alle verschachtelt sind.

Würde bei uns nicht funktionieren, da haben wir oft fünf Blöcke und mehr, und die immer tiefer hineinzuschachteln wäre einfach grausam.