Clean Code

public Object get() {
return booleanValue;
}
}
Once again the tests passed. So both 
get
and
set
deploy to the 
BooleanArgumentMarshaler
!
This allowed me to remove the old 
getBoolean
function from 
ArgumentMarshaler
, move the
protected
booleanValue
variable down to 
BooleanArgumentMarshaler
, and make it 
private
.
I did the same pattern of changes for 
Strings
. I deployed both 
set
and
get
, deleted the
unused functions, and moved the variables.
private void setStringArg(char argChar) throws ArgsException {
currentArgument++;
try {
stringArgs.get(argChar).
set
(args[currentArgument]);
} catch (ArrayIndexOutOfBoundsException e) {
valid = false;
errorArgumentId = argChar;
errorCode = ErrorCode.MISSING_STRING;
throw new ArgsException();
}
}

---

219
String Arguments
...
public String getString(char arg) {
Args.ArgumentMarshaler am = stringArgs.get(arg);
return am == null ? "" : 
(String)
am.
get
();
}
...
private abstract class ArgumentMarshaler {
private int integerValue;
public void setInteger(int i) {
integerValue = i;
}
public int getInteger() {
return integerValue;
}
public abstract void set(String s);
public abstract Object get();
}
private class BooleanArgumentMarshaler extends ArgumentMarshaler {
private boolean booleanValue = false;
public void set(String s) {
booleanValue = true;
}
public Object get() {
return booleanValue;
}
}
private class StringArgumentMarshaler extends ArgumentMarshaler {