Clean Code

private int intValue = 0;
public void set(String s) 
throws ArgsException {
try {
intValue = Integer.parseInt(s);
} catch (NumberFormatException e) {
throw new ArgsException();
}
}
public Object get() {
return intValue;
}
}

---

221
String Arguments
Of course, the tests continued to pass. Next, I got rid of the three different maps up at
the top of the algorithm. This made the whole system much more generic. However, I
couldn’t get rid of them just by deleting them because that would break the system.
Instead, I added a new 
Map
for the 
ArgumentMarshaler
and then one by one changed the
methods to use it instead of the three original maps.
public class Args {
...
private Map booleanArgs = 
new HashMap();
private Map stringArgs = 
new HashMap();
private Map intArgs = 
new HashMap();
private Map marshalers = 
new HashMap();
...
private void parseBooleanSchemaElement(char elementId) {
ArgumentMarshaler m = new BooleanArgumentMarshaler();
booleanArgs.put(elementId, m);
marshalers.put(elementId, m);
}
private void parseIntegerSchemaElement(char elementId) {
ArgumentMarshaler m = new IntegerArgumentMarshaler();
intArgs.put(elementId, m);
marshalers.put(elementId, m);
}
private void parseStringSchemaElement(char elementId) {
ArgumentMarshaler m = new StringArgumentMarshaler();
stringArgs.put(elementId, m);