提问人:Aayush Mahajan 提问时间:6/1/2018 更新时间:6/1/2018 访问量:28
通过引用传递向量会使迭代算法以不同的方式运行
Passing vector by reference makes iterative algorithm run differently
问:
这让我感到困惑。我正在用C++实现荷兰国旗问题。我最初忘记通过引用传递我的数组,但在每一步都显示数组,这帮助我得出结论,我的算法工作正常。为了修复向量最终未分区的微小错误,我更改了函数定义以传递向量作为引用,这不知何故产生了错误。
我以前从未遇到过这样的事情。代码不是递归的,因此当函数运行时,它是在向量的副本上运行还是在实际向量上运行都无关紧要。
代码如下:
#include<iostream>
#include<vector>
#include<algorithm>
#include<iomanip>
using namespace std;
void optArr(vector<int> arr){
for(auto i: arr){
cout<<setw(3)<<i<<" ";
}
cout<<endl;
}
void _partition(vector<int> arr, int lo, int hi, int &i, int &k){ //Vector passed by value
i=lo-1, k=hi+1;
int ptr=lo, pivot=arr[hi];
while(ptr<k){
optArr(arr);
cout<<"i: "<<i<<" ptr: "<<ptr<<" k: "<<k<<endl;
if(arr[ptr]<pivot){
swap(arr[++i],arr[ptr++]);
}else if(arr[ptr] == arr[pivot]){
ptr++;
}else{
swap(arr[--k],arr[ptr]);
}
}
}
int main(){
vector<int> arr = {57,22,85,17,11,04,17,93,1,17,25,17};
int i,k;
_partition(arr, 0, arr.size()-1, i, k);
optArr(arr);
cout<<"i: "<<i<<" ptr: "<<0<<" k: "<<k<<endl;
return 0;
}
向量按值传递时的输出:
57 22 85 17 11 4 17 93 1 17 25 17
i: -1 ptr: 0 k: 12
17 22 85 17 11 4 17 93 1 17 25 57
i: -1 ptr: 0 k: 11
17 22 85 17 11 4 17 93 1 17 25 57
i: -1 ptr: 1 k: 11
17 25 85 17 11 4 17 93 1 17 22 57
i: -1 ptr: 1 k: 10
17 17 85 17 11 4 17 93 1 25 22 57
i: -1 ptr: 1 k: 9
17 17 85 17 11 4 17 93 1 25 22 57
i: -1 ptr: 2 k: 9
17 17 1 17 11 4 17 93 85 25 22 57
i: -1 ptr: 2 k: 8
1 17 17 17 11 4 17 93 85 25 22 57
i: 0 ptr: 3 k: 8
1 17 17 17 11 4 17 93 85 25 22 57
i: 0 ptr: 4 k: 8
1 11 17 17 17 4 17 93 85 25 22 57
i: 1 ptr: 5 k: 8
1 11 4 17 17 17 17 93 85 25 22 57
i: 2 ptr: 6 k: 8
1 11 4 17 17 17 17 93 85 25 22 57
i: 2 ptr: 7 k: 8
57 22 85 17 11 4 17 93 1 17 25 17
i: 2 ptr: 0 k: 7
您可以看到输出是正确的,直到最后一个输出是正确的,该输出是通过 main 函数调用的。当我看到这一点时,我决定通过修改_partition签名来通过引用向量传递为:void _partition(vector<int> &arr, int lo, int hi, int &i, int &k)
这产生了以下输出:
57 22 85 17 11 4 17 93 1 17 25 17
i: -1 ptr: 0 k: 12
17 22 85 17 11 4 17 93 1 17 25 57
i: -1 ptr: 0 k: 11
25 22 85 17 11 4 17 93 1 17 17 57
i: -1 ptr: 0 k: 10
17 22 85 17 11 4 17 93 1 25 17 57
i: -1 ptr: 0 k: 9
1 22 85 17 11 4 17 93 17 25 17 57
i: -1 ptr: 0 k: 8
1 22 85 17 11 4 17 93 17 25 17 57
i: 0 ptr: 1 k: 8
1 93 85 17 11 4 17 22 17 25 17 57
i: 0 ptr: 1 k: 7
1 17 85 17 11 4 93 22 17 25 17 57
i: 0 ptr: 1 k: 6
1 4 85 17 11 17 93 22 17 25 17 57
i: 0 ptr: 1 k: 5
1 4 85 17 11 17 93 22 17 25 17 57
i: 1 ptr: 2 k: 5
1 4 11 17 85 17 93 22 17 25 17 57
i: 1 ptr: 2 k: 4
1 4 11 17 85 17 93 22 17 25 17 57
i: 2 ptr: 3 k: 4
1 4 11 17 85 17 93 22 17 25 17 57
i: 2 ptr: 0 k: 3
As you can see, this behaves abnormally right from the third line of output. I apologize for such a vague question, but I have no idea how else to go about looking for a solution to such an unusual problem.
答:
0赞
Igor Tandetnik
6/1/2018
#1
In your example, and exhibits undefined behavior by way of accessing an index out of bounds.pivot == 17
arr[pivot]
评论
0赞
Aayush Mahajan
6/1/2018
Wow, I don't know how I missed something so trivial. Thanks!
评论