インスタンスフィールドの初期化方法

Question

クラスのインスタンスフィールドを初期化する場合、次の3つが考えられますが、どれが一番スマートなんでしょうか。


クラスQ1はインスタンス変数int xを持つとします。

１．コンストラクタ内で初期化
e.g. Q1() {this.x = 1;}

２．宣言時に初期化
e.g. int x = 1;

３．インスタンス初期化子内で初期化
e.g.
{
x = 1;
}
Q1(){}

komi1341 · Accepted Answer

スマート、という表現が主観的なので正解はなさそうに思います。強いて言えば、一つのプログラム内で意味もなく複数のやり方を混ぜて使うのはスマートじゃないだろうな、というくらいです。

参考までに、私は基本的にコンストラクタ内で初期化しています。staticな定数なら宣言時に初期化。インスタンス初期化子は使ったことがありませんし、人のプログラムでも見たことがありません。

hrm_mmm · Answer

私は、趣味プログラマなので、「こうやってる」というだけですが、
3番は、初期化子の前に、宣言は必須です。よって３番は冗長なだけであり、２番の方がわかりやすくバグの危険が減ると思います。
あと、初期化データのためによほど複雑な計算があるなら、初期化子内でやることもあるのかもしれないけど、初期値にそんな複雑なデータを持たせる必要はいままでなかったので、初期化子自体使ったことがありません。
１番は、コンストラクタでの初期化時に、引数などから初期化用データをとりたいときに使います。
定数代入なら、２番にすることが多いです。eclipseを使ってると、変数の宣言部分参照で、どんなデータが入ってるかもすぐわかるので。

komi1341 · Answer

いろいろな方の意見を聞けるといいですね。
追加の質問の件のみ回答させて頂きます。

> 宣言時に初期化かつコンストラクタでも同じ値で初期化するのは、意味のないことでしょうか。

逆にお聞きするようですが、そうすることに意味またはメリットはありますか？　私には保守性の面でのデメリットしか見えてきません。
例えばあるとき、別の値で初期化したくなったらどうするのでしょう。宣言部とコンストラクタ内と、両方修正するのでしょうか。単純に考えて面倒です。動作上はコンストラクタの方のみ値を変えれば想定通りになるでしょうが、宣言部と違う値で初期化されているのを他人が見たら、理解に苦しむでしょう。1年も放置すれば、自分のプログラムも他人が書いたものみたいに感じるものなので、未来の自分を困惑させることになるかもしれません。
そういうデメリットを上回るぐらいのメリットがないなら、いずれか一つの方法で初期化した方がよいと思います。

インスタンスフィールドの初期化方法

スマート、という表現が主観的なので正解はなさそうに思います。

私は、趣味プログラマなので、「こうやってる」というだけですが、

いろいろな方の意見を聞けるといいですね。

関連するカテゴリからQ&Aを探す

デイリーランキングこのカテゴリの人気デイリーQ&Aランキング

マンスリーランキングこのカテゴリの人気マンスリーQ&Aランキング