我的C ++中的mergesort代码有问题。我不知道为什么,但当我从Merge_Sort调用它们时,这些函数不能正常工作。他们分开工作。
当我输入这样的列表时:H-> 1-> 2-> 8-> 4-> 7-> 6-> 33-> NULL
作为排序列表,我得到这样的结果:H-> 33-> 4-> 6-> NULL。
void Split(node *&H, node *&h1, node *&h2)
{
if (H != NULL)
{
node *P = H;
bool check = 0;
while (H != NULL)
{
P = H;
H = H->next;
if (check) {
AddLastNode(h2,P);
}
else
{
AddLastNode(h1, P);
}
check=!check;
}
}
}
void Merge(node *&H, node *&h1, node *&h2)
{
node *p1=h1, *p2=h2;
node *temp = H;
while (h1!=NULL && h2!=NULL)
{
p1 = h1;
p2 = h2;
if (p1->val < p2->val)
{
h1 = h1->next;
temp = p1;
}
else
{
h2 = h2->next;
temp = p2;
}
temp->next = H;
H = temp;
}
if (h1 == NULL)
{
temp = h2;
temp->next = H;
H = temp;
}
else if (h2 == NULL)
{
temp = h1;
temp->next = H;
H = temp;
}
}
void Merge_Sort(node *&H)
{
node *h1, *h2;
if (H && H->next)
{
h1 = h2 = NULL;
Split(H, h1, h2);
Merge_Sort(h1);
Merge_Sort(h2);
Merge(H, h1, h2);
}
}
答案 0 :(得分:0)
问题在于递归调用Merge_Sort
。你在指针上调用它,所以最后它们变成了1个长度而其他数据丢失了。我认为这是一个糟糕的算法。
您可以使用STL容器和算法更轻松地完成此操作。使用std::list
作为容器,将其分开。然后在分隔的容器上使用list方法sort
,并在新容器或h1,h2之一上创建merge
。
这是我的合并算法代码:
#include <iostream>
#include <string>
#include <algorithm>
#include <list>
void TraceList( const std::list<int32_t>& lst, const std::string& title = "" )
{
if ( !title.empty() )
{
std::cout << title << std::endl;
}
std::copy(lst.begin(), lst.end(), std::ostream_iterator<int16_t>(std::cout, ","));
std::cout << std::endl;
}
int main()
{
//Input test data. You can use here your input code
const uint32_t SIZE_START = 10;
int16_t ArrInputData[SIZE_START] = {4, 45, 12, 78, 11, 17, 2, 58, 79, 10 };
std::list<int32_t> StartList;
std::copy(ArrInputData, ArrInputData + SIZE_START, std::back_inserter(StartList));
//Separate input data on parts
std::list<int> h1;
std::list<int> h2;
bool CounterFlag = true;
std::for_each(StartList.begin(), StartList.end(), [&CounterFlag, &h1, &h2]( int16_t val )
{
CounterFlag ? h1.push_back(val) : h2.push_back(val);
CounterFlag = !CounterFlag;
});
//Sort separated parts
h1.sort();
h2.sort();
TraceList(h1, "H1 list:");
TraceList(h2, "H2 list:");
//Merge separated
std::list<int32_t> FinalMergedList = h1; //Copy one of separated to new list
FinalMergedList.merge(h2);
TraceList(FinalMergedList, "MergeFinal list:");
return 0;
}
结果是:
H1 list:
2,4,11,12,79,
H2 list:
10,17,45,58,78,
MergeFinal list:
2,4,10,11,12,17,45,58,78,79,
Program ended with exit code: 0
如果您喜欢std::list
的解决方案,那么您也应该考虑其他容器。当你在尾部和头部之间插入时,std::list
是好的,但它在排序时很糟糕。可能是您应该使用任意具有随机访问权限的容器,它们在排序时更快,例如std::vector
。