Diverse Fragen (Style)

[DirkB]

Wutz schrieb:

Doch. Besser abstürzen (möglichst noch mit aussagekräftiger Meldung) als unbemerkt mit falschen Daten weitermachen.

Wenn man noch eine Fehlermeldung hin bekommt, kann man das Programm auch beenden.

[Diverses]

Nun gut, ob man das Programm jetzt crashen lassen will bzw bei falscher Eingabe beendet oder eine Fehlermeldung ausgibt hängt denke ich mal vom Programm ab.

Da ich hier in der Uni erst mal nur so kleine Snippets abgeben muss nehme ich mal stark an dass sowas wie user eingabe auf herz und Niere prüfen zu vernachlässigen ist.

Ich merke mir aber dass ich im Ernstfall schauen sollte, dass ich die Eingaben überprüfe bevor ich weiter mache.

[Diverses]

Mir kam jetzt noch eine Frage auf:

1. Tenärer Operator - ist der grundsätzlich böse oder darf man ihn dezent anwenden?

Bsp. return a == b ? c : b;

Das würde ich jetzt spontant schreiben, anstatt per if-else die jeweiligen return Werte hinzuschreiben.
C ist hierbei eine konstante also auch kein Ausdruck.

Hier gibts kein Gesetzt, mich würd interessieren ob Ihr persönlich das als unsauber sehen würde.

2. "Überladene" Rückgabewerte - Wir sollten hier mal (etwas her) ein kleines Snippet abgeben, in der "Freund"-Zahlen berechnet werden sollen.

So soll der Benutzer Ober- und Untergrenze eingeben und man soll dann für jede Zahl in diesem Bereich schauen, ob diese Zahl eine Freundzahl besitzt und wenn sie eine solche besitzt beide ausgeben (x ist freund von y).

Dazu schrieb ich mir also eine Funktion im Stil int friendOf(int x) .

Die returned den friend von x oder 0 wenn es keinen gibt.

Dazu hieß es, der Rückgabewert der Funktion sei "überladen" also mehrdeutig, nicht eindeutig, nicht intuitiv und daher schlecht / unsauber.

Würdet ihr da zustimmen? Ich bin das Verhalten eigentlich gewohnt - beispielsweise aus find() Funktionen usw.

Mir würde spontan dann auch gar nicht einfallen, wie es schöner / sauberer gelöst wäre.

[Techel]

1. Klar kannst du den benutzen, solange es nicht zu unübersichtlich wird.
2. Versteh ich nicht ganz...

[Diverses]

Danke für deine Antwort

Zu 1.:
Okay, dann bin ich beruhigt. Ich finde den an manchen Stellen eigentlich echt angenehm und auch übersichtlich zu benutzen.
Aber ich dachte, dass er eventuell allgemein verpönt ist, weil der Dozent damals schon sehr abgeneigt reagierte ("Wenn man den Operator verwendet, dann weiß man schon, dass was schief gelaufen ist - den benutzt man nur, wenn man gar nicht mehr weiter weiß").

Zu 2.:
So genau weiß ich das ehrlich gesagt auch gar nicht.

Ich denke, dass er mit "überladen" meinte, dass der Rückgabe-wert der Funktion zwei verschiedene Bedeutungen haben "Kann".
Einmal gibt er den "Freund" der übergebenen Zahl zurück und einmal fungiert er sozusagen als "hasFriend" indem er 0 zurück gibt, wenn kein "Freund" gefunden.

Find ich jetzt aber auch komisch, ich dachte das wäre Usus.

[SeppJ]

Zu 2: Das ist tatsächlich unschön, aus den genannten Gründen. Man kann in deinem Beispiel nicht unterscheiden, ob die Grenze nun 0 ist oder ob ein Fehler aufgetreten ist. Es wird manchmal trotzdem gemacht, vor allem bei älteren Funktionen aus der Standardbibliothek, die man sich nicht unbedingt zum Vorbild nehmen sollte.

Mögliche Abhilfen in diesem Fall:
-Funktionsergebnis über Funktionsparameter zurück geben und Fehlercode als Rückgabewert der Funktion
-Globaler Fehlerindikator, wie errno.
-sich mit longjmp so etwas wie die Exceptions aus anderen Sprachen selber programmieren

[Diverses]

Also tatsächlich gilt das, sehr interessant!

Die Alternativen überraschen mich allerdings etwas, nur um das jetzt richtig verstanden zu haben:

Vorzuziehen wäre in diesem Fall also solch eine Konstruktion:

int friendOf(int param, int* result) {
  // Success
  *result = friendValue;
  return 1;

  // failure
  return 0;
}

// main
if (friendOf(i, &result)) {
  printf("%d is friend of %d\n", i, result);
}

Finde ich überraschend und entgeht auch meiner Intuition, aber da ich auf keine wirkliche Programmier-Erfahrung zurückblicken kann, bin ich durchaus gewillt es zu glauben

Die Sache mit dem Exceptions nachbauen wäre wohl für Snippets ein overkill. Findet das denn sonst bei euch Verwendung?

[Diverses]

Krieg ich hier noch ein abschließendes Fazit bitte?

Ich würd sehr gerne wissen ob ich richtig verstanden habe, wie der bevorzugte Weg auszusehen hat!

[Wutz]

Immer sachte mit deinen Ansprüchen.
Du willst eine Druckbetankung als Extrakt der dir hier gegebenen Antworten angereichert mit allen Erfahrungen langjähriger Praktiker und vielleicht noch vollständige Auflistungen aus der Standardbibliothek, wo und warum es dort anders gemacht wird?
Was zahlst du?

[Diverses]

Ich hätte ehrlich gesagt eine abschließende Meinung zu der konkreten, genannten Thematik.

Ich brauche keine Auflistung irgendwelcher Standardfunktionen mit Erklärung.

SeppJ hatte hier auf mein konkretes Beispiel reagiert und bessere Alternativen genannt - ich würde jetzt nur noch gerne wissen ob diese richtig verstanden habe.

Ich weiß jetzt leider nicht, was dich zu deinem Beitrag bewegt hat, aber ich hoffe mal, dass es ehrliches Missverständnis wär.

[SeppJ]

Ja, dein Beispiel entspricht einer der drei genannten Methoden.

[Diverses]

SeppJ schrieb:

Ja, dein Beispiel entspricht einer der drei genannten Methoden.

Danke.

[TocToc]

@Diverses damit musst du leben. Wutz gibt immer so qualitativ hochwertige Antworten auf Leute die es nicht besser wissen wie er

[Diverses]

TocToc schrieb:

@Diverses damit musst du leben. Wutz gibt immer so qualitativ hochwertige Antworten auf Leute die es nicht besser wissen wie er

Alles klar

Solange nicht die Mehrheit der Meinung ist, dass ich Unverschämtes wissen wollte, bin ich beruhigt!

[Bitmapper]

int friendOf(int param, int* result) {
  // Success
  *result = friendValue;
  return 1;

  // failure
  return 0;
}

// main
if (friendOf(i, &result)) {
  printf("%d is friend of %d\n", i, result);
}

Du hast vergessen zu dereferenzieren:

printf("%d is friend of %d\n", i, *result);

[Diverses]

Bitmapper schrieb:

Du hast vergessen zu dereferenzieren:

Ne, ich übergebe doch &result .

[Bitmapper]

Diverses schrieb:

Bitmapper schrieb:

Du hast vergessen zu dereferenzieren:

Ne, ich übergebe doch &result .

Stimmt, dachte du hättest in der main auch einen pointer.