没有上下文类型信息的重载函数 |无法解析重载函数“swap”基于到类型“int”的转换；

Question

我正在尝试编写自己的冒泡排序算法作为练习。我不明白这两个错误消息。谁能指出我的代码有问题吗？

// Bubble sort algorithm
#include <iostream>
#include <iomanip>
using namespace std;

void bubbleSort(int array[], int arraySize); // bubbleSort prototype

int main(void)
{
        const int arraySize = 10;
        int array[arraySize] = {2,3,6,5,7,8,9,3,7,4};

        cout << "Unsorted: ";
        for(int i = 0; i < arraySize; ++i)
                cout << setw(5) << array[i];

        cout << "Sorted: " << bubbleSort(array, arraySize);
}

void bubbleSort(int array[], int arraySize)
{
        const int max = arraySize;
        int swap = 0;

        for(int i = 0; i < max; ++i)
        {
                if(array[i] > array[i + 1])
                {
                        swap = array[i + 1];
                        array[i + 1] = array[i];
                        array[i] = swap;
                }
                else
                        break;
        }
}

Answer 1

我看到您正在使用

using namespace std;

So 当您键入时

array[i] = swap;

编译器无法消除您是指的是 std::swap 函数还是 int swap 变量的歧义。事实上，它看起来像是假设您正在引用该函数并尝试以某种方式将其转换为 int 类型。尝试将变量重命名为其他名称。

一般来说，尽量远离 using 指令，以避免像这样的名称冲突。

Answer 2

array[i] = swap;

这条线引起了问题。最好更改 swap 局部变量的名称，因为在 std 命名空间中已经存在一个同名的函数，该函数由行 引入作用域无论如何，应该避免使用命名空间 std; 。

我还建议您在 if 块内声明变量，实际使用它：

if(array[i] > array[i + 1])
{
     //declare temp here where it is actually used!
     int temp = array[i + 1]; 
     array[i + 1] = array[i];
     array[i] = temp;
}

最佳实践：通过延迟局部变量的声明来减少局部变量的范围，这意味着在实际使用它们的地方声明它们。不要在函数的开头声明它们。

解决代码中问题的另一种方法是为编译器提供一个上下文，您可以通过这样做（虽然我不会建议这个解决方案；这只是为了让您知道< /em>):

array[i] = (int)swap; //giving compiler contextual type information

当你将 swap 转换为 int 时，编译器可以知道 swap 指的是局部变量，而不是函数。在 std 命名空间中定义。

Answer 3

cout << "Sorted: " << bubbleSort(array, arraySize);

该函数的返回类型是void。没有什么可打印的。如果需要打印排序后的数组，则需要在函数调用后迭代数组元素。