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下载