Question

我们正在使用宏包装器来绑定参数函数。

#define bindWhereClause(fieldName, fieldDataType, fieldData) _bindWhereClause(fieldName, fieldDataType, sizeof(fieldData), &fieldData)

void _bindWhereClause(const char *name, int dataType, int dataSize, void *data)
{
  // Implementation
}

Database.bindWhereClause( "FIRST_NAME", SQL_VARCHAR, name.getFirstName());

当我尝试使用函数作为参数（如上所述）调用宏时，我收到错误消息“error：non-lvalue in anary`＆amp;'”。

我可以使用普通变量调用宏，例如

Database.bindWhereClause( "FIRST_NAME", SQL_VARCHAR, firstName);

如何解决这个问题？我是否需要使用内联函数而不是宏？

提前感谢您的帮助。

谢谢， Mathew Liju

Answer 1

你不能取一个函数返回值的地址 - 它是短暂的。

您需要使用实数变量：

Nametype first_name = name.getFirstName();
Database.bindWhereClause( "FIRST_NAME", SQL_VARCHAR, first_name);

// ... and maybe name.setFirstName(first_name); here

使用内联函数会使其编译，但实际上不可能工作。据推测，接下来是：

Database.execute();

...期望您之前传递的地址仍然有效。如果使用内联函数而不是宏，那么这些对象将不再存在，因为它们只是已经退出的内联函数的本地函数。

Answer 2

对于这种情况，您可以使用内联函数。它通常要好得多，你应该使用内联函数，除非绝对需要使用宏。

Answer 3

宏扩展为：

_bindWhereClause("FIRST_NAME", SQL_VARCHAR, sizeof(name.getFirstName()), &name.getFirstName())

您无法获取函数返回值的地址，因此无效。你可以获取变量或参数的地址，所以是的，把它变成一个带有实际参数的函数就行了。

Answer 4

为什么不在Database周围编写一个包装类。

查看装饰器模式：http://en.wikipedia.org/wiki/Decorator_pattern

错误：一元“＆amp;”中的非左值在C ++中

4 个答案: