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

フリオ さんが書きました:　
　とりあえず、気なったことだけを、

>例えば" both of japan grrfrbsebs korea great "という文章だったら"both of japan and korea great "
　この、" and " は何処から出てきたのでしょうか。

解答ありがとうございます。このandは完全に私のミスです。"both of japan korea great "となればokでjapanとkoreaの間の文字列は完全に削除するということが目的です

>削除するのはjapanからkoreaの間の文字列のみで
　ということは、"japanese somesome korean" だと、"japankorean" になるのでしょうか。

そういう事になります。よろしくお願いします。

　

フリオ さんが書きました:　
　それから、FindAndDeleteString 関数は、
calloc を使うことも含めて、仕様が決まっているのでしょうか。

当方Ｃ言語ペーペーのものですので、「関数の仕様がきまる」という事がどういう事かわからないのですが、ある友達からは、このcallocの部分はいらない、もしくは間違っていると言われました。
　

ペーペー さんが書きました: 解答ありがとうございます。このandは完全に私のミスです。"both of japan korea great "となればokでjapanとkoreaの間の文字列は完全に削除するということが目的です

ペーペー さんが書きました: "japanese somesome korean" だと、"japankorean" になるのでしょうか。

box さんが書きました:

ペーペー さんが書きました: 解答ありがとうございます。このandは完全に私のミスです。"both of japan korea great "となればokでjapanとkoreaの間の文字列は完全に削除するということが目的です

ここと、

ペーペー さんが書きました: "japanese somesome korean" だと、"japankorean" になるのでしょうか。

こことで食い違いがあることに気づいていますか？

japanとkoreaの間の文字列を完全に削除する、という内容は、
japanとkoreaとの間に空白を付ける／付けないのどちらでしょうか？
ここが、まさに関数の「仕様（の一部）」です。

non さんが書きました:もしかしたら、想定が甘いかも知れませんが、こんなんではいかがでしょう。
変数をグローバルにするなら、引数を渡したり、値を返す必要はないのですが。

コード:

char* FindAndDeleteString(char* buf, char* head, char* tail)
{
	char *p,*q,*r;
	r=buf;
	while(1){
		p=strstr(r,head);
		q=strstr(p,tail);
		if(p==NULL || q==NULL)
			break;
		p+=strlen(head);
		r=p+strlen(tail);
		while(*q!='\0')
			*p++=*q++;
		*p='\0';
	}
	return buf;
}

ペーペー さんが書きました:ハッカーさん返信ありがとうございます。実際にコードをこちらでコンパイルさせていただいたのですが、最初のjapanとkoreaの間の文字列は削除されているのですが、それ以降のjapanとkoreaの間の文字列の削除が行われていません・・・　ハッカーさんの環境ではちゃんと動作しているのでしょうか？？

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <time.h>
void chop_crlf(char* buf);
char* FindAndDeleteString(char* buf, char* head, char* tail);
FILE *fp;
const char* ifile = "input.txt";
#define HEAD    "japan"
#define TAIL    "korea"
char *p,buf[5040]; 




int main(int argc, char* argv[])
{


	FILE *hp = fopen("output.txt","w");
	if ( hp == NULL ){
		fprintf(stderr,"cannot output to file.\n");}


	if ( ( fp = fopen(ifile,"r") ) == NULL ) {
		fprintf(stderr,"%s: no such file or directory.\n", ifile);
		return -1;
	}


	// 文字列の検索（文字列の置換）
	rewind(fp);
	while ( fgets(buf,5040,fp) ) {
		chop_crlf(buf);

		p = FindAndDeleteString(buf,(char*)HEAD,(char*)TAIL);

		fprintf(stdout,"%s\n",p);
	}

	fclose(fp);
	fclose(hp);  
	return 0;
}

char* FindAndDeleteString(char* buf, char* head, char* tail)
{
    char *p,*q,*r;
    r=buf;
    while(1){
        p=strstr(r,head);
        q=strstr(p,tail);
        if(p==NULL || q==NULL)
            break;
        p+=strlen(head);
        r=p+strlen(tail);
        while(*q!='\0')
            *p++=*q++;
        *p='\0';
    }
    return buf;
}
// 改行コードの除去
void chop_crlf(char* buf)
{
	for ( unsigned i=0; i<strlen(buf); i++ ) {
		if ( buf[i] == '\r' || buf[i] == '\n' ) buf[i] = 0;
	}
}

