Ah, mas esse é um código feio e não orientado a objetos. No momento em que você vê "if / else" e "typeof", você deve estar pensando em polimorfismo. Este é o caminho errado a seguir. Acho que os genéricos são seus amigos aqui.
Com quantos tipos você planeja lidar?
ATUALIZAR:
Se você está falando apenas sobre String e int, aqui está uma maneira de fazer isso. Comece com a interface XmlGenerator (basta com "foo"):
package generics;
public interface XmlGenerator<T>
{
String getXml(T value);
}
E a implementação 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(">");
}
}
Se eu executar isso, obtenho o seguinte resultado:
integer: <java.lang.Integer>42<java.lang.Integer>
string : <java.lang.String>foobar<java.lang.String>
Não sei se é isso que você tinha em mente.