Oh, pero eso es un código feo, no orientado a objetos. En el momento en que veas "if / else" y "typeof", deberías estar pensando en polimorfismo. Este es el camino equivocado. Creo que los genéricos son tus amigos aquí.
¿Con cuántos tipos planeas lidiar?
ACTUALIZAR:
Si solo está hablando de String e int, aquí hay una forma en que puede hacerlo. Comience con la interfaz XmlGenerator (suficiente con "foo"):
package generics;
public interface XmlGenerator<T>
{
String getXml(T value);
}
Y la implementación concreta XmlGeneratorImpl:
package generics;
public class XmlGeneratorImpl<T> implements XmlGenerator<T>
{
private Class<T> valueType;
private static final int DEFAULT_CAPACITY = 1024;
public static void main(String [] args)
{
Integer x = 42;
String y = "foobar";
XmlGenerator<Integer> intXmlGenerator = new XmlGeneratorImpl<Integer>(Integer.class);
XmlGenerator<String> stringXmlGenerator = new XmlGeneratorImpl<String>(String.class);
System.out.println("integer: " + intXmlGenerator.getXml(x));
System.out.println("string : " + stringXmlGenerator.getXml(y));
}
public XmlGeneratorImpl(Class<T> clazz)
{
this.valueType = clazz;
}
public String getXml(T value)
{
StringBuilder builder = new StringBuilder(DEFAULT_CAPACITY);
appendTag(builder);
builder.append(value);
appendTag(builder, false);
return builder.toString();
}
private void appendTag(StringBuilder builder) { this.appendTag(builder, false); }
private void appendTag(StringBuilder builder, boolean isClosing)
{
String valueTypeName = valueType.getName();
builder.append("<").append(valueTypeName);
if (isClosing)
{
builder.append("/");
}
builder.append(">");
}
}
Si ejecuto esto, obtengo el siguiente resultado:
integer: <java.lang.Integer>42<java.lang.Integer>
string : <java.lang.String>foobar<java.lang.String>
No sé si esto es lo que tenías en mente.