Programmierstil

Fragender schrieb:
> Mache ich die if-Bedingung dann direkt in die Funktion rein oder führe
> ich sie in diesem Fall nur bedingt aus, sodass die if klar ersichtlich
> ist.

Das kommt darauf an. Wie heißt die Funktion, was gibt sie vor zu tun, 
wie ist sie dokumentiert, was würde der Leser spontan erwarten wenn er 
das sieht, wie verhalten sich ähnliche Funktionen in anderen Teilen 
deines Codes, ist es naheliegend es so zu tun, wird es einfacher lesbar, 
führt es insgesamt zu weniger code?

Fragender schrieb:
> Was ist der gängige Programmierstil?

Egal, in welcher Progarmmiersprache, die Antwort auf solche Fragen ist 
immer 42.

Oliver

Fragender schrieb:
> ....
> Da gibt es eine Funktion, die Daten schickt, wenn sie ausgeführt wird.
> Jetzt möchte ich dazu programmieren, dass die Daten nur geschickt
> werden, wenn sie sich auch wirklich geändert haben.
> Mache ich die if-Bedingung dann direkt in die Funktion rein oder führe
> ich sie in diesem Fall nur bedingt aus, sodass die if klar ersichtlich
> ist.
> Was ist der gängige Programmierstil?

Mit anderen Worten: Du willst die function aufrufen und dann in der 
function prüfen ob sie was machen soll?

Ruf die function dann auf, wenn Du sicher bist dass Du sie brauchst!

Wenn es Dir irgendwie falsch vorkommt dann ist es wahrscheinlich falsch. 
Oder falsch dokumentiert. Oder falsch benannt. Scheib es so hin daß es 
offensichtlich wird und sich richtig anfühlt.

Wenn die Funktion in etwa "writeData()" heißt, würde ich als 
Außenstehender erwarten, dass die Daten immer geschrieben werden, sobald 
ich die Funktion aufrufe. "updateData()" oder "writeNewData()" klingen 
für mich eher so, dass nur geänderte Daten geschrieben werden.

Je, nachdem, wie oft welcher Fall benötigt wird (alle Daten schreiben 
oder nur Neue) könnte man der writeData() eine Wrapper-Funktion 
schreiben, die dann eben writeData() nur aufruft, wenn neue Daten da 
sind. Was natürlich wieder Sprünge und Zeit braucht.

Insgesamt ist es nicht festgelegt, wie man es umsetzt. Es sollte nur für 
andere ersichtlich sein, was eine Funktion tut.

Fragender schrieb:

> Was ist der gängige Programmierstil?

Je nach Komplexität kannst Du da auch 3 Funktionen draus machen:
- Eine die Sendet
- Eine die Daten vergleicht
- und eine, die Daten vergleicht und Kopie / Hash der Daten erstellt und 
sendet.

ausserdem schrieb:
> Sorry, ich bin kein fernsehgucker. Musste erst Wikipedia fragen, was 42
> bedeutet. Kryptisches mit Kryptischem erklären? Hilft das?
Zur 42: https://de.wikipedia.org/wiki/Per_Anhalter_durch_die_Galaxis

Was dir der Schreiber damit sagen wollte: Ohne
ein paar Code-Schnipsel ist es sehr schwer, darauf
eine Antwort zu geben. Warum schreibst du den
Code nicht so hin, wie du denkst und fragst uns
dann hier um unsere Meinung? Wäre einfacher.

merciless

42 - die Antwort auf die Frage der Fragen : Was ist der Sinn von Allem.
Siehe : "Hitchhiker guide through the galaxis", oder so
Stellt die ultimaive Antwort auf fast alle Fragen dar.

Die Frage der Fragen wurde dann anders gestellt. Die Frager im Buch 
haben dann die Erde gebaut und der Prozess zur Beantwortung laeuft immer 
noch.

G4st schrieb:
> Ach und wenn jemand mit "Kommt drauf an" antwortet, brauchst du gar
> nicht erst weiterlesen. Diese Leute erwarten eine vollständige
> Spezifikation bevor sie sich herablassen eine Antwort zu geben. Wenn man
> allerdings alles im Detail spezifizieren kann, braucht man eine Antwort
> in der Regel auch nicht mehr.

Wenn ich jetzt frage, was ich für mein Auto tanken muss, ist, denke ich, 
die Aussage durchaus berechtigt, dass es auf mein Auto ankommt, dass ich 
habe. Die Randbedingungen zum Problem zu benennen ist niemals falsch und 
ja, oft ergibt sich die Lösung aus den Randbedingungen ganz von alleine. 
Der Spruch, dass es hilfreich sei, mal einen Schritt zurück zu machen 
und das ganze Bild zu betrachten, kommt nicht von ungefähr ;)

Fragender schrieb:
> Ich wollte mal kurze Frage zum Design des Codens stellen.

"Design des Codens" ... aua!

> Da gibt es eine Funktion, die Daten schickt, wenn sie ausgeführt wird.

