分配-指针到指针

Question

我想寻求分配方面的帮助....我把这本功课上学了...我必须编写程序，该程序将加载一个G矩阵和第二个G矩阵，并在第二个G矩阵中搜索第一个G矩阵的存在数量G矩阵...。但是，当我尝试运行我的程序时，出现了分段错误消息...在此先感谢。 该程序应如何工作的示例...

...

输入所需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更改为just 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() ，以及在search()对strlen()的前三个调用中。

您的get_searched_g_matrix()可能会在没有返回值的情况下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使用带括号的signle语句被认为更安全，但我建议尽可能避免使用括号，或者至少使用较少的换行符。 括号用于编译器，人们喜欢使用制表符。

2）有几种while在你的语句search功能应该写成for秒。

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

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