StringUtilの改善要望

• Reporter: kidotaka
• Owner: (None)
• Type: Feature Requests
• Status: Open
• Component: (None)
• MileStone: (None)
• Priority: 5 - Medium
• Severity: 5 - Medium
• Resolution: None
• File: 3

Details Reply

kidotakaと言います。

半角=>全角に部分で、半角と全角を保持する2種類のStringで
実現されていますが、あの箇所は変換用のHashMap<Character
(半角),Character(全角)>を用意して実装にした方が早いです。

StringでindexOfで位置を求めてマッピングするやり方である
と、どうしても後ろの方にある文字は遅くなりますが、
HashMapの場合はそういう劣化はほとんどありません。
2種類のStringはアルファベットを先に列挙しているので、カ
タカナは基本的に変換が遅くなる実装になってます。

メモリ的には悪化しますが、検討されてはいかがでしょうか？

あと、私が似たようなクラスを作った際、半角=>全角のマッ
ピングはプロパティに出して、staticイニシャライザで初期
化するようにしていました。
prefix.<半角のユニコード>=<全角のユニコード>
あとは念のためCollections.unmodifiableMapを用いて参照専
用の処置をするなど。

他に、
nextvalue = new Character(chars[i + 1]);
if (nextvalue.equals(new Character('ﾞ'))) {...
} else if (nextvalue.equals(new Character('ﾟ'))) {...
こう書いてある箇所は、オブジェクトを作らずにcharのまま
比較するべきかと思います。

Ticket History (3/5 Histories)

2008-05-12 07:59 Updated by: kidotaka

• File 2321: StringUtil.java is attached

2008-05-12 08:02 Updated by: kidotaka

• File 2322: StringUtil.java is attached

Comment Reply

Logged In: YES
user_id=31181

簡単な修正イメージとしてファイルを添付します。

2008-05-14 16:13 Updated by: kimuraku

Comment Reply

Logged In: YES
user_id=31940

ご要望ありがとうございます。

半角全角のマッピングに関しては、ご指摘の通り
Mapを利用するのが良いと思います。

次期バージョンアップ時には是非検討させていただきます。

今後ともTERASOLUNAをよろしくお願いいたします。

2008-05-14 16:14 Updated by: kimuraku

Comment Reply

Logged In: YES
user_id=31940

ご要望ありがとうございます。

半角全角のマッピングに関しては、ご指摘の通り
Mapを利用するのが良いと思います。

次期バージョンアップ時には是非検討させていただきます。

今後ともTERASOLUNAをよろしくお願いいたします。

2008-05-18 01:59 Updated by: kidotaka

• File 2328: StringUtil.java is attached

Comment Reply

Logged In: YES
user_id=31181

未テストで思い切りバグらせてましたが、
既存のテストケースを実施して動く状態にしてみました。