文字列操作のやり方について

Question

お世話になっております。

教本を見ながら、
-（ハイフン)でコマンドライン引数で受け取った文字列を区切り、
1つずつ表示させるプログラムを書いています。
（※アルゴリズムの考え方の勉強のため、splitや正規表現の使用は意図的に避けています。）

一応仕様通りに動くものはできたのですが、後学のため
他のやり方も知りたいと思い、質問させていただきました。

そこで思ったのが、変数のindexを1つだけにすることは
できるのでしょうか？(＝indexかindex2をなくす)
（「変数の数はなるべく少ない方が良い」と書籍で以前読んだので...）

他のやり方でも構いませんので、アドバイスを頂けると幸いです。

package nosplit;

public class NoSplit {

	public static void main(String[] args) {

		int index = 0;

		int index2 = 0;

		String word = args[0];

		while (true) {

			index2 = word.indexOf("-", index);

			if(!(-1 == index2)){

				System.out.println(word.substring(index,index2));

				index = index2 + 1;

			} else {

				index = word.lastIndexOf("-");

				System.out.println(word.substring(index + 1));

				break;
			}

		}
	}
}

Comments

[swordone]

このコードで仕様通りに動くとは思えません。そもそもbeginIndexはどこ？

Answer 1

「変数の数はなるべく少ない方が良い」と書籍で以前読んだので...

変数の数が多いと挙動を追いづらくなるのは確かですが、
減らすと可読性が上がる、バグが出づらくなるとは一概には言えません。

どちらかと言うと、変数の影響範囲を狭める方が実際的でしょう。
文字列を指定の文字で分割し、分割後の配列を返すstaticメソッドを書いてみると便利です。

static String[] mySplit(String src, String delimiter) {
     String[] ret = new String[src内のdelimiterの個数 + 1];
     ...

     return ret;
}

納得のいくメソッドができたら、
mainメソッド内の mySplit(str, ",") を str.split(",") に差し替えれば良いです。

Answer 2

文字列を文字に分解し、それがハイフンなら改行しそれ以外ならそのまま出力します。

public static void main(String[] args) {
    args[0].chars().forEach(i -> System.out.print(i == '-' ? System.lineSeparator() : (char)i));
}

Answer 3

やるんだとしたら、「残りの文字列」を毎回生成することとかかな…

package nosplit;

public class NoSplit {

    public static void main(String[] args) {

        int index = 0;

        String word = args[0];

        while ((index = word.indexOf("-")) >= 0) {

            System.out.println(word.substring(0, index));

            word = word.substring(index + 1);

        }

        System.out.println(word)
    }
}

ただやはりStringの生成はコストがかかるので、変数を減らすことにこだわるよりもStringの生成回数が少ないほうを選んだほうがいいです。

Comments

[orangeLover]

迅速なご回答、ありがとうございます。
なるほど...ケースバイケース、ということなのでしょうか？

＞Stringの生成回数
これは
String something = "";
のことでしょうか？

Answer 4

変数を1つだけにしてみました。

package teratail;

public class NoSplit {

    public static void main(String[] args) {
        if (args.length == 0 || args[0].length() == 0) {
            return;
        }

        for (char c : args[0].toCharArray()) {
            if (c == '-') {
                c = '\n';
            }
            System.out.print(c);
        }
        System.out.println();
    }
}

substring() を使おうとすると、start, end の２つの変数が必要になります。
String を char[] に分解して、一文字ずつ処理をするようにするなら変数は１つで済みます。