0

我收到 -Weffc++ 发出的警告，这似乎是错误的。我可以用第二双眼睛来确认：

template<template<class> class CLASS_TYPE, typename T>
class some_class
{
typedef CLASS_TYPE<T> class_type;

public:
   virtual ~some_class() {};
   virtual class_type& operator++() = 0;
};

template<typename T>
class other_class
:
public some_class<other_class, T>
{
public:
   virtual ~other_class() {};
   other_class<T>& operator++() {
      return *this;
   };
};

int main() {

   return 0;
}

警告是：

main.cpp:8:39: warning: prefix ‘some_class<CLASS_TYPE, T>::class_type& some_class<CLASS_TYPE, T>::operator++()’ should return ‘some_class<CLASS_TYPE, T>&’ [-Weffc++]
    virtual class_type& operator++() = 0;

经测试g++ (GCC) 4.9.3。

更新

添加了一个附加类以提供实现示例。警告本身是正确的，但我认为我不同意警告出现在纯虚函数上，因为它是另一个类的接口。

@Frerich Raabe 提供了必要的说明，说明为什么 g++ 认为我违反了 Effective C++ 设置的规则，并且我已经接受了这个答案。

为了使警告静音，我添加了以下内容：

#pragma GCC diagnostic push
#pragma GCC diagnostic ignored "-Weffc++"
virtual class_type& operator++() = 0;
#pragma GCC diagnostic pop
4

1 回答 1

1

编译器是正确的。您的operator++重新实现应该返回一个与 相同类型的值*this，即 a some_class<...>。事实上，operator++()end in的许多实现

return *this;

有关更详细的讨论，请参阅此答案中标记为“一元算术运算符”的部分。

于 2016-02-12T14:55:12.870 回答