如何解决为什么我会看到这种奇怪的行为?
代码是针对0/1背包问题的递归动态编程。
因此,让我首先说一下代码看起来是正确的,因为当我运行它时,它会显示结果,但是仅当我取消对printf
行的注释(请参阅突出显示的部分)并且与解决方案无关时(我仅用于测试目的),我觉得这很奇怪。有人可以告诉我为什么会这样吗。
int main() {
int DP_Recursive(int W,static int wt[],static int val[],int n,static int dp[][]);
static int wt[5] = { 5,10,20,30 };
static int val[5] = { 50,60,100,120 };
static int dp[5][60]; //marker 2-D array
for (int i = 0; i <= 5; i++) {
for (int w = 0; w <= 50; w++) {
dp[i][w] = -1;
}
}
printf("The total loot is %d$.",DP_Recursive(50,wt,val,2,dp));
}
//Recursive D.P. solution
int DP_Recursive(int W,static int dp[5][60]) {
//-------**HIGHLIGHTED PART**-----------
printf("%d",dp[2][30]);
//--------------------------
//Base case
if (n == 0 || W == 0)
return 0;
if (dp[n][W] != -1)
return dp[n][W];
if (wt[n-1] > W) {
dp[n][W] = DP_Recursive(W,n - 1,dp);
return dp[n][W];
} else {
dp[n][W] = max(val[n-1] + DP_Recursive(W - wt[n-1],n-1,dp),DP_Recursive(W,dp));
}
return dp[n][W];
}
解决方法
代码中存在多个问题:
-
[主要] 的
DP_recursive
原型不正确:在{{1 }}函数(在int DP_Recursive(int W,int wt[],int val[],int n,static int dp[5][60])
主体之外)以及函数定义本身。 -
[次要] 不需要声明
main()
中的数组。 -
[主要] 初始化循环执行了一个步骤:
main
使用main()
从static
到{{1 }} 包括的。在两个循环中都应使用for (int i = 0; i <= 5; i++)
而不是i
。根据经验,在使用0
之前,请务必检查两次,并始终希望排除上限。 -
[主要] 内部初始化循环使用
5
而不是<
作为上限,从而导致该数组被部分初始化并导致错误的结果(未定义)行为),而不是<=
。最好使用<=
宏来获取数组中元素的数量。 -
[次要] ,您应使用换行符(
50
)结束输出 -
[未成年人]
60
在成功终止后应返回110
。 -
您没有发布
countof
的定义,建议将其定义为函数。 -
包含文件
\n
也丢失了。
这是修改后的版本:
main()
版权声明:本文内容由互联网用户自发贡献,该文观点与技术仅代表作者本人。本站仅提供信息存储空间服务,不拥有所有权,不承担相关法律责任。如发现本站有涉嫌侵权/违法违规的内容, 请发送邮件至 dio@foxmail.com 举报,一经查实,本站将立刻删除。