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，因此将创建一个副本，并且只对进行更改，即副本：

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