Как соединить две модели в коллекции?

Question

[AlexandrMa]

Есть две модели Компания и Контакт. Ранее я в Компании хранил массив Контактов. Но тк это две независимые модели, то решил удалить метод addContact из Компании. Вынести его в класс коллекции. Но тут сложности возникли.

class Company{

  public int $id;
  public string $name;
  //public array $contacts;

  public function __construct(int $id, string $name)
  {
    $this->id = $id;
    $this->name = $name;
  }

  /*public function addContact(Contact $contact){
    $this->contacts[] = $contact;
  }*/

}


class Contact{

  public int $id;
  public string $name;

  public function __construct(int $id, string $name)
  {
    $this->id = $id;
    $this->name = $name;
  }

}


class CompanyCollection
{
  public array $data;

  public function getData():array
  {
    $this->merge();
    return $this->data;
  }

  public function merge()
  {
    for($i = 0; $i <= 5; $i++){

      $companyId = $i+1;

      $this->addCompany(new Company($companyId, 'CompanyName'));

      for($j = 0; $j < 3; $j++){
        $contactId = $j + 1;
        $this->addContact(new Contact($contactId, 'ContactName'), $i);
      }

    }
  }

  public function addCompany(Company $company){
    $this->data[] = $company;
  }

  public function addContact(Contact $contact, int $position){
    $this->data[$position][] = $contact;
  }

}

В классе коллекции получилась какая-то чушь. Как правильно соединить?

Answer 1

[Дмитрий Гординский]

for($i = 0; $i <= 5; $i++){
Почему пять, почему не десять?
for($j = 0; $j < 3; $j++){
А почему не пять?

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

получилась какая-то чушь

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

Comments

[AlexandrMa]

foreach это чисто для примера я сделал. 5 компаний, по 3 контакта. Это просто пример.
Меня волнует именно реализация закомментированного метода, который в классе Comany

Answer 2

[Григорий Васильков]

Коллекция это про пачку.

Создав класс коллекции компаний и запихнув туда контакты ты сказал примерно так "несколько компаний имеют общие контакты в рамках этой пачки компаний".

Иногда это имеет смысл, но это такие редкие случаи, что так не надо делать. "независимость" модели это про другое, это про то что контакты являются частью одной задачи из ТЗ, а компании - другой. Но это не мешает одним иметь другое в подчинении, если в тз написано что компания имеет контакты.

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

Comments

[AlexandrMa]

У меня просто два класса. И хотелось бы иметь третий класс, которые бы выдавал массив объектов компаний, каждая из которых содержала бы в себе массив контактов. В классе Company был метод закомментированный. И я просто решил вынести его в отдельный класс, который соединял бы как раз фирмы и контакты.

[Григорий Васильков]

В вашем случае получилось смешать сервис, собирающий информацию, и объект, содержащий информацию, и получилось нечто среднее не делающее не то и не другое.

Сделайте класс, который умеет наполнять компании контактами и верните из функции массив компаний, каждая из которых наполнена контактами. Контакты в коллекции компаний дословно означают именно то, что контакты общие для всего объекта, а не то, что это каждая компания обладает ими, т.к. здесь неясно разделение какие контакты к какой компании

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

[AlexandrMa]

Григорий Васильков,

Сделайте класс, который умеет наполнять ...

Круто, спасибо. Подумаю над классом расширения.