StringBufferからStringへキャストする

StringBufferからStringへキャストする方法としてどちらを用いるのが一般的なのでしょうか
StringBuffer sb
として
public String ***(***)メソッドの最後に

１. return sb.toString()とする
２. return new String(sb)とする

私は１を使っていたのですが、
Javaの規約だか処理能力が良い方法だかに
厳しい人が２を使っていたので少し戸惑う程度ですが…
回答よろしくお願いします。

No.1 回答者： aton 回答日時： 2002/02/27 15:10

以下の2つのクラスを作って簡単に実験してみました。

==========
class STest1
{
public static void main(String[] args)
{
System.out.println(getString(args[0]));
}

static String getString(String str)
{
StringBuffer sb = new StringBuffer(str);
return sb.toString();
}
}
==========
class STest2
{
public static void main(String[] args)
{
System.out.println(getString(args[0]));
}

static String getString(String str)
{
StringBuffer sb = new StringBuffer(str);
return new String(sb);
}
}
==========

バイトコードのサイズは，STest1.classが605バイト，STest2.classが627バイト。これをそれぞれ javap -c してみると，当然ですが違いはgetString()メソッドの最後の方だけで，STest1のほうが，

8 astore_1
9 aload_1
10 invokevirtual #7 <Method java.lang.String toString()>
13 areturn

であるのに対し，STest2のほうは，

8 astore_1
9 new #7 <Class java.lang.String>
12 dup
13 aload_1
14 invokespecial #8 <Method java.lang.String(java.lang.StringBuffer)>
17 areturn

となっていました。これを見るかぎり，toString()を使ったほうが効率が良く，new String() を使うメリットは何も考えられないのですが…。

No.2 回答者： web5 回答日時： 2002/03/03 17:19

StringBufferソースを見ると、

return new String(sb);
のようにtoString()がオーバーライドされていたと思います。

厳しい人が２を使っていたというのは、
その方が正確・・・というか、理解しやすいからでしょう。
１を使う人は、StringBufferのtoString()が、
オーバーライドされているということを知らない人が多いと思います。
ちなみに、私も２の書き方をします。

また、バイトコードのサイズを比較されていた方もおられますが、
現在はHDDもメモリも安価になり、大量に積んでいるマシンが多いため、
あまり気にすることはないように思います。

sb.toString()を使ってtoString()内部で何がされているかわからないよりも、
new String(sb)として、Stringオブジェクトを返しているという明示的なわかりやすさを得る方がいいと思います。

No.3 ベストアンサー 回答者： bo-ken 回答日時： 2002/03/05 15:55

toString()メソッドは全てのClassのスーパクラスであるObjectクラスのメソッドです。

その他のクラスのtoString()メソッドは、これをオーバライドしたに過ぎません。
toString()メソッドの用途はそのオブジェクトを視覚的にあらわすときに利用します。オブジェクトのダンプイメージです。
今回はStringBufferクラスのtoString()メソッドのためダンプイメージがStringそのもののため問題ありませんが、
「Stringにキャストする」と明示的に処理するには２を使用するのが正解と思います。
格クラスのtoString()メソッドはDEBUGで画面やファイルに出力するときに使用すると考えたほうが賢明と思います。

No.4 回答者： noname#16216 回答日時： 2002/03/05 22:26

参考URLに、この問題についての議論が行われています

そこでも議論されているように正しいのは２の方だと思います。
未だにJava自身のソースや「プログラミング言語Java 第３版」の中でも、sb.toString()が
例として上がっているようですが。

ちなみに、個人的に１を使っていました。
１つ勉強になりました

参考URL：http://java-house.jp/ml/archive/j-h-b/012934.html

この回答へのお礼

まとめてのお礼になり申し訳ないです。

皆様の意見を私なりに解釈してこうなりました。

・String toString()はObjectメソッドを継承しているクラスなので戻り値がStringである事はJavaをそこそこ触れれば誰もがわかる
・str = strinBuffer.toString();がたまたまStringへキャストした値を返していた。
・newを使用するよりも動作的にも早かった。
と言う理由で（１）が浸透してしまった(orしつつあった)

しかし、strinBuffer.toString();が将来#StringBuffer:6542154等ということになりかねないし、StringBufferクラスを継承させて使おうものならtoString()オーバーライドされればTheEnd…
その点、new String(StringBuffer)ならば確実にキャストされる。
ですので（２）を使用するのが正しい…

atonさん、web5さん、bo-kenさん、spoonyさん、本当にありがとうございました。

お礼日時：2002/03/08 14:07