报错信息提示存在heap-buffer-overflow的问题,这通常是由于数组或向量访问越界造成的。
报错代码
//力扣1005.k次取反后最大化的数组和
class Solution {
public:
static bool cmp(int a,int b){
if(abs(a)>abs(b)) return true;//绝对值降序排序,因为要先处理大的负数
}
int largestSumAfterKNegations(vector<int>& nums, int k) {
sort(nums.begin(),nums.end(),cmp);//绝对值降序排序
for(int i=0;i<nums.size();i++){
//第一次遍历的时候,只在负数上消耗次数!
if(nums[i]<0&&k>0){
nums[i]=-nums[i];
k--;
}
if(k==0) break;
}
//如果此时还有k没消耗完,看看k是奇数or偶数,如果是偶数就不用管了
if(k%2==1){
//如果是奇数,直接最小值取反就是结果
nums[nums.size()-1]=- nums[nums.size()-1];
}
for(int i:nums){//遍历Nums
sum+=nums[i];
}
return sum;
}
};
问题出现在最后的遍历循环中:
for(int i:nums){
sum+=nums[i];//错误写法,i这里本身就代表了数组元素
}
这段代码的含义是遍历nums中的每一个元素,i是元素的值,而不是元素的索引。
所以在sum+=nums[i];这里,实际上访问的是数组的第i个元素,而不是当前元素。当i的值大于nums的长度时,就会发生访问越界,导致heap-buffer-overflow的错误。
应该直接将i加到sum中,如下所示:
for(int i:nums){
sum += i;//正确写法,i这里就是数组元素i的值
}