简化__VA_ARGS__问题:无法提取NULL

时间:2012-08-05 18:47:51

标签: c++ variadic args

我收到了以下代码:

头文件:

  #define html(...) anyHtml(__VA_ARGS__, NULL);
  const char* Html::anyHtml(const char* arg, ...);
  #define body(cssClass, ...) anyBody(cssClass, __VA_ARGS__, NULL);
  const char* Html::anyBody(const char* cssClass, const char* arg, ...);

cpp文件:

const char* Html::anyHtml(const char* arg, ...) {
string temp = initOpen + tagToString.at(HTML) + end + lf + lf;
va_list arguments;
for (va_start(arguments, arg); arg != NULL; arg = va_arg(arguments, const char*)) {
temp += arg + lf;
}
va_end(arguments);
temp += deOpen + tagToString.at(HTML) + end + lf;
string *returnThis = new string(temp);
return (*returnThis).c_str();
}

const char* Html::body(const char* cssClass, const char* arg, ...) {
string temp = initOpen + tagToString.at(BODY) + " class=\"" + cssClass + "\"" + end + lf + lf;
va_list arguments;
for (va_start(arguments, arg); arg != NULL; arg = va_arg(arguments, const char*)) {
temp += arg + lf +lf;
}
va_end(arguments);
temp += deOpen + tagToString.at(BODY) + end + lf;
string *returnThis = new string(temp);
return (*returnThis).c_str();
}

主文件:

Html *html = new Html();

cout << html -> html("test","test","test");
cout << html -> body("thisClass","test","test");

工作得很好!!

但是:

Html *html = new Html();

cout << html -> html(
    "hello",
    html -> body("thisClass","content")
    );

不工作.. 如果我在头文件

中取消对anyBody的宏的注释
//#define body(cssClass, ...) anyBody(cssClass, __VA_ARGS__, NULL);

可能是这样的:

Html *html = new Html();

cout << html -> html(
    "hello",
    html -> body("thisClass","content",NULL)
    );

现在我的问题:

是否可以嵌套一些宏,如

#define body(cssClass, ...) anyBody(cssClass, __VA_ARGS__, NULL);

他们之间?

我已经读过,整个“变量参数计数”的东西今天仍然没有在cpp中真正设计过吗?

如果你看看函数anyHtml()和anyBody()。对于这个问题,这是一个“好”的解决方案,还是某种肮脏的新手攻击?我仍然是新来的cpp与perl和java一起出现...为此我想得到一些反馈以进一步了解它..

现在......够了: - )

欢迎你告诉我你想说什么..: - )

----编辑

好的,对不起..

Html *html = new Html();

cout << html -> html(
    "hello",
    html -> body("thisClass","content")
    );

输出应为

<html>
hello
<body class="thisClass">
content
</body>
</html>

但它给出了

C2143 Syntax Error missing ')' before ';'
C2143 Syntax Error missing ')' before ';'
C2059 Syntax Error: ')'

如果我同时使用

#define html(...) anyHtml(__VA_ARGS__, NULL);
#define body(cssClass, ...) anyBody(cssClass, __VA_ARGS__, NULL);

如果我做

cout << html -> html("hello","test");
cout << html -> body("thisClass","content");

这些错误没有出现..

没有这个宏我必须输入:

 cout << html -> html(
    "hello",
    html -> body("thisClass","content",NULL)
    );

非常特别我知道..: - (

---编辑

感谢您解决我的问题!

宏定义中的分号解决了嵌套问题...

函数anyHtml()和body()......

使用

是否正确
string temp = ...

并在字符串操作后将其强制转换?

还是很脏?

2 个答案:

答案 0 :(得分:1)

你的宏中有无关的分号。删除它们:

#define html(...) anyHtml(__VA_ARGS__, NULL)
#define body(cssClass, ...) anyBody(cssClass, __VA_ARGS__, NULL)

如果你不删除它们,最终会出现语法错误,因为扩展会变成这样的:

cout << html -> anyHtml(
    "hello",
    html -> anyBody("thisClass", "content", NULL);, NULL);;

编辑:您问:

  

使用string temp = ...并在字符串操作后将其强制转换是否正确?还是很脏?

您没有投射temp,而是在动态分配的内存中创建它的副本。虽然这允许您传回有效的C字符串,但它会产生内存泄漏,因为您将无法在您创建的对象上调用delete

您只需更改例程即可返回std::string,然后您可以直接返回temp,而无需动态分配。

答案 1 :(得分:1)

你的宏定义后面有分号,所以你的参数列表中最后会出现一个分号

cout << html -> html(
  "hello",
  html -> body("thisClass","content")
  );