Question

这是我正在尝试的简单冒泡排序：

template<class T>
void bubbleSort(T *begin, T *end) {
    for (auto index = begin + 1; index != end; ++index) {
        for (auto bubble = begin; bubble != end - 1; ++bubble) {
            if (*bubble > *(bubble + 1)) {

                const T temp = *bubble;
                *bubble = *(bubble + 1);
                *(bubble + 1) = temp;
            }
        }
    }
}

这个版本似乎有效（在它的所有泡沫排序荣耀中）。顺便说一句，这是我正在测试它的类，如果它有帮助：

class Numbers {
    int max;
    int *numbers;

public:
    Numbers(initializer_list<int> initialList) : max { initialList.size() }, numbers { new int[max] }
    {
        int index = 0;
        for (auto it = initialList.begin(); it != initialList.end(); ++it, ++index) {
            numbers[index] = *it;
        }
    }

    int operator *(int index) { return numbers[index]; }
    int *begin() { return &numbers[0]; }
    int *end() { return &numbers[max]; }
};

我尝试做的是使用std::swap在内循环中编写手动交换，如下所示：

for (auto bubble = begin; bubble != end - 1; ++bubble) {
    if (*bubble > *(bubble + 1)) swap (bubble, bubble + 1);
}

但由于某种原因，编译器告诉我：

error C2665: 'std::swap' : none of the 3 overloads could convert all the argument types

为什么？

Answer 1

swap通过引用获取其参数。在代码的第一个版本中，您（正确地）编写：

const T temp = *bubble;
*bubble = *(bubble + 1);
*(bubble + 1) = temp;

现在考虑如何交换，例如，两个整数：

const int temp = a;
a = b;
b = temp;
// or more simply
swap(a, b);

因此，您的swap应反映您在第一个正确版本中所做的解除引用：

swap(*bubble, *(bubble + 1));
//   ^ here   ^ and here

Answer 2

std::swap将参考作为参数。

你正在指点。

你应该这样做：

swap ( *bubble, *(bubble + 1) );
//     ^        ^

我们在这里取消引用指针以使其有效。

Answer 3

您需要取消引用：

swap (*bubble, *(bubble + 1));

为什么std :: swap不能在模板项上工作？

3 个答案: