streamについて

Question

/**
*文字列を分割して変換する
*/
private String changeword(String word, String name) {
    	StringBuilder changingWord= new StringBuilder();
    	String[] searchWord = word.split("");
    	Arrays.stream(searchWord).forEach(w -> {
    		if (w　== "a") {
    			changingWord.append(w.replace(w, "A"));
    		} else if (w　== "b")  {
    			changingWord.append(w.replace(w, "C"));
    		} else if  (w　== "c") {
    			changingWord.append(w.replace(w, "0"));
    		} else {
    			changingWord.append(w.replace(w, "$$"));
    		}
    	});
    	String result = name+ "=",changingWord.toString());
    	return result;
    }

上記の文字列を分割し、変換した結果を表示するメソッドをラムダ式をより使用し、記述できますか？
例えば、if文をfilter()を使用したり、最後にcollect()を使用したり、
String[] searchWord = word.split("");の記述をStringJoinerクラスを使用したりして、実装できますか？
よりラムダ式を使用して、記述したいですが、イメージがわきません。
教えていただけるとありがたいです。
よろしくお願い致します。

Answer 1

質問者さんは、1文字ずつ分離するのにsplit("")を使っておられますが、それは直接的でないように感じました。文字列を1文字ずつのストリームへ変換するString#codePointsメソッド(※)がありますのでそれを用いた方がより直接的であるように感じます。

import java.util.stream.Collectors;

public class StreamTest {
	public static void main(String[] args) {
		System.out.println(changeWord("abcd", "NAME"));

	}
	
	static private String changeWord(String word, String name) {
		return name + "=" + word.codePoints()
			.mapToObj(StreamTest::mapCodePointToString)
			.collect(Collectors.joining());
	}
	
	// このメソッドを定義せずともchangeWord内に記述してもかまわないです。
	// 単に個人的好みで分けて定義したに過ぎません。
	static private String mapCodePointToString(int codePoint) {
		switch (codePoint) {
			case 'a': return "A";
			case 'b': return "C";
			case 'c': return "0";
			default: return "$$";
		}
	}
}

---

※:似たメソッドにString#charsがありますが、これはサロゲートペアをばらばらの要素に分解してしまうため、携帯用絵文字や拡張領域の漢字などcharが2つで一文字となるような文字が有り得ることを考えるとString#codePointsの方が素直な選択のように思います。

Comments

[KSwordOfHaste]

自分は見易さからストリーム処理の外側でname,"="を連結しました。しかし、効率のことを配慮すればroot_jpさん回答のようにjoiningへ指定する方がベターではないかと推測します。

[root_jp]

> String#codePoints
うおーこんな便利なメソッドがあったなんて。
@since 1.8
知らなかった。。ありがとうございますｗ

[swordone]

CharSequenceのデフォルトメソッドになってて目立ちにくいですからね。

Answer 2

一旦文字列に変換するのが嫌だったので

public class Q99560 {
	public static void main(String[] args) {
		System.out.println(changeWord("abcd", "NAME"));

	}

	static private String changeWord(String word, String name) {
		return IntStream.concat(new StringBuilder(name).append('=').codePoints(),
								word.codePoints().flatMap(Q99560::mapCodePoint))
				.collect(StringBuilder::new, StringBuilder::appendCodePoint, StringBuilder::append)
				.toString();
	}

	static private IntStream mapCodePoint(int codePoint) {
		switch (codePoint) {
		case 'a':
			return IntStream.of('A');
		case 'b':
			return IntStream.of('C');
		case 'c':
			return IntStream.of('0');
		default:
			return IntStream.of('$', '$');
		}
	}
}

最後の変換までStringを避けようとしていたらやたら冗長になった気がする

Comments

[KSwordOfHaste]

joiningにまかせず、collectの3引数版を用いてint(codepoint)のまま連結する点が面白いと思いました。
ただmapCodePointでIntStreamを生成する際に少なくとも1文字あたり2つオブジェクトがnewされるので効率がどうなのか興味がわきました。

[swordone]

まあIntStreamだとCollectorが使えませんからね…
1文字あたり2つのオブジェクトとは？ofに渡しているのはプリミティブな値なので
オブジェクトは1つしかnewされていないと思いますが…

[KSwordOfHaste]

IntStream.ofの実装をちょこっと見ただけですが、Streams.IntStreamBuilderImplとIntPipeline.Headの２つの小さなオブジェクトがnewされているようでした。

Answer 3

private String changeword(String word, String name) {
	return Arrays.stream(word.split("")).map(w -> {
		switch (w) {
		case "a":
			return "A";
		case "b":
			return "C";
		case "c":
			return "0";
		default:
			return "$$";
		}
	}).collect(Collectors.joining("", name + "=", ""));
}