ER-084 Avoid using text labels in "switch" statements

Severity1
Enabledyes
Waivable
Violation
switch (baseVal) {
	case TXT_1 :
		retVal = INT_2;
		break;
	case "Option 2" :
		retVal = INT_7;
		break;
	default :
		retVal = INT_1;
		break;
}
Fix
if (TXT_1.compareTo(baseVal)==0) {
	retVal = INT_2;
} else if (TXT_1.compareTo(baseVal)==0) {
	retVal = INT_7;
} else {
	retVal = INT_1;
}


Hammurapi 3 Copyright © 2004 Hammurapi Group. All Rights Reserved.