無駄な記述をなくしたい

Question

前提・実現したいこと

お世話になります。
初めての質問でお見苦しい点多々あると思いますが、ご容赦下さい。
ネットで見かけたJavaの問題を解いております。

改善したい点

一応コンパイル・実行ともにでき、やりたいことはできたのですが、
該当のソースコードの★の部分が同じ処理をしているため、
str.indexOf(args[1])を1回だけ使うように
書き換えることはできないかと思い、今回質問させていただきました。

該当のソースコード

class Mondai3{
	public static void main(String[]args){

		String str = args[0];		
		
		//★
		int num = str.indexOf(args[1]);

		while(num != -1){
			str = (str.substring(0,num) + args[2] + str.substring(num + args[1].length()));
			
			//★
			num = str.indexOf(args[1]);
		}

		System.out.println(str);
	}
}

/*
コマンドライン引数↓
たけやぶにたけたてかけた たけ 竹
実行結果↓
竹やぶに竹たてかけた
*/

補足情報（FW/ツールのバージョンなど）

Eclipce

Comments

[m.ts10806]

タイトルには要件を書いて下さい。 https://teratail.com/help/question-tips#questionTips3-1 タイトルから質問内容が想像できないもの、タイトルと質問内容が食い違っているものはスムーズな回答を得られなくなります。

Answer 1

replaceを使わずに、省略したいなら、こんな感じ？

class Mondai3{
    public static void main(String[]args){

        String str = args[0];
        int num;

        while((num = str.indexOf(args[1])) != -1){
            str = (str.substring(0,num) + args[2] + str.substring(num + args[1].length()));
        }

        System.out.println(str);
    }
}

Comments

[nekocat]

やはりnumの宣言を外に出すのがよさそうですね！
ありがとうございます。

Answer 2

1回探索したところは2度探索しないように改良
ついでにコマンドライン引数の内容がどれがどれかわかりにくいため変数名を付けた

class Mondai3{
    public static void main(String[]args){

        String str = args[0];
        String target = args[1];
        String replacement = args[2];      

        int num = 0;

        while((num = str.indexOf(target, num)) != -1){
            str = (str.substring(0,num) + replacement + str.substring(num + target.length()));
            num += replacement.length();
        }

        System.out.println(str);
    }
}

Answer 3

自分でループを回す場合、無限ループにならないようにする注意が必要です。
失眠分のコードに、
たけやぶにたけたてかけた たけ 竹たけ
をコマンドライン引数に指定したらどうなるでしょう？

ループさせずに済ます方法と、ループをさせる方法の２つを書いてみました。

import java.util.Arrays;
import java.util.List;

public class Mondai3 {
    public static void main(String[] args) {
        List<String> words = Arrays.asList(args[0].split(args[1]));
        System.out.println(String.join(args[2], words.toArray(new String[words.size()])));

        final int src_len = args[1].length();
        if (src_len == 0) {
            System.out.println(args[0]);
            return;
        }
        String ans = "";
        int p = 0;
        while (p < args[0].length()) {
            if (args[0].substring(p).startsWith(args[1])) {
                ans += args[2];
                p += src_len;
            } else {
                ans += args[0].charAt(p);
                p += 1;
            }
        }
        System.out.println(ans);
    }
}
/*
 * 入力: たけやぶにたけたてかけた たけ 竹 
 * 出力: 竹やぶに竹たてかけた
 * 
 * 入力: たけやぶにたけたてかけた たけ 竹たけ 
 * 出力: 竹たけやぶに竹たけたてかけた
 */

Answer 4

Javaでやるのは文化的に受け入れられるかわからないですが、ループ自体を無限ループにしておいて、途中脱出にすればindexOfの記載は1箇所で済ませられます。

        while(true){
            int num = str.indexOf(args[1]);
            if(num == -1) break;
            str = (str.substring(0,num) + args[2] + str.substring(num + args[1].length()));
        }

Comments

[nekocat]

ご回答ありがとうございます。
Javaの教科書でも見たことがあったので大丈夫だと思います！
（ただbreakの条件があるのに無限ループとは…？と見るたびに混乱します（笑））
ありがとうございました！

Answer 5

breakすればいいじゃない。

while(true){
    int num = str.indexOf(args[1]);
    
    if(num == -1){
      break;
    }
    
    str = (str.substring(0,num) + args[2] 
         + str.substring(num + args[1].length()));

}

Comments

[nekocat]

ご回答ありがとうございます。
無限ループですね！（でもこの場合はbreakしているので無限ではないのでしょうか...???）
また違った発想だったので面白かったです！ありがとうございます。

Answer 6

replace関数を使うのはダメなんでしょうか？

class Mondai3{
    public static void main(String[]args){
        System.out.println(args[0].replace(args[1], args[2]));
    }
}

Comments

[nekocat]

ご回答ありがとうございます。
今回アルゴリズムを学ぼうと思っており、replaceは使わないように
やってみようと試行錯誤中なのです。
ご助言ありがとうございます！

Answer 7

String#replaceを使えば数行で全体を代替可能です。

System.out.println(
    args[0].replace(args[1], args[2])
);

Comments

[nekocat]

ご回答ありがとうございます。
今回アルゴリズムを学ぼうと思っており、replaceは使わないように
やってみようと試行錯誤中なのです。
ご助言ありがとうございます！

Answer 8

ループの中でstrが変化する以上、２個めのindexOfは省略できないですね

Comments

[nekocat]

ご回答ありがとうございます。
そうですよね...書き方に一工夫いりそうです。。。