1

Доброго времени суток. Имеется такой метод добавления микроблога в контроллере:

/**
 * Написание микроблога
 * @param Request $request
 * @return \Illuminate\Routing\Redirector|\Illuminate\View\View
 */
public function writeMicroblog(Request $request) {
    $model = new Microblog();
    $user = new User();

    $title = 'Написать микроблог';

    if ($request->has('write'))
    {
        $this->validate($request, [
            'title' => 'sometimes|min:3|max:50',
            'text' => 'required|min:3|max:5000',
        ]);

        $model->writeMicroblog($request);
        $user->addValues(Auth::user()->id, 5, Config::get('ratings.mb.new')); // Прибавка баллов и рейтинга
        return redirect('/microblogs/' . DB::getPdo()->lastInsertId());
    }

    return view('microblogs/write', ['title' => $title]);
}

И, соответственно модель.

class Microblog extends Model {
    protected $table = 'microblogs';
    protected $dates = ['created_at', 'updated_at'];

    /**
     * Добавление микроблога
     * @param Request $request
     */
    public function writeMicroblog(Request $request, $type = 'post')
    {
        $microblog = new Microblog();

        $microblog->author_id = Auth::user()->id;
        $microblog->title = $request->get('title');
        $microblog->text = $request->get('text');
        $microblog->type = $type;

        $microblog->save();
    }
}

Недавно узнал, что так делать вообще нельзя, т.е. Request в модели вообще не нужен, хотя я всё делал согласно документации. Если это действительно так, как можно улучшить код? Я так понимаю, в контроллере нужно вызывать Microblog::create(), но как тогда данные из Request передать в этот самый create?

Заранее спасибо.

2 ответа 2

1

Посмотрите что написано о вставке данных в базу в документации Laravel

У вас простой случай и все можно сделать в контроллере:

public function writeMicroblog(Request $request)
{
    $user = new User();

    $title = 'Написать микроблог';

    if ($request->has('write')) {
        $this->validate($request, [
            'title' => 'sometimes|min:3|max:50',
            'text' => 'required|min:3|max:5000',
        ]);

        $microblog = Microblog::create([
            'author_id' => Auth::user()->id,
            'title' => $request->get('title'),
            'text' => $request->get('text'),
            'type' => 'post'

        ]);
        $microblog->save();

        $user->addValues(Auth::user()->id, 5, Config::get('ratings.mb.new')); // Прибавка баллов и рейтинга
        return redirect('/microblogs/' . $microblog->id);
    }

    return view('microblogs/write', ['title' => $title]);
}

так же можно вынести валидацию в Form Request.

8
  • По хорошему могли бы написать - вынести всю магию в переменные.
    – borodatych
    8 июл 2016 в 14:10
  • смысл в том чтобы показать как надо работать с моделью, а по хорошему я бы и валидацию по другому сделал. Не вижу смысла плодить бесполезные переменные 8 июл 2016 в 14:40
  • Видите, а я думаю по другому - если взялся переделывать и показывать как надо, то уж покажи до конца, как говорится. Переменный, ладно, но валидацию то уж подскажите. И меня чему-нибудь научите.
    – borodatych
    8 июл 2016 в 14:47
  • валидация в примере уместна, но при чтобы не писать валидаторы в контроллерах можно использовать [Form Request Validation] (laravel.com/docs/5.2/validation#form-request-validation) 8 июл 2016 в 14:58
  • Спасибо за ссылку, добавьте это в ответ, что бы я смог сменить голос.
    – borodatych
    8 июл 2016 в 15:00
-1

Согласно документации и код контроллера можно прямо в роутере написать.
Вот будет чуть по эстетичнее.

Контроллер:

/**
 * Написание микроблога
 * @param Request $request
 * @return \Illuminate\Routing\Redirector|\Illuminate\View\View
 */
public function writeMicroblog(Request $request) {
    $model = new Microblog();
    $user = new User();

    $title = 'Написать микроблог';

    if ($request->has('write'))
    {
        $data = $request->all();
        $this->validate($request, [
            'title' => 'sometimes|min:3|max:50',
            'text' => 'required|min:3|max:5000',
        ]);

        $model->writeMicroblog($data);
        $user->addValues(Auth::user()->id, 5, Config::get('ratings.mb.new')); // Прибавка баллов и рейтинга
        return redirect('/microblogs/' . DB::getPdo()->lastInsertId());
    }

    return view('microblogs/write', ['title' => $title]);
} 

Модель:

class Microblog extends Model {

    protected $table = 'microblogs';
    protected $dates = [ 'created_at', 'updated_at' ];

    /**
     * Добавление микроблога
     * @param Request $request
     */
    public function writeMicroblog( $data, $type = 'post' ) {

        $microblog = new Microblog();

        $microblog->author_id = Auth::user()->id;
        $microblog->title     = $data->get( 'title' );
        $microblog->text      = $data->get( 'text' );
        $microblog->type      = $type;

        $microblog->save();
    }
}

В этом месте: $data = $request->all();, только синтаксис уточните...

5
  • В модели создавать новый объект этой же модели который сам себя сохранит? это же Active Record, не надо так извращаться 8 июл 2016 в 14:04
  • Вы неверное не подумали, что я не знаком с фреймворком и мог не догадаться про Microblog::create. Ответил конкретно на вопрос ТС. Не правильно?
    – borodatych
    8 июл 2016 в 14:15
  • смысл не в Microblog::create, можно сделать сохранение как у вас, но только не в модели, а в контроллере 8 июл 2016 в 14:35
  • 1
    Вы хотите сделать ТТУК? По мне дак и так нормально. Можно было конечно и так: $microblog = new static();, но у ТС вопрос был в другом.
    – borodatych
    8 июл 2016 в 14:44
  • В простом сохранении данных нет логики, поэтому нет необходимости выделять для этого отдельные методы в модели и в контроллере такой код уместен. Если же у вас много логики в контроллере - нужно выделить еще один слой в приложении, например в виде сервисов. 8 июл 2016 в 14:50

Ваш ответ

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

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