timeout：被监控的命令转储核心

Question

我正在在线编译器中运行此代码，显示转储核心错误该程序找到两对的最高乘积。

#include <bits/stdc++.h>
using namespace std;
void maximumproductpair(int arr[],int n){
sort(arr,arr+n);
reverse(arr,arr+n);
int prod;
for(int i=0; i<n; i++){
cout<<arr[i];
}
prod=prod*arr[0];
prod=*arr[1];
cout<<prod;
}

int main() 
{
int t,n;
int arr[n];
cin>>t;
cout<<"\n";
cin>>n;
cout<<"\n";
  while(t--){
  for(int i=0; i<n;i++){
  
     cin>>arr[i];
   }
 }
maximumproductpair(arr,n);
  return 0;
}

在代码中一切正常，那么为什么会出现这个错误？

Answer 1

错误 1

问题是您的t 和n 变量未初始化。这意味着t 和n 都具有垃圾值。当你写的时候：

int t,n;
int arr[n];//UNDEFINED BEHAVIOR because n has garbage value

您有未定义的行为，因为n 具有垃圾值。

这就是为什么建议这样做

始终在本地/块范围内初始化内置类型。

错误 2

第二，注意在 C++ 中数组的大小必须是一个编译时常数。所以以下面的代码sn-ps为例：

int n = 10;
int array[n]; //INCORRECT

上面的正确写法是：

const int n = 10;
int array[n]; //CORRECT 

同样，

int n;
cin >> n;
int array[n]; //INCORRECT becasue n is not a constant expression

另请注意，某些编译器提供编译器扩展，让您拥有可变长度数组。你可以在Why aren't variable-length arrays part of the C++ standard?阅读更多关于它的信息。

如果您想将数组的大小作为用户的输入，那么您应该使用 动态大小的容器，例如 std::vector：

int n;
cin >>n;
std::vector<int> array(n);// CORRECT, this creates a vector of size n of elements of type int

另见Why should I not #include <bits/stdc++.h>?。