Präprozessor Textersetzung defines Prototypen

[Imperator (Gast)]

Servus,
ich möchte mein Programm portabler machen. Will irgendwie nicht so 
richtig funktionieren.

//file1.h Prototypen
#ifndef FILE1_H
#define FILE1_H
extern void x_set(void);
extern void x_get(void);
extern void x_clear(void);
...
#endif

//file1.c Definition
#include "file1.h"
void x_set(void){}
void x_get(void){}
void x_clear(void){}
...

//file2.h neue Deklaration, Namensgebung
#ifndef FILE2_H
#define FILE2_H

#ifdef FILE1_H
#include "file1.h"
#define y         x
#else
#error "You must use either FILE1_H or ... define!"
#endif

void test(void);

extern void y_set(void);
extern void y_get(void);
extern void y_clear(void);
...
#endif


//file2.c Funktionen Aufruf
#include "file2.h"
void test(void){
  y_set();   //oder x_set() sollte hier gehen
  y_get();
  y_clear();
}

Wie könnte man mithilfe des Präprozessors die Namensgebung schlau 
ändern?
Oder sollte man immer den kompletten Namen ersetzen:

#define y_set       x_set
#define y_get       x_get
#define y_clear     x_clear

[PittyJ (Gast)]

Ich verzichte, bis auf die Include-Guards, komplett auf Defines und 
Macros.
Man kann damit viel zu viel kaputt machen. Programme die keiner 
versteht, unvorhersehbare Effekte etc.

Auf keine Fall die Funktionsnamen über Preprozessor ändern. Spätestens 
in einer Woche blickt da keiner mehr durch!

[Rufus Τ. F. (rufus)]

Imperator schrieb:
> #ifdef FILE1_H
> #include "file1.h"

Das widerspricht sich. "FILE1_H" ist definiert, wenn "file1.h" 
eingebunden wurde, also wird "file1.h" nicht eingebunden, denn FILE1_H 
ist nicht definiert ...


> #define y         x

Das funktioniert nicht, nicht um y_set aus x_set zu gewinnen oder 
umgekehrt.


Wozu soll das ganze wirklich gut sein?

"Vereinfachen" tut's jedenfalls so nichts.

[Imperator (Gast)]

Es geht um eine led library. Ich möchte mithilfe eines einzelnen 
defines, unabhängig verschiedener led-typen, dasselbe Programm 
ansteuern.

Ich bin auch weiter gekommen.

#define y_set       x_set
#define y_get       x_get
#define y_clear     x_clear

Funktioniert gut. Einzelne Literale wollte der Präprozessor nicht 
ersetzen.

Wenn man halt auf y_set(); 'open Declation' geht, dann wandert man auf 
das dazugehörige define x_set. Wiederum ein weiteres 'open Declation' 
spring auf die gewünschte Deklaration.

Ich bin jetzt zufrieden.

[a.s (Gast)]

PittyJ schrieb:
> Ich verzichte, bis auf die Include-Guards, komplett auf Defines und
> Macros.

Naja, dann wirst Du wohl C++ schreiben
oder nur kleinen und Plattform- und Varianten-unabhängigen Code 
schreiben.

[Jörg W. (dl8dtl) (Moderator)]

Imperator schrieb:
> Einzelne Literale wollte der Präprozessor nicht ersetzen.

Geht mit dem Verkettungsoperator ##

#define X_set(x) x ## _set

...
X_set(a) // => a_set

Aber wie oben schon geschrieben wurde: wenn man das nicht sehr sauber
und klar strukturiert und dokumentiert, dann blickt da später keiner
mehr durch.

[Bernd K. (prof7bit)]

Imperator schrieb:
> Ich bin auch weiter gekommen.
> #define y_set       x_set
> #define y_get       x_get
> #define y_clear     x_clear

Hör auf, bitte! Ich ahne daß sich das ganz schnell zu einer ganz üblen 
Code-Obfuskation auswachsen könnte. Das bekommst Du später kaum noch weg 
und die damit einhergehenden Beschwerden können wirklich höllisch sein.

Wenn Du Schreibarbeit für langweiligen und sich zig fach selbstähnlich 
wiederholenden Boilerplate sparen willst dann denk doch mal drüber nach 
code (echten Code, keine Makros) von einem übersichtlichen Python (oder 
sonstigen) Script erzeugen zu lassen, du kannst dann da tonnenweise 
getter und setter automatisch erzeugen lassen und solange das Ergebnis 
dann als echte .c oder .h ohne Makros dort steht kann man es sogar lesen 
und schmerzfrei debuggen.

[Imperator (Gast)]

@Jörg Wunsch (dl8dtl) (Moderator)

Tatsächlich. Es läuft.

#define disp_clear_all_leds(disp)     disp ## _clear_all_leds

//Aufruf
disp_clear_all_leds(apa102)(10);

Vielen Dank. Das Läuft super. Aber ich denke ich bleibe lieber bei copy 
und past. Der Aufruf sieht doch etws bescheiden aus.

Obwohl, wenn man noch weiter verschachtelt:

clear_all_leds    disp_clear_all_leds(apa102)

//Aufruf
clear_all_leds(10);

Sieht schon super aus und funktioniert auch. Aber die IDE findet die 
Funktionsdefinition nicht.

[Bernd K. (prof7bit)]

Imperator schrieb:
> Sieht schon super aus und funktioniert auch. Aber die IDE findet die
> Funktionsdefinition nicht.

Siehst Du, es fängt schon an. Hör auf bevor es noch schlimmer wird.

[PittyJ (Gast)]

a.s schrieb:
> PittyJ schrieb:
>> Ich verzichte, bis auf die Include-Guards, komplett auf Defines und
>> Macros.
>
> Naja, dann wirst Du wohl C++ schreiben
> oder nur kleinen und Plattform- und Varianten-unabhängigen Code
> schreiben.

Ich verdiene sein 25 Jahren mein Geld mit C/C++ Code.
C++ hilft schon viel weiter.
Und ich habe z.B. eine Bibliothek, die läuft auf ARM, Mips und 64 Bit 
Intel.
Und alles ohne Macros.

[Jörg W. (dl8dtl) (Moderator)]

PittyJ schrieb:
> C++ hilft schon viel weiter.

Allerdings sind Template-Fehlermeldungen oft mindestens genauso
kryptisch wie Makro-verursachte Fehlermeldungen.

[Rolf Magnus (Gast)]

Jörg W. schrieb:
> PittyJ schrieb:
>> C++ hilft schon viel weiter.
>
> Allerdings sind Template-Fehlermeldungen oft mindestens genauso
> kryptisch wie Makro-verursachte Fehlermeldungen.

Gerade bezüglich Makros haben sich die Compiler-Meldungen bei gcc in den 
letzten Versionen deutlich verbessert.