FlightCompanyFactory::getSubject() - Code Metrics - Inspection of "Dev" - st-universe/core - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — master (#2250)

by Janko

created 2025-09-03 10:43 UTC

FlightCompanyFactory::getSubject() A

↳ Parent: FlightCompanyFactory

Complexity

Conditions	3
Paths	3

Size

Total Lines	13
Code Lines	7

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	7
CRAP Score	3.0175

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	3
eloc	7
nc	3
nop	1
dl	0
loc	13
ccs	7
cts	8
cp	0.875
crap	3.0175
rs	10
c	1
b	0
f	0

<?php

namespace Stu\Module\Spacecraft\Lib\Movement;

use Doctrine\Common\Collections\ArrayCollection;
use Doctrine\Common\Collections\Collection;
use Stu\Module\Ship\Lib\FleetWrapperInterface;
use Stu\Module\Spacecraft\Lib\Movement\Component\PreFlight\PreFlightConditionsCheckInterface;
use Stu\Module\Spacecraft\Lib\SpacecraftWrapperInterface;

class FlightCompanyFactory
{
    public function __construct(private PreFlightConditionsCheckInterface $preFlightConditionsCheck) {}

    public function create(SpacecraftWrapperInterface $leadWrapper): FlightCompany
    {
        $subject = $this->getSubject($leadWrapper);

        return new FlightCompany(
            $subject,
            $this->getMembers($subject),
            $this->preFlightConditionsCheck
        );
    }

    private function getSubject(SpacecraftWrapperInterface $leadWrapper): SpacecraftWrapperInterface|FleetWrapperInterface
    {
        $fleetWrapper = $leadWrapper->getFleetWrapper();
        if ($fleetWrapper === null) {
            return $leadWrapper;
        }

        $fleetLeader = $fleetWrapper->getLeadWrapper()->get();

        //TODO check for isFleetLeader too?
        return $fleetLeader === $leadWrapper->get()

            ? $fleetWrapper
            : $leadWrapper;
    }

    /** @return Collection<int, covariant SpacecraftWrapperInterface> */
    private function getMembers(SpacecraftWrapperInterface|FleetWrapperInterface $subject): Collection
    {
        return $subject instanceof FleetWrapperInterface
            ? $subject->getShipWrappers()
            : new ArrayCollection([$subject->get()->getId() => $subject]);
    }
}


1		<?php
2
3		namespace Stu\Module\Spacecraft\Lib\Movement;
4
5		use Doctrine\Common\Collections\ArrayCollection;
6		use Doctrine\Common\Collections\Collection;
7		use Stu\Module\Ship\Lib\FleetWrapperInterface;
8		use Stu\Module\Spacecraft\Lib\Movement\Component\PreFlight\PreFlightConditionsCheckInterface;
9		use Stu\Module\Spacecraft\Lib\SpacecraftWrapperInterface;
10
11		class FlightCompanyFactory
12		{
13	1	public function __construct(private PreFlightConditionsCheckInterface $preFlightConditionsCheck) {}
14
15	6	public function create(SpacecraftWrapperInterface $leadWrapper): FlightCompany
16		{
17	6	$subject = $this->getSubject($leadWrapper);
18
19	6	return new FlightCompany(
20	6	$subject,
21	6	$this->getMembers($subject),
22	6	$this->preFlightConditionsCheck
23	6	);
24		}
25
26	6	private function getSubject(SpacecraftWrapperInterface $leadWrapper): SpacecraftWrapperInterface\|FleetWrapperInterface
27		{
28	6	$fleetWrapper = $leadWrapper->getFleetWrapper();
29	6	if ($fleetWrapper === null) {
30		return $leadWrapper;
31		}
32
33	6	$fleetLeader = $fleetWrapper->getLeadWrapper()->get();
34
35		//TODO check for isFleetLeader too?
36	6	return $fleetLeader === $leadWrapper->get()
		0 ignored issues – show introduced 2025-04-10 07:26 UTC by Report Bug Copy Issue Report The condition `$fleetLeader === $leadWrapper->get()` is always `false`. Loading history...
37	6	? $fleetWrapper
38	6	: $leadWrapper;
39		}
40
41		/** @return Collection<int, covariant SpacecraftWrapperInterface> */
42	6	private function getMembers(SpacecraftWrapperInterface\|FleetWrapperInterface $subject): Collection
43		{
44	6	return $subject instanceof FleetWrapperInterface
45	6	? $subject->getShipWrappers()
46	6	: new ArrayCollection([$subject->get()->getId() => $subject]);
47		}
48		}
49

st-universe / core

Pull Request — master (#2250)

FlightCompanyFactory::getSubject() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like