0

У меня есть в базе данных 2 таблицы, новости и история. История само собой выводится на главной странице.
Так вот у меня есть форма, я отправляю post запрос, и у меня должно добавлять одновременно в две базы данных, в историю и новости.
Как это реализовать более грамотно? Чтобы к примеру не писать вот так в одном контроллере:

public function addNews()
{
    $history = new History;
            $history->admin_id = Auth::user()->id;
            $history->title = $request->title;
            $history->action = "Добавил новость";
            $history->url_address = "news/create";

    $addNews = new News;

       $addNews->author_id = Auth::user()->id;
            $addNews->title = $request->title;
            $addNews->description = $request->description;

...

То по моему мнению так не грамотно писать будет, так как не соответствует solid принципам.

1
  • 1
    Как по мне все норм, куда тут еще упрощать. Смотришь на код и все быстро понятно, это самое главное
    – Paul Wall
    15 июл 2018 в 3:46

1 ответ 1

1

1.Сразу предлагаю Вам вынести валидацию в отдельный класс, поскольку здесь Вы получаете все данные из $request, который сейчас даже нигде не описан. Примерно так:

public function addNews(StoreNewsRequest $request)
{
...
}

2.Историю лучше вести с привязкой через полиморфические связи. Таким образом, в БД всегда будет храниться история взаимодействий с данной новостью, а Вы сможете легко посмотреть, кто и что делал с этой новостью на протяжении всего времени. Больше про полиморфические связи в Eloquent здесь.

3.Писать в коде прямым текстом вот так: $history->action = "Добавил новость"; - зло. Лучше это вынести в константу модели History:

class History 
{
   const USER_ADDED_NEWS = "Добавил новость";
}    
...
$history->action = History::USER_ADDED_NEWS;

В заключении посоветую почитать эти best practices, явно много нового и полезного для себя найдете :)

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.