|
Anzeige
|
Funktion so ok?Hallo, ich habe folgende Funktion:
Die Funktion prüft, ob der Parameter Frequency.ist im zulässigen Bereich liegt. Dafür hat die Funktion 2 Rückgabewerte, jedoch drei Austrittspunkte: 1. Frequenz ok => GD_OK 2. Frequenz nicht ok, aber noch nicht lang genug nicht ok => GD_OK 3. Frequenz lang genug nicht ok => GD_ERROR Normalerweise hat ja eine Funktion nur einen Austrittspunkt oder geht das so in Ordnung?
Gast
#4987658
Ingo L. schrieb: > Normalerweise hat ja eine Funktion nur einen Austrittspunkt oder geht > das so in Ordnung? Spricht nix dagegen. Solange die Funktion das tut was du willst. Ich bevorzuge:
oder
Mehrere Returns bei mehreren Abbruchbedingungen finde ich auch klarer. Hängt von der im Projekt festgelegten "Coding Guidelines" ab. Ich habe lieber Funktionen mit nur einen Austrittspunkt un würde dies so angehen:
(und ich würde noch * Frequency.ist der Funktion als Parameter mitübergeben * GD_OK, GD_ERROR usw in einem enum typedef-en und das als Rückgabetype angeben * checken ob es nicht Frequency.ist >= ISLAND_GD_FREQUENCY_MIN sein muss.
Gast
#4987674
Das geht so in Ordnung, wenn ihr alle so programmiert.
Bei dieser Funktion wäre es mit einem Austrittspunkt aber
übersichtlicher.
Da Du z.B. mit geschweiften klammern "{}" sehr sparsam bist, nehme ich,
dass Du nicht mit Lint oder Compiler-Warnungen arbeitest. Daher wird es
auch kaum jemanden interessieren (also kein Peer-Review, Misra, ...)
Das explizite Setzen von ret in jedem Pfad erlaubt es statischen Analysetools festzustellen, ob man ein bewusstes setzen vergessen hat. Mit GD_OK als Default entfielen 2 Zeilen. Nochmal: Ich habe nichts gegen mehrere Austrittspunkte. Zumal es Anwendungen gibt, die Code und Schachteltiefe enorm reduzieren. Das ist aber hier nicht der Fall.
Gast
#4987675
...und viel länger sollte die Funktion aus Gründen der Übersicht IMHO auch nicht werden, wenn sie mehrere return-Statements enthält. Oder anders ausgedrückt: Längere Funktionen als ca. eine Bildschirmseite sollten IMHO nur dann mehrere return-Statements haben, wenn man das sowohl auf der ersten als auch auf der letzten Bildschirmseite der Funktion offensichtlich sieht. Etwa im Stil eines "Guards": http://stackoverflow.com/questions/36707/should-a-function-have-only-one-return-statement Sonst besteht zu leicht die Gefahr, dass man bei der Wartung nicht alle return-Statements findet. MfG, Arno Danke erstmal Eric B. schrieb: > * Frequency.ist der Funktion als Parameter mitübergeben Ok, habe ich gemacht, auch wenns nicht nötig ist, spart 2 Byte Flash ;) > * GD_OK, GD_ERROR usw in einem enum typedef-en und das als > Rückgabetype angeben Nee, ich mag enums nicht gerne > checken ob es nicht Frequency.ist >= ISLAND_GD_FREQUENCY_MIN sein > muss. Ja ok, is n Argument...
Gast
#4987682
Statt ret o.ä. mitzuschleppen, kann man einmal am Ende prüfen, ob die Fehlerbedingung gegeben ist:
Wieder mal erstaunlich wie viele Lösung es für so eine triviale Funktion geben kann... Antwort schreiben |
Anzeige
|
