ObjectFactory с аргументом класса <?>

Question

ObjectFactory с аргументом класса <?>

1

Я разрабатываю фабрику, которая может создавать объекты типа Ability с помощью параметра Class. Код для этой фабрики:

public static final class AbilityFactory {
    private AbilityFactory(){}

    public static <T extends Class<A>, A extends Ability> A getAbility(T clazz){
        A ability = null;
        try{
            ability = clazz.newInstance();
        } catch(InstantiationException | IllegalAccessException ex){
            ex.printStackTrace();
        }
        return ability;
    }
}

Код клиента выглядит примерно так:

Ability.AbilityFactory.getAbility(Firebolt.class));

Единственная проблема, которую я подумал об этом дизайне, заключается в том, что класс Ability является абстрактным, и вы можете передать что-то вроде этого:

Ability.AbilityFactory.getAbility(Ability.class));

что приводит к ошибке java.lang.InstantiationException.

Я хотел знать, могу ли я сделать некоторые ограничения, такие как "Расширяет способность, но не способность", или это просто плохой дизайн, и я должен передать, например, Enum в качестве аргумента в заводский метод для предотвращения этой ситуации.

Zarial 05 фев. 2015, в 17:15

Источник

1

Я не понимаю, почему это может быть проблемой, просто не передавайте Ability.class в качестве параметра.
FINDarkside 05 фев. 2015, в 15:53
1

Если все ваши подклассы Ability имеют конструктор по умолчанию, и вам нужен объект Class чтобы создать его экземпляр с помощью этой фабрики, тогда какой смысл? Почему бы просто не использовать новый оператор и избегать рефлексии?
Balázs Édes 05 фев. 2015, в 15:53
0

@FINDarkside хорошо, тогда все назначение дженериков исчезает. Вы не можете определить неправильный аргумент во время компиляции.
Zarial 05 фев. 2015, в 15:58
0

@ bali182 потому что вместо Чемпиона <Azae> azae = new Azae (новый Firebolt (), новый Rend ()); Я хочу Champion <Azae> azae = new Azae (Firebolt.class, Rend.class);
Zarial 05 фев. 2015, в 16:03
0

Тогда, возможно, вы захотите взглянуть на некоторые структуры внедрения зависимостей, такие как Guice или Dagger . Они делают это гораздо более надежным и зрелым способом, чем то, что вы или я можем собрать за несколько часов;)
Balázs Édes 05 фев. 2015, в 18:14
0

@ bali182 Спасибо, но я делаю это с целью изучения)
Zarial 05 фев. 2015, в 21:35

Показать ещё 4 комментария

Теги:

java

class

generics

1 ответ

Ещё вопросы

Я не понимаю, почему это может быть проблемой, просто не передавайте Ability.class в качестве параметра.
Если все ваши подклассы Ability имеют конструктор по умолчанию, и вам нужен объект Class чтобы создать его экземпляр с помощью этой фабрики, тогда какой смысл? Почему бы просто не использовать новый оператор и избегать рефлексии?
@FINDarkside хорошо, тогда все назначение дженериков исчезает. Вы не можете определить неправильный аргумент во время компиляции.
@ bali182 потому что вместо Чемпиона <Azae> azae = new Azae (новый Firebolt (), новый Rend ()); Я хочу Champion <Azae> azae = new Azae (Firebolt.class, Rend.class);
Тогда, возможно, вы захотите взглянуть на некоторые структуры внедрения зависимостей, такие как Guice или Dagger . Они делают это гораздо более надежным и зрелым способом, чем то, что вы или я можем собрать за несколько часов;)
@ bali182 Спасибо, но я делаю это с целью изучения)

llogiq · Accepted Answer · 2015-02-05T14-23-00.000Z

2

Лучший ответ

Единственное, что вы могли бы сделать, это предоставить интерфейс маркера, допустим, ConcreteAbility и <T extends Ability & ConcreteAbility>. Затем вам нужно реализовать ConcreteAbility со всеми конкретными возможностями.

llogiq 05 фев. 2015, в 14:23

0

Кажется, это хороший совет, но теперь я не могу передать аргумент в конструкторе моего чемпиона. До того, как он стал публичным Чемпионом (Class <? Extends Ability <C >> firstAbility) {...} и после того, как я изменил свою фабрику, следуя вашему совету: public static <T расширяет Class <A>, A расширяет Ability & ConcreteAbility> A getAbility (T clazz) {...} Мне нужно в моем конструкторе что-то вроде Class <? расширяет Ability <C> и ConcreteAbility> , но это не вариант из-за неправильного синтаксиса. Можете ли вы посоветовать мне обходной путь?
Zarial 05 фев. 2015, в 16:20
1

Ну, это действительно помогло мне. Я забыл, что мы можем использовать параметры именованного типа в конструкторах тоже. Спасибо!) Теперь код для моего конструктора: public <T расширяет класс <A>, A расширяет Ability & ConcreteAbility> Champion (T firstAbility, T secondAbility) {...}
Zarial 05 фев. 2015, в 16:31
1

Btw. вам не нужен T, если у вас действительно нет собственной реализации Class - что, учитывая, что java.lang.Class является окончательным, крайне маловероятно. Таким образом, ваш конструктор может быть упрощен до public <A extends Ability & ConcreteAbility> Champion(Class<A> firstAbility…)
llogiq 05 фев. 2015, в 16:49
0

@IIogiq Да, большое спасибо! Это упрощает все!
Zarial 05 фев. 2015, в 16:56

Показать ещё 2 комментария