C++ 尝试插入排序失败？

Question

这段代码直接翻译自维基百科上的伪代码。然而它不交换任何元素。有人可以告诉我发生了什么事吗？

#include<iostream>
#include<string>
#include<fstream>
#include<cctype>
#include<vector>
using namespace std;

void sortowanie(vector<int> v)
{
    for(int i=1; i<=(int)v.size()-1; i++)
    {
        for(int j=i; j>0; j--)
        {
            if(v[j-1] > v[j])
            {
                int x = v[j-1];
                v[j-1] = v[j];
                v[j] = x;
            }
            else
            {
                break;
            }
        }
    }
}

int main()
{
    vector<int> v;
    v.push_back(4);
    v.push_back(2);
    v.push_back(15);
    v.push_back(234);
    v.push_back(21);
    v.push_back(6);
    v.push_back(3);
    v.push_back(53);
    v.push_back(23);
    v.push_back(1);

    sortowanie(v);
    for(int i=0; i<=(int)v.size()-1; i++)
    {
        cout << " -> " << v[i];
    }
    cout << endl;

    return 0;
}

Answer 1

您正在按值传递 std::vector，因此将制作一个副本，并且只会在 that 副本上进行更改：

void sortowanie(vector<int> v)

添加一个 & 以使用 pass-by-reference，这样函数内对 std::vector 的任何更改也将应用于原始 std::vector：

void sortowanie(vector<int>& v)

完整代码：

#include<iostream>
#include<vector>
using namespace std;

void sortowanie(vector<int>& v)
{
    for(size_t i = 1; i < v.size(); i++)
    {
        for(int j = i; j > 0; j--)
        {
            if(v[j-1] > v[j]) { swap(v[j-1], v[j]); }
            else  {break;}
        }
    }
}

int main()
{
    vector<int> v({4,2,15,234,21,6,3,53,23,1});

    sortowanie(v);
    for(int x : v)
    {
        cout << x << " ";
    }
    cout << "\n";

    return 0;
}

此外，使用 swap()（顺便说一句，也使用 pass-by-reference）和初始化列表将大大缩短程序。

输出：1 2 3 4 6 15 21 23 53 234

C++ 尝试插入排序失败？

C++ experimenting with insertion sort fail?

c++

sorting

insertion-sort