Comments 13
php artisan migrate
а не php artisan migration
php artisan make:model ModelName -mrc
и будет вам и миграции и контроллерРоуты жуть, почему не ресурсы?
И в целом материал какой-то устаревший, например:
<li><a href="{{URL::route('create_album_form')}}">Создание нового альбома</a></li>
сейчас просто <li><a href="{{route('create_album_form')}}">Создание нового альбома</a></li>
Статья о том, как не нужно писать на Laravel, да и вообще на PHP
В миграции стоило бы добавить nullable
Потому что при создании альбома поле description не обязательное по валидатору и этот код вызовет ошибку при пустом значении description
Про CodeStyle, похоже, никто не слышал
Про CRUD через php artisan make:resource и соответствующие роуты написали выше. И еще куча всего
Статья пойдет как для ну очень новичка, ИМХО
UPD
Зачем во вьюхах
Можно же наследовать,
В миграции стоило бы добавить nullable
$table->text('description'); // описание нашего альбома
Потому что при создании альбома поле description не обязательное по валидатору и этот код вызовет ошибку при пустом значении description
$album = Album::create(array(
'name' => $request->get('name'),
'description' => $request->get('description'),
'cover_image' => $filename,
));
Про CodeStyle, похоже, никто не слышал
Про CRUD через php artisan make:resource и соответствующие роуты написали выше. И еще куча всего
Статья пойдет как для ну очень новичка, ИМХО
UPD
Зачем во вьюхах
@include('includes.header')
?Можно же наследовать,
@extends('site.layouts.main')
@section('content')
@endsection
Данная статья действительно рассчитана на новичков, все верно
Для новичков больше вреда чем полезности. Навалено в кучу ИМХО автора.
Начиная работу с Ларавелем перерыл кучу подобных мануалов. Пока тим лид не обьяснил на пальцах структуру где, что лежит и за что отвечает тот или иной кусок кода — в голове была жуткая каша.
Начиная работу с Ларавелем перерыл кучу подобных мануалов. Пока тим лид не обьяснил на пальцах структуру где, что лежит и за что отвечает тот или иной кусок кода — в голове была жуткая каша.
Чутка добавлю.
В первой статье рассказывалось про кодогенераторы, но почему-то здесь контроллер написан вручную. Я понимаю, что новичкам не обязательно знать про ресурсы, но банальные конвенции именования методов лучше соблюдать, раз уж Вы учите их работе с фреймворком. Потом будет проще переучиться.
Удаление обязательно делать методом POST(лучше, конечно, через method_field('DELETE'), но списываем на новичков) и вкидывать csrf-token. Зачем-то же он нужен…
Конструкцию array(...) не видел уже миллион лет :)
Переменная $uploadSuccess присваивается, но нигде не используется. Там вообще лучше обернуть в try...catch, но, опять же, новичкам хватит и простой проверки if ($uploadSuccess) {...}.
Это вообще огонь :)
$album = Album::with('Photos')->find($id);
$albums = Album::with('Photos')->get();
Ну и про code style уже говорили. Вообще не понимаю людей, жадных на пробелы…
В первой статье рассказывалось про кодогенераторы, но почему-то здесь контроллер написан вручную. Я понимаю, что новичкам не обязательно знать про ресурсы, но банальные конвенции именования методов лучше соблюдать, раз уж Вы учите их работе с фреймворком. Потом будет проще переучиться.
Удаление обязательно делать методом POST(лучше, конечно, через method_field('DELETE'), но списываем на новичков) и вкидывать csrf-token. Зачем-то же он нужен…
Конструкцию array(...) не видел уже миллион лет :)
Переменная $uploadSuccess присваивается, но нигде не используется. Там вообще лучше обернуть в try...catch, но, опять же, новичкам хватит и простой проверки if ($uploadSuccess) {...}.
Это вообще огонь :)
$album = Album::with('Photos')->find($id);
$albums = Album::with('Photos')->get();
Ну и про code style уже говорили. Вообще не понимаю людей, жадных на пробелы…
Нарисовать сову очень просто!
Собираюсь к вам на курс, а тут такое… Я хотел-бы научиться так не делать.
Благодарю всех за комментарии. Все замечания переданы автору, для того, чтобы в последующих публикациях такого не повторялось.
php artisan make:controller AlbumsController -r
Что бы такого ужаса не было в именах функций
Sign up to leave a comment.
Laravel: объясняем основные понятия. Часть вторая: «Практика»