требования

читая ваш вопрос я нашел следующие требования зоопарк:

1. поиск животных по их названию вида
2. определить порядок видов
3. легко добавлять животных в будущем

1. Поиск животных по их названию вида

как вы упомянули, существует нежелательное разделение между строкой "LION" и класс Lion. В эффективной Java, пункт 50, следующей строки:

строки являются плохими заменителями других типов значений. Когда часть данных поступает в программу из файла, из сети или с клавиатуры, она часто находится в Строковой форме. Существует естественная тенденция оставлять это так, но эта тенденция оправдана только в том случае, если данные действительно являются текстовыми по своей природе. Если он числовой, его следует перевести в соответствующий числовой тип, например int, float, или BigInteger. Если это ответ на вопрос " да " или "нет", его следует перевести в boolean. В более общем плане, если есть соответствующий тип значения, будь то примитив или ссылка на объект, вы должны использовать его; если нет, вы должны написать его. Хотя этот совет может показаться очевидным, он часто нарушается.

поэтому, вместо того, чтобы искать животных по их видам имя вы должны искать животных по их видам экземпляр.

2. Определение вида

чтобы определить порядок видов, вам нужно использовать коллекцию с предсказуемым порядком итераций, например, упомянутый LinkedHashMap.

3. Легко добавлять животных в будущем

добавление животных в настоящее время состоит из трех шагов, которые вы упомянули. Побочным эффектом использования перечисления является то, что только человек с доступом к исходному коду имеет возможность добавлять новые виды (как the Species перечисление должно быть расширено).

Теперь рассмотрим, Species.LION, это вид Lion класса. Обратите внимание, что это отношение семантически такое же, как отношение между классом и его экземпляров. Поэтому гораздо более элегантным решением было бы использовать Lion.class как вида Lion. Это также уменьшает количество шагов для добавления животного, поскольку вы получаете вид бесплатно.

анализ других частей код

зоопарк

в вашем предложении, зоопарк несет ответственность за создание животных. Следствием этого является то, что каждый зоопарк когда-либо созданный должен использовать все определенные животные (из-за Species перечисление) и используйте указанный заказ, не будет никаких изменений среди зоопарка. Лучше отделить создание животных от зоопарка, чтобы обеспечить большую гибкость как животных, так и зоопарк.

животные

из-за их гибкости интерфейсы должны быть предпочтительнее абстрактных классов. Как объясняется в эффективной Java, пункт 18:

пункт 18. Предпочитайте интерфейсы абстрактным классам

язык программирования Java предоставляет два механизма для определения типа, который позволяет использовать несколько реализаций: интерфейсы и абстрактные классы. Наиболее очевидным различием между этими двумя механизмами является что абстрактные классы могут содержать реализации для некоторых методов, а интерфейсы-нет. Более важным отличием является то, что для реализации типа, определенного абстрактным классом, класс должен быть подклассом абстрактного класса. Любой класс, который определяет все необходимые методы и подчиняется общему контракту, может реализовать интерфейс, независимо от того, где класс находится в иерархии классов. Поскольку Java разрешает только одно наследование, это ограничение на абстрактные классы сильно ограничивают их использование в качестве определений типов.

циклические ссылки

в вашем вопросе вы упоминаете, что животное также должно иметь ссылку на зоопарк, в котором оно находится. Это вводит круговую ссылку, которой следует избегать как можно больше.

одним из многих недостатков круговых ссылок является:

круговые ссылки класса создают высокое соединение; оба занятия должны быть перекомпилированы каждый раз, когда любой из них меняется.

пример реализации

public interface Animal {}

public class Zoo {
  private final SetMultimap<Class<? extends Animal>, Animal> animals;

  public Zoo() {
    animals = LinkedHashMultimap.create();
  }

  public void addAnimal(Animal animal) {
    animals.put(animal.getClass(), animal);
  }

  @SuppressWarnings("unchecked") // the cast is safe
  public <T extends Animal> Set<T> getAnimals(Class<T> species) {
    return (Set<T>) animals.get(species);
  }
}

использование

static class Lion implements Animal {}

static class Zebra implements Animal {}

final Zoo zoo = new Zoo();
zoo.addAnimal(new Lion());
zoo.addAnimal(new Zebra());
zoo.addAnimal(new Lion());

zoo.getAnimals(Lion.class); // returns two lion instances

zoo.getSpeciesOrdering(); // returns [Lion.class, Zebra.class]

Обсуждение

реализация выше имеет поддержку нескольких экземпляров животных для каждого вида, так как это, кажется, имеет больше смысла для зоопарка. Если требуется только одно животное, подумайте об использовании гуава по ClassToInstanceMap вместо SetMultimap.

создание животных не рассматривался как часть проблемы проектирования. Если нужно построить более сложных животных, подумайте об использовании шаблон построителя.

добавление животного теперь так же просто, как создание нового класса, который реализует Animal интерфейс и добавление его в зоопарк.