Question

我已经创建了一个DP程序，但是问题是当我不使用备注时我得到正确的答案。引入记忆后，我会针对某些问题开始得到错误的答案

这是C ++ 14中的代码，其中记忆已关闭（通过注释）

#include <iostream>
#include <math.h>
#include<algorithm>

using namespace std;
int max_Number_of_turns;
int dp[9999][1000];
int changeTheDigit(int n, int d) {
    int rem = n % (int) (pow(10, 4 - d));
    n /= (pow(10, 4 - d));
    int x = n % 10;
    n /= 10;
    if (x == 9) x = 0;
    else x = x + 1;
    n = n * (10) + x;
    n = n * (pow(10, 4 - d)) + rem;
    return n;
}

int minMax(int n, int t) {
    int ans =0;
    //if(dp[n][t]>=0) { return dp[n][t];}
    if (t > max_Number_of_turns) return n;

    int N;
    for (int i = 0; i < 4; i++) {
        N = changeTheDigit(n, i + 1);
        if (t % 2 == 0) {
            //Manish chance
            if(ans==0) ans=minMax(N, t+1);
            else ans = min(ans, minMax(N, t + 1));
        } else {
            //Nitish Chance
            ans = max(ans, minMax(N, t + 1));
        }
    }
    //cout << ans << endl;
    dp[n][t]=ans;
    return ans;
}

using namespace std;

int main() {
    int T, N, M;
    cin >> T;
    while (T--) {
        cin >> N >> M;
        max_Number_of_turns=M;
        for(int i=0;i<9999;i++)
            for(int j=0;j<1000;j++)
                dp[i][j]=-1;
        if(minMax(N,1)>N){
            cout << "Nitish" << endl;
        }
        else{
            cout << "Manish" << endl;
        }
    }
    return 0;
}

打开备忘录注释（即从该行中删除注释）

if(dp[n][t]>=0) { return dp[n][t];}

，我的代码将为某些问题提供错误答案例如，让我们考虑输入

1
4569 12

最初的正确解决方案是Manish 但是如果我打开记忆功能，我的解决方案就是Nitish

您能建议我这里我做错了吗

另外，一个有趣的事实是，如果将DP代码从

if(dp[n][t]>=0) { return dp[n][t];}

到

if(dp[n][t]>0) { return dp[n][t];}

那一切都很好

Answer 1

您的问题是n和/或t的值未检查，因此可能导致数组出现越界问题。您可以看到，如果在minMax函数的开头插入以下内容：

if (n < 0 || n >= 9999) cout << "n invalid at " << n << '\n';
if (t < 0 || t >= 1000) cout << "t invalid at " << t << '\n';

使用示例输入运行该命令会在输出结果之前发出警告：

n invalid at 9999
n invalid at 9999
n invalid at 9999

要解决此问题，您只能确保仅在有足够存储空间时才使用备忘录，首先是检查值：

if (n >= 0 && n < 9999 && t >= 0 && t < 1000 && dp[n][t] >= 0)
    return dp[n][t];

，其次，当存储值时：

if (n >= 0 && n < 9999 && t >= 0 && t < 1000)
    dp[n][t] = ans;

如果将递归添加了记忆，则解决方案错误

1 个答案: