王宁凯的个人博客

Laravel 最佳实践

2018.08.09

Laravel 最佳实践

这不是对 Laravel 改编的 SOLID 原则、模式等。这是在现实生活中Laravel项目中通常被忽略的最佳实践。

目录

单一责任原则

[丰富的模型, 简单的控制器](#丰富的模型, 简单的控制器)

验证

业务逻辑应该放在服务类中

不要重复你自己(DRY)

[最好倾向于使用 Eloquent 而不是 Query Builder 和原生的 SQL 查询。要优先于数组的集合](#最好倾向于使用 Eloquent 而不是 Query Builder 和原生的 SQL 查询。要优先于数组的集合)

批量赋值

[不要在 Blade 模板中执行查询并使用关联加载(N + 1 问题)](#不要在 Blade 模板中执行查询并使用关联加载(N + 1 问题))

给予方法或变量一个优雅的描述比注释代码更好。

[不要把 JS 和 CSS 放在 Blade 模板中,也不要将任何 HTML 放在 PHP 类中](#不要把 JS 和 CSS 放在 Blade 模板中,也不要将任何 HTML 放在 PHP 类中)

在代码中使用配置和语言文件、常量,而不是固定写死

使用社区推荐的标准Laravel工具

遵循Laravel命名规范

运用更短、可读性更强的语法

[使用 IoC 容器或 facades 代替 new Class()](#使用 IoC 容器或 facades 代替 new Class())

[不要直接从 .env 文件获取数据](#不要直接从 .env 文件获取数据)

以标准格式存储日期,必要时就使用访问器和修改器来修改日期格式

其它好的实践

单一责任原则

一个类和一个方法应该只有一个职责。

不好的实践:

public function getFullNameAttribute()
{
    if (auth()->user() && auth()->user()->hasRole('client') && auth()->user()->isVerified()) {
        return 'Mr. ' . $this->first_name . ' ' . $this->middle_name . ' ' . $this->last_name;
    } else {
        return $this->first_name[0] . '. ' . $this->last_name;
    }
}

好的实践:

public function getFullNameAttribute()
{
    return $this->isVerifiedClient() ? $this->getFullNameLong() : $this->getFullNameShort();
}

public function isVerifiedClient()
{
    return auth()->user() && auth()->user()->hasRole('client') && auth()->user()->isVerified();
}

public function getFullNameLong()
{
    return 'Mr. ' . $this->first_name . ' ' . $this->middle_name . ' ' . $this->last_name;
}

public function getFullNameShort()
{
    return $this->first_name[0] . '. ' . $this->last_name;
}

丰富的模型, 简单的控制器

如果你使用Laravel查询构造器或原始 SQL语句 来查询,请将全部的与数据库相关的逻辑放到Eloquent 模型或存储库类中。

坏的实践:

public function index()
{
    $clients = Client::verified()
        ->with(['orders' => function ($q) {
            $q->where('created_at', '>', Carbon::today()->subWeek());
        }])
        ->get();

    return view('index', ['clients' => $clients]);
}

好的实践:

public function index()
{
    return view('index', ['clients' => $this->client->getWithNewOrders()]);
}

class Client extends Model
{
    public function getWithNewOrders()
    {
        return $this->verified()
            ->with(['orders' => function ($q) {
                $q->where('created_at', '>', Carbon::today()->subWeek());
            }])
            ->get();
    }
}

验证

将验证从控制器中移动到请求类中。

不好的实践:

public function store(Request $request)
{
    $request->validate([
        'title' => 'required|unique:posts|max:255',
        'body' => 'required',
        'publish_at' => 'nullable|date',
    ]);

    ....
}

好的实践:

public function store(PostRequest $request)
{    
    ....
}

class PostRequest extends Request
{
    public function rules()
    {
        return [
            'title' => 'required|unique:posts|max:255',
            'body' => 'required',
            'publish_at' => 'nullable|date',
        ];
    }
}

业务逻辑应该放在服务类中

一个控制器必须仅有一个职责,所以请将业务逻辑放到服务类中。

不好的实践:

public function store(Request $request)
{
    if ($request->hasFile('image')) {
        $request->file('image')->move(public_path('images') . 'temp');
    }
    
    ....
}

好的实践:

public function store(Request $request)
{
    $this->articleService->handleUploadedImage($request->file('image'));

    ....
}

class ArticleService
{
    public function handleUploadedImage($image)
    {
        if (!is_null($image)) {
            $image->move(public_path('images') . 'temp');
        }
    }
}

不要重复你自己 (DRY)

尽可能的复用代码. SRP(单一职责原则)正在帮助你避免重复。当然,这也包括了 Blade 模板、Eloquent 的范围等。

不好的实践:

public function getActive()
{
    return $this->where('verified', 1)->whereNotNull('deleted_at')->get();
}

public function getArticles()
{
    return $this->whereHas('user', function ($q) {
            $q->where('verified', 1)->whereNotNull('deleted_at');
        })->get();
}

好的实践:

public function scopeActive($q)
{
    return $q->where('verified', 1)->whereNotNull('deleted_at');
}

public function getActive()
{
    return $this->active()->get();
}

public function getArticles()
{
    return $this->whereHas('user', function ($q) {
            $q->active();
        })->get();
}

最好倾向于使用 Eloquent 而不是 Query Builder 和原生的 SQL 查询。要优先于数组的集合

Eloquent 可以帮助你编写可读性高和可维护性好的代码。此外,Eloquent 也拥有很棒的内置工具,比如软删除、事件、范围等。

不好的实践:

SELECT *
FROM `articles`
WHERE EXISTS (SELECT *
              FROM `users`
              WHERE `articles`.`user_id` = `users`.`id`
              AND EXISTS (SELECT *
                          FROM `profiles`
                          WHERE `profiles`.`user_id` = `users`.`id`) 
              AND `users`.`deleted_at` IS NULL)
AND `verified` = '1'
AND `active` = '1'
ORDER BY `created_at` DESC

好的实践:

Article::has('user.profile')->verified()->latest()->get();

批量赋值

不好的实践:

$article = new Article;
$article->title = $request->title;
$article->content = $request->content;
$article->verified = $request->verified;
// Add category to article
$article->category_id = $category->id;
$article->save();

好的实践:

$category->article()->create($request->all());

不要在 Blade 模板中执行查询并使用关联加载(N + 1 问题)

不好的实践 (对于100个用户,下面需要 101 次查询):

@foreach (User::all() as $user)
    {{ $user->profile->name }}
@endforeach

好的实践 (对于100个用户,下面只需 2 次查询):

$users = User::with('profile')->get();

...

@foreach ($users as $user)
    {{ $user->profile->name }}
@endforeach

给予方法或变量一个优雅的描述比注释代码更好。

不好的实践:

if (count((array) $builder->getQuery()->joins) > 0)

更好的实践:

// 判断是否有更多的连接
if (count((array) $builder->getQuery()->joins) > 0)

好的实践:

if ($this->hasJoins())

不要把 JS 和 CSS 放在 Blade 模板中,也不要将任何 HTML 放在 PHP 类中

不好的实践:

let article = `{{ json_encode($article) }}`;

好的实践:

<input id="article" type="hidden" value="{{ json_encode($article) }}">

或者

<button class="js-fav-article" data-article="{{ json_encode($article) }}">{{ $article->name }}<button>

在一个 Javascript 文件中:

let article = $('#article').val();

最好的方法是使用专门的PHP到JS包来传输数据。

在代码中使用配置和语言文件、常量,而不是固定写死

不好的实践:

public function isNormal()
{
    return $article->type === 'normal';
}

return back()->with('message', 'Your article has been added!');

好的实践:

public function isNormal()
{
    return $article->type === Article::TYPE_NORMAL;
}

return back()->with('message', __('app.article_added'));

使用社区推荐的标准Laravel工具

最好使用内置的 Laravel 功能和社区软件包,而不是其他第三方软件包和工具。因为将来与你的应用程序一起工作的开发人员都需要学习新的工具。另外,使用第三方软件包或工具的话,如果遇到困难,从 Laravel 社区获得帮助的机会会大大降低。不要让你的客户为此付出代价!

任务标准工具第三方工具
授权PoliciesEntrust, Sentinel and other packages
前端编译Laravel MixGrunt, Gulp, 3rd party packages
开发环境HomesteadDocker
部署Laravel ForgeDeployer and other solutions
单元测试PHPUnit, MockeryPhpspec
浏览器测试Laravel DuskCodeception
数据库操作EloquentSQL, Doctrine
模板BladeTwig
数据操作Laravel collectionsArrays
表单验证Request classes3rd party packages, validation in controller
认证Built-in3rd party packages, your own solution
API 认证Laravel Passport3rd party JWT and OAuth packages
创建 APIBuilt-inDingo API and similar packages
数据库结构操作MigrationsWorking with DB structure directly
本地化Built-in3rd party packages
即时用户接口Laravel Echo, Pusher3rd party packages and working with WebSockets directly
初始化测试数据Seeder classes, Model Factories, FakerCreating testing data manually
计划任务Laravel Task SchedulerScripts and 3rd party packages
数据库支持MySQL, PostgreSQL, SQLite, SQL ServerMongoDB

遵循Laravel命名规范

遵从 PSR 标准.

另外, 请遵循 Laravel 社区接受的命名规范:

类型规则好的实践不好的实践
Controller单数ArticleControllerArticlesController
Route复数articles/1article/1
Named route带点符号的蛇形命名users.show_activeusers.show-active, show-active-users
Model单数UserUsers
hasOne or belongsTo relationship单数articleCommentarticleComments, article_comment
All other relationships复数articleCommentsarticleComment, article_comments
Table复数article_commentsarticle_comment, articleComments
Pivot table按字母顺序排列的单数模型名称article_useruser_article, articles_users
Table column带着模型名称的蛇形命名meta_titleMetaTitle; article_meta_title
Model property蛇形命名$model->created_at$model->createdAt
Foreign key带_id后缀的单数型号名称article_idArticleId, id_article, articles_id
Primary key-idcustom_id
Migration-2017_01_01_000000_create_articles_table2017_01_01_000000_articles
Method小驼峰命名getAllget_all
Method in resource controllerstoresaveArticle
Method in test class小驼峰命名testGuestCannotSeeArticletest_guest_cannot_see_article
Variable小驼峰命名$articlesWithAuthor$articles_with_author
Collection具描述性的复数形式$activeUsers = User::active()->get()$active, $data
Object具描述性的单数形式$activeUser = User::active()->first()$users, $obj
Config and language files index蛇形命名articles_enabledArticlesEnabled; articles-enabled
View蛇形命名show_filtered.blade.phpshowFiltered.blade.php, show-filtered.blade.php
Config蛇形命名google_calendar.phpgoogleCalendar.php, google-calendar.php
Contract (interface)形容词或名词AuthenticatableAuthenticationInterface, IAuthentication
Trait形容词或名词NotifiableNotificationTrait

运用更短、可读性更强的语法

不好的实践:

$request->session()->get('cart');
$request->input('name');

好的实践:

session('cart');
$request->name;

更多实例:

通用语法简短可读性强的语法
Session::get('cart')session('cart')
$request->session()->get('cart')session('cart')
Session::put('cart', $data)session(['cart' => $data])
$request->input('name'), Request::get('name')$request->name, request('name')
return Redirect::back()return back()
is_null($object->relation) ? $object->relation->id : null }optional($object->relation)->id
return view('index')->with('title', $title)->with('client', $client)return view('index', compact('title', 'client'))
$request->has('value') ? $request->value : 'default';$request->get('value', 'default')
Carbon::now(), Carbon::today()now(), today()
App::make('Class')app('Class')
->where('column', '=', 1)->where('column', 1)
->orderBy('created_at', 'desc')->latest()
->orderBy('age', 'desc')->latest('age')
->orderBy('created_at', 'asc')->oldest()
->select('id', 'name')->get()->get(['id', 'name'])
->first()->name->value('name')

使用 IoC 容器或 facades 代替 new Class()

new Class() 语法创建类时,不仅使得类与类之间紧密耦合,还加重了测试的复杂度。推荐改用 IoC 容器或 facades。

不好的实践:

$user = new User;
$user->create($request->all());

好的实践:

public function __construct(User $user)
{
    $this->user = $user;
}

....

$this->user->create($request->all());

不要直接从 .env 文件获取数据

将数据传递给配置文件,然后使用辅助函数 config() 在应用程序中使用数据。

不好的实践:

$apiKey = env('API_KEY');

好的实践:

// config/api.php
'key' => env('API_KEY'),

// Use the data
$apiKey = config('api.key');

以标准格式存储日期,必要时就使用访问器和修改器来修改日期格式

不好的实践:

{{ Carbon::createFromFormat('Y-d-m H-i', $object->ordered_at)->toDateString() }}
{{ Carbon::createFromFormat('Y-d-m H-i', $object->ordered_at)->format('m-d') }}

好的实践:

// Model
protected $dates = ['ordered_at', 'created_at', 'updated_at']
public function getSomeDateAttribute($date)
{
    return $date->format('m-d');
}

// View
{{ $object->ordered_at->toDateString() }}
{{ $object->ordered_at->some_date }}

其它好的实践

千万不要在路由文件中放置任何逻辑。

在 Blade 模板中最小化 vanilla PHP 的使用。