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

yilaozhuang 2009-02-19 02:17:51
/* 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?

高手给帮忙分析一下原因,这个问题我碰到了好多次了,不知道怎么解决。
...全文
91 13 打赏 收藏 转发到动态 举报
写回复
用AI写文章
13 条回复
切换为时间正序
请发表友善的回复…
发表回复
passionboy03 2009-02-19
  • 打赏
  • 举报
回复
“其实你这样还是有问题的,例如
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]
passionboy03 2009-02-19
  • 打赏
  • 举报
回复
宏定义在编译的时候会直接会用后面的内容完全替换变量,特别是在定义数值计算公式时,千万要细心些,把该加的括号要加上……
[Quote=引用 8 楼 yilaozhuang 的回复:]
引用 1 楼 ch1oE 的回复:
#define OO_FAIL -1
后面不用加分号 -- 因为macro在做完常量运算后会直接替换过去 -- 导致你的 if(ret == -1;) ..
所以会错~


是不是这种常数的宏定义都不需要后面的分号??请高手再给说一下
[/Quote]
passionboy03 2009-02-19
  • 打赏
  • 举报
回复
我以前笔试遇到过这样的问题,让我找错了,楞是盯着看了老长时间就是不知道错哪儿了,面试官提示我看下宏,豁然开朗……呵呵
[Quote=引用 5 楼 yilaozhuang 的回复:]
引用 1 楼 ch1oE 的回复:
#define OO_FAIL -1
后面不用加分号 -- 因为macro在做完常量运算后会直接替换过去 -- 导致你的 if(ret == -1;) ..
所以会错~


多谢多谢,是这个原因。这下省去了调试过程中的好多问题了。
[/Quote]
ch1oE 2009-02-19
  • 打赏
  • 举报
回复
[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)
这样的常量运算,一般是运算完,然后再替换。不会出现冗余的计算~
waizqfor 2009-02-19
  • 打赏
  • 举报
回复
[Quote=引用 8 楼 yilaozhuang 的回复:]
引用 1 楼 ch1oE 的回复:
#define OO_FAIL -1
后面不用加分号 -- 因为macro在做完常量运算后会直接替换过去 -- 导致你的 if(ret == -1;) ..
所以会错~


是不是这种常数的宏定义都不需要后面的分号??请高手再给说一下
[/Quote]
宏是预编译 跟#include <xxx.h>一样 他们都不需要加';'
yilaozhuang 2009-02-19
  • 打赏
  • 举报
回复
[Quote=引用 1 楼 ch1oE 的回复:]
#define OO_FAIL -1
后面不用加分号 -- 因为macro在做完常量运算后会直接替换过去 -- 导致你的 if(ret == -1;) ..
所以会错~
[/Quote]

是不是这种常数的宏定义都不需要后面的分号??请高手再给说一下
waizqfor 2009-02-19
  • 打赏
  • 举报
回复
[Quote=引用 5 楼 yilaozhuang 的回复:]
引用 1 楼 ch1oE 的回复:
#define OO_FAIL -1
后面不用加分号 -- 因为macro在做完常量运算后会直接替换过去 -- 导致你的 if(ret == -1;) ..
所以会错~


多谢多谢,是这个原因。这下省去了调试过程中的好多问题了。
[/Quote]
LZ看错误的时候 注意看错误信息 都会把你的错误位置 和什么错误指明的
waizqfor 2009-02-19
  • 打赏
  • 举报
回复

#define OO_FAIL -1;
==============>
#define OO_FAIL -1 //宏是不需要加';'的
yilaozhuang 2009-02-19
  • 打赏
  • 举报
回复
[Quote=引用 1 楼 ch1oE 的回复:]
#define OO_FAIL -1
后面不用加分号 -- 因为macro在做完常量运算后会直接替换过去 -- 导致你的 if(ret == -1;) ..
所以会错~
[/Quote]

多谢多谢,是这个原因。这下省去了调试过程中的好多问题了。
bfhtian 2009-02-19
  • 打赏
  • 举报
回复
[Quote=引用 1 楼 ch1oE 的回复:]
#define OO_FAIL -1
[/Quote]
没看到这个
yilaozhuang 2009-02-19
  • 打赏
  • 举报
回复
再说一下,如果把25行的OO_FAIL换成-1
if(ret==-1),所有编译都能通过。请教高手给指点下原因
bfhtian 2009-02-19
  • 打赏
  • 举报
回复
一般是提示上面一行有错
ch1oE 2009-02-19
  • 打赏
  • 举报
回复
#define OO_FAIL -1
后面不用加分号 -- 因为macro在做完常量运算后会直接替换过去 -- 导致你的 if(ret == -1;) ..
所以会错~

65,211

社区成员

发帖
与我相关
我的任务
社区描述
C++ 语言相关问题讨论,技术干货分享,前沿动态等
c++ 技术论坛(原bbs)
社区管理员
  • C++ 语言社区
  • encoderlee
  • paschen
加入社区
  • 近7日
  • 近30日
  • 至今
社区公告
  1. 请不要发布与C++技术无关的贴子
  2. 请不要发布与技术无关的招聘、广告的帖子
  3. 请尽可能的描述清楚你的问题,如果涉及到代码请尽可能的格式化一下

试试用AI创作助手写篇文章吧