遇到一个有趣的问题

dj狂人

遇到一个有趣的问题 [复制链接]

1. if(Rc_C.ARMED)

   LED_ON; //"LED_ON 是一个宏：#define LED_ON GPIO_ResetBits(GPIOB, GPIO_Pin_8);"，LED_OFF也是

   else

   LED_OFF;

复制代码

就这样很普通的语句居然报错了，让我郁闷了一会 而给 if 加上花括号就没有报错了：

1. if(Rc_C.ARMED)

   {

   LED_ON;

   }

   else

   LED_OFF;

复制代码

到这我大概知道报错的原因了，我试着把“LED_ON；”的分号去掉，果然，编译通过。 很明显出现这种报错的原因是 在“#define LED_ON GPIO_ResetBits(GPIOB, GPIO_Pin_8);”这个宏中已经 带了一个分号，如果在使用这个宏的时候又加分号就变成了两条语句：

if(Rc_C.ARMED) if(Rc_C.ARMED)

LED_ON; 等价于 GPIO_ResetBits(GPIOB, GPIO_Pin_8)；；

else else

LED_OFF; GPIO_SetBits(GPIOB, GPIO_Pin_8)；； 不得不说写代码一点都马虎不得

sint27

个人有时候也会出现这种错误，所以宏定义的时候要想想怎么被调用的

mylsf

用宏定义的语句，先加大括号，保险

仙猫

不错。
●#define 后面别写分号；
●若 #define 是复合句，干脆在 #define 行就加上括号，给调用者省事；
●养成 if语句内哪怕只一句也加花括号的习惯，一般多几个括号没什么坏处，只会使代码易读。

dj狂人

仙猫 发表于 2015-7-28 10:39
不错。
●#define 后面别写分号；
●若 #define 是复合句，干脆在 #define 行就加上括号，给调用者省事； ...

恩恩，这种习惯是好。不过我个人有点毛病就是如果语句不长的话，花括号是见一个灭一个，因为那样看起来感觉简洁。这算洁癖吗

dj狂人

mylsf 发表于 2015-7-28 10:37
用宏定义的语句，先加大括号，保险

嗯  这是比较规范的做法。之前我对很多人这样做不是很理解，感觉有点多此一举。现在懂了，代码还是规范点好，不然一不小心程序死哪了都不知道

仙猫

dj狂人 发表于 2015-7-28 10:45
恩恩，这种习惯是好。不过我个人有点毛病就是如果语句不长的话，花括号是见一个灭一个，因为那样看起来感 ...

看来是有点洁癖哈……不过未必是坏事。
兵无常势，水无常形，把什么都压抑死这个世界就出不了彩了。
被公认为能排唐诗首席的崔颢作的《黄鹤楼》，平仄多处出律，是犯格律诗大忌的。但若规规矩矩，或许中华文化瑰宝里就少了这首好诗了！

weizhongc

你的宏定义本来就有点问题

dj狂人

weizhongc 发表于 2015-7-28 11:04
你的宏定义本来就有点问题

嗯  确实。一般都是不加分号的。

dj狂人

仙猫 发表于 2015-7-28 11:03
看来是有点洁癖哈……不过未必是坏事。
兵无常势，水无常形，把什么都压抑死这个世界就出不了彩了。
被 ...

嗯  是不能有太多的框框架架。特别是对编程人员，太多框架在一定程度上对思维是一种限制，容易让人缺乏创新。当然不要太多框架不是让人不需要太过于严谨，那是不同的概念了。对于编程人员来说保持严谨是必须的。

okhxyyo

；虽小但是不容忽视呀~

qiushenghua

所以才会有do{}while(0)的语句出现…

dj狂人

qiushenghua 发表于 2015-7-28 11:22
所以才会有do{}while(0)的语句出现…

这个怎么说

qiushenghua

dj狂人 发表于 2015-7-28 11:26
这个怎么说

#define functiion() do{\
language1();\
language2();\
}while(0)

调用的时候:
functions();

dj狂人

qiushenghua 发表于 2015-7-28 11:30
#define functiion() do{\
language1();\
language2();\
}while(0)

调用的时候:
functions();

do()while()  应该应该不是因为这个才出现的吧。只能说do{}while()刚好适合这样用

qiushenghua

dj狂人 发表于 2015-7-28 11:41
do()while()  应该应该不是因为这个才出现的吧。只能说do{}while()刚好适合这样用

while(0)

栾鸥

我习惯“#define”定义不加符号。

dj狂人

qiushenghua 发表于 2015-7-28 12:41
while(0)

为什么不直接用花括号呢，像这样子

#define functiion() {\
language1();\
language2();\
}

lcofjp

dj狂人 发表于 2015-7-28 14:07
为什么不直接用花括号呢，像这样子

#define functiion() {\
language1();\
language2();\
}

do {...}while(0)    +    ;    ===>一条语句
{...}                    +    ;    ===>两条语句
你可以把这个调用想象在没有大括号的if语句里面，就会有和本帖主题一样的错误。

dj狂人

lcofjp 发表于 2015-7-28 16:26
do {...}while(0)    +    ;    ===>一条语句
{...}                    +    ;    ===>两条语句
你可 ...

懂了，C语言果然博大精深