是否可以去掉下面代码中的模板?
想法是使用 std::priority_queue
和通过自制抽象类实现 Compare
概念的类。
我遇到的一些问题是:
- 优先级队列不会用抽象比较初始化。
- 从
Compare
继承的类没有默认构造函数,因为它们都有特定的参数。 Algorithm
负责初始化Compare
实例使用的数据结构。- 编辑:
算法
可能存在多个实现,使用不同的数据结构需要比较
,类的用户应该是能够决定他想使用哪个比较。
我不喜欢下面给出的设计,因为实现 Algorithm
的人并不是真的被迫调用 Compare::attach
并且 Compare
类应该因此有状态管理(此处未显示)。
它还强制用户指定一个比较模板并传递相应的实例(因为编译器无法在 Algorithm
的构造函数中推断它)。 make_algo
函数可以缓解这种情况,但我宁愿避免这种奇特的构造(不是那种奇特的,但仍然如此)。
有没有一种设计可以让我在Algorithm
而不是模板中使用Compare
抽象类?
#include <vector>
#include <queue>
class Compare
{
private:
std::vector<int>* _costs;
protected:
std::vector<int>& costs() {return *_costs;}
int cost(const int i) const {return (*_costs)[i];}
public:
Compare() : _costs(nullptr) {}
void attach(std::vector<int> & costs_) {_costs = &costs_;}
virtual bool operator()( const int a, const int b ) const = 0;
};
struct CompCosts : public Compare
{
virtual bool operator()( const int a, const int b ) const
{
return this->cost(a) < this->cost(b);
}
};
struct CompEps : public Compare
{
const int eps;
CompEps(const int e) : Compare(), eps(e) {}
virtual bool operator()( const int a, const int b ) const
{
return this->cost(a)-eps < this->cost(b)+eps;
}
};
template<typename C>
struct Algorithm
{
C & comp;
Algorithm(C& comp_) : comp(comp_) {}
virtual void operator()() const = 0;
};
template<typename C>
struct Algo : public Algorithm<C>
{
Algo(C & comp_) : Algorithm<C>(comp_) {}
virtual void operator()() const
{
std::vector<int> costs;
this->comp.attach(costs);
std::priority_queue<int, std::vector<int>, C > queue(this->comp);
}
};
template<typename C>
Algo<C> make_algo(C& comp)
{
return Algo<C>(comp);
}
int main()
{
CompCosts compc;
Algo<CompCosts> algo0(compc);
auto algo1 = make_algo(compc);
CompEps compe(1);
Algo<CompEps> algo2(compe);
auto algo3 = make_algo(compe);
}
EDIT2:由于Barry提出的解决方案对每个人来说可能不是显而易见的,这里是相应的代码:
#include <iostream>
#include <vector>
#include <queue>
#include <functional>
#include <cassert>
struct Compare
{
virtual bool operator()( const int a, const int b, const std::vector<int>& costs ) const =0;
};
struct CompCosts : public Compare
{
virtual bool operator()( const int a, const int b, const std::vector<int>& costs ) const
{
return costs[a] < costs[b];
}
};
struct CompEps : public Compare
{
const int eps;
CompEps(const int e) : Compare(), eps(e) {}
virtual bool operator()( const int a, const int b, const std::vector<int>& costs ) const
{
return costs[a]-eps < costs[b]+eps;
}
};
struct Algorithm
{
std::function<bool(const int, const int, const std::vector<int>& costs )> comp;
Algorithm(
std::function<
bool(const int, const int, const std::vector<int>& costs )
> comp_
) : comp(comp_) {}
virtual void operator()() const = 0;
};
struct Algo : public Algorithm
{
Algo(std::function<bool(const int, const int, const std::vector<int>& costs )> comp_) : Algorithm(comp_) {}
virtual void operator()() const
{
std::vector<int> costs{3,2,1};
using namespace std::placeholders;
std::function<bool(const int, const int)> f = std::bind(comp, _1, _2, std::cref(costs));
std::priority_queue<int, std::vector<int>, std::function<bool(const int, const int )> > queue(f);
}
};
int main()
{
CompCosts compc;
Algo algo0(compc);
algo0();
CompEps compe(0);
Algo algo2(compe);
algo2();
}
最佳答案
自 priority_queue
持有 Compare
类型的对象, 那里的任何多态性尝试都会导致切片和失败。但是无论如何,多态性确实是对问题的过度思考(就像我在之前的修订版中所做的那样)。
只需使用类型删除。你所有的比较器都会有一个 bool operator()(int, int)
,比如:
struct CompCosts
{
std::vector<int> const& costs;
bool compare(int lhs, int rhs) const
{
return costs[lhs] < costs[rhs];
}
};
所以它们都可以由 std::function<bool(int, int)>
处理:
using Q = std::priority_queue<int, std::vector<int>, std::function<bool(int, int)>>;
然后创建您的 Q
无论你想使用哪个比较器:
Q queue_by_cost(CompCosts{costs});
Q queue_by_eps(CompEps{costs});
关于c++ - 使用自制的 `std::priority_queue` 初始化 `std::Compare` 时避免使用模板?,我们在Stack Overflow上找到一个类似的问题: https://stackoverflow.com/questions/37465772/