Clean Code

booleanArgs.put(elementId, m);
marshalers.put(elementId, m);
}
And now we can delete the 
boolean
map.
public class Args {
...
private Map booleanArgs = 
new HashMap();
private Map stringArgs = 
new HashMap();
private Map intArgs = 
new HashMap();
private Map marshalers = 
new HashMap();
...
Next, I migrated the 
String
and
Integer
arguments in the same manner and did a little
cleanup with the 
booleans
.
private void parseBooleanSchemaElement(char elementId) {
marshalers.put(elementId, 
new BooleanArgumentMarshaler()
);
}
2.
To prevent further surprises of this kind, I added a new unit test that invoked all the FitNesse tests.

---

225
String Arguments
private void parseIntegerSchemaElement(char elementId) {
marshalers.put(elementId, 
new IntegerArgumentMarshaler()
);
}
private void parseStringSchemaElement(char elementId) {
marshalers.put(elementId, 
new StringArgumentMarshaler()
);
}
...
public String getString(char arg) {
Args.ArgumentMarshaler am = 
marshalers
.get(arg);
try {
return am == null ? "" : (String) am.get();
} catch (ClassCastException e) {
return "";
}
}
public int getInt(char arg) {
Args.ArgumentMarshaler am = 
marshalers
.get(arg);
try {
return am == null ? 0 : (Integer) am.get();