我有一个使用operator==
实现nullptr
的自定义类。
这是我的代码愚蠢到一个简单的例子:
#include <cstdint>
#include <iostream>
class C {
private:
void *v = nullptr;
public:
explicit C(void *ptr) : v(ptr) { }
bool operator==(std::nullptr_t n) const {
return this->v == n;
}
};
int main()
{
uint32_t x = 0;
C c(&x);
std::cout << (c == nullptr ? "yes" : "no") << std::endl;
C c2(nullptr);
std::cout << (c2 == nullptr ? "yes" : "no") << std::endl;
return 0;
}
代码按预期工作,但g ++(版本6.2.1)给出了以下警告:
[Timur@Timur-Zenbook misc]$ g++ aaa.cpp -o aaa -Wall -Wextra
aaa.cpp: In member function ‘bool C::operator==(std::nullptr_t) const’:
aaa.cpp:12:36: warning: parameter ‘n’ set but not used [-Wunused-but-set-parameter]
bool operator==(std::nullptr_t n) const {
^
我做错了什么?
注意:我正在使用-Wall -Wextra
。
答案 0 :(得分:9)
对于为什么会发生这种情况并不是真正的答案,但无论如何,n
只有nullptr
可以有什么价值?
返回this->v == nullptr
并使参数未命名会删除警告:
bool operator==(std::nullptr_t) const {
return this->v == nullptr;
}
修改强>
将n
声明为右值引用,或作为const左值引用也会删除警告:
bool operator==(std::nullptr_t&& n) const {
return this->v == n;
}
bool operator==(const std::nullptr_t& n) const {
return this->v == n;
}
<强> EDIT2:强>
可以在this question中找到更多关于未使用变量警告的方法(thx @ShafikYaghmour将其指向评论中)。上面的例子涵盖了“隐含”的方式。
显式解决方案可用,但由于有效使用参数,因此IMHO看起来不那么连贯。经过测试的显式解决方案包括:
bool operator==(std::nullptr_t n) const {
(void)n;
return this->v == n;
}
#define UNUSED(expr) do { (void)(expr); } while (0)
bool operator==(std::nullptr_t n) const {
UNUSED(n);
return this->v == n;
}
GCC的非便携式解决方案:
bool operator==(__attribute__((unused)) std::nullptr_t n) const {
return this->v == n;
}