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

Question

[verygoodboy]

Есть 2 однометодных контроллера с похожим кодом:
NewPostUpdate и OldPostUpdate. Эти контроллеры работают с одной моделью Post. Эти контроллеры принимают параметром $request и $id.

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

public function __invoke(Request $request, $id)
{
$data = $request->all();
$newPost = Post::where('id', $id)->firstOrFail();

if($data['date_time'] > now()){
 // обновляем модель
// создаем связанную модель для названия файла
// сохраняем полученный файл
} elseif ($data['date_time'] < now()){
 // обновляем модель
// создаем связанную модель для названия файла
// сохраняем полученный файл
} else {
 // обновляем модель
// создаем связанную модель для названия файла
// сохраняем полученный файл
}
return redirect()->back()->with('success', 'Saved');
}

Фактически этот код дублируется в обеих контроллерах.

Подскажите, пожалуйста, можно ли как то вынести этот код за пределы этих контроллеров, передать параметром туда провалидированные из request данные и $id Post, сохранить там данные и вернуть обратно ответ?

Если можно, то куда правильнее было бы перенести этот код (модель Post, или action...), и каким должен быть ответ выполнения этой функции (как его обработать в контроллере)?

Comments

[Дмитрий]

Черт опять __invoke. Зачем?

[Сергей delphinpro]

Если оба контроллера (или метода) обновляют одну и ту же модель, да еще и код у них одинаковый, то возникает закономерный вопрос: А нафига здесь вообще два контроллера (метода)?

[verygoodboy]

Сергей delphinpro, код не совсем одинаковый, но есть кусок одинакового кода. Я вынес этот код в action.

[Сергей delphinpro]

verygoodboy, фигня. Одна модель – один метод обновления.

[jazzus]

verygoodboy, а если логика поменяется контроллеры будешь переименовывать или новые добавлять? OldActivePostWithImageUpdateController))

Answer 1

[Tony V]

Вариантов много:
Можно сделать базовый контроллер с кодом, а в этих двух его наследовать
Можно сделать отдельный Action и запускать его из контроллеров
Подробнее можно посмотреть тут, всё достаточно хорошо описано

Comments

[verygoodboy]

Спасибо, смотрел это видео. Попробую перенести в action

Answer 2

[jazzus]

Можно без этих извращений с контроллерами. Один ресурсный PostController с методом update, где первым параметром реквест, вторым Post.