outrank_error 2014-06-16 11: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 2014-07-25 20:17:28  

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

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

Twitter認証

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

[Java] 連番

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

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

...

package com.renban.erq053.czp008;

/**...

鑑賞する »

[Java] もうコメント書くなお前

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

改修案件でぶち当たったソース マジでこ...

/**
  *
  *   〇〇の値を取得する
  *
  */
pu...

鑑賞する »

[Java] ぼくの考えた最強のリスト

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

やればできる

import java.util.ArrayList;
import java...

鑑賞する »