宏定义问题

[WuYL7812]

#define DOCVIEWER_TO_CLIP_COORDINATION(CX1, CY1, CX2, CY2, X1, Y1, X2, Y2) \
do{\
CX1 = X1;\
CY1 = Y1;\
CX2 = X2;\
CY2 = Y2;\
if(CX1 < DOCVIEWER_SCREEN_X) CX1 = DOCVIEWER_SCREEN_X;\
if(CX2 > DOCVIEWER_SCREEN_END_X) CX2 = DOCVIEWER_SCREEN_END_X;\
if(CY1 < DOCVIEWER_SCREEN_Y) CY1 = DOCVIEWER_SCREEN_Y;\
if(CY2 > DOCVIEWER_SCREEN_END_Y) CY2 = DOCVIEWER_SCREEN_END_Y;\
}while(0);

在这个宏里，while循环式做什么用的呀，我觉得只要加个大括号就可以了

[freshui]

这样的好处很明显,比如我们可以使用
if(a)
DOCVIEWER_TO_CLIP_COORDINATION(CX1, CY1, CX2, CY2, X1, Y1, X2, Y2);
else
DOCVIEWER_TO_CLIP_COORDINATION(CX1, CY1, CX2, CY2, X1, Y1, X2, Y2);
这样的表达式不会出错。
仅仅使用花括号就不行。

但是如果只有一个语句，完全没有必要。

[freshui]

还有, 你的也写错了
while(0) 后面不能跟分号

[hzcpig]

do{}while(0)的宏定义就是为了能在宏后面加分号。

如果只是单纯的{}的宏，后面不能添加分号。

[dogygb]

quote:
--------------------------------------------------------------------------------
这样的好处很明显,比如我们可以使用
if(a)
DOCVIEWER_TO_CLIP_COORDINATION(CX1, CY1, CX2, CY2, X1, Y1, X2, Y2);
else
DOCVIEWER_TO_CLIP_COORDINATION(CX1, CY1, CX2, CY2, X1, Y1, X2, Y2);
这样的表达式不会出错。
仅仅使用花括号就不行。

但是如果只有一个语句，完全没有必要。
-----------------------------------------------------------------------------------


为什么仅仅使用花括号就不行呢 ？？？

[dogygb]

就是分号问题？

[winner8080]

还是不明白,
没中要害

[yutaooo]

恩。值得研究一下。如下的两段(#1 #2)一样吗？

// #1:
do {
x = 1;
y = 1;
z = 1;
}
while(0);


// #2:
{
x = 1;
y = 1;
z = 1;
}

[freshui]

while(0) 后面不能有分号

比如
#define Add(a, b) do{ a++; a+b;
}while(0)

if(a)
add(a,b);
else
...
如果while(0)后面有分号, 那么会有两个连续的分号, 这样if else就不配对了

同理 如果没有do while(0) 只是和大括号
展开变成:
if(a)
{}
;
else
...
也会多出一个分号, 同样if else不配对


主要是为防止这个的.

[freshui]

呵呵 这下说的应该够明白了吧 :)

[dogygb]

hzcpig() 说的倒不错~~~可以加“；”在后边了

[poo2004]

这是一种预防性措施。如下：
1。定义一个宏:
#define SUM(a, b) a = a + 1; b = a + b;
那么在if-else结构中处理就会有问题：
if(a)
SUM(a, b)；
else
exit(1);
经过预处理后，这段代码就会变成：
if(a)
a = a + 1;
b = a + b;
else
exit(1);
编译这段代码时将出错。
2。如果在上面那个宏加个{}，预处理后将会成这样：
if(a)
{
a = a + 1;
b = a + b;
};
else
exit(1);
还是通不过。

[intuition444]

和这个函数是一样的

DOCVIEWER_TO_CLIP_COORDINATION(CX1, CY1, CX2, CY2, X1, Y1, X2, Y2)
do{
CX1 = X1;
CY1 = Y1;
CX2 = X2;
CY2 = Y2;
if(CX1 < DOCVIEWER_SCREEN_X) CX1 = DOCVIEWER_SCREEN_X;
if(CX2 > DOCVIEWER_SCREEN_END_X) CX2 = DOCVIEWER_SCREEN_END_X;
if(CY1 < DOCVIEWER_SCREEN_Y) CY1 = DOCVIEWER_SCREEN_Y;
if(CY2 > DOCVIEWER_SCREEN_END_Y) CY2 = DOCVIEWER_SCREEN_END_Y;
}while(0);

[xiantongyuan]

可能是为了以后的扩展吧，不过在这里好象是一样的。

[yutaooo]

为了产生一个复合语句。while本身没作用。但没有的话，复合语句如何写出来呢？

[yutaooo]

算是个技巧，戏法吧。

[jixingzhong]

一些防护性措施。

由于宏的 简单替换性，
完全没有任何检查，
所以可能宏被和其他宏或者代码叠加，
得到一些非预期的结果。

使用 do{}while(0) 可以显式的把这段宏所代表的代码独立封装，
不会被其他宏或者代码重叠造成非预期结果。

[WuYL7812]

to yutaooo(旦见散分需快进, 莫等分尽空遗恨! → 今天,你散了没有) ( )
为了产生一个复合语句。while本身没作用。但没有的话，复合语句如何写出来呢？
------------------------------------------------------------------------
怎么不能写了？