0-1背包代码中的错误.我的代码中有什么错误

Bug in the 0-1 Knapsack code. What is the error in my code?

本文关键字:错误 代码 什么 我的 背包      更新时间:2023-10-16

我在实践部分用极客解决了这个问题,但这个代码在我不理解的地方失败了

#include <iostream>
using namespace std;
int main() {
//code
int T,N,W;
cin>>T;//The number of test cases
while(T-->0){
cin>>N>>W;//N for number of items and W for capacity
int dyna[N+1][W+1] = {{0,0}};//table to store values
int val[N+1],wt[N+1];//val stores value and wt for weight
for(int i = 1;i <= N;i++)cin>>val[i];
for(int i = 1;i <= N;i++)cin>>wt[i];
for(int i = 0;i <=N;i++){
for(int j = 0;j <= W;j++){
if(i==0||j==0)dyna[i][j]=0;
else if(wt[i]<=W)
dyna[i][j] = max(dyna[i-1][j],val[i]+dyna[i-1][j-wt[i]]);
else
dyna[i][j] = dyna[i-1][j];
}
}
cout<<dyna[N][W]<<endl;
}
return 0;
}

首先,您有单独的N和W长度,这是多余的,因为它们应该总是相等的。不过,我认为你观察到的问题是你的else if。您检查当前的权重是否小于项目数,这实际上没有意义。它可能应该类似于else if(j-wt[i]>=0),因为您正在尝试确定是否可以从当前和j中减去当前权重wt[i],并获得有效的(在边界内(结果。我想你会得到一个segfault或类似的错误,因为你试图访问dyna[i-1][j-wt[i]],例如,当ij为1,wt[i]为10时,因为dyna[0][-9](可能(是非分配内存。