Wer kann mir dieses Define erklären?

OP #4717131
Hey Leute,

wer wäre mal so nett und erklärt mir das folgende #define?

#define TEST()  (PORTD &= ~(1<<PD1)); (PORTD |= ( (1<<PD3) | (1<<PD7) ))

Die Trennung duch Semikolom im define ist mir unbekannt. Ich würde 
denken, das es folgendes bedeutet:


PD1 auf 0 setzen, ansonsten nichts verändern

und

PD3 und PD7 auf 1 setzen, ansonsten nichts verändern.

Richtig soweit?

Wie wäre es aber, wenn ich nicht nur an PORTD etwas verändern will 
sondern auch an PORTB?

Kann ich dann in einem #define in einer Zeile so schreiben (sofern meine 
Interpretation überhaupt richtig ist):

#define TEST()  (PORTD &= ~(1<<PD1)); (PORTD |= ( (1<<PD3) | (1<<PD7) 
)); (PORTB |= (1<<PB3))

?
Moderator Persönliche Seite
Hans R. schrieb:

> Die Trennung duch Semikolom im define ist mir unbekannt.

Nun, die rechte Seite eines #define ist ja weiter nichts als das,
was bei Aufruf des Makros dann als Text stattdessen eingesetzt
wird.  Insofern kannst du natürlich dort erst einmal beliebige
Zeichen stehen haben.

Allerdings ist ein Semikolon innerhalb runder Klammern in der Tat
seltsam, und in vielen Fällen dürfte die Verwendung dieses Makros
wohl zu einem Syntaxfehler führen.

> Ich würde
> denken, das es folgendes bedeutet:

Das war wohl die Intention.

Besser würde man es so schreiben:
1
#define TEST() do { PORTD &= ~(1<<PD1)); (PORTD |= ( (1<<PD3) | (1<<PD7) ); } while(0)

Diese do … while(0)-Konstruktion erscheint auf den ersten Blick
seltsam, hat aber den Vorteil, dass sie syntaktisch in der Tat ein
einziges Statement ist.  Da sie einen Block innerhalb geschweifter
Klammern beinhaltet, darf man darin wiederum weitere Statements mit
Semikolon trennen.

Der Vorteil dieses Konstrukts kommt vor allem dann zum Tragen, wenn
man schreiben möchte:
1
if (bedingung)
2
  TEST();
3
else
4
  MACHWASANDERES();

> Wie wäre es aber, wenn ich nicht nur an PORTD etwas verändern will
> sondern auch an PORTB?

Welche Sub-Statements man da mit Semikolons trennt, ist dem Compiler
am Ende wurscht.
Gast
Hans R. schrieb:
> wer wäre mal so nett und erklärt mir das folgende #define?
>
> #define TEST()  (PORTD &= ~(1<<PD1)); (PORTD |= ( (1<<PD3) | (1<<PD7) ))

Der Präprozessor ersetzt auf Grund dieser /define/-Direktive in deinem 
Quelltext
1
TEST()
 durch
1
(PORTD &= ~(1<<PD1)); (PORTD |= ( (1<<PD3) | (1<<PD7) ))
bevor er an durch den Compiler geschickt wird.
Persönliche Seite
Spätestens, wenn das Macro so verwendet wird
1
if (condition)
2
  TEST();
 
macht man ne Bauchlandung.  Falls eine inline-Funktion wie
 
1
static inline void __attribute__((__always_inline__))
2
TEST (void)
3
{
4
  PORTD &= ~(1<<PD1);
5
  PORTD |= (1<<PD3) | (1<<PD7);
6
}
 
ausscheidet, wählt man besser folgendes Konstrukt:
 
1
#define TEST()                    \
2
  do {                            \
3
    PORTD &= ~(1<<PD1);           \
4
    PORTD |= (1<<PD3) | (1<<PD7); \
5
  } while (0)
Gast
Jörg W. schrieb:
> Huh schrieb:
>> Ich denke, daß es NICHT innerhalb einer Klammer ist.
>
> OK, verzählt.
>
> Aber der Rest passt schon, und Johann tutet ja ins gleiche Horn –
> wobei er völlig Recht hat, dass eine inline-Funktion übersichtlicher
> ist.

Das stimmt. Da hat er recht.
OP
Wenn ich das so schreibe, dann meckert der GCC-Compiler:

Error    expected identifier or '(' before 'do'
Error    expected identifier or '(' before 'while'


#define Test()
do
{
  PORTD &= ~(1<<PD2);
  PORTD |= (1<<PD3);
        PORTB |= (1<<PB3);
} while (0)
Moderator Persönliche Seite
Joe F. schrieb:
> ja, ok, aber wer vor 'else' ein semikolon setzt ist selbst schuld.

Aha.  Du meinst also
1
if (cond)
2
  a = sin(x)
3
else
4
  a = cos(x);

wäre legales C?

Wenn du das nicht meinst, dann solltest du wohl deine obige Aussage
nochmal überdenken …

Nimm's einfach, wie es ist: dieser do…while(0)-Konstrukt im Body eines
Makros ist das übliche Mittel dafür, dass man den Makro wie einen
normalen Funktionsaufrauf in einer if-Anweisung benutzen kann.

Im übrigen würde es bei dem ganz oben gezeigten Makro in diesem Falle
auch gar nicht helfen, wenn man das Semikolon vom Aufrufer des Makros
in den Makro verlagert: nach dem if steht dann trotzdem nur eine
Anweisung, obwohl zwei gewünscht sind.

Ja, immer geschweifte Klammern nach einem if schreiben ist defensiv
und hilft ebenfalls (und ist daher in manchen Programmierrichtlinien
vorgeschrieben), aber die do…while(0)-Variante hilft eben auch dann,
wenn derjenige, der das if schreibt, sowas nicht von sich aus tut.

Und weil's noch nicht oft genug geschrieben worden ist: eine
inline-Funktion benimmt sich immer syntaktisch wie eine Funktion,
auch ohne derartig seltsam anmutende Tricks, und sie ist auch noch
in der Regel einfacher zu lesen als ein verzwackter Makro.  Man sollte
einer solchen also nach Möglichkeit den Vorzug gegeben.

Antwort schreiben

Eine eigene Frage? Dieser Thread ist über 10 Jahre alt. Stell sie in einem eigenen Thread – dort lesen mehr Leute mit, und sie wird öfter und schneller beantwortet.

Neuen Thread starten

Eine Lösung, Ergänzung oder Rückmeldung zu diesem Thread? Dann antworte hier.

Formatierung
  • [c]C-Code[/c]
  • [code]Code in anderen Sprachen, ASCII-Zeichnungen[/code]
  • [math]Formel in LaTeX-Syntax[/math]
  • [[Titel]] – Link zu einem Artikel
  • Verweis auf einen anderen Beitrag: Rechtsklick auf dessen Beitrags-ID (#8078628), „Adresse kopieren“, in den Text einfügen

Mehr zur Formatierung

Längeren Sourcecode als Anhang, nicht in den Text. JPG nur für Fotos und Scans; Zeichnungen und Screenshots als PNG oder GIF. Siehe Bildformate.

Schon registriert? Anmelden

Wird nicht angezeigt. Wir schicken dir einen Link; sobald du damit bestätigst, erscheint dein Beitrag.

Mit dem Abschicken legst du einen kostenlosen Account an und erkennst die Nutzungsbedingungen und die Datenschutzerklärung an.