c - 複数の文字列を格納するために2次元配列を使用する

Question

部分文字列が文字列内にあるかどうかを検索する関数を C で作成しましたが、それは問題ありませんが、文字列の配列で使用するとエラーが発生します。このコードを見てください：

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

int substring(char*, char*);

main()
{
    char student[100][7];
    int counter = 0;
    int finish =1;
    char s1[4];
    while(finish){
        printf("please Enter Student Number: \n");
        scanf("%s", student[counter]);
        counter++;
        printf("Do you want to exit? 1/0");
        scanf("%d", &finish);
    }
    printf("Now, You can search in Student numbers\n");
    printf("Enter a number to search: ");
    scanf("%s", s1);
    for(int i  = 0; i < counter; i++){
        printf("%d : %s\n", i, student[i]);
        if(substring(student[i],s1) == 1)
            printf("%s", student[i]);
    }

    getch();
}

int substring(char *s1,char *s2)
{
    int f=0;
    for(; *s1 !='\0';)
    {
        if(*s2=='\0')
            break;
        for(;*s2 !='\0';)
        {
            if(*s1==*s2)
            {
                f=1;
                s1 ++;
                s2 ++;
            }
            else
            {
                f=0;
                s1++;
                break;
            }
        }
    }
    if(f==0)
        return 0;
    else
        return 1;
    getch();
}

Answer 1

これは、単純なステート マシンを使用して部分文字列をテストする方法です。

int substring(const char *s1, const char *s2)
{
    enum
    {
        search, start_match
    } state = search;
    const char *m;

    while(*s1 != '\0')
    {
        switch(state)
        {
        case search:
            if(*s2 == '\0')
                return 0;
            else if(*s2 == *s1)
            {
                state = start_match;
                m = s2 + 1;
            }
            break;
        case start_match:
            if(*m == '\0')
                return 1;
            else if(*m != *s1)
                state = search;
            else
                ++m;
        }

        ++s1;
    }

    return 0;
}

また、標準のstrstr関数を使用することもできます。