outrank_error outrank_error 2014-06-16 20:01:15

[Java] そのメソッドは必要なのか このエントリーをはてなブックマークに追加

投稿者からのアピールポイント

改修に次ぐ改修で魔境と化しているコード
その中のある一幕

public class FooX{
	public void foo(){
		FooA obj = getFooA();
		obj.bar();
	}
	public FooB getFooA(){
		return new FooB();
	}
}

public class FooA{
	public void bar(){
		//処理が書いてある
	}
}

public class FooB extends FooA{
	public void bar(){
		//処理が書いてある
		super.bar();
	}
}

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

コメント(1)

#1 InternalServerE InternalServerE 2014-07-26 05:17:28  

ケースバイケースな印象。
例えば、FooX#getFooA() を子クラスからオーバーライドされる前提で作っているなら、割とアリな気もする。
メソッドの戻り値がFooBな点は少し気になるけども。

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

Twitter認証

このウンコードに臭った人は、こちらのウンコードにも臭ってます

[Java] 連番

このエントリーをはてなブックマークに追加

もはや人間が読むものではない。

...

package com.renban.erq053.czp008;

/**...

鑑賞する »

[C] switch-caseは現場の臭いがする

このエントリーをはてなブックマークに追加

"とりあえず"で書いた 2010 年 4...

// switch-case にしたいが、関数で値を取得しないといけないため、
...

鑑賞する »

[C] すげーちゃんと動く!

このエントリーをはてなブックマークに追加

結合試験2日前に脱退したうんkプログラマ...

return 0;

鑑賞する »