Implementiere ich eine auf Generika basierende Java-Fabrik richtig?

Question

Ich glaube nicht, dass ich das Fabrikmuster richtig implementiere, weil die Application Klasse' createDocument Methode akzeptiert jeden Klassentyp, nicht nur Unterklassen von Document .

Mit anderen Worten, gibt es eine Möglichkeit, die createDocument Methode, um nur Unterklassen von Document ?

• Dokument.java

  package com.example.factory;
  
  public abstract class Document {
      public Document() {
          System.out.println("New Document instance created: " + this.toString());
      }
  }

• DrawingDocument.java

  package com.example.factory
  
  public class DrawingDocument extends Document {
      public DrawingDocument() {
          System.out.println("New DrawingDocument instance created: " this.toString());
      }
  }

• Anwendung.java

  package com.example.factory;
  
  public class Application {
      public <T> T createDocument(Class<T> documentClass) {
          try {
              return documentClass.newInstance();
          } catch (InstantiationException e) {
              throw new IllegalArgumentException(e);
          } catch (IllegalAccessException e) {
              throw new IllegalArgumentException(e);
          }
      };
  }

• Main.java

  package com.example.factory;
  
  public static void main(String[] args) {
      Application application = new Application();
      application.createDocument(DrawingDocument.class);
  }

Answer 1

Sie sollten gebunden Ihre Generika so, dass sie nur T's verwenden, die Document erben. Beispiel:

public class Application {
    //Add extends Document after T
    public static <T extends Document> T createDocument(Class<T> documentClass) throws InstantiationException, IllegalAccessException {
        return documentClass.newInstance();
    };
}

Answer 2

Der Code sieht gut aus. In einer echten Implementierung sollte die Factory-Methode nicht so deklariert werden, dass sie eine der Reflection-bezogenen Ausnahmen auslöst. Und Sie werden wahrscheinlich sowieso einen anderen Code haben, um das Dokument zu erstellen.

Die Methode faxtory sollte eine Class<? extends Document> als Parameter, so dass man ihn nicht auffordern kann, eine String zum Beispiel.

[update:] Codebeispiel:

public Document createDocument(Class<? extends Document> clazz) {
  try {
    return clazz.newInstance();
  } catch (InstantiationException e) {
    throw new IllegalArgumentException(e);
  }
}

Answer 3

Wo liegt die Beschränkung auf den Dokumententyp in der Fabrik? Versuchen Sie

public <T extends Document> T createDocument(Class<T> documentClass) throws InstantiationException, IllegalAccessException {
    return documentClass.newInstance();
};