Question

我必须编写一个程序来确定最长的公共子序列。

输入：

第一个参数是一个文件，每行包含两个字符串，以分号分隔。您可以假设每个测试用例只有一个唯一的子序列。 e.g。

XMJYAUZ; MZJAWXU

输出：

最长的共同子序列。确保您打印的每一行都没有尾随空格。 e.g。

MJAU

我正在使用Dev C ++ ..它编译得很好！...但这个问题是programming challenge，当我提交答案时，它会向我显示分段错误！

我写了以下代码，我得到了一个Seging Fault，我错了吗？

#include<stdio.h>
#include<stdlib.h>
#include<string.h>
char str1[100],str2[100];
int len1;
int len2;
void printLCS(char b[len1][len2],char str1[],int i,int j)
{
    if(i==0 || j==0)
    return;
    if(b[i][j]=='c')
    {
        printLCS(b,str1,i-1,j-1);
        printf("%c",str1[i-1]);
    }
    else if(b[i][j]=='l')
    printLCS(b,str1,i,j-1);
    else
    printLCS(b,str1,i-1,j);
}
void Seq(char str1[],char str2[])
{

    int i,j;
    len1=strlen(str1);
    len2=strlen(str2);
    int LCS[len1+1][len2+1];
    char b[len1][len2];
    for(i=0;i<=len1;i++)
    {
        LCS[i][0]=0;
    }
    for(j=0;j<=len2;j++)
    {
        LCS[0][j]=0;
    }
    for(i=1;i<=len1;i++)
    {
        for(j=1;j<=len2;j++)
        {
            if(str1[i-1]==str2[j-1])
            {
                LCS[i][j]=1+LCS[i-1][j-1];
                b[i][j]='c';
            }
            else if(LCS[i-1][j]>=LCS[i][j-1])
            {
                LCS[i][j]=LCS[i-1][j];
                b[i][j]='u';
            }
            else
            {
                LCS[i][j]=LCS[i][j-1];
                b[i][j]='l';
            }
        }
    }
    printLCS(b,str1,len1,len2);
}
int main(int argc,char *argv[])
{
    if(argc!=2)
    {
        printf("Invalid Number of Arguments:\n");
        exit(0);
    }
    FILE *fp;
    fp=fopen(argv[1],"r");
    if(fp==NULL)
    {
        printf("File can't be opened:\n");
        exit(0);
    }
    char c;
    c=fgetc(fp);
    while(c!=EOF)
    {
        int k=0;
        if(c=='\n')
        c=fgetc(fp);
        while(c!=';')
        {
            str1[k]=c;
            k++;
            c=fgetc(fp);
        }
        str1[k]='\0';
        c=fgetc(fp);
        k=0;
        while(c!=EOF && c!='\n')
        {
            str2[k]=c;
            k++;
            c=fgetc(fp);
        }
        str2[k]='\0';
        Seq(str1,str2);
        printf("\n");
        if(c==EOF)
        {
            break;
        }
        else
        c=fgetc(fp);
    }
    return 0;
}

Answer 1

我不知道这个网站的系统但是; 编译没有错误，结果是真的。

You didnt close file. Maybe memory leak etc. didnt allowed by site.

并且，不要使用全局变量，除非你不知道另一个解决方案

这个用法非常非常糟糕！无论如何，ISO C90禁止这样做

int len1;
int len2;
void printLCS(char b[len1][len2]...

祝你好运。

Answer 2

如果您可以访问Mac或Linux系统，那么有一个名为valgrind的奇妙工具可以帮助您跟踪这些类型的错误：基本上它在虚拟机中运行您的程序并监视它是什么读取和写入内存。

虽然我无法编译你的代码，但我对这个for循环非常怀疑：

   for(i=1;i<=len1;i++)
{
    for(j=1;j<=len2;j++)
    {
        if(str1[i-1]==str2[j-1])
        {
            LCS[i][j]=1+LCS[i-1][j-1];
            b[i][j]='c';
        }
        else if(LCS[i-1][j]>=LCS[i][j-1])
        {
            LCS[i][j]=LCS[i-1][j];
            b[i][j]='u';
        }
        else
        {
            LCS[i][j]=LCS[i][j-1];
            b[i][j]='l';
        }
    }
}

C和C ++中的数组从0开始，因此您感兴趣的最大偏移量可能是strlen - 1。尝试将for循环更改为

for(i=1;i<len1;i++)
{
  for(j=1;j<len2;j++) 
  {
     ...
  }
}

最长公共子序列分段故障

2 个答案: