Question

我想请求帮助分配......我把这个功课送到学校......我必须编写程序，它将加载一个G矩阵和第二个G矩阵，并将搜索第二个G矩阵的存在数量第一个G矩阵....但是，当我尝试运行我的程序时，我收到了Segmentation故障消息...提前谢谢。示例程序应该如何工作....

...

输入所需g矩阵的行数： 3 输入想要的g矩阵：
121212个
212121个
121212个
要搜索的G矩阵：
12121212121212
21212121212121
12121212123212
21212121212121
12121212121212
G矩阵发现8次。

...

这是我的代码：

#include <stdio.h>
#include <stdlib.h>
#include <string.h>


char * get_line(void) // get line
{
    char * string;
    if((string = (char *)malloc(100 * sizeof(char))) == NULL)
    {
        printf("Nedostatek pameti.\n"); // not enough memory
        exit(1);
    }
    int i = 0;
    while((string[i] = fgetc(stdin)) != '\n')
    {
        i++;
        if((i % 100) == 0)
        {
            if((string = (char *)realloc(string, 100 * ( i - 1 ) * sizeof(char))) == NULL)
            {
                printf("Nedostatek pameti.\n"); // not enough memory
                exit(1);
            }
        }
    }
    return ( string );
}

char ** get_wanted_g_matrix(int pocetradek /*number of lines*/) // get wanted g matrix
{
    char ** string;
    printf("Zadejte hledanou matici:\n");
    int i = 0;
    if(( * string = (char ** )malloc(100 * sizeof(char *))) == NULL)
    {
        printf("Nedostatek pameti.\n"); // not enough memory
        exit(1);
    }
    while(i <= (pocetradek - 1))
    {
        string[i] = get_line();
        if((i > 1) && (*string[i-1] != strlen(*string[i])))
        {
               printf("Nespravny vstup.\n"); // not enough memory
               exit(1);
        }
        printf("%s", string[i]);
        i++;
        if((i % 100) == 0)
        {
            if((* string = (char **)realloc(* string, 100 * ( i - 1 ) * sizeof(char *))) == NULL)
            {
                printf("Nedostatek pameti.\n"); // not enough memory
                exit(1);
            }
        }
    }
    return (string);
}

int get_number_of_lines(void) // get number of lines
{
    int number_of_lines;
    printf("Zadejte pocet radek hledane matice:\n"); // enter the number of lines of wanted g matrix
    if(scanf("%d", &number_of_lines) != 1)
    {
        printf("Nespravny vstup.\n"); // Error
        exit(1);
    }
    return ( number_of_lines );
}

char ** get_searched_g_matrix(void) // get wanted g matrix
{
    char ** string;
    printf("Matice, ktera bude prohledana:\n"); // G matrix to be searched
    int i = 0;
    if(( * string = (char ** )malloc(100 * sizeof(char *))) == NULL)
    {
        printf("Nedostatek pameti.\n"); // not enough memory
        exit(1);
    }
    while(!feof(stdin))
    {
        string[i] = get_line();
        if((i > 1) && (*string[i-1] != strlen(*string[i])))
        {
               printf("Nespravny vstup.\n"); // error
               exit(1);
        }
        printf("%s", string[i]);
        i++;
        if((i % 100) == 0)
        {
            if((* string = (char **)realloc(* string, 100 * ( i - 1 ) * sizeof(char *))) == NULL)
            {
                printf("Nedostatek pameti.\n"); // not enough memory
                exit(1);
            }
        }
    }
    if(feof(stdin))
    {
    return string;
    }
}
int search( char ** string1, char ** string2 ) // search
{
    int string1width = strlen(*string1[0]);
    int string2width = strlen(*string2[0]);
    int string2height = strlen(**string2);
    int number_of_lines = get_number_of_lines();
    unsigned int g = 0, h = 0, i2, j2, l = 0, i = 0, j;
            while( i <= (string2height - 2) )
            {
                j = 0;
                while( j <= string2width - 2 )
                {
                    g = 0; h = 0;
                    if(string2[i][j] == string1[g][h])
                    {
                        i2 = i;
                        while((g <= number_of_lines - 1) && (i2 <= string2height - 2))
                        {
                            j2 = j; h = 1;
                            while(((string2[i2][j2] == string1[g][h]) && (j2 <= string2height - 2)) && (h <= string1width - 2))
                            {
                                j2++;
                                h++;
                            }
                            if(h != string1width - 1)
                            {
                                break;
                            }
                            if(g == number_of_lines - 1)
                            {
                                l++;
                                break;
                            }
                            i2++;
                            g++;
                        }
                    }
                    j++;
                }
                i++;
            }
            return ( l );
}

int main(void)
{
    char ** string1;
    char ** string2;
    int number_of_lines = get_number_of_lines();
    string1 = get_wanted_g_matrix(number_of_lines);
    string2 = get_searched_g_matrix();

    if(feof(stdin))
    {
                printf("Matice nalezena %d krat.\n", search( ** string1, **string2 )); // G matrix found %d times.
    }

    return 0;
}

Answer 1

在此代码中：

char ** get_wanted_g_matrix(int pocetradek /*number of lines*/) // get wanted g matrix
{
    char ** string;
    printf("Zadejte hledanou matici:\n");
    int i = 0;
    if(( * string = (char ** )malloc(100 * sizeof(char *))) == NULL)

您取消引用string，但它未初始化，因此您正在写入内存中的随机位置。将* string更改为string。这同样适用于此：

if((* string = (char **)realloc(* string, 100 * ( i - 1 ) * sizeof(char *))) == NULL)

..以及get_searched_g_matrix()中的相应行。

在这一行：

    if((i > 1) && (*string[i-1] != strlen(*string[i])))

当您传递char时，您正在将strlen()传递给char *。我怀疑你的意思只是strlen(string[i])，但这条线似乎有点荒谬。同样的问题也出现在get_searched_g_matrix()中，以及strlen()中对search()的前三次调用。

get_searched_g_matrix()可能会在没有返回值的情况下失败 - 如果feof(stdin)不正确，您需要考虑要返回的内容。

在main()中，您对search()的调用会传递char个值，但该函数需要char **。你可能意味着：

printf("Matice nalezena %d krat.\n", search( string1, string2 ));

（以上内容不足以修复您的代码 - 您似乎也有一些逻辑问题。但这是一个必要的开始。）

将来，您应该在启用更高级别警告的情况下编译代码，然后修复编译器识别的问题（如果您使用gcc，则使用-Wall进行编译）。

Answer 2

只是一些评论和风格提示：

我会说它应该做的很大（并且很难遵循），尝试简化一下。

1）节省垂直空间，现在大多数人都有宽屏幕，当你看不到相应的结束支架时，这是非常令人讨厌的。

1.1）检查错误情况非常好，但尝试使用

之类的东西

void err(const char* msg){
    printf("\n\nFATAL ERROR: %s\n", msg);
    exit(1);
};

这样你就可以了

if (!(x = malloc(sz))) 
    err("Not enough memory!");

1.2）虽然在if中使用括号作为单一语句被认为更安全，但我建议尽可能避免使用它们，或者至少使用较少的换行符。括号用于编译器，人们可以使用标签。

2）while函数中有多个search语句应写为for。

3）为什么你需要两个不同的函数来读取矩阵？一个人就够了。

4）正如@caf指出的那样，输入函数也有错误。在进一步前测试每个功能。在您一次编写整个程序之前需要多年的经验。

分配 - 指向指针

2 个答案: