FileMutations::addFileMutation() - Code Metrics - Inspection of "Version 1" - DaveLiddament/sarb - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — master (#69)

by Dave

created 2020-09-05 21:00 UTC

FileMutations::addFileMutation() A

↳ Parent: FileMutations

Complexity

Conditions	2
Paths	2

Size

Total Lines	10
Code Lines	6

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	2
eloc	6
nc	2
nop	1
dl	0
loc	10
rs	10
c	0
b	0
f	0

<?php

/**
 * Static Analysis Results Baseliner (sarb).
 *
 * (c) Dave Liddament
 *
 * For the full copyright and licence information please view the LICENSE file distributed with this source code.
 */

declare(strict_types=1);

namespace DaveLiddament\StaticAnalysisResultsBaseliner\Domain\HistoryAnalyser\UnifiedDiffParser;

use Webmozart\Assert\Assert;

/**
 * Holds FileMutation objects. Use the.
 */
class FileMutations
{
    /**
     * @var FileMutation[]
     */
    private $fileMutations;

    /**
     * FileMutations constructor.
     *
     * @param FileMutation[] $fileMutations
     */
    public function __construct(array $fileMutations)
    {
        $this->fileMutations = [];
        foreach ($fileMutations as $fileMutation) {
            $this->addFileMutation($fileMutation);
        }
    }

    /**
     * Returns FileMutations for the given file name. Or null if there are no file mutations for that file in the diff.
     */
    public function getFileMutation(NewFileName $newFileName): ?FileMutation
    {
        $newFileNameAsString = $newFileName->getFileName();

        return $this->fileMutations[$newFileNameAsString] ?? null;
    }

    private function addFileMutation(FileMutation $fileMutation): void
    {
        if ($fileMutation->isDeletedFile()) {
            return;
        }
        $newFileNameAsString = $fileMutation->getNewFileName()->getFileName();
        $alreadyExists = array_key_exists($newFileNameAsString, $this->fileMutations);
        Assert::false($alreadyExists, "Multiple new files with name [$newFileNameAsString]");

        $this->fileMutations[$newFileNameAsString] = $fileMutation;
    }

    /**
     * Returns number of FileMutations (only usecase is for testing).
     */
    public function getCount(): int
    {
        return count($this->fileMutations);
    }
}


1			<?php
2
3			/**
4			* Static Analysis Results Baseliner (sarb).
5			*
6			* (c) Dave Liddament
7			*
8			* For the full copyright and licence information please view the LICENSE file distributed with this source code.
9			*/
10
11			declare(strict_types=1);
12
13			namespace DaveLiddament\StaticAnalysisResultsBaseliner\Domain\HistoryAnalyser\UnifiedDiffParser;
14
15			use Webmozart\Assert\Assert;
16
17			/**
18			* Holds FileMutation objects. Use the.
19			*/
20			class FileMutations
21			{
22			/**
23			* @var FileMutation[]
24			*/
25			private $fileMutations;
26
27			/**
28			* FileMutations constructor.
29			*
30			* @param FileMutation[] $fileMutations
31			*/
32			public function __construct(array $fileMutations)
33			{
34			$this->fileMutations = [];
35			foreach ($fileMutations as $fileMutation) {
36			$this->addFileMutation($fileMutation);
37			}
38			}
39
40			/**
41			* Returns FileMutations for the given file name. Or null if there are no file mutations for that file in the diff.
42			*/
43			public function getFileMutation(NewFileName $newFileName): ?FileMutation
44			{
45			$newFileNameAsString = $newFileName->getFileName();
46
47			return $this->fileMutations[$newFileNameAsString] ?? null;
48			}
49
50			private function addFileMutation(FileMutation $fileMutation): void
51			{
52			if ($fileMutation->isDeletedFile()) {
53			return;
54			}
55			$newFileNameAsString = $fileMutation->getNewFileName()->getFileName();
56			$alreadyExists = array_key_exists($newFileNameAsString, $this->fileMutations);
57			Assert::false($alreadyExists, "Multiple new files with name [$newFileNameAsString]");
58
59			$this->fileMutations[$newFileNameAsString] = $fileMutation;
60			}
61
62			/**
63			* Returns number of FileMutations (only usecase is for testing).
64			*/
65			public function getCount(): int
66			{
67			return count($this->fileMutations);
68			}
69			}
70

DaveLiddament / sarb

Pull Request — master (#69)

FileMutations::addFileMutation() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like