PHP: PHP metrics. Инструмент оценки сложности и maintainability кода. Часть 1 | Глазами фрилансера - HD

Как всегда топ.Интересный инструмент.Спасибо Саш, такого контента на ютубе не встречал.

— Советую на главной странице блок с Maintability/complexity проверять без комментариев, так как аннотации и комментарий к каждому методу, по моему мнению, сильно завышают показатель индекса поддерживаемости. К примеру, если добавить комментарий к каждому методу, свойству и условному оператору, то у вас этот «улей» будет практически весь зелёный. Но, как сами понимаете, это лишь иллюзия :)
— Volume до 8000 на класс — наверно, это много, даже как для критического показателя. И не обращаем внимание на цвет. Даже при хороших показателях, они будут иметь синий фон. Смотрим только на число.
— Самым полезным для меня оказалась цикломатическая сложность классов и методов. Сразу было видно, с чего начать рефакторинг, ну или, как минимум, на что обратить внимание. И помним, главное — не фанатеть :)

— А вот пример «плохо поддерживаемого класса». Я догадываюсь, почему так выходит. Слишком много публичных методов. Но выносить relations в какой-нибудь trait у меня рука не поднялась. Меня, кстати, удивило то, что scopes в Laravel прописываются в Model, а не в ModelQuery. В Yii2 это показалось более логичным, так как это автоматически разгружало модель. Хотя только что посмотрел и в Laravel это делается также простым переопределением query() с возвратом Query builder для определённой модели. Только вот примеров таких я ещё не встречал, по этому и подумал, что так просто не принято.
Но суть не в этом, а том, что класс плохо поддерживаемый :)

class Box extends Model
{
use SoftDeletes;
use SeoMetaModelTrait;

protected $table = 'box';

protected $casts = [
'name' => 'array',
'description' => 'array',
'date' => AsDate::class,
'datetime' => AsDatetime::class,
'image' => AsImage::class. ':widen_100|widen_500',
'images_list' => AsImages::class. ':widen_100|widen_500',
];

public function brand(): BelongsTo
{
return $this->belongsTo(Brand::class, 'brand_id')->withTrashed();
}

public function variations(): HasMany
{
return $this->hasMany(Variation::class, 'box_id')->orderBy('sort_index');
}

public function categories(): BelongsToMany
{
return $this->belongsToMany(Category::class, 'box_category_ref', 'box_id', 'category_id')->withTrashed();
}

public function tags(): BelongsToMany
{
return $this->belongsToMany(Tag::class, 'box_tag_ref', 'box_id', 'tag_id')->withTrashed();
}

public function scopePublished(Builder $query): void
{
$query->where('datetime', '<=', date('Y-m-d'));
}
}

Всех с новым годом!

А как вставлен ниже php дока смайлик с подписью yakut?

Класс!!! Очень интересно!!! Надо попробовать. И очень интересно продолжение.

Спасибо за урок! Какая досада, я полагал, что мой слог выразителен и лаконичен.:)
Как я понял, более всего phpmetrics невзлюбил классы, чьи методы используют переменные, находящиеся в глобальной видимости.
Я много раз читал, о том что это плохо. Но всё равно пользуюсь, ибо на практике c ними легче, если их немного.
Например, $env, $user. Это вроде как «Будешь так делать, прыщи вырастут. А от Денди кинескоп сядет».:)
Но есть вещи, которые меня заставили пересмотреть свои взгляды. Например, классы раздутые обилием методов.
Это произошло в силу того, что не очень хорошо получилось освоить ООП. Вместо наследования я избыточно использую композицию.
В php вообще сложно осваивать ООП. Вникнуть в эту парадигму помог опыт написания кода на C# в Unity.

Интересно было бы узнать всякие частности: 1) как вы развертываете приложение, в случаях, когда это зависит от вас (laradock? собственная сборка докера? laravel sail?), 2) что с дебагом (что-то стандартное или специальное) 3) если микросервисы, то как вписываете туда ларавель (lumen или как-то по-другому) 4) интересны также внешние сервисы, которые используете (postman, diagram.net и т. п.)