我有一个跨平台的应用程序，在我的几个函数中，并不是所有传递给函数的值都被利用。因此我从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 ?(我讨厌这样做，因为它改变了程序流中的某些东西，从而使编译器警告静音)。
有正确的方法吗?
当前回答
你可以使用__unused来告诉编译器这个变量可能不会被使用。
- (void)myMethod:(__unused NSObject *)theObject
{
// there will be no warning about `theObject`, because you wrote `__unused`
__unused int theInt = 0;
// there will be no warning, but you are still able to use `theInt` in the future
}
其他回答
总是注释掉参数名是否不安全?如果不是，你可以做一些
#ifdef _MSC_VER
# define P_(n) n
#else
# define P_(n)
#endif
void ProcessOps::sendToExternalApp(
QString sAppName, QString sImagePath,
qreal P_(qrLeft), qreal P_(qrTop), qreal P_(qrWidth), qreal P_(qrHeight))
它不那么难看。
这工作得很好，但需要c++ 11
template <typename ...Args>
void unused(Args&& ...args)
{
(void)(sizeof...(args));
}
一个同事刚给我指了这个小宏
为了方便起见，我将包括下面的宏。
#ifdef UNUSED
#elif defined(__GNUC__)
# define UNUSED(x) UNUSED_ ## x __attribute__((unused))
#elif defined(__LCLINT__)
# define UNUSED(x) /*@unused@*/ x
#else
# define UNUSED(x) x
#endif
void dcc_mon_siginfo_handler(int UNUSED(whatsig))
当前的解决方案是最好的——如果不使用参数名，就注释掉它。这适用于所有编译器，因此不需要使用预处理程序专门为GCC做这件事。
无宏且可移植的声明一个或多个参数为未使用的方法:
template <typename... Args> inline void unused(Args&&...) {}
int main(int argc, char* argv[])
{
unused(argc, argv);
return 0;
}
