Я правильно ли реализую основанную на генериках Java factory?

Я не верю, что правильно внедряю шаблон factory, потому что метод Application class 'createDocument принимает любой тип класса, а не только подклассы Document.

Другими словами, есть ли способ, которым я могу ограничить метод createDocument, принимать только подклассы Document?

Document.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());
    }
}

Application.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);
}

Ответ 1

Вы должны bound ваш общий тип, чтобы использовать только T, наследующий документ. Пример:

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

Ответ 2

Код выглядит хорошо. В реальной реализации метод factory не должен объявляться, чтобы бросать любое из связанных с отражением исключений. И вы, вероятно, будете иметь какой-то другой код в любом случае для создания документа.

Метод faxtory должен принимать Class<? extends Document> в качестве своего параметра, так что нельзя попросить его создать String, например.

[обновление:] Пример кода:

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

Ответ 3

Где ограничение на тип документа в factory? Попробуйте

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