[Java] 宣言したら必ず new して代入しなければならないらしい-ウンコード・マニア

sk44_ 2012-08-17 10:46:54

[Java] 宣言したら必ず new して代入しなければならないらしい

なんかしらんけど以前やたらと見かけた。 new 要らんだろ。

// 意味のない new
List<Foo> l = new ArrayList<Foo>();
l = someMethod();

使い方ヒント: 「これは臭う」という行を見付けたら、各行の smell をクリックしてマーキングしておきましょう(要Twitter OAuth認証)

#1 InternalServerE 2012-08-17 12:51:42

ああ、ありますねぇ。 nullで初期化しといて即代入、っていうケースも良く見かける。

#3 math_neko 2012-08-17 14:34:04

null で初期化して try ～ catch の中で代入ってのは Stream 系とかで良くやりますが、Collection ではねぇ…w

#4 poad1010 2012-08-20 17:22:34

null初期化ならまだいいけど、このケースは頻繁に実行されるとGCさんコンニチハなので「不要」ではなく、「やるべきではない」かと。

#6 k5n6 2012-08-21 09:54:17

あるある。Javaならnewだからまだ意識する人もいるけど、PHPだと平気でarray()とかやる人いるね。

#7 tacticsrealize 2013-04-14 11:01:58

上のarray()の亜種だけど、Stringを""で初期化して即代入とか

コメント投稿には、twitter認証が必要です。

Twitter認証

« Java一覧に戻る

実際にあった某システムの超重要なマスター...

create table item_master (
 A varchar(2...

画面の左側がわりと無駄な気がします。

public int hoge(){
    int result = 0;
...

これを書いた先輩に「英語で書きましょうよ...

class 会員 {
    private int 会員番号;
    p...