インナークラスへのリファクタリング

あえてインナークラスに向かってリファクタリングしたい状況があるとする。

package test;
public class Foo {
}

package test;
public class Hoge {
	public static void hoge() {
	}
}


package test2;
import test.Hoge;
public class Main {
	public static void main(String[] args){
		Hoge.hoge();
	}
}

この状態から、Fooクラスの中にHogeクラスを移動したいとする。

eclipseのリファクタリングにはインナークラスへのリファクタリングが無いため、手順としては

１、Fooの中に空のHogeクラスを作成する。
２、メソッドの移動で、HogeクラスのメソッドをFooの中のHogeクラスに移動する。

の２ステップ。

これでうまく行くが、悲しいことにMainクラスがこうなってしまう。

package test2;
import test.Hoge;
public class Main {
	public static void main(String[] args){
		test.Foo.Hoge.hoge();
	}
}

この場合、手順増やすことできれいにリファクタできる。

１、Fooの中に空のHoge2クラスを作成する。
２、メソッドの移動で、HogeクラスのメソッドをFooの中のHoge2クラスに移動する。
３、Hogeクラスを削除する。
４、Fooの中のHoge2クラスを、Hogeクラスにリネームする

Hogeクラスを削除しないと、なぜか４の手順でエラーが出る。（たぶんeclipseのバグ）
そのまま無視して進めればうまくリファクタできるが、なにぶん気持ち悪い。

おすすめはこれ。

１、Fooの中に空のHogeクラスを作成する。
２、メソッドの移動で、HogeクラスのメソッドをFooクラスに移動する。
３、Fooクラスのメソッドを、Fooの中のHogeクラスに移動する。

こちらだと元のHogeを削除しなくてもエラーにならない。

上記２つの手順だと、

package test2;
import test.Foo.Hoge;
public class Main {
	public static void main(String[] args){
		Hoge.hoge();
	}
}

とリファクタリングされる。

ただ、もしこのようにリファクタリングしたい場合は

package test2;
import test.Foo;
public class Main {
	public static void main(String[] args){
		Foo.Hoge.hoge();
	}
}

上記のどの手順でもうまく行かないので、泣く泣くコーディングする必要がある。

enjoy!