高手帮忙给分析下错误原因

[yilaozhuang]

/* allocate memory space for psbob_db_descr.*/
psbob_db_descr=(DB_DESCR *)malloc(sizeof(DB_DESCR));
/* open database */
24行 ret=oodbopen("psbob",sys_name,bob_name,real_srv,W,psbob_db_descr);
25行 if(ret== OO_FAIL)
{
printf("oodbopen failed\n");
free(psbob_db_descr);
printf("*********************error %d occur at line %d\n",_oodb_errno,__LINE__);
return PWRFLOW_FAIL;
}

其中
#define OO_FAIL -1;

psbob_open.cpp(24) :error C2143: syntax error : missing ')' before ';'
psbob_open.cpp(24) : error C2059: syntax error : ')'
psbob_open.cpp(25) : warning C4390: ';' : empty controlled statement found; is this the intent?

高手给帮忙分析一下原因，这个问题我碰到了好多次了，不知道怎么解决。

[ch1oE]

#define OO_FAIL -1
后面不用加分号 -- 因为macro在做完常量运算后会直接替换过去 -- 导致你的 if(ret == -1;) ..
所以会错～

[bfhtian]

一般是提示上面一行有错

[yilaozhuang]

再说一下，如果把25行的OO_FAIL换成-1
if(ret==-1)，所有编译都能通过。请教高手给指点下原因

[bfhtian]

[Quote=引用 1 楼 ch1oE 的回复:]
#define OO_FAIL -1
[/Quote]
没看到这个

[yilaozhuang]

[Quote=引用 1 楼 ch1oE 的回复:]
#define OO_FAIL -1
后面不用加分号 -- 因为macro在做完常量运算后会直接替换过去 -- 导致你的 if(ret == -1;) ..
所以会错～
[/Quote]

多谢多谢，是这个原因。这下省去了调试过程中的好多问题了。

[waizqfor]

#define      OO_FAIL        -1; 
==============>
#define      OO_FAIL        -1 //宏是不需要加';'的

[waizqfor]

[Quote=引用 5 楼 yilaozhuang 的回复:]
引用 1 楼 ch1oE 的回复:
#define OO_FAIL -1
后面不用加分号 -- 因为macro在做完常量运算后会直接替换过去 -- 导致你的 if(ret == -1;) ..
所以会错～


多谢多谢，是这个原因。这下省去了调试过程中的好多问题了。
[/Quote]
LZ看错误的时候 注意看错误信息 都会把你的错误位置 和什么错误指明的

[yilaozhuang]

[Quote=引用 1 楼 ch1oE 的回复:]
#define OO_FAIL -1
后面不用加分号 -- 因为macro在做完常量运算后会直接替换过去 -- 导致你的 if(ret == -1;) ..
所以会错～
[/Quote]

是不是这种常数的宏定义都不需要后面的分号？？请高手再给说一下

[waizqfor]

[Quote=引用 8 楼 yilaozhuang 的回复:]
引用 1 楼 ch1oE 的回复:
#define OO_FAIL -1
后面不用加分号 -- 因为macro在做完常量运算后会直接替换过去 -- 导致你的 if(ret == -1;) ..
所以会错～


是不是这种常数的宏定义都不需要后面的分号？？请高手再给说一下
[/Quote]
宏是预编译 跟#include <xxx.h>一样 他们都不需要加';'

[ch1oE]

[Quote=引用 8 楼 yilaozhuang 的回复:]
引用 1 楼 ch1oE 的回复:
#define OO_FAIL -1
后面不用加分号 -- 因为macro在做完常量运算后会直接替换过去 -- 导致你的 if(ret == -1;) ..
所以会错～


是不是这种常数的宏定义都不需要后面的分号？？请高手再给说一下
[/Quote]

主要是这样，macro是直接做替换，以你的这个为例：
#define OO_FAIL -1
就是将OO_FAIL全部替换为"-1"
如果你用
#define OO_FAIL -1;
就将所有的OO_FAIL全部替换为"-1;"

其实你这样还是有问题的，例如
int a = 1 OO_FAIL;
结果是什么？结果是 a = 0 -- 因为后面的表达式被替换为了1-1

一般使用是用括号来做限定，例如：
#define OO_FAIL (-1)
这样上面那句话就编译不过了～

另外就是#define OO_FAIL (-1 * 5 * 5 * 5)
这样的常量运算，一般是运算完，然后再替换。不会出现冗余的计算～

[passionboy03]

我以前笔试遇到过这样的问题，让我找错了，楞是盯着看了老长时间就是不知道错哪儿了，面试官提示我看下宏，豁然开朗……呵呵
[Quote=引用 5 楼 yilaozhuang 的回复:]
引用 1 楼 ch1oE 的回复:
#define OO_FAIL -1
后面不用加分号 -- 因为macro在做完常量运算后会直接替换过去 -- 导致你的 if(ret == -1;) ..
所以会错～


多谢多谢，是这个原因。这下省去了调试过程中的好多问题了。
[/Quote]

[passionboy03]

宏定义在编译的时候会直接会用后面的内容完全替换变量，特别是在定义数值计算公式时，千万要细心些，把该加的括号要加上……
[Quote=引用 8 楼 yilaozhuang 的回复:]
引用 1 楼 ch1oE 的回复:
#define OO_FAIL -1
后面不用加分号 -- 因为macro在做完常量运算后会直接替换过去 -- 导致你的 if(ret == -1;) ..
所以会错～


是不是这种常数的宏定义都不需要后面的分号？？请高手再给说一下
[/Quote]

[passionboy03]

“其实你这样还是有问题的，例如
int a = 1 OO_FAIL;
结果是什么？结果是 a = 0 -- 因为后面的表达式被替换为了1-1 ”

一般也不会这么使用的吧？
更形象点：int a = 1 + OO_FAIL;
应该会报错……
[Quote=引用 10 楼 ch1oE 的回复:]
引用 8 楼 yilaozhuang 的回复:
引用 1 楼 ch1oE 的回复:
#define OO_FAIL -1
后面不用加分号 -- 因为macro在做完常量运算后会直接替换过去 -- 导致你的 if(ret == -1;) ..
所以会错～


是不是这种常数的宏定义都不需要后面的分号？？请高手再给说一下


主要是这样，macro是直接做替换，以你的这个为例：
#define OO_FAIL -1
就是将OO_FAIL全部替换为"-1"
如果你用
#define OO_FAIL -1;
就将所有的O…
[/Quote]