teratail header banner
teratail header banner
質問するログイン新規登録

回答編集履歴

2

追記に対して

2017/05/15 17:51

投稿

swordone
swordone

スコア20675

answer CHANGED
@@ -1,1 +1,5 @@
1
- replaceAllを使うことに固執せず、一文字ずつチェックして文字列を組み直してみてはいかがでしょう。
1
+ replaceAllを使うことに固執せず、一文字ずつチェックして文字列を組み直してみてはいかがでしょう。
2
+ ###追記に助言
3
+ - 何度も文字列を連結するなら、Stringに対する+演算子を使うより、StringBuilderを使ったほうが良い
4
+ - ループのたびにMatcherを作成しなおしているのが無駄。前回マッチ時の終端位置を取得し、同じMatcherを使いまわせるようにするべき
5
+ - 「何とマッチしたか」はgroupメソッドで取得できる。

1

誤字

2017/05/15 17:51

投稿

swordone
swordone

スコア20675

answer CHANGED
@@ -1,1 +1,1 @@
1
- replaseAllを使うことに固執せず、一文字ずつチェックして文字列を組み直してみてはいかがでしょう。
1
+ replaceAllを使うことに固執せず、一文字ずつチェックして文字列を組み直してみてはいかがでしょう。