kalanis\simple_vin\Support\Manufacturer

Complexity

Total Complexity

10

Size/Duplication

Total Lines	30
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
B	desc()	0	8	8
A	__construct()	0	13	1
A	__toString()	0	3	1

<?php

namespace kalanis\simple_vin\Support;


final class Manufacturer implements \Stringable
{
    public function __construct(
        public readonly string $code,
        public readonly string $brand,
        public readonly string $types = '',
        public readonly string $model = '',
        public readonly string $manufacturer = '',
        public readonly string $country = '',
        public readonly ?int   $from = null,
        public readonly ?int   $to = null,
        public readonly string $notes = '',
        public readonly bool   $canCompareDates = false,
    )
    {
    }

    public function desc(): string
    {
        return
            ($this->manufacturer ? $this->manufacturer . ' - ' : '')
            . $this->brand . ' ' . $this->model
            . ($this->types ? (' ' . $this->types) : '')
            . (($this->from || $this->to) ? (' (' . ($this->from ?: '') . '-' . ($this->to ?: '') . ')') : '')

The expression $this->from of type integer|null is loosely compared to true; this is ambiguous if the integer can be 0. You might want to explicitly use !== null instead.

The expression $this->to of type integer|null is loosely compared to true; this is ambiguous if the integer can be 0. You might want to explicitly use !== null instead.

            . ($this->country ? ', ' . $this->country : '');
    }

    public function __toString(): string
    {
        return $this->desc();
    }
}