169 похожих чатов

Всем привет, есть два метода, оба рабочие, какой подход будет

считаться более профессиональным? Стоит ли так усложнять всё в коде или простота и читаемость являются более правильными. Тут Создание обычного поста. Первый код: public function store(Request $request)
{
$request->validate([
'title' => 'required',
'body' => 'required',
'file' => 'nullable|file|mimes:pdf,doc,docx|max:2048',
'image' => 'nullable|image|mimes:jpeg,png,jpg,gif|max:2048',
]);


$post = new Post([
'title' => $request->input('title'),
'body' => $request->input('body'),
'user_id' => auth()->user()->id,
]);

if ($request->hasFile('file')) {
$file = $request->file('file');
$fileName = time() . '_' . $file->getClientOriginalName();
$file->move(public_path('uploads'), $fileName);
$post->file = $fileName;
}
if ($request->hasFile('image')) {
$image = $request->file('image');
$imageName = time() . '_' . $image->getClientOriginalName();
$image->storeAs('public/images', $imageName);
$post->image = 'storage/images/' . $imageName;
}

$post->save();

return redirect()->route('posts.index')->with('success', 'Пост успешно создан');

}
второй код: public function store(StoreRequest $request)
{
Post::create($request->validated());

return redirect()->route('posts.index')->with('success', 'Пост успешно создан.');
}

1 ответов

19 просмотров

В общем - валидация - да, должна быть в отдельном месте, но в ней не должно быть сохранения файла, что я вижу со второго варианта

Похожие вопросы

Обсуждают сегодня

а через ESC-код ?
Alexey Kulakov
29
30500 за редактор? )
Владимир
47
Чёт не понял, я ж правильной функцией воспользовался чтобы вывести отладочную информацию? но что-то она не ловится
notme
18
У меня есть функция где происходит это: write_bit(buffer, 1); write_bit(buffer, 0); write_bit(buffer, 1); write_bit(buffer, 1); write_bit(buffer, 1); w...
~
13
Недавно Google Project Zero нашёл багу в SQLite с помощью LLM, о чём достаточно было шумно в определённых интернетах, которые сопровождались рассказами, что скоро всех "ибешни...
Alex Sherbakov
5
в JclConsole объявлено так: function CtrlHandler(CtrlType: DWORD): BOOL; stdcall; - где ваше объявление с stdcall? у вас на картинке нет stdcall
Karagy
8
Как передать управляющий символ в открытую через CreateProcess консоль? Собсна, есть процедура: procedure TRedirectThread.WriteData(Data: OEMString); var Written: Cardinal;...
Serjone
6
Ребят в СИ можно реализовать ООП?
Николай
33
program test; {$mode delphi} procedure proc(v: int32); overload; begin end; procedure proc(v: int64); overload; begin end; var x: uint64; begin proc(x); end. Уж не знаю...
notme
6
у вас два процесса. один посылает другому сигнал. у вас есть код обоих процессов? если всё не так - расскажите как оно на самом деле. а именно кто кому чего, есть-ли консоли,...
Karagy
6
Карта сайта