使用C++程序合并排序没有得到正确的输出
Merge Sort using C++ Program not getting correct output
我不知道这段代码有什么问题,但我花了太多时间来找出问题,但仍然无法解决,我认为复制数组有一些错误,因为其他所有事情似乎都是正确的
请检查此代码-
#include<iostream>
using namespace std;
void MergeArray(int arr[],int lb,int mid,int ub){
int i=lb;
int j=mid+1;
int k=0;
int newarr[ub-lb+1];
//condition required for comparison between the split parts
while(i<=mid && j<=ub){
if(arr[i] < arr[j]){
newarr[k]=arr[i];
k++;
i++;
}
//basically a[j]<a[i] in this else condition
else{
newarr[k]=arr[j];
k++;
j++;
}
}
//all th left out elements in a[i] when a[j] is finished are added to newarr
while(i<=mid){
newarr[k]=arr[i];
k++;
i++;
}
//all th left out elements in a[j] when a[i] is finished are added to newarr
while(j<=ub){
newarr[k]=arr[j];
k++;
j++;
}
//copying all the elements of newarr to original arr
//i think this part has something messed up
for(int i=lb;i<=ub;i=i+1){
arr[i]= newarr[i];
}
}
void MergeElements(int arr[], int lb,int ub){
int mid;
if(lb<ub){
mid=(lb+ub)/2;
//spliting into 2 arts**
MergeElements(arr,lb,mid);
MergeElements(arr,mid+1,ub);
//merging in sorted order**
MergeArray(arr,lb,mid,ub);
}
}
int main(){
int n;
cout<<"enter the size of the array"<<endl;
cin>>n;
int arr[n];
cout<<"please enter the elements of the array"<<endl;
for(int i=0;i<n;i++){
cout<<"enter the element no."<<i<<endl;
cin>>arr[i];
}
MergeElements(arr,0,n-1);
cout<<"tSorted Array Elements"<<endl;
for(int i=0;i<n;i++){
cout<<arr[i]<<"t";
}
return 0;
}
我想我无法正确获取数组,因为根据我的说法,其他一切似乎都是正确的,请检查
好吧,我将绕过您在C++
中使用可变长度数组 (VLA) 时遇到的问题(目前 -标准C++
中不允许使用 VLA),首先,发布您的问题的解决方案。这是(正如您在评论中正确"猜测"的那样)在这个循环中:
//i think this part has something messed up
for(int i=lb;i<=ub;i=i+1){
arr[i]= newarr[i];
}
在这里,尽管i
索引(从给定的下限开始,lb
)对于arr
数组是正确的,但对于newarr
数组来说却不正确!这是在本地创建的,大小ub - lb + 1
(正确),但索引从零开始- 因此您需要删除newarr
的lb
偏移量:
for (i = lb; i <= ub; i++) { // NOTE: You've already declared "int i" - a new one will give a 'hides previous declaration' warning
arr[i] = newarr[i - lb]; // *** You need to remove the lower-bound offset!
}
关于C++
中的VLA问题:我相信GCC/g++
支持这些,但是,如果你想遵守标准C++
,你应该使用std::vector。因此,代替:
int newarr[ub - lb + 1];
用:
std::vector<int> newarr(size_t(ub - lb + 1));
并在main
函数中类似地使用std::vector<int> arr(n);
。对于代码的最小更改,您仍然可以保留void MergeElements(int arr[], int lb, int ub)
签名,但要使用std::vector
调用它,您需要提供第一个元素的地址。因此,main
使用它:
MergeElements(&arr[0], 0, n - 1);
请随时要求进一步澄清和/或解释。
此注释下方的循环
// copying all the elements of newarr to original arr
对源和目标使用相同的索引 - 它不应该。 要复制的newarr[]
内容从索引0
开始,而它在arr[]
中的目标区域从索引lb
开始。
编写将数组 newarr 复制到 arr 的循环,如下所示
//copying all the elements of newarr to original arr
//i think this part has something messed up
for(int i=lb, k = 0;i<=ub; i++, k++){
arr[i]= newarr[k];
}
请注意,可变长度数组不是标准的C++功能。您可以使用标准容器std::vector<int>
代替可变长度数组newarr
。此外,使用标准算法使 functionMergeArray 的实现更加简单。给你
void MergeArray(int arr[],int lb,int mid,int ub){
std::vector<int> newarr(ub-lb+1 );
std::merge( arr + lb, arr + mid + 1, arr + mid + 1, arr + ub + 1, newarr.begin() );
std::copy( newarr.begin(), newarr.end(), arr + lb );
}
- 如何将c++程序的一些输出传递给shell,以便在shell中使用
- 使用C++程序合并排序没有得到正确的输出
- 显示错误输出的简单数组排序程序
- 更改.cpp程序的输入文件中数据的位置会意外更改输出
- 如何从Windows应用程序输出到标准?
- 我写了一个C++程序来模拟Enigma机器.我没有得到输出
- 如何使用C++读取另一个程序中的源代码输出
- 为什么我的C++程序的程序集输出充满了 .ascii,没有汇编代码?
- 为什么这个程序的输出不如预期
- 在由Sublime文本3编译后在cmd上显示Java程序输出
- C++ 每次运行程序时我都会"nan"输出的问题
- 如何从 Win32 C++ 应用程序输出到父控制台窗口?
- 这是使用回溯的 nqueen 问题,但我使用了动态 2d 数组,我的程序编译良好,但不返回任何输出
- 而循环:简单的除法程序输出零,不明白为什么
- 如何在 html 页面中插入 cgi 应用程序的输出?
- C 中的输出程序无明显的原因是白色的
- C++:更改简单输出程序的字体
- C++ 简单的输入/输出程序 + 枚举器
- C++字母等效数字代码输出程序
- 编译输出程序集