[AS7] Probleme mit Präprozessor Direktiven

[Draco (Gast)]

Ich habe hier ein "Problem", welches bestimmt kein größeres ist.

Ich arbeite in Atmel Studio 7 und auf einem MEga32u4 und komme irgendwie 
mit den Präprozessor Abfragen nicht so klar. Hintergrund: Es gibt die 
Software für den µC welchen es aber in zwei verschiedenen Ausführungen 
gibt (Mit unterschiedlichen Anzeigeelementen). Die Software ist aber im 
Grunde die gleiche, so das sich zwei unterschiedliche Projekte nicht 
lohnen.

Momentan habe ich es mit ner normalen If Abfrage gelößt, was aber im 
Grunde quark ist, da der jeweils andere Code auf dem µC sinnlos ist, da 
nie gebraucht:

void SetGearPosition(uint8_t Gear)
{

  if (ucVersion == 0)
  {
  DDRB &= ~(1<<4);
  DDRD &= ~((1<<0)|(1<<1)|(1<<2)|(1<<4)|(1<<7));
  DDRE &= ~(1<<6);
    
  switch (Gear)
  {
    case 0:
    DDRB |= (1<<4);
    break;
    case 1:
    DDRE |= (1<<6);
    break;
    case 2:
    DDRD |= (1<<7);
    break;
    case 3:
    DDRD |= (1<<4);
    break;
    case 4:
    DDRD |= (1<<0);
    break;
    case 5:
    DDRD |= (1<<1);
    break;
    case 6:
    DDRD |= (1<<2);
    break;
    default:
    break;
  }
  }

  if (ucVersion == 1)
  {
  if(Gear >= 0 && Gear <= 9)
  {
    DracoMAX7219_send(Gear);
  }
  }
}

Wenn ich dies aber mit Präprozessorabfragen versuche, schlage ich aber 
fehl (Ich muss zugeben das ich das auch vorher nie gemacht habe):

#define __DracoV0_ 1
#define __DracoV1_ 0

//...

void SetGearPosition(uint8_t Gear)
{

#ifdef __DracoV0_

  DDRB &= ~(1<<4);
  DDRD &= ~((1<<0)|(1<<1)|(1<<2)|(1<<4)|(1<<7));
  DDRE &= ~(1<<6);
    
  switch (Gear)
  {
    case 0:
    DDRB |= (1<<4);
    break;
    case 1:
    DDRE |= (1<<6);
    break;
    case 2:
    DDRD |= (1<<7);
    break;
    case 3:
    DDRD |= (1<<4);
    break;
    case 4:
    DDRD |= (1<<0);
    break;
    case 5:
    DDRD |= (1<<1);
    break;
    case 6:
    DDRD |= (1<<2);
    break;
    default:
    break;
  }
#endif //__DracoV0_



#ifdef __DracoV1_

  if(Gear >= 0 && Gear <= 9)
  {
    DracoMAX7219_send(Gear);
  }
#endif //__DracoV1_
}

Er springt dennoch durch beide Zweige und kompiliert auch alle beide 
zweige. Jemand einen Rat?!

[Wolfgang (Gast)]

Draco schrieb:
> Ich arbeite in Atmel Studio 7 und auf einem MEga32u4 und komme irgendwie
> mit den Präprozessor Abfragen nicht so klar.

Erstmal: Präprozessorabfragen haben nichts mit Atmel Studio sondern nur 
mit der Programmiersprache etwas zu tun.

Draco schrieb:
> #define _DracoV0 1
> #define _DracoV1 0
> ...
> #ifdef __DracoV0_
> ...

> Er springt dennoch durch beide Zweige und kompiliert auch alle beide
> zweige.

Wenn du beide definierst (egal wie), wird natürlich auch der Code für 
beide Fälle erzeugt. Du könntest eines der #define auskommentieren.

[Draco (Gast)]

Ahhhh..... vielen Dank :D

Zum Thema C / Atmel Studio etc... das heißt quasi das es 
Compilerunabhängig ist und in allen C Compilern gleich behandelt wird?! 
Gut zu wissen, danke.


Ich habe das auch jetzt geblickt, hätte ich das nach meinem "Beispiel" 
machen wollen hätte ich es mit #if (_DracoV1 == 1) machen müssen. Aber 
so ist das mit dem einen Define ja auch wesentlich übersichtlicher. 
Danke dir fürs Licht ins Dunkel bringen :)

[Daniel (Gast)]

Nimm anstatt #ifdef ein #if und dein Problem sollte gelöst sein...

Ansonsten kannst du auch einfach ein define auskommentieren.

Gruß,
Daniel

[Wolfgang (Gast)]

Draco schrieb:
> Zum Thema C / Atmel Studio etc... das heißt quasi das es
> Compilerunabhängig ist und in allen C Compilern gleich behandelt wird?!

Da bringst du mehrere Dinge durcheinander.
1. Atmel Studio ist die Entwicklungsumgebung.
2. C ist die Programmiersprache
3. Die #define haben nichts mit dem Compiler zu tun
4. Zu C gehört ein Präprozessor, der soetwas wie #define abarbeitet, 
bevor der C Compiler den Source Code überhaupt zu Gesicht bekommt.

Vielleicht solltest du mal ein Blick in ein C Grundlagenbuch werfen.

[Rufus Τ. F. (rufus)]

#define __DracoV0_ 1
#define __DracoV1_ 0

//...

void SetGearPosition(uint8_t Gear)
{

#ifdef __DracoV0_

Beide Macros sind definiert, deswegen wird #ifdef immer glücklich.

Alternative:

#define __DracoV0_ 
// #define __DracoV1_ 

//...

void SetGearPosition(uint8_t Gear)
{

#ifdef __DracoV0_

Ja, es genügt, ein leeres Macro zu definieren, damit #ifdef 
funktioniert.

Weitere Alternative:

#define __DracoVersion_ 1

//...

void SetGearPosition(uint8_t Gear)
{

#if __DracoVersion_ == 1

Abschließend: Macros mit zwei vorangestellten Unterstrichen sind 
verboten, die gehören ausschließlich dem Compiler. Das solltest Du Dir 
daher besser abgewöhnen.

[Draco (Gast)]

Rufus Τ. F. schrieb:
> Abschließend: Macros mit zwei vorangestellten Unterstrichen sind
> verboten, die gehören ausschließlich dem Compiler. Das solltest Du Dir
> daher besser abgewöhnen.

Danke, werde ich mir einprägen. Gibt es da ungeschriebene Gesetze wie 
man diese am besten bennent?!