プログラムの間違っている箇所が分かりません

質問

文字列の置換プログラムを作りたいのですが、下記のプログラムでは正常に動きませんでした。
例えば、「call」の「ll」を「sell」と変換しようとするとエラーがでてしまいます。
プログラミング初心者なので、どこが間違っているのかよく分からないです。
以下、問題のあるプログラムを記述します。

#include <stdio.h>

void change(char *in, char *bef, char *af){

　　　　char tmp[100];
　　　　int i = 0, j = 0;
　　　　int flg=0;
　　　　　　　　for(i = 0; *(af + i) != '\0'; i++){

　　　　　　　　tmp[i] = *(af + i);
　　　　}
　　　　tmp[i] = '\0';
　　　　for(j = 0; *(in + j) != '\0'; j++){
　　　　if(flg == 0 && *(bef + j) == '\0'){
　　　　　　　　　　　　flg = 1;
　　　　}
　　　　　　　　if(flg == 1){
　　　　　　　　　　　　tmp[i++] = *(in + j);
　　　　　　　　　　　　tmp[i] = '\0';
　　　　　　　　}
　　　　}
　　　　for(j = 0; j<=i; j++){
　　　　　　　　*(in + j) = tmp[j];
　　　　}
}

int check(char *in, char *bef){

　　　　int i=0;
　　　　int flg=0;

　　　　while(*(bef+i)!='\0'){
　　　　　　　　　　　　if(*(in+i)=='\0' || *(in+i)!=*(bef+i)){
　　　　　　　　　　　　flg=1;
　　　　　　　　　　　　break;
　　　　　　　　}
　　　　　　　　i++;
　　　　}
　　　　return flg;
}

int main(){
　　　　char in[100];
　　　　char bef_ch[20];
　　　　char af_ch[20];
　　　　int i=0;

　　　　printf("文字列を入力して下さい。\n");

　　　　gets(in);
　　　　printf("置換したい文字列を入力して下さい。\n");
　　　　gets(bef_ch);

　　　　printf("置換後の文字列を入力して下さい。\n");
　　　　gets(af_ch);

　　　　while(in[i]!='\0'){
　　　　　　　　if(check(&in[i], &bef_ch[0]) == 0){
　　　　　　　　　　　　change(&in[i], &bef_ch[0], &af_ch[0]);
　　　　　　　　}
　　　　　　　　i++;
　　　　}

　　　　printf("置換結果を示します。\n");
　　　　printf("%s\n",in);

　　　　return 0;
}

回答No.4

wadachi5 · ベストアンサー

こんにちは

回答としましては、以下私なりに改良したソースを挙げておきます。

－－－改良したソース－－－（//コメントのある行）
#include <stdio.h>

int change(char *in, char *bef, char *af){ //文字列長さを返すようにした

char tmp[100];
int i = 0, j = 0;
int flg=0;
int aflength = 0; //afの文字列長さ
for(i = 0; *(af + i) != '\0'; i++){
tmp[i] = *(af + i);
}
tmp[i] = '\0';
aflength = i;
for(j = 0; *(in + j) != '\0'; j++){
if(flg == 0 && *(bef + j) == '\0'){
flg = 1;
}
if(flg == 1){
tmp[i++] = *(in + j);
tmp[i] = '\0';
}
}
for(j = 0; j<=i; j++){
*(in + j) = tmp[j];
}
return aflength; //置き換えられたafの文字長を返す
}

int check(char *in, char *bef){

int i=0;
int flg=0;

while(*(bef+i)!='\0'){
if(*(in+i)=='\0' || *(in+i)!=*(bef+i)){
flg=1;
break;
}
i++;
}
return flg;
}

int main(){
char in[100];
char bef_ch[20];
char af_ch[20];
int i=0;

printf("文字列を入力して下さい。\n");

gets(in);
printf("置換したい文字列を入力して下さい。\n");
gets(bef_ch);

printf("置換後の文字列を入力して下さい。\n");
gets(af_ch);

while(in[i]!='\0'){
if(check(&in[i], &bef_ch[0]) == 0){
i += change(&in[i], &bef_ch[0], &af_ch[0]); //置き換えられた分だけポインタを進める
} else {
++i;
}

}

printf("置換結果を示します。\n");
printf("%s\n",in);

return 0;
}

ーーーーー以上

原因としては、メインルーチンで「call」の「ll」を「sell」に置き換え後の処理がなされていないため永久ループに入ってしまうため発生していました。

１）call
２）casell←ここでllがsellに置き換えられる
３）casesell←さらにsell自身のllをsellで置きかえる「
４）casesesell←さらにsell自身のllをsellで置きかえる
５）・・・・・・・←同上の繰り返し

これを避けるため置き換え後は、置き換え文字分でポインタを進める事にしました。

以下、追伸

なお、質問にある使い方「例えば、「call」の「ll」を「sell」と変換しようとすると」とある部分は、下記の様に質問されると回答者が理解しやすく、早々な回答を得られる可能性が高まります。

（以下、本件の質問例）

■プログラムを実行後、下記の様な操作を行うとエラーが発生してしまいます。

文字列を入力して下さい。
call[enter]
置換したい文字列を入力して下さい。
ll[enter]
置換後の文字列を入力して下さい。
sell[enter]←直後にエラーが発生

■期待する動作としては、下記の通りです。

文字列を入力して下さい。
call
置換したい文字列を入力して下さい。
ll
置換後の文字列を入力して下さい。
sell
置換結果を示します。
casell

お礼

回答ありがとうございます。
とても分かりやすく、良く理解できました。
書き方のアドバイスもわざわざ書いて下さって、ありがとうございました。

回答No.3

noname#62605

続きです。
break;の意味をよーく考えてみてください。

回答No.2

noname#62605

ちょっと質問の内容が理解できないので、整理して補足要求をお願いします。

＞「call」の「ll」を「sell」と変換しようとするとエラーがでてしまいます。
→この文章の表現だと「ll」は変換されていないですよね？？
　たとえば「call」の「ll」以外を置き換えて「sell」にするとか
　「call」を「sell」に置き換えるか（全部置き換え）によってプログラミングの仕方（元をたどると仕様）が変わってくると思います。
　さらに踏み込めば「all」と入力した場合はどういう結果を求めたいのか。

ポイントとしては文字列を入力してください→置換したい文字列を入力してください→置換後の文字列を入力するステップが正しいかどうか。
check()およびchange()のパラメータの渡し方が正しいか。中の処理が正しいか。
change()関数で処理が行われますか？？
これをチェックすれば大丈夫だと思います。

補足

回答ありがとうございます。
もう一度関数の中の処理が正しいか確認してみます。
求めたい結果については、

文字列を入力して下さい。
aabbcc
置換したい文字列を入力して下さい。
bb
置換後の文字列を入力して下さい。
ebb
置換結果を示します。
aaebbcc

という結果を求めていますが、置換結果を表示する部分でエラーが出ます。

回答No.1

Tacosan

なんか毎度書いてるような気がするんですが,
・どのような入力に対してどのような出力を期待したところ
・実際にはどのような結果となったのか
くらいは書いてもいいんじゃないかなぁ.

補足

言葉足りずですいません。

文字列を入力して下さい。
aabbcc
置換したい文字列を入力して下さい。
bb
置換後の文字列を入力して下さい。
ebb

と入力するとエラーが出るということです。
この場合の求めたい結果はaaebbccで、bbがebbと入れ替われば成功です。