JavaのSimpleDateFormatの使いまわしは可能か

Question

前提

Javaを勉強中の者です。
Date型から文字列へ変換する処理を調べています。
SimpleDateFormatはスレッドセーフではないため、インスタンス変数として宣言するのではなく、メソッド内にローカル変数として宣言しないといけないと記事で読みました。

疑問

SimpleDateFormatは内部でCalendarのインスタンス変数を保持しており、formatを実行するたびにそれが書き換えられる仕様とのことです。
この仕様は、たとえば以下のコードのように、異なる変数に対してSimpleDateFormatを使いまわしたりするのに影響があるでしょうか。
あくまでdate1の処理が終わってからdate2のformatが開始するため、SimpleDateFormatのインスタンスの状態が変わっても上書きされ問題がない認識です。

public void test(Date date1, Date date2) {
  SimpleDateFormat sdf = new SimpleDateFormat("yyyy-MM-dd");
  String dateStr1 = sdf.format(date1);
  String dateStr2 = sdf.format(date2);
}

Comments

[m.ts10806]

実際に試したコードではどうだったのでしょう。
あまりオブジェクトを別の用途を使いまわすということはないと思いますが。。

[退会済みユーザー]

ありがとうございます。
テストコードでSimpleDateFormatを異なる変数で使いまわしてみても、結果は問題なく出ていました。
ただ、使いまわし自体が本来の使い方でない可能性もあるのかなと思い、質問させていただきました。

>あまりオブジェクトを別の用途を使いまわすということはないと思いますが。。
やはり、各変数のためにSimpleDateFormatを個別に宣言してあげるのが一般的でしょうか。
このあたりプログラミングの経験が浅くよくわかっておりません。。。

[m.ts10806]

役割が違うなら別のオブジェクトとして使いたいですが、使うならせめて初期化したいところ。

SimpleDateFormat sdf = new SimpleDateFormat("yyyy-MM-dd");
String dateStr1 = sdf.format(date1);
sdf = new SimpleDateFormat("yyyy-MM-dd");
String dateStr2 = sdf.format(date2);

[退会済みユーザー]

ありがとうございます。
大変参考になりました。

Answer 1

SimpleDateFormatはスレッドセーフではないため、インスタンス変数として宣言するのではなく、メソッド内にローカル変数として宣言しないといけないと記事で読みました。

提示されているtestメソッドはスレッドは関係していそうにないので、SimpleDateFormatをインスタンス変数とするかローカル変数とするかというはなしなら、わたしは、ローカル変数として定義します。

SimpleDateFormatのインスタンスを生成するコストが非常に大きいのであれば(とっても時間がかかるとか)、インスタンス変数として定義することも考えますが、変数のスコープを狭めることを優先します。インスンタンス変数のスコープはそのクラス全体。ローカル変数のスコープはそのメソッドが終了するまで。後者を選びます。そしてこのケースならインスタンスは使いまわします。つまり、件のtestメソッドままとしますね。

---

追記。こういう考え方もできると思います。

定義したクラスをスレッドセーフなものにしたいか、否か。

class Hogege {
  private final SimpleDateFormat sdf = new SimpleDateFormat("yyyy-MM-dd");

  public void test(Date date1, Date date2) {
    String dateStr1 = sdf.format(date1);
    String dateStr2 = sdf.format(date2);
  }
}

上記では、スレッドセーフではないクラスのインスタンスをインスタンス変数としているので、(synchronizedで適宜同期化しなければ)Hogegeクラスはスレッドセーフではなくなる。

スレッドセーフではないクラスのインスタンスをインスタンス変数としていなければ、スレッドセーフなクラスになりうる。

class Hogege {
  public void test(Date date1, Date date2) {
    SimpleDateFormat sdf = new SimpleDateFormat("yyyy-MM-dd");
    String dateStr1 = sdf.format(date1);
    String dateStr2 = sdf.format(date2);
  }
}

Comments

[退会済みユーザー]

ご回答ありがとうございます！
どのような筋道で考えればよいのか、とても参考になりました。

Answer 2

SimpleDateFormatはスレッドセーフではないため、インスタンス変数として宣言するのではなく、メソッド内にローカル変数として宣言しないといけないと記事で読みました。

よければURLを貼ってください。

さすがにそれは言い過ぎなのではという気がします。
別スレッドからアクセスする可能性がゼロならば、何の問題もないです。

そもそもスレッドセーフなクラスのほうが少数派ですよね？
ArrayListはスレッドセーフじゃないから、インスタンス変数に持ったらダメですか？そんなことはないですよね？

スレッドセーフな DateTimeFormatter との対比で、（やや大げさに）そのように言われてしまうのかもしれません。

たとえば以下のコードのように、異なる変数に対してSimpleDateFormatを使いまわしたりするのに影響があるでしょうか。

この例なら100%使いまわしますね。2度newするほうがおかしいでしょう。

（DateFormatでですが）ドキュメントでもそのように使っています。
DateFormat (Java Platform SE 8 )

---

私ならインスタンス変数に持っちゃいますが「simpledateformat スレッドセーフ」とかで検索したら大量にヒットするってことは、（知っていたとしても？）はまりやすい問題なのかもしれません（SimpleDateFormatが特別はまりやすい理由は見当がつきませんが^^;

Comments

[退会済みユーザー]

ご回答ありがとうございます！

私が参考にしていたのは下記のブログです。
SDFをThreadLocalにするなどほかの対処法もあるようですが、まだきちんと理解できていないのでこれから勉強進めます。
http://tech-blog.tsukaby.com/archives/172

> （DateFormatでですが）ドキュメントでもそのように使っています。
こちらチェックできていませんでした。
JavaDoc大事なんですね...これからはまず確認します。
ありがとうございました。

[TN8001]

SimpleDateFormatが悪者？というわけではなく、
「スレッドを扱う場合は細心の注意を払おう」ということですね^^