最近维护一个基于laravel的后台项目时,发现通过artisan执行任何命令都很慢,并且还会报错无法连接数据库,即使是执行php artisan optimize 这样的命令也是。这很让人疑惑,为什么执行optimize也要连接数据库并且执行效率这么慢?我在数据库报错的地方打印了调用栈,终于发现了问题。
laravel的artisan会调用Illuminate\Foundation\Console\Kernel 类来处理artisan输入的命令,该类第284行的getArtisan方法会创建一个Artisan对象,并调用Artisan对象的resolveCommands方法,参数是我们自己通过Kernel类绑定的所有Command类名,在resolveCommands方法中laravel会将我们绑定的所有Command类实例化,而项目中很多Command类的构造方法里有如下写法:
class Example extends Command {
public function __construct() {
parent::__construct();
$this->db = app('db')->connection('xxxxx')->getPdo();
$this->anotherDb = app('db')->connection('xxxxx')->getPdo();
...
}
}
这就导致不论我们通过artisan执行什么命令,laravel都会把所有Command类实例化,并且连接一遍所有数据库,导致脚本执行缓慢。合理的写法不应该在__construct()方法里进行连接数据库、redis等操作,避免实例化太多无用的资源,拖慢运行速度。
但是我也对laravel产生了一个疑问,artisan每次执行都要把所有Command实例化一遍,这么操作合理吗,为什么选择这样实现?继续往后看可以看到Symfony\Component\Console\Application 类的find()方法里会便利所有Command对象,判断哪个Command对象可以处理本次调用,为什么不把$signature变成静态成员,这样不是不需要实例化每个Command吗?
|