Правильно ли я реализую основанную на обобщениях фабрику Java? - PullRequest
Новая
       20

Правильно ли я реализую основанную на обобщениях фабрику Java?

14 голосов
/ 31 мая 2011

Я не верю, что правильно реализую шаблон фабрики, потому что метод 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);
}

Ответы [ 3 ]

18 голосов
/ 31 мая 2011

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

public class Application {
    //Add extends Document after T
    public static <T extends Document> T createDocument(Class<T> documentClass) throws InstantiationException, IllegalAccessException {
        return documentClass.newInstance();
    };
}
6 голосов
/ 31 мая 2011

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

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

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

public Document createDocument(Class<? extends Document> clazz) {
  try {
    return clazz.newInstance();
  } catch (InstantiationException e) {
    throw new IllegalArgumentException(e);
  }
}
3 голосов
/ 31 мая 2011

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

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