Question

代码是针对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 wt[], static int val[], int n, 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, wt, val, n - 1, dp);
        return dp[n][W];
    } else {
        dp[n][W] = max(val[n-1] + DP_Recursive(W - wt[n-1], wt, val, n-1, dp),
                       DP_Recursive(W, wt, val, n-1, dp));
    }
    return dp[n][W];
}

Answer 1

代码中存在多个问题：

[主要] 的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()

为什么我会看到这种奇怪的行为？

1 个答案: