Eclipse重构——Replace TypeCode with Class

《重构——改善既有代码》

Motivation

ü  类型码不是类型安全的，终究只是一堆数字或者一个字符串。使用类代替类型码，可以做更好的编译检查。

ü  如果类型码不影响宿主类的行为，则优先考虑使用Replace Type Code with Class

Mechanics

ü  为类型码建立一个类。类中应该定义一组静态变量，表示允许被创建的实例。

ü  修改源类实现，将类型码替换为上面新建的类。

Eclipse重构菜单中没有直接的对应选项，一般主要应用下面两个可逆操作,完成类型码变量的替换:

[Inline]                        CLAIMED  -->  PermissionState.CLAIMED

[Extract Constant]  PermissionState.CLAIMED --> CLAIMED

 

表示类型的Class，一般可以是enum， 静态内部类，或者一般的类。

1. 使用enum

 

         private enum PermissionState {

		REQUESTED("REQUESTEDd"),

		CLAIMED("CLAIMEDd"),

		DENIED("DENIED"),

		GRANTED("GRANTED");

		

		String mState;

		private PermissionState(String state) {mState = state;}

		private String getState() { return mState; }

// enum 有valueof(String arg)作为setState()

	}

2. 静态内部类

 

 

	static class PermissionState {

		public final static PermissionState REQUESTED = new PermissionState("REQUESTED");

		public final static PermissionState CLAIMED = new PermissionState("CLAIMED");

		public final static PermissionState DENIED = new PermissionState("DENIED");

		public final static PermissionState GRANTED = new PermissionState("GRANTED");

		public final static PermissionState UNKNOWN = new PermissionState("UNKNOWN");	

		

		String mState;

		// 构造函数应该private

		private PermissionState(String state) {

			mState = state;

		}



		public String getState() {

			return mState;

		}



	}

3. 一般类，意味着有继承体系

 

 

abstract class PermissionState {

	public final static String REQUESTED = "REQUESTED";

	public final static String CLAIMED = "CLAIMED";

	public final static String DENIED = "DENIED";

	public final static String GRANTED = "GRANTED";

	public final static String UNKNOWN = "UNKNOWN";	

	

	// String mState;  // no field mState

	public abstract String getState() ;  // override by subclass

	public void setState() { //switch REQUESTED, CLAIMED 

	}

}

一般两个特征总是必不可少，getState， setSate。 进一步，因为类型码是有限可列举的，所以setState最好是可以使用HashMap实现，static Hashmap<typecode,  class>.

 

over

完整的重构代码包here下载