我在下面的代码中遇到了一个针对背包问题的贪婪算法的分段错误。我以前从未成功解决过分段错误,虽然我已经看过它们,所以我很感激一些帮助。
运行调试器时得到的消息是没有“malloc.c”。当我运行valgrind时,我得到一个“4号无效读取”。在那个和bug的性质之间,我猜我正在尝试访问一个不存在的向量元素。但是我已经尝试了各种我能想到的方法来尝试确保循环在迭代向量时没有超越它的界限。
(我在没有使用基于C ++ 11范围的for循环的情况下完成了这个并且仍然得到了相同的错误。对于for循环的参数,它似乎并不重要,它仍然会抛出一个分段错误。)
提前致谢。
#include <iostream>
#include <vector>
#include <algorithm>
#include <stdlib.h>
using namespace std;
bool decreaseSort(int a, int b)
{
return a > b; //sort by decreasing value for better performance
}
double get_optimal_value(int capacity, vector<int> weights, vector<int> values) {
sort(weights.begin(),weights.end(), decreaseSort);
sort(values.begin(),weights.end(), decreaseSort);
vector<int> ourKnapsack(values.size(), 0);
double totalValue = 0.0;
int ourInput = 0;
for (auto i : ourKnapsack){
int ourValue = values.at(i);
int ourWeight = weights.at(i);
double unitValue = (double)ourValue/ourWeight;
if (capacity == 0)
return totalValue;
if (weights.at(i) < capacity){
ourInput = weights.at(i);
}
else {
ourInput = capacity;
}
totalValue = totalValue * (ourInput * unitValue);
weights.at(i)-=ourInput;
ourKnapsack.at(i)+=ourInput;
capacity-=ourInput;
}
return totalValue;
}
int main() {
int n = 3;
int capacity = 50;
vector<int> values(n);
values = {60,100,120};
vector<int> weights(n);
weights = {20,50,30};
double optimal_value = get_optimal_value(capacity, weights, values);
std::cout.precision(10);
std::cout << optimal_value << std::endl;
return 0;
}
答案 0 :(得分:2)
sort(values.begin(),weights.end(), decreaseSort);
那是你的问题,就在那里。糟糕。
显然,它应该是values.end()
,而不是weights.end()
。