Verwendung Methode des generischen Typs

Question

Ich schreibe gerade einen XML-Konverter für ein Supply-Chain-Projekt. Wir verwenden Anfragen und Bestellungen.

Der Konverter hat mehrere Methoden, die zur Zeit dasselbe tun, aber separat für Anfragen und Bestellungen implementiert werden.

ich geschaffen habe daher eine abstrakte Klasse Wartbarkeit des Codes zu verbessern und einen generischen Typen verwendet:

public abstract class AbstractConverter<T extends BusinessObject> 

Dann habe ich die spezifischen Implementierungen für den eigentlichen Wandler

public class OrderConverter extends AbstractConverter<Order> 
public class RequestConverter extends AbstractConverter<Request> 

Wie gesagt , Ich habe mehrere Methoden in den zwei spezifischen Klassen, die im Grunde dasselbe tun, also möchte ich sie natürlich in der abstrakten Klasse haben. Ich habe hinzugefügt nun die folgende Methode die abstrakte Klasse:

protected Comment createComment(T obj) { 
    String remark; 
    if (obj instanceof Order) { 
     remark = ((Order) obj).getRemark(); 
    } else if (obj instanceof Request) { 
     remark = ((Request) obj).getRequestRemark(); 
    } else { 
     throw new IllegalArgumentException("This method does not support objects of the type " + obj.getClass().toString()); 
    } 
    return new Comment(remark); 
} 

Meine Frage ist nun: ist dies der Weg zu gehen, oder gibt es eine elegantere Möglichkeit Generika in diesem Zusammenhang zu benutzen?

Ich brauche das gelöst, aber ich möchte auch guten Stil verwenden.

Answer 1

Die natürliche objektorientierte Lösung ist createComment eine abstrakte Methode

protected abstract Comment createComment(T obj); 

und lassen Sie die Subklassen es implementieren zu machen:

public class OrderConverter extends AbstractConverter<Order> { 
    protected Comment createComment(Order order) { 
      return new Comment(order.getRemark()); 
    } 
} 

public class RequestConverter extends AbstractConverter<Request> { 
    protected Comment createComment(Request request) { 
      return new Comment(request.getRequestRemark()); 
    } 
} 

Answer 2

Ich würde vorschlagen, die getRemark Methode zu einer Schnittstelle, die sowohl Request und Order implementiert.

Auf diese Weise können Sie einfach überprüfen, ob das eingehende generische Objekt eine Instanz der Schnittstelle ist.

protected Comment createComment(T obj) { 
    if (obj instanceof Remarkable) { 
     return new Comment(((Remarkable) obj).getRemark()); 
    } 
    throw new IllegalArgumentException("This method does not support objects of the type " + obj.getClass().toString()); 
}