Question

我使用动态编程方法编写了这段代码，我认为逻辑很好，但代码没有显示结果。代码如下：

#include <iostream>
using namespace std;

void LCS(int input1[], int input2[], int n, int m) {
    int L[n + 1][m + 1];  /*This matrix stores the length of common subsequences*/
    for (int i = 0; i <= n; i++) {
        for (int j = 0; j <= m; j++) {
            if (i == 0 || j == 0)
                L[i][j] = 0;
            else if (input1[i - 1] == input2[j - 1]) 
                L[i][j] = 1 + L[i - 1][j - 1]; 
            else
                L[i][j] = max(L[i - 1][j], L[i][j - 1]);
        }
    }
    int index = L[n][m];
    int lcs[index];
    int i = n, j = m;
    while (i > 0 && j > 0) {
        if (input1[i - 1] == input2[j - 1]) {
            lcs[index - 1] = input1[i - 1];
            i--;
            j--;
            index--;
        } else if (L[i - 1][j] > L[i][j - 1])
            i--;
        else
            j--;

    }
    for (int i = 0; i < index; i++){
        cout << lcs[i];
    }

}
int main() {
    int n, m;
    cin >> n >> m;
    int input1[n], input2[m]; /*two arrays from which longest subsequnce is to be found*/
    for (int i = 0; i < n; i++)
        cin >> input1[i];
    for (int i = 0; i < m; i++)
        cin >> input2[i];
    LCS(input1, input2, n, m);
    return 0;
}

代码终止而不显示任何结果！

我甚至切换到另一个IDE，但它是相同的。这有什么问题？

Answer 1

您正在修改index变量。创建它的副本并修改它。我在这里使用了temp。

int index = L[n][m];
int temp = index;
int lcs[index];
int i = n, j = m;
while (i > 0 && j > 0) {
    if (input1[i - 1] == input2[j - 1]) {
        lcs[temp - 1] = input1[i - 1];
        i--;
        j--;
        temp--;
    } else if (L[i - 1][j] > L[i][j - 1])
        i--;
    else
        j--;

}
for (int i = 0; i < index; i++){
    cout << lcs[i];
}

在您的版本index中，如果要打印结果，则会减少为零，因此不会打印任何内容。

最长公共子序列未显示结果

1 个答案: