Sinnvoller Rückgabewert

[Mehrle (Gast)]

Was ist sinnvoll für einen Rückgabewert.
Brauche jeweils einen für erfolgreich abgearbeitet sowie vorzeitig 
abgebrochen.

Sicherlich kann ich nun return 1 oder 0 machen, nur was ist so das 
gängige Verfahren?

return 1  // wenn ausgeführt wurde
return 0  // wenn es abgebrochen wurde

?

[Ulli N. (Gast)]

Meistens nimmt man negative Werte für Fehler und 0 für OK. Kannst du 
aber halten wie du willst.

Vielleicht willst du aber auch noch unterscheiden, wo der Fehler 
passiert ist, oder Fehlerklassen (z.B. tolerant weitermachen oder ganze 
Aktion abbrechen). Wenn du z.B. int als Rückgabetyp nimmst, kannst du 
damit ja schon eine ganze Menge "kodieren".

[Daniel A. (daniel-a)]

Nimm einen enum. Der lässt sich dann beliebig erweitern, falls zukünftig 
weitere statuscodes zurückgegeben werden, z.B. Pausiert

[Mehrle (Gast)]

Danke.
Umfangreiche Fehlercodes gibt es nicht. Es gibt nur die Unterscheidung, 
ob Wert kleiner oder größer ist als der gewünschte Bereich.
-2 // Wert unterschritten
-1 // Wert überschritten
 0 // Wert iO

reicht hier aus.

[Bernhard F. (bernhard_fr)]

typedef enum vorgang
{
     erfolgreich,       // entspricht 0
     unterschritten,
     ueberschritten
                        // moegliche Erweiterungen
}vorgang_t;

Klingt für mich übersichtlich und man muss nicht raten bzw zusätzlich 
kommentieren, warum jetzt die Funktion X auf -2 abgefragt wurde.

[Christian H. (netzwanze)]

Wenn Du in C programmierst und wirklich nur OK und Fehler zurückgeben 
willst, verwendest Du 0 für Fehler und 1 für OK.

Beispiel:

int funktionErfolgreich() {
   ...
   return 1;
}

if (funktionErfolgreich()) {
  ...
}

...
if (!funktionErfolgreich()) {
  ...
}

Auf diese Art sparst Du dir die Vergleichsoperation.

if (funktionErfolgreich() == 1) {

Liest sich nicht so flüssig.

Alternativ (also in Deinem Fall) kann man auch sehr gut ENUM verwenden.

if (torSchliessen() == TOR_GESCHLOSSEN) {

}
...
if (torSchliessen() == TOR_KLEMMT) {

}

Nimm sinnvolle Namen. Dann ist der Code auch flüssig lesbar.

Auch fallen Tippfehler schneller auf.

if (funktion() == -3) { ...

Ist so weit korrekt - auch wenn -3 nicht als Rückgabe existiert.
"TOR_KEMT" wird Dir aber gnadenlos um die Ohnen gehauen.

[eagle user (Gast)]

Christian H. schrieb:
> verwendest Du 0 für Fehler und 1 für OK.

sag bitte, dass das ein Tippfehler war.

[Walter Tarpan (Gast)]

eagle user schrieb:
>> verwendest Du 0 für Fehler und 1 für OK.
>
> sag bitte, dass das ein Tippfehler war.

Tja....da gibt es zwei Philosophien mit jeweils negierter Logik.

[eagle user (Gast)]

Walter Tarpan schrieb:
> eagle user schrieb:
>>> verwendest Du 0 für Fehler und 1 für OK.
>>
>> sag bitte, dass das ein Tippfehler war.
>
> Tja....da gibt es zwei Philosophien mit jeweils negierter Logik.

da können wir ja froh sein, dass Computer nur 0 und 1 kennen.
oh, Moment...


FunFact: unter VMS gilt: 1 = OK, 2,4,6... = Fehler, 0 = undefiniert :)

[Dirk B. (dirkb2)]

Mehrle schrieb:
> Danke.
> Umfangreiche Fehlercodes gibt es nicht. Es gibt nur die Unterscheidung,
> ob Wert kleiner oder größer ist als der gewünschte Bereich.
> -2 // Wert unterschritten
> -1 // Wert überschritten
>  0 // Wert iO
>
> reicht hier aus.

Dann kannst du auch
  -1 // Wert unterschritten
   0 // Wert iO
  +1 // Wert überschritten
nehmen.

Ist evtl. intuitiver