Laravel: разбираем основные понятия. Часть третья: «Заключительная»

[NikitchenkoSergey]

Почему Вы не использовали запрос с валидацией (php artisan make:request)?
Ну и код бы причесать — убрать копипасту вида $request->get('album_id'), добавить phpDoc, проверить результат перемещения файла, определиться с неймингом переменных — сейчас есть и camelCase и snake_case ($destinationPath и $random_name).

[vdem]

Да, автору бы в первую очередь ознакомиться с PSR по части code style (уж если сам Laravel этим стандартам следует). Сложновато читать кашу из символов и отступов, поставленных непонятно по какому алгоритму :)

[wscms]

Что сразу бросается в глаза
1. @if (Auth::check()) ... @endif можно смело заменить на @auth ... @endauth и @guest ... @endguest

2. Роуты можно группировать, используя общую middleware
Route::group([ 'middleware' => 'auth'], function () {
Route::get('/createalbum', array('as' => 'create_album_form','uses' => 'AlbumsController@getForm'));
Route::post('/createalbum', array('as' => 'create_album','uses' => 'AlbumsController@postCreate'));
});

3. Вы до сих пор используете include в шаблонах, вместо наследования. С наследованием удобнее.

4. Очень удобная штука Laravel Collective. Кстати, в нее можно передавать модель, а не отдельные поля формы.

[fauther_i]

Как по мне, для читабельности такой вариант получше:

Route::get('/createalbum', 'AlbumsController@getForm')->name('create_album_form');

вместо этого:

Route::get('/createalbum', array('as' => 'create_album_form','uses' => 'AlbumsController@getForm'));