包装谓词函子

Question

我想知道关于谓词函子包装的约定和最佳实践。例如，给定的一类，如：

class Timer 
{ 
public: 
    Timer(const std::string& name, int interval); 
    bool nameIs(const std::string& name) const; 
private: 
    std::string name_; 
    int interval_; 
}; 

在TimerVec类使用的是（在一种情况下）：

class TimerVec 
{ 
public: 
    typedef std::vector<Timer>::iterator iterator;`` 
    <... ctors, etc ...> 
    iterator findByName(const std::string& name); 
private: 
    std::vector<Timer> timers_; 
}; 

并具有谓词函子像：

class TimerNameIs 
{ 
public: 
    TimerNameIs(const std::string& name) : name_(name) {} 
    bool operator()(const Timer& t) { return t.nameIs(name_); } 
private: 
    const std::string& name_; 
}; 

我可以想想一些地方可以放上函子代码，有些是：

1. 紧接在定时器声明之后的头文件中
2. 嵌套在定时器内（即，所以裁判成为Timer::TimerNameIs）
3. 内TimerVec（目前唯一的用户）
4. 嵌套在提前为TimerVec::findByName实施（再次仅它使用的地方）的匿名命名空间

虽然任何这些会我相当喜欢＃2，但这不是我见过的。有没有具体的理由支持特定的选择？

Answer 1

这是开放辩论。我更喜欢创建一个嵌套类。通过这种方式，仅用于处理特定类型对象的仿函数是该对象内的命名空间范围。

我也通常命名谓词match_xxx其中xxx是我匹配的参数。

机智：

class Timer 
{ 
    // ... 
public: 
    class match_name : public std::unary_function<Timer, bool> 
    { 
    public: 
    match_name(const std::string& name) : name_(name) {} 
    bool operator()(const Timer& t) { return t.nameIs(name_); } 
    private: 
    const std::string& name_; 
    }; 
}; 

...这正是如此使用：

std::find_if(v.begin(), v.end(), Timer::match_name("Flibbidy")); 

在这个代号为6个月后看的时候我喜欢这种方法，因为Timer::match_name("Flibbidy")语义是非常清楚的。

我也很小心从std::unary_function派生我的仿函数（虽然我上面的派生可能有参数颠倒）。

Answer 2

我个人在它自己的头文件和cpp文件中。在TimerNameIs头文件使用#include "Timer.h"：

#include "Timer.h" 
#include <string> 

class TimerNameIs 
{ 
    public: 
     TimerNameIs(const std::string& name) : name_(name) {} 
     bool operator()(const Timer& t) { return t.nameIs(name_); } 
    private: 
     const std::string& name_; 
}; 

这样做，你从一个到另一个隔离定时器和TimerNameIs。