[AVR/C++] Ruzie met preprocessor

[TommyboyNL]

Het volgende stukje code levert de volgende error op: "error: expected unqualified-id before 'else'", het regelnummer klopt echter van geen kant.

code:

//#define klimaatcontroller
#if (defined klimaatcontroller)
  if (FIFOFull(temp_buff)||(sensortype==2)) {
#else
  if (FIFOFull(temp_buff)) {
#endif
    blah;
  }

Volgens mij zou dit gewoon goed moeten werken, aangezien na de preprocessor slechts één van mijn if-statements over blijft. Ik krijg het echter niet werkend, en zit met mijn handen in het haar. Wat zie ik over het hoofd?

[Herko_ter_Horst]

Als het regelnummer niet naar dit stukje code wijst, zit de fout dan niet gewoon eerder of later ergens? Mist er niet ergens een }?

[eek]

Moet het niet #if defined(klimaatcontroller) zijn?

[TommyboyNL]

Als ik bovenstaande code vervang door het volgende stukje, compileert de boel wel succesvol, en werkt de boel ook zoals zou moeten (afgezien van de functionaliteit die de bovenste define aan moet zetten natuurlijk):

code:

//#define klimaatcontroller
  if (FIFOFull(temp_buff)) {
    blah;
  }

Het enige verschil zijn de preprocessor statements en de alternatieve "if" regel.
Ook als ik de code uit mijn startpost gebruik en één van de twee if-statements comment, compileert de boel goed.

@eek:
Volgens mij niet, zoals het hierboven staat, staat het op andere plekken in mijn code ook, waar het prima lijkt te werken. Daat gaat het echter om variabelen die al dan niet nodig zijn, en niet om het kiezen van verschillende if-statements.

[ Voor 8% gewijzigd door TommyboyNL op 20-03-2015 20:11 ]

[Gleighton]

Kijk eens goed naar http://www.cplusplus.com/doc/tutorial/preprocessor/

[MSalters]

Doe maar niet; cppreference.com is betrouwbaarder.

De fout in het stukje code is een haakje op de verkeerde plek, dat moet zijn: #if defined (klimaatcontroller)

De foutmelding over "else" zit uiteraard ergens anders, vandaar ook dat het regelnummer niet klopt. De code in de TS bevat nota bene niet eens een "else" maar alleen een "#else".

[TommyboyNL]

Bedankt voor alle tips. Ik heb geprobeerd met de plaats van de haakjes te spelen, wat geen oplossing gaf. Ook #ifdef gebruiken in plaats van "#if defined" leverde geen oplossing. Uiteindelijk heb ik het zo opgelost:

code:

//#define klimaatcontroller
  if (FIFOFull(temp_buff)
#ifdef klimaatcontroller
  ||(sensortype==2)
#endif          
  ) {
    blah;
  }

Ik ben er niet heel happy mee aangezien de leesbaarheid belabberd is (en ik HAAT WORN-code), maar het werkt ten minste.

[Rmg]

code:

#ifdef klimaatcontroller
    if (FIFOFull(temp_buff)||(sensortype==2)) {
#else
    if (FIFOFull(temp_buff)) 
#endif

waarom geen ifdef gebruiken?

[ Voor 9% gewijzigd door Rmg op 23-03-2015 14:26 ]

[TommyboyNL]

Rmg schreef op maandag 23 maart 2015 @ 14:21:

code:

#ifdef klimaatcontroller
    if (FIFOFull(temp_buff)||(sensortype==2)) {
#else
    if (FIFOFull(temp_buff)) 
#endif

waarom geen ifdef gebruiken?

TommyboyNL schreef op maandag 23 maart 2015 @ 12:53:
Ook #ifdef gebruiken in plaats van "#if defined" leverde geen oplossing.

[farlane]

Je gebruikt nu toch ook #ifdef? Je doet iets fout, maar ik zie het zo niet. Hoe ziet klimaatcontroller er uit? Undef je 'em ergens?

Om de leebaarheid te vergroten kun je de voorwaarde natuurlijk ook naar een (inline) functie of macro halen en daar het conditionele stuk in doen, dat leest al heel wat prettiger in je 'main flow'.

[ Voor 50% gewijzigd door farlane op 23-03-2015 15:38 ]

[.oisyn]

MSalters schreef op zondag 22 maart 2015 @ 00:54:
Doe maar niet; cppreference.com is betrouwbaarder.

De fout in het stukje code is een haakje op de verkeerde plek, dat moet zijn: #if defined (klimaatcontroller)

Nope. De defined operator vereist geen haakjes. En uiteraard mag je sowieso haakjes om de gehele expressie zetten.

quote: http://en.cppreference.com/w/cpp/preprocessor/conditional

If the expression contains unary operators in form defined identifier or defined (identifier), it is evaluated first. The result is 1 if the identifier was defined as a macro name and 0 otherwise.

[TommyboyNL]

Hoe de rest van mijn programma er uitziet is volgens mij niet van belang. Dit is de eerste keer dat ik in mijn main loop preprocessor commando's gebruik.
Het maffe is dat volgens mij beide stukken code hieronder het zelfde (werkende) resultaat op moeten leveren (eveneens met de allereerste regel uncommented), maar dat dus niet doen.

code:

//#define klimaatcontroller
#ifdef klimaatcontroller
  if (FIFOFull(temp_buff)||(sensortype==2)) {
#else
  if (FIFOFull(temp_buff)) {
#endif
    blah;
  }

code:

//#define klimaatcontroller
  if (FIFOFull(temp_buff)
#ifdef klimaatcontroller
  ||(sensortype==2)
#endif          
  ) {
    blah;
  }

[farlane]

Het zou ook hetzelfde moeten opleveren, daarom denk ik dat het probleem ergens anders zit.

[TommyboyNL]

Ik heb volgens mij wel wat preprocessor directives genest, kan dat het probleem zijn?

[farlane]

Nee, tenzij er iets fout gaat met de nesting. Kun je wat meer code posten?