Ähm. Die "Daten die eine Funktion schickt, wenn sie ausgeführt wird", 
nennt man üblicherweise Returnwert oder - wenn man zwanghaft 
eindeutscht - Rückgabewert. Und der Typ des Rückgabewerts ist fest. Er 
ist ein Teil der Funktions→Signatur, ebenso wie der Funktionsname.

> Jetzt möchte ich dazu programmieren, dass die Daten nur geschickt
> werden, wenn sie sich auch wirklich geändert haben.

Eine Funktion kann nicht "nichts" zurückgeben. Wenn der Rückgabewert 
z.B. vom Typ int ist, dann muß sie einen int zurückgeben. Was du 
aber machen kannst: du kannst einen bestimmten Wert, z.B. 0 dahingehend 
festlegen, daß er "nichts geändert" bedeutet.

> Mache ich die if-Bedingung dann direkt in die Funktion rein oder führe
> ich sie in diesem Fall nur bedingt aus, sodass die if klar ersichtlich
> ist.

Man kann eine Funktion nicht "bedingt ausführen". Wenn du außerhalb der 
Funktion feststellen kannst, daß sich nichts geändert hat, dann brauchst 
du die Funktion natürlich nicht aufzurufen. Aber dann ist deine ganze 
Frage sinnlos. Wenn wir hingegen mal annehmen, daß die Entscheidung, 
ob sich etwas geändert hat, nur innerhalb der Funktion getroffen 
werden kann, daß mußt du die Funktion in jedem Fall aufrufen. Und 
brauchst dann natürlich ein if(), um auszuwerten ob die Funktion 
vielleicht "es hat sich nichts geändert" zurück gegeben hat. Und 
innerhalb der Funktion ein weiteres if(), damit sie das tut.

Im Normalfall wird man diese Prüfung so früh wie möglich machen, damit 
man im Fall des Falles möglichst viel Code einsparen kann und die 
Sonderbedingung über möglichst wenige Aufrufebenen transportieren muß.

Name H. schrieb:
> der Prozess zur Beantwortung laeuft immer
> noch.

Nein. Die Erde wurde fünf Minuten vor Ablauf von den Vogonen gesprengt, 
um einer Hyperraumumgehungsstraße Platz zu machen.

Axel S. schrieb:
>> Da gibt es eine Funktion, die Daten schickt, wenn sie ausgeführt wird.
>
> Ähm. Die "Daten die eine Funktion schickt, wenn sie ausgeführt wird",
> nennt man üblicherweise Returnwert oder - wenn man zwanghaft
> eindeutscht - Rückgabewert. Und der Typ des Rückgabewerts ist fest. Er
> ist ein Teil der Funktions→Signatur, ebenso wie der Funktionsname.

Ich denke eher, dass Fragender wirklich eine Funktion meint, die Daten 
irgendwo hinsendet. Da kann es durchaus sinnvoll sein, diese nur bei 
Änderung neu zu übertragen, wenn die Gegenstelle die Daten speichert.

svensson schrieb:
>> Eine Funktion kann nicht "nichts" zurückgeben.
> In C kann eine Funktion auch vom Typ "void" sein, dann gibt sie keine
> Daten zurück.

Dann gibt sie niemals irgendwas zurück.
Und die Frage des TE erübrigt sich.


Alex D. schrieb:
> Axel S. schrieb:
>> Die "Daten die eine Funktion schickt, wenn sie ausgeführt wird",
>> nennt man üblicherweise Returnwert oder - wenn man zwanghaft
>> eindeutscht - Rückgabewert.
>
> Ich denke eher, dass Fragender wirklich eine Funktion meint, die Daten
> irgendwo hinsendet.

Oh. DU meinst im Sinne von: "eine Funktion, die eine Ein-/Ausgabeinheit 
(z.B. UART) bedient, um Daten zu verschicken". Hmm, ja. So kann man das 
auch interpretieren. Fand ich angesichts der Wortwahl des TE jetzt nicht 
naheliegend.

Aber auch dann gilt, was ich schrieb:
> Im Normalfall wird man diese Prüfung so früh wie möglich machen, damit
> man im Fall des Falles möglichst viel Code einsparen kann und die
> Sonderbedingung über möglichst wenige Aufrufebenen transportieren muß.

Thomas E. schrieb:
> Die Erde wurde fünf Minuten vor Ablauf von den Vogonen gesprengt,
> um einer Hyperraumumgehungsstraße Platz zu machen.

<Trump> So sad! </Trump>

svensson schrieb:
> Dann ist es streng genommen keine Funktion mehr, sondern eine Prozedur.

In C gibt es keine Prozeduren.

BabyYouCanDriveMyCar schrieb:
>> Dann gibt sie niemals irgendwas zurück.
>
> Meine Nachbarin ist dann auch eine Funktion. Die gibt auch nie wieder
> etwas zurück.

Hat deine Nachbarin dann wenigstens Seiteneffekte? Hat sie irgendeine 
Wirkung auf Dich? Denn wenn nicht kannst Du sie ersatzlos wegoptimieren!