我试图实现合并排序算法。首先,我尝试创建合并方法。我使用矢量和利用迭代器。但while
函数中的merge
正文会导致此错误:
“./ a.out”由信号SIGSEGV(地址边界错误)终止
这是代码
#include <iostream>
#include <vector>
std::vector<int> merge(std::vector<int> &, std::vector<int> &);
int main()
{
std::vector<int> vect {1,3,5,7};
std::vector<int> vect2 {2,3,6,8};
std::vector<int> temp_vect = merge(vect, vect2);
for(int num: temp_vect) {
std::cout << num << std::endl;
}
}
std::vector<int> merge(std::vector<int> & first_vect, std::vector<int> & second_vect)
{
std::vector<int> sorted_vect;
auto sorted_it = sorted_vect.begin();
auto first_it = first_vect.begin(), second_it = second_vect.begin();
while(first_it != first_vect.end() || second_it != second_vect.end()) {
if(*first_it < *second_it) {
sorted_vect.push_back(*first_it);
first_it++;
} else if(*first_it > *second_it) {
sorted_vect.push_back(*second_it);
second_it++;
} else {
sorted_vect.push_back(*first_it);
sorted_vect.push_back(*second_it);
first_it++; second_it++;
}
}
if(first_it == first_vect.end()) {
//end of first_vect reached
//inserting the rest of second_vect
sorted_vect.insert(sorted_vect.end() - 1, second_it, second_vect.end());
} else if (second_it == second_vect.end()) {
//end of second_vect reached
//inserting the rest of first_vect
sorted_vect.insert(sorted_vect.end() - 1, first_it, first_vect.end());
}
return sorted_vect;
}
答案 0 :(得分:1)
通常在进程尝试访问未分配给该特定进程的内存时引发SIGSEGV。引发SIGSEGV的主要原因是取消引用无效指针。在您的情况下,当您取消引用first_it或second_it以检查其值时,会发生这种情况,当您到达该列表的末尾时。
您至少需要进行三项更改:
1: line 23- while(first_it != first_vect.end() && second_it != second_vect.end())
2: line 40- sorted_vect.insert(sorted_vect.end(), second_it, second_vect.end());
3: line 44- sorted_vect.insert(sorted_vect.end(), first_it, first_vect.end());
1:您应该检查是否到达任何列表的末尾:您应该使用和(&amp;&amp;)not或(||)。这将删除SIGSEGV错误。
2&amp; 3:您应该将剩余的剩余列表添加到合并列表的末尾,而不是在最后一个元素之前。
可选更改:
1-您不需要sorted_it变量(您没有使用它)
2-你不必检查是否相等,如果值相等,你可以插入其中任何一个,循环的下一次迭代就会发挥作用。
3-您不必检查哪个列表到达目的地,您可以将每个列表的其余部分合并。
以上所有内容都反映在这里:
std::vector<int> merge(std::vector<int> & first_vect, std::vector<int> & second_vect)
{
std::vector<int> sorted_vect;
auto first_it = first_vect.begin(), second_it = second_vect.begin();
while(first_it != first_vect.end() && second_it != second_vect.end()) {
if(*first_it < *second_it) {
sorted_vect.push_back(*first_it);
first_it++;
} else {
sorted_vect.push_back(*second_it);
second_it++;
}
}
sorted_vect.insert(sorted_vect.end(), first_it, first_vect.end());
sorted_vect.insert(sorted_vect.end(), second_it, second_vect.end());
return sorted_vect;
}
答案 1 :(得分:0)
while(first_it != first_vect.end() || second_it != second_vect.end())
你正在迭代:
first_it
等于first_vect.end()
AND
second_it
等于second_vect.end()
这两个条件都必须是真的。例如,如果first_it
已达到first_vect.end()
,则迭代将继续,直到second_it
到达其向量的末尾。
if(*first_it < *second_it) {
正如我刚刚解释的那样,first_it
现在可以等于其向量的end()
迭代器。在这种情况下,在此处取消引用会导致未定义的行为。
同样的事情适用于第二个向量的迭代器。
你的逻辑没有考虑迭代器到达其向量的end()
的可能性。
一种解决方法是将||
转换为&&
,然后在末尾添加其他代码,以将任意一个向量中的任何剩余值附加到输出向量。
解决此问题的另一种方法是调整if()
条件以考虑迭代器可能是end()
值的可能性,并相应地继续。对于第一个矢量:
if(second_it == second_vect.end() ||
(first_it != first_vect.end() && *first_it < *second_it)) {
但是这个逻辑在这里变得有些复杂(两个if
条件必须以这种方式检查两个迭代器的边缘情况。)