プログラマ専用SNS ミクプラ

コード:

#include<stdio.h>
#include<stdlib.h>
#include<string.h>
#define str 300

int match_str(char *a, char *b);
int main(int argc,char *argv[])
{
	FILE *fp;
	char a[str] = {'\0'};
	int cnt,num;
	
	if(argc != 2)
	{
		return 1;
	}
	fp = fopen(argv[1],"r");
	
	if(fp == NULL)
	{
		printf("ファイル %s が見つかりません\n",argv[1]);
		return 1;
	}
	while(fgets(a,str,fp)!= NULL)
	{
		cnt++;
		printf("%d\t:",cnt);
		num = match_str(a,argv[2]);
		
		if(num == 1)
		{
			printf("%s",a);
		}
	}	
	fclose(fp);
	printf("\n");
}
int match_str(char *a, char *b)
{
	int i = 0;
	int j = 0;
	int maxa,maxb;
	int flg = 0;

	while(flg == 0)
	{
		if(a[i] == '\0')
		{
			maxa = i;
			flg == 1;
		}
		i++;
	}
	i = 0;
	while(flg == 0)
	{
		if(b[i] == '\0')
		{
			maxb = 0;
			flg = 1;
		}
		i++;
	}
	for(i=0;i<maxa;i++)
	{
		if(0==strncmp(&a[i],&b[j],1))
		{
			if(maxb-1 == j)
			{
				return 1;
			}
			else
			{
				j++;
			}
		}
	}
	return 0;
}
		

コード:

#include<stdio.h>
#include<stdlib.h>
#include<string.h>
#define str 300

int match_str(char *a, char *b);
int main(int argc,char *argv[])
{
	FILE *fp;
	char a[str] = {'\0'};
	int cnt,num; /* cntを初期化していない */
	
	if(argc != 2) /* 条件が不自然。argc[2]を利用しているので、argc != 3の方が適するのでは？ */
	{
		return 1;
	}
	fp = fopen(argv[1],"r");
	
	if(fp == NULL)
	{
		printf("ファイル %s が見つかりません\n",argv[1]);
		return 1;
	}
	while(fgets(a,str,fp)!= NULL)
	{
		cnt++;
		printf("%d\t:",cnt); /* 初期化していない自動変数の値を使っているのでundefined behavior */
		num = match_str(a,argv[2]);
		
		if(num == 1)
		{
			printf("%s",a);
		}
	}	
	fclose(fp);
	printf("\n");
	/* 新しいCでは大丈夫だが、戻り値を明示的に書いたほうがわかりやすい */
}
int match_str(char *a, char *b)
{
	int i = 0;
	int j = 0;
	int maxa,maxb;
	int flg = 0;

	while(flg == 0)
	{
		if(a[i] == '\0')
		{
			maxa = i;
			flg == 1; /* 意味のない文。flg = 1;の間違いでは？そもそも素直にfor(;;)でループしてここではbreak;でいいのでは？ */
		}
		i++;
	}
	i = 0;
	/* ここに来るということはflgが0でなくなったということだが、flgを初期化していない */
	while(flg == 0)
	{
		if(b[i] == '\0')
		{
			maxb = 0; /* 詳しい仕様を追っていないが、iではなく0でいいのか？ */
			flg = 1; /* 同様に素直にwhile(flg == 0)ではなくfor(;;)を使ってここではbreak;でいいのでは？ */
		}
		i++;
	}
	for(i=0;i<maxa;i++)
	{
		if(0==strncmp(&a[i],&b[j],1)) /* 素直にa[i] == b[j]でいいのでは？ */
		{
			/* maxbが初期化されていない場合はundefined behavior */
			/* maxb=0の場合はjがオーバーフローしなければ成り立たない(符号付き整数のオーバーフローはundefined behavior)が、
			 * j++;は高々maxa回(<=INT_MAX回)しか実行されず、jの初期値は0なので、オーバーフローは
			 * (デバッガや宇宙線などによるメモリの書き換えや、処理系の不都合による誤動作が無い)通常の実行では起こらない。
			 * よって、この式は通常の実行では常に偽である。 */
			if(maxb-1 == j)
			{
				return 1;
			}
			else
			{
				j++;
			}
		}
	}
	return 0;
}

コード:

#include<stdio.h>
#include<stdlib.h>
#include<string.h>
#define str 300
 
int match_str(char *a, char *b);
int main(int argc,char *argv[])
{
    FILE *fp;
    char a[str] = {'\0'};
    int cnt = 0,num; 
    
    if(argc != 3) 
    {
        return 1;
    }
    fp = fopen(argv[2],"r");
    
    if(fp == NULL)
    {
        printf("ファイル %s が見つかりません\n",argv[1]);
        return 1;
    }
    while(fgets(a,str,fp)!= NULL)
    {
        cnt++;
        num = match_str(a,argv[1]);
        
        if(num == 1)
        {
            printf("%d:%s",cnt,a);
        }
    }   
    fclose(fp);
}
int match_str(char *a, char *b)
{
    int i = 0;
    int j = 0;
    int maxa,maxb;
    int flg = 0;
 
    while(flg == 0)
    {
        if(a[i] == '\0')
        {
            maxa = i;
            flg = 1; 
        }
        i++;
    }
    i = 0;
	flg = 0;
   
    while(flg == 0)
    {
        if(b[i] == '\0')
        {
            maxb = i; 
            flg = 1; 
        }
        i++;
    }
    for(i=0;i<maxa;i++)
    {
        if(a[i] == b[j]) 
        {
            if(maxb-1 == j)
            {
                return 1;
            }
            else
            {
                j++;
            }
        }
    }
    return 0;
}

コード:

match_str("kanozyogafuraguwooraretara", "gaworare")

コード:

!!strstr("kanozyogafuraguwooraretara", "gaworare")