使用 DP 查找 LCS

Finding LCS using DP

我已经使用动态规划找到最长公共子序列 b/w 两个字符串。代码有什么问题。为什么它总是给出答案为0?

#include<bits/stdc++.h>
using namespace std;

int dp[20][20];
void initialize()
{
    for(int i=0;i<20;i++)
        for(int j=0;j<20;j++)
            dp[i][j]=-1;
}
int lcs(string a,string b,int m,int n)
{
    if(m==0||n==0) 
        return 0;
    if(dp[m][n]!=-1) 
        return dp[m][n];
    if(a[m-1]==b[n-1]) 
        return  dp[m-1][n-1] = 1+lcs(a,b,m-1,n-1);
    if(a[m-1]!=b[n-1])
        return dp[n-1][m-1]=max(dp[n][m-1]=lcs(a,b,n,m-1),dp[n-1][m]=lcs(a,b,n-1,m));
}
int main()
{
    string a="AGGTAB",b="GXTXAYB";

    cout<<lcs(a,b,a.length(),b.length());

}
  1. 您忘记调用 initialize()
  2. 第18行,应该是dp[m][n],不是dp[m-1][n-1]
  3. 注释了第 19 行代码,因为不需要 & 使代码与所有编译器兼容

i.e., some compiler may give warning: control reaches end of non-void function [-Wreturn-type]

  1. 在第 20 行更改了一些代码,因为您似乎混淆了变量 m 和 n。

代码:

#include<bits/stdc++.h>
using namespace std;
int dp[20][20];
void initialize()
{
    for(int i=0;i<20;i++)
        for(int j=0;j<20;j++)
            dp[i][j]=-1;
}
int lcs(string a,string b,int m,int n)
{
    if(m==0||n==0) 
        return 0;
    if(dp[m][n]!=-1) 
        return dp[m][n];
    if(a[m-1]==b[n-1]) 
        return dp[m][n] = 1+lcs(a,b,m-1,n-1);
    //if(a[m-1]!=b[n-1])
    return dp[m][n]=max(lcs(a,b,m-1,n),lcs(a,b,m,n-1));
}   
int main()
{
    string a="AGGTAB",b="GXTXAYB";
    initialize();
    cout<<lcs(a,b,a.length(),b.length());
}

输出:

4