C-Präprozessor #define SQR (x) (x * x)

Question

Ich denke, ich werde verrückt. Ich lerne C-Vorprozessor und ich kann dieses Problem nicht herausfinden, I definiert SQR (X) (X * X) absichtlich ohne Klammern

#include <stdio.h> 
#define SQR(x) (x * x) 
int main() 
{ 
    int counter; /* counter for loop */ 
    for(counter = 0; counter < 5; ++counter) 
    { 
     printf("x %d, x squared %d\n",counter+1, SQR(counter+1)) 
    } 
    return (0); 
} 

in einem Moment in Schleifenzähler = 3. Dann sollte es sein 3 + 1 * 3 + 1 und das ist gleich 7, aber die Ausgabe sagt 5. Hier Ausgabe des Codes oben:

x 1, x squared 1 
x 2, x squared 3 
x 3, x squared 5 
x 4, x squared 7 
x 5, x squared 9 

Process returned 0 (0x0) execution time : 0.020 s 
Press any key to continue. 

ich eindeutig etwas fehlt bin, aber ich kann es nicht verstehen aus.

Answer 1

Wenn Zähler = 3, wird als gedruckt, weil Sie counter+1 drucken. Verwendung:

printf("x %d, x squared %d\n",counter, SQR(counter+1)); 

, um zu sehen, was der tatsächliche Wert des Zählers ist.

Answer 2

Abgesehen von dem offensichtlichsten Fehler, der in Ignaus 'Antwort aufgezeigt wurde, haben Sie hier einen zweiten, heimtückischeren und gefährlicheren Fehler. Haben die Substitution, die der Präprozessor tun auf SQR:

SQR(counter+1) 

wird:

counter+1 * counter+1 

, die ausgeführt wird, als ob es so geschrieben wurden:

counter + (1 * counter) + 1 

Das ist natürlich falsch und nicht was du beabsichtigst. Bei Makros sollten Sie immer die Argumente in Klammern einschließen. So sollte Ihr Makro wie folgt aussehen:

#define SQR(x) ((x) * (x)) 

Jetzt tun die Substitution wieder und Sie erhalten:

((counter+1) * (counter+1)) 

was richtig ist. Machen Sie es sich zur Gewohnheit immer Makro Argumente in Klammern zu enthalten, um diese Art von Fehlern zu vermeiden.