C++ 为什么在向量为空时显示运行时错误?
我有下面的代码,如果一个向量有超过0个元素,我会尝试找到它的最小值,否则返回-1 但是,当向量为空时,我得到以下运行时错误: 向量下标超出范围C++ 为什么在向量为空时显示运行时错误?,c++,vector,runtime-error,C++,Vector,Runtime Error,我有下面的代码,如果一个向量有超过0个元素,我会尝试找到它的最小值,否则返回-1 但是,当向量为空时,我得到以下运行时错误: 向量下标超出范围 ints=0; 字符串s1=“”; int n; cin>>n; INTC[10]; 维生素串[10]; 向量mask1; 对于(int i=0;i10,那么您的第一个for循环会出现缓冲区溢出。为什么不使用std::vector和std::vector而不是int[10]和string[10]?或者更好,因为您总是同时使用c和维生素,然后使用单个std
ints=0;
字符串s1=“”;
int n;
cin>>n;
INTC[10];
维生素串[10];
向量mask1;
对于(int i=0;i
int-minimum=mask1[0];
先访问mask1
的第一个索引,然后再检查它是否为空。如果它为空(即大小为0),则会导致未定义的行为
您应该在检查mask1是否为空后放置此声明。在此行中:
int-minimum=mask1[0];
先访问mask1
的第一个索引,然后再检查它是否为空。如果它为空(即大小为0),则会导致未定义的行为
您应该在检查mask1是否为空后放置此声明。在检查向量是否为空之前,请先执行int-minimum=mask1[0];
操作。很抱歉,最后一条评论,我现在将尝试此操作!如果mask1
为空,则int-minimum=mask1[0];
为将来参考,请使用mask1.at(0)
抛出一个异常,该异常可以处理并且更容易检测。另外,仅供参考,如果用户为n
输入值>10
,那么您的第一个for
循环会出现缓冲区溢出。为什么不使用std::vector
和std::vector
而不是int[10]
和string[10]
?或者更好,因为您总是同时使用c
和维生素
,然后使用单个std::vector
,其中struct
将int
和std::string
保存在一起。您可以使用int minimum=mask1[0]在检查向量是否为空之前。;
对不起,上次的评论,我现在就试试!如果mask1
为空,则int-minimum=mask1[0];
is。为了将来参考,请使用mask1.at(0)
抛出一个异常,该异常可以处理并且更容易检测。另外,仅供参考,如果用户为n
输入值>10
,那么您的第一个for
循环会出现缓冲区溢出。为什么不使用std::vector
和std::vector
而不是int[10]
和string[10]
?或者更好,因为您总是同时使用c
和维生素
,然后使用单个std::vector
,其中结构将int
和std::string
放在一起。订阅超出范围是未定义的行为;“导致超出范围错误”是误导性的,因为人们可能认为程序以某种定义的方式作出反应,例如抛出异常。订阅超出范围是未定义的行为;“导致越界错误”是误导性的,因为人们可能认为程序以某种定义的方式作出反应,例如抛出异常。
int S = 0;
string s1 = "";
int n;
cin >> n;
int c[10];
string vitamins[10];
vector <int> mask1;
for (int i = 0;i < n;i++)
{
cin >> c[i];
cin >> vitamins[i];
}
for (int a = 0;a < (1 << n);a++)
{
for (int b = 0;b < n;b++)
{
if (a & (1 << b))
{
s1 += vitamins[b];
S += c[b];
}
}
if(s1.find('A') != string::npos &&s1.find('B') != string::npos && s1.find('C') != string::npos)
{
mask1.push_back(S);
}
S = 0;
s1 = "";
}
int minimum = mask1[0];
int size = mask1.size();
if(mask1.empty())
{
cout<<-1<<endl;
}
else
{
for (int m = 1;m < size;m++)
{
if (minimum > mask1[m])
{
minimum = mask1[m];
}
}
cout << minimum << endl;
}