naohiro19 さんが書きました:ぺーぺーさんがNo.4で満たすコードを.NET FrameworkのC#で書くと以下のようになります。

コード:

using System;
class RemoveTest
{
    public static void Main(string[] argv)
    {
        string str = "both of japan grrfrbsebs korea great";
        string str2 = "";
        Console.WriteLine("削除前の文字列: {0}", str);
        string removeText = "grrfrbsebs"; //削除する文字列
        
        //文字列を検索
        int cnt = str.IndexOf(removeText,removeText.Length);
        
        //文字列が見つからなかった
        if( cnt == -1)
        {
            Console.WriteLine("指定した文字列は見つかりませんでした");
        }

        //見つかったら削除
        str2 = str.Remove(cnt,removeText.Length);
        Console.WriteLine("削除後の文字列:{0}", str2);           
    }
}

ISLe さんが書きました:サンプルコード書いてみました。
バッファの文字列を書き換えます。

コード:

#include <stdio.h>
#include <string.h>
int main(void)
{
	char buf[] = "japan somesome korea friends japan somesomerr korea friends japan somesomerw korea friends japan";
	const char *ja = "japan";
	const char *kr = "korea";
	char *top, *end, *p;

	top = end = buf;
	while (top = strstr(top, ja)) {
		top += strlen(ja);

		end = strstr(top, kr);
		if (end == NULL) break;

		p = top;
		while (*(p++) = *(end++));
	}
	printf("%s\n", buf);
	return 0;
}

non さんが書きました:

ペーペー さんが書きました:ハッカーさん返信ありがとうございます。実際にコードをこちらでコンパイルさせていただいたのですが、最初のjapanとkoreaの間の文字列は削除されているのですが、それ以降のjapanとkoreaの間の文字列の削除が行われていません・・・　ハッカーさんの環境ではちゃんと動作しているのでしょうか？？

ハンドルはハッカーさんではないのですが・・・・nonです。
私の環境ではうまくいきます。VC++2008です。環境に影響しないように作ったつもりでしたが。strcpyは環境に影響すると思ったので使ってないし。
参考までにmainを含めた、全文です。なお、出力はstdoutにしています。

コード:

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <time.h>
void chop_crlf(char* buf);
char* FindAndDeleteString(char* buf, char* head, char* tail);
FILE *fp;
const char* ifile = "input.txt";
#define HEAD    "japan"
#define TAIL    "korea"
char *p,buf[5040]; 




int main(int argc, char* argv[])
{


	FILE *hp = fopen("output.txt","w");
	if ( hp == NULL ){
		fprintf(stderr,"cannot output to file.\n");}


	if ( ( fp = fopen(ifile,"r") ) == NULL ) {
		fprintf(stderr,"%s: no such file or directory.\n", ifile);
		return -1;
	}


	// 文字列の検索（文字列の置換）
	rewind(fp);
	while ( fgets(buf,5040,fp) ) {
		chop_crlf(buf);

		p = FindAndDeleteString(buf,(char*)HEAD,(char*)TAIL);

		fprintf(stdout,"%s\n",p);
	}

	fclose(fp);
	fclose(hp);  
	return 0;
}

char* FindAndDeleteString(char* buf, char* head, char* tail)
{
    char *p,*q,*r;
    r=buf;
    while(1){
        p=strstr(r,head);
        q=strstr(p,tail);
        if(p==NULL || q==NULL)
            break;
        p+=strlen(head);
        r=p+strlen(tail);
        while(*q!='\0')
            *p++=*q++;
        *p='\0';
    }
    return buf;
}
// 改行コードの除去
void chop_crlf(char* buf)
{
	for ( unsigned i=0; i<strlen(buf); i++ ) {
		if ( buf[i] == '\r' || buf[i] == '\n' ) buf[i] = 0;
	}
}