这是我第一次在这里发帖,我是c ++的新手,正试图实现最大子数组问题
问题::调用函数max_subarray时,它永远不会返回并导致无输出
我尝试过的事情:我将问题缩小到cross_max_subarray函数,因为当我从max_subarray函数中删除函数调用或注释掉cross_max_subarray中的所有代码后,输出将正常打印< / p>
我想知道的事情:关于代码中问题所在的提示或答案会很不错,如果我尝试在命令行的第一行打印任何内容,我也会感到困惑该功能..它仍然没有输出..它不能执行该打印命令吗?
我希望我能很好地正确提问,这是下面的代码:
#include <iostream>
#include <cmath>
#include <vector>
using namespace std;
vector<int> cross_max_subarray(vector<int> v,int low,int high,int mid){
int lt_sum = -20000;
int sum = 0;
int lt_index = mid;
for(int i = mid; i>=low; i--){
sum += v[i];
if(sum > lt_sum){
i = lt_index;
sum = lt_sum;
}
}
int rt_sum = -20000;
sum = 0;
int rt_index = mid;
for(int i = mid; i<=high; i++){
sum += v[i];
if(sum > rt_sum){
i = rt_index;
sum = rt_sum;
}
}
std::vector<int> a = {lt_index, rt_index, lt_sum+rt_sum};
return a;
}
vector<int> max_subarray(vector<int> v,int low,int high){
cout << "gkjreor";
if(low == high){
std::vector<int> a = {low, high, a[low]};
return a;
}
int mid = (low+high)/2;
vector<int> ll_lh_ls = max_subarray(v, low, mid);
vector<int> rl_rh_rs = max_subarray(v, mid+1, high);
vector<int> cl_ch_cs = cross_max_subarray(v, low, high, mid);
int ls = ll_lh_ls[2];
int rs = rl_rh_rs[2];
int cs = cl_ch_cs[2];
if(ls>=rs && ls>=cs){
std::vector<int> a = {ll_lh_ls[0], ll_lh_ls[1], ls};
return a;
} else if(rs>=ls && rs>=cs){
std::vector<int> a = {rl_rh_rs[0], rl_rh_rs[1], rs};
return a;
} else{
std::vector<int> a = {cl_ch_cs[0], cl_ch_cs[1], cs};
return a;
}
}
int main(){
//number of elements
int n;
cin >> n;
std::vector<int> v(n);
// taking input
for(int i = 0; i<n; i++){
cin >> v[i];
}
// test print
for(int i = 0; i<n; i++){
cout << v[i] << " ";
}
// tried to flush output buffer
cout << endl;
// this is the function call that does not exit
std::vector<int> l_h_s = max_subarray(v, 0, n-1);
cout << l_h_s[2] << "\n";
return 0;
}
答案 0 :(得分:1)
您需要使用调试器或在代码中使用一些调试语句来了解程序的执行位置。我不太了解cross_max_subarray函数的意图。但是在函数内部,您正在for循环中修改'i'值。这在第二个for循环中造成了问题。 i = rt_index;当rt_index为0时,此循环变为无限。
答案 1 :(得分:1)
一些错误加在一起使调试程序变得更加困难。一个快速的技巧是尝试尽可能多地构建代码,并在编写代码时对其进行测试。这可以帮助您尽早发现问题。
我必须更改的第一个问题是我在为向量分配{}
时遇到问题。由于某种原因,我的编译器不喜欢它。如果您没有遇到此问题,请忽略此问题。我还注意到尽管您在顶部声明了std::vector
,但您仍在使用using namespace std
,因此我通过在所有std::
前面删除了vectors
来解决了这一问题。
我遇到的第二个主要问题是在cross_max_subarray
内部。问题是您分配的变量方向错误。 i = lt_index
做到了这一点,因此您总是在更改i
而不是您想要的lt_index
。这可能会导致无限循环,因为i
总是= = mid
每个循环。当我使用该函数时,我注意到您在两个求和中都将mid
号相加,因此最终将其相加两次。为了解决这个问题,我从mid+1
开始。
我注意到的另一个问题是,您本质上是在分配a[2] = a[low]
。这是一个问题,因为a
尚未填充任何东西,因此a[low]
只是垃圾数据。我敢肯定,您只是想做a[2] = v[low]
,因为v
是实际上保存数据的向量。
您所有的错误似乎都很容易解决,而我经常成为他们的受害者。有时C ++可能会让人很困惑,但请不要灰心。
这是工作代码:
#include <iostream>
#include <cmath>
#include <vector>
using namespace std;
vector<int> cross_max_subarray(vector<int> v,int low,int high,int mid){
int lt_sum = -20000;
int sum = 0;
int lt_index = mid;
for(int i = mid; i>=low; i--){
sum += v[i];
if(sum > lt_sum){
lt_index = i;
lt_sum = sum;
}
}
int rt_sum = -20000;
sum = 0;
int rt_index = mid;
for(int i = mid+1; i<=high; i++){
sum += v[i];
if(sum > rt_sum){
rt_index = i;
rt_sum = sum;
}
}
vector<int> a(3);
a[0]= lt_index;
a[1]= rt_index;
a[2]= lt_sum+rt_sum;
return a;
}
vector<int> max_subarray(vector<int> v,int low,int high){
// cout << "gkjreor";
vector<int> a(3);
if(low == high){
a[0]= low;
a[1]= high;
a[2]= v[low];
return a;
}
int mid = (low+high)/2;
vector<int> ll_lh_ls = max_subarray(v, low, mid);
vector<int> rl_rh_rs = max_subarray(v, mid+1, high);
vector<int> cl_ch_cs = cross_max_subarray(v, low, high, mid);
int ls = ll_lh_ls[2];
int rs = rl_rh_rs[2];
int cs = cl_ch_cs[2];
if(ls>=rs && ls>=cs){
a[0]= ll_lh_ls[0];
a[1]= ll_lh_ls[1];
a[2]= ls;
} else if(rs>=ls && rs>=cs){
a[0]= rl_rh_rs[0];
a[1]= rl_rh_rs[1];
a[2]= rs;
} else{
a[0]= cl_ch_cs[0];
a[1]= cl_ch_cs[1];
a[2]= cs;
}
return a;
}
int main(){
//number of elements
int n;
cin >> n;
vector<int> v(n);
// taking input
for(int i = 0; i<n; i++){
cin >> v[i];
}
// test print
for(int i = 0; i<n; i++){
cout << v[i] << " ";
}
// tried to flush output buffer
cout << endl;
// this is the function call that does not exit
std::vector<int> l_h_s = max_subarray(v, 0, n-1);
cout << l_h_s[2] << "\n";
return 0;
}
这是我得到的输出:
bash-4.3$ ./a.out
5
1
2
-20
3
4
1 2 -20 3 4
7