合并排序 - 向量不排序

Question

我正在尝试对std::vector<int> 执行合并排序。向量中的每个 int 对应于另一个向量 std::vector<Node> 中的一个索引。每个节点都有一个深度。我正在尝试根据深度进行排序。

我正在使用我找到的一些合并排序代码。使用整数数组可以正常工作，所以我认为它可以在我的代码中正常工作。这是我的代码的缩短版本：

.h 文件：

class Log{
public:
    static std::vector<int> a;
};

.cpp 文件：

std::vector<int> Log::a;

int getNodeDepth (int index, std::vector<cv::ml::DTrees::Node::Node> nodeList, std::vector<int> treeList) {
    //returns the node's depth
}

void exchange(int i, int j) {
    int t = Log::a[i];
    Log::a[i] = Log::a[j];
    Log::a[j] = t;
}

void compare(int i, int j, std::vector<cv::ml::DTrees::Node::Node> nodeList) {
    if (getNodeDepth(nodeList[Log::a[i]], nodeList) > (getNodeDepth(nodeList[Log::a[j]], nodeList)))
    exchange(i, j);
}

/**
 * lo is the starting position and
 * n is the length of the piece to be merged,
 * r is the distance of the elements to be compared
 */
void oddEvenMerge(int lo, int n, int r, std::vector<cv::ml::DTrees::Node::Node> nodeList) {
    int m = r * 2;
    if (m < n) {
        oddEvenMerge(lo, n, m, nodeList); // even subsequence
        oddEvenMerge(lo + r, n, m, nodeList); // odd subsequence
        for (int i = lo + r; i + r < lo + n; i += m)
            compare(i, i + r, nodeList);
    } else
        compare(lo, lo + r, nodeList);
}

/**
 * sorts a piece of length n of the array
 * starting at position lo
 */
void oddEvenMergeSort(int lo, int n, std::vector<cv::ml::DTrees::Node::Node> nodeList) {
    if (n > 1) {
        int m = n / 2;
        oddEvenMergeSort(lo, m, nodeList);
        oddEvenMergeSort(lo + m, m, nodeList);
        oddEvenMerge(lo, n, 1, nodeList);
    }
}

int mergeSort(std::vector<int> treeList, std::vector<cv::ml::DTrees::Node::Node> nodeList) {
    Log::a = treeList;
    int i, n = sizeof(Log::a) / sizeof(Log::a[0]);
    for (i = 0; i < n; i++)
        std::cout << std::setw(3) << Log::a[i];
    std::cout << std::endl;
    oddEvenMergeSort(0, n, nodeList);
    //print sorted list
    for (i = 0; i < Log::a.size(); i++)
        std::cout << Log::a[i] << "  *   ";
    std::cout << std::endl;

    return(0);
}

请注意，变量nodeList 就在那里，因为深度方法需要它。 输出看起来像只触摸了矢量的前半部分。向量后半部分的任何项目都不会被交换。我再次检查以确保深度是正确的，并且正在交换正确的东西。它只是没有完成工作。任何想法为什么？

Answer 1

sizeof(Log::a) / sizeof(Log::a[0]);

这没有得到Log::a中的元素个数！它获取std::vector 类型的大小（以字节为单位），这与包含的元素的数量无关，然后除以元素的大小。这会给你一些垃圾。

这个成语适用于数组，std::size 更好，但还不是标准的——你可以像这样轻松地自己写：

template <typename T, std::size_t N>
std::size_t arraySize(T (&)[N])
{
    return N;
}

（如果你需要它作为编译时间常数，还有更复杂的版本。）

使用Log::a.size(); 获取std::vector 的大小。

（我可能遗漏了其他问题，但这个问题很突出。）