Всем Добрый день. Имеется класс Catalog, у него одно поле - массив телефонов, у каждого телефона есть несколько полей: марка, цвет, цена. Мне нужно клонировать каталог. Я имплементирую интерфейс Cloneable и переопределяю метод clone:
 @Override
 protected Object clone() throws CloneNotSupportedException {
   UniversalCatalog uc = (UniversalCatalog) super.clone();
   for (int i = 0; i < this.phones.length; i++) {
     uc.phones[i] = (Phone) this.phones[i].clone();
   }
   return uc;
 }
Но если я клонирую каталог и потом меняю например у одного телефона цвет, то он меняется и в другом каталоге. Что я не так делаю?

ребят, как вызвать окошкоbranches в идее? там контрол+шифт и чето еще непонятное, вроде ' , но чето так не вызывается

1) Использовать clone() и Cloneable - не очень хороший тон по сути, в реальной жизни этого никто не делает (проще создать конструктор, принимающий объект такого же типа)
2) Выглядит так, как будто ты ещё и не переопределил clone() в классе Phone

переопределил. Клонировать телефоны получается. Если в каталоге удаляешь один телефон, второй каталог не меняется. Но вот если менять какое-то поле в телефоне в одном каталоге, то меняется и в другом

Видимо, плохо переопределил.

1) Перестать использовать clone()
2) Исправить код clone() в телефонах, как - не знаю, потому что я его не видел.