在带有迭代器和模板的 C++ 中使用 std::sort

Question

我试图让 std::sort 在使用模板和稍微复杂的数据结构时工作。 我似乎无法弄清楚为什么以下命令会导致错误

sort(graph->edge[0], graph->edge[(graph->E)-1], myComp<T>);

这背后的数据结构如下所示。

template <typename T>
class Edge
{
    public:
    int src, dest;
    T weight;
};

template <typename T>
class Graph_krus
{
    public:
    int V, E;
    Edge<T>* edge;
};

template <typename T>
Graph_krus<T>* createGraph(int V, int E)
{
    Graph_krus<T>* graph = new Graph_krus<T>;
    graph->V = V;
    graph->E = E;
    graph->edge = new Edge<T>[E];
    return graph;
}

template <typename T>
bool myComp(Edge<T>* a, Edge<T>* b)
{
    bool greater = a->weight > b->weight;
    return greater;
}

这会导致许多构建时错误。 来自 Xcode 的错误似乎是

“在‘std::__1::iterator_traits >’中没有名为‘difference_type’的类型”

“在‘std::__1::iterator_traits >’中没有名为‘value_type’的类型”

“二进制表达式的无效操作数（'Edge' 和 'Edge'）”

“不能递减‘Edge’类型的值”

和更多

Answer 1

std::sort需要迭代器或指针。 graph->edge[0]返回一个Edge对象引用而不是一个指针。 您需要传递指针：

std::sort(graph->edge, graph->edge + graph->E-1, myComp<T>);    

假设您想要对所有边缘进行排序，您的第二个指针需要成为列表末尾的边缘：

std::sort(graph->edge, graph->edge + graph->E, myComp<T>);

下一个问题是myComp需要引用而不是指针：

template <typename T>
bool myComp(const Edge<T>& a, const Edge<T>& b)
{
    bool greater = a.weight > b.weight;
    return greater;
}

正如其他人指出的那样，用std::vector替换原始指针数组会简化您的代码并使其更安全：

#include <algorithm>
#include <vector>
#include <memory>

template <typename T>
class Edge
{
    public:
    int src, dest;
    T weight;
};

template <typename T>
struct Graph_krus
{
    int V;
    std::vector<Edge<T>> edge;
};

template <typename T>
std::unique_ptr<Graph_krus<T>> createGraph(int V, int E)
{
    auto graph = std::make_unique<Graph_krus<T>>();
    graph->V = V;
    graph->edge.resize(E);
    return graph;
}
template <typename T>
bool myComp(const Edge<T>& a, const Edge<T>& b)
{
    bool greater = a.weight > b.weight;
    return greater;
}

int main()
{
    auto graph = createGraph<int>(1,1);
    std::sort(graph->edge.begin(), graph->edge.end(), myComp<int>);    
}