我有一个跨平台的应用程序，在我的几个函数中，并不是所有传递给函数的值都被利用。因此我从GCC得到一个警告，告诉我有未使用的变量。
绕开警告的最佳编码方式是什么?
在函数周围使用#ifdef ?
#ifdef _MSC_VER
void ProcessOps::sendToExternalApp(QString sAppName, QString sImagePath, qreal qrLeft, qreal qrTop, qreal qrWidth, qreal qrHeight)
#else
void ProcessOps::sendToExternalApp(QString sAppName, QString sImagePath, qreal /*qrLeft*/, qreal /*qrTop*/, qreal /*qrWidth*/, qreal /*qrHeight*/)
#endif
{
这是如此丑陋，但似乎是编译器更喜欢的方式。
还是把函数末尾的变量赋值为0 ?(我讨厌这样做，因为它改变了程序流中的某些东西，从而使编译器警告静音)。
有正确的方法吗?
当前回答
c++ 17更新
在c++ 17中，我们获得了[[maybe_unused]]属性，该属性包含在[dcl.attr.unused]中。
属性令牌maybe_unused表示一个名称或实体可能是故意未使用的。应当 每个属性列表中最多出现一次，且不允许出现属性参数子句。 .．. 例子: [[maybe_unused]] void f([[maybe_unused]] bool thing1， [[maybe_unused]] bool thing2 { [[maybe_unused]] bool b = thing1 && thing2; 断言(b); ｝ 不管是否定义了NDEBUG，实现都不应该警告b未使用。-end示例]
示例如下:
int foo ( int bar) {
bool unused_bool ;
return 0;
}
clang和gcc都使用-Wall -Wextra为bar和unused_bool生成诊断。
添加[[maybe_unused]]将使诊断静默:
int foo ([[maybe_unused]] int bar) {
[[maybe_unused]] bool unused_bool ;
return 0;
}
现场观看。
在c++ 17
在c++ 11中，可以使用lambda表达式(通过Ben Deane)捕获未使用的变量来形成UNUSED宏的另一种形式:
#define UNUSED(x) [&x]{}()
lambda表达式的直接调用应该被优化掉，给出如下示例:
int foo (int bar) {
UNUSED(bar) ;
return 0;
}
我们可以在godbolt中看到调用被优化了:
foo(int):
xorl %eax, %eax
ret
其他回答
使用UNREFERENCED_PARAMETER(p)可以工作。我知道它是在Windows系统的WinNT.h中定义的，也可以很容易地为gcc定义(如果它还没有)。
未引用参数(p)定义为
#define UNREFERENCED_PARAMETER(P) (P)
在WinNT.h。
在GCC和Clang中，你可以使用__attribute__((未使用))预处理器指令来实现你的目标。 例如:
int foo (__attribute__((unused)) int bar) {
return 0;
}
大多数时候，使用预处理器指令被认为是邪恶的。理想情况下，你应该像避开害虫一样避开它们。记住，让编译器理解你的代码很容易，让其他程序员理解你的代码就难多了。到处都有几十个这样的案例，这让你以后很难阅读，也让别人现在很难阅读。
一种方法可能是将参数放在某种类型的参数类中。然后，您可以只使用变量的一个子集(实际上相当于为0赋值)，或者为每个平台对该参数类有不同的专门化。然而，这可能并不值得，你需要分析它是否合适。
如果你能阅读不可能的模板，你可能会在“例外c++”一书中找到高级技巧。如果阅读您代码的人能够掌握书中所教的疯狂内容，那么您将拥有易于阅读的漂亮代码。编译器也会很清楚你在做什么(而不是通过预处理隐藏所有内容)
无宏且可移植的声明一个或多个参数为未使用的方法:
template <typename... Args> inline void unused(Args&&...) {}
int main(int argc, char* argv[])
{
unused(argc, argv);
return 0;
}
一种更简洁的方法是只注释掉变量名:
int main(int /* argc */, char const** /* argv */) {
return 0;
}
