Как соединить работу двух методов?

Question

[Max Ba]

Есть класс Customer
Метод create() создает запись о клиенте
Метод createContact() создает запись в таблице контактов. Контактов может быть несколько у одного клиента.

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

Единственное что пришло в голову, сделать так:

if(isset($_POST['create'])){
  try{
    $last_id = $customer->create($_POST); //метод возвращает новый номер записи
    if(!empty($last_id)){
      $customer->createContact($last_id, $_POST); //передаем его в метод добавления контакта
    }
  }catch(AppException $e){
		
  }
}

Может быть есть иные способы?

PS. пробовал также добавлять createContact внутрь create, внутри класса

Comments

[Евгений]

Вызов create добавляли внутри класаа или как? Не совсем понял...

[Max Ba]

Евгений, да, внутри create добавлял createContact. Не уверен, что это было правильно.

[Евгений]

Max Ba, надо было попробывать вызвать метод...

[Max Ba]

Евгений, это работало. Я не уверен, что такая логика должна быть зашита в метод)
В контроллере хотя бы видно, что при создании основной записи, данные о контакте добавляются тоже.
В общем я понял) Нормально было сделать. Внутри метода)))

Answer 1

[FanatPHP]

1. В контроллере ничего подобного в принципе никогда быть не должно. Ни двух методов, ни одного. Никакой работы с БД в контроллере в принципе быть не может. Контроллер должен только принять данный из формы и вызвать метод модели. Только не убогого класса для маппинга таблицы из БД в класс РНР, который называют моделью новички, а нормальной модели, реелизующей всю бизнес-логику приложения.
2. По-хорошему, это код для репозитория, поскольку он работает с базой данных. Но конкретная реализация зависит от того как реализован класс customer

В идеале цепочка вызовов должна быть такая
Контроллер вызывает инстанс хелпера (или сервиса) CustomerHelper.
В этот хелпер через конструктор передается инстанс CustomerRepository и присваивается свойству класса.
В методе Create класса CustomerHelper вызывается метод Create класса CustomerRepository.
И в этом методе уже и происходит искомая транзакция.

Comments

[Max Ba]

У нас огромная разница в знаниях))
Спасибо за информацию. Не уверен, что разберусь, но по кускам думаю что-то да сделаю.

PS
вот так было ранее

class Customer {

  private $db;

  public function __construct(DB $db){
      $this->db = $db;
  }

  public function create(array $data){

      //проверки $data
        
      $this->db->insert('...');  //return bool

      //способ 1
      return $this->db->last_insert_id(); //return int
        
      //способ 2
      $last_id = $this->db->last_insert_id();
        
      if(!empty($last_id)){
          $this->contactAdd($id, $data);
      }
  }

  public function contactAdd($id, array $data){

      //проверки $data

      return $this->db->insert('...'); //return bool
  }

}

[Max Ba]

Кстати, у меня и не было работы с БД в контроллере)))

[FanatPHP]

Выложены слайды Дмитрия Елисеева с PHPRussia, https://drive.google.com/file/d/1rLyfjYvsnmkmeu99V...

Без сопровождения рассказчика очень тяжелдо, конечно, но рекомендую посмотреть все-таки.
Начиная с 79 слайда
Он рассказывает, что фреймворк - это тонкая оболочка, ее можно менять как перчатки, если бизнес-логика выведена из контроллеров. И объясняет, почему.

Answer 2

[kafkiansky]

Если добавляете сразу в 2 таблицы, то оберните в транзакцию и тогда вызываете один метод надежно

Comments

[Max Ba]

Транзакцию то сделать не вопрос) Тут именно в логике взаимодействия двух методов. В комменте выше обсудили, что метод createContact добавить в метод create. и тогда при вызове одного метода, создаются сразу все необходимые.

[kafkiansky]

Max Ba, так если транзакция, то конечно в одном методе, а так как работе с базой данных в контроллере не место, то в контроллере вы должны вызвать уже метод create, в котором вызывается транзакция, ну и так далее.

[Max Ba]

mad_maximus, так а где у меня работа с БД в контроллере? Мб я не понимаю что-то....

[kafkiansky]

Max Ba, вы пишете, что не знаете, соединять ли create и createContact, и при этом знаете, что у вас транзакция. Ну так где ещё вы оформите транзакцию, если не в модели? В таком случае контроллер уже вызовет единственный метод, который все разрешит сам