App\Support\Mechanics\ChooseMechanic

Complexity

Total Complexity

5

Size/Duplication

Total Lines	24
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
A	forOS()	0	16	5

<?php

namespace App\Support\Mechanics;

class ChooseMechanic
{
    /**
     * Get the mechanic for the host operating system.
     *
     * @param string|null $os
     * @return Mechanic
     */
    public static function forOS($os = null)
    {
        $os = $os ?: PHP_OS;

        switch (true) {

It seems like you are loosely comparing stristr($os, 'WIN') of type string to the boolean true. If you are specifically checking for a non-empty string, consider using the more explicit !== '' instead.

It seems like you are loosely comparing stristr($os, 'LINUX') of type string to the boolean true. If you are specifically checking for a non-empty string, consider using the more explicit !== '' instead.

It seems like you are loosely comparing stristr($os, 'DAR') of type string to the boolean true. If you are specifically checking for a non-empty string, consider using the more explicit !== '' instead.

            case stristr($os, 'DAR'):
                return app(MacOs::class);

            case stristr($os, 'WIN'):
                return app(Windows::class);

            case stristr($os, 'LINUX'):
                return app(Linux::class);

            default :
                return app(Untrained::class);
        }
    }
}