Spreadsheet::replace() - Code Metrics - Inspection of "make sure to not add the extension twice" - Baptouuuu/Spreadsheet - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 1718fc...f98f5b )

by Baptiste

created 2016-07-16 14:09 UTC

Spreadsheet::replace() A

↳ Parent: Spreadsheet

Complexity

Conditions	2
Paths	2

Size

Total Lines	14
Code Lines	8

Duplication

Lines	14
Ratio	100 %

Code Coverage

Tests	7
CRAP Score	2

Importance

Changes

Metric	Value
dl	14
loc	14
ccs	7
cts	7
cp	1
rs	9.4285
c	0
b	0
f	0
cc	2
eloc	8
nc	2
nop	1
crap	2

<?php
declare(strict_types = 1);

namespace Spreadsheet;

use Spreadsheet\Exception\{
    SheetAlreadyExistsException,
    SheetNotFoundException
};
use Innmind\Immutable\{
    MapInterface,
    Map
};

final class Spreadsheet implements SpreadsheetInterface
{
    private $name;
    private $sheets;

    public function __construct(string $name)
    {
        $this->name = $name;
        $this->sheets = new Map('string', SheetInterface::class);
    }

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

    /**
     * {@inheritdoc}
     */
    public function sheets(): MapInterface
    {
        return $this->sheets;
    }

    public function add(SheetInterface $sheet): SpreadsheetInterface
    {
        if ($this->has($sheet->name())) {
            throw new SheetAlreadyExistsException;
        }

        $self = new self($this->name);
        $self->sheets = $this->sheets->put(
            $sheet->name(),
            $sheet
        );

        return $self;
    }

    public function replace(SheetInterface $sheet): SpreadsheetInterface
    {
        if (!$this->has($sheet->name())) {
            throw new SheetNotFoundException;
        }

        $self = new self($this->name);
        $self->sheets = $this->sheets->put(
            $sheet->name(),
            $sheet
        );

        return $self;
    }

    public function has(string $sheet): bool
    {
        return $this->sheets->contains($sheet);
    }

    public function get(string $sheet): SheetInterface
    {
        return $this->sheets->get($sheet);
    }
}


1			<?php
2			declare(strict_types = 1);
3
4			namespace Spreadsheet;
5
6			use Spreadsheet\Exception\{
7			SheetAlreadyExistsException,
8			SheetNotFoundException
9			};
10			use Innmind\Immutable\{
11			MapInterface,
12			Map
13			};
14
15			final class Spreadsheet implements SpreadsheetInterface
16			{
17			private $name;
18			private $sheets;
19
20	14		public function __construct(string $name)
21			{
22	14		$this->name = $name;
23	14		$this->sheets = new Map('string', SheetInterface::class);
24	14		}
25
26	11		public function name(): string
27			{
28	11		return $this->name;
29			}
30
31			/**
32			* {@inheritdoc}
33			*/
34	9		public function sheets(): MapInterface
35			{
36	9		return $this->sheets;
37			}
38
39	11	View Code Duplication	public function add(SheetInterface $sheet): SpreadsheetInterface
40			{
41	11		if ($this->has($sheet->name())) {
42	1		throw new SheetAlreadyExistsException;
43			}
44
45	11		$self = new self($this->name);
46	11		$self->sheets = $this->sheets->put(
47	11		$sheet->name(),
48			$sheet
49			);
50
51	11		return $self;
52			}
53
54	2	View Code Duplication	public function replace(SheetInterface $sheet): SpreadsheetInterface
55			{
56	2		if (!$this->has($sheet->name())) {
57	1		throw new SheetNotFoundException;
58			}
59
60	1		$self = new self($this->name);
61	1		$self->sheets = $this->sheets->put(
62	1		$sheet->name(),
63			$sheet
64			);
65
66	1		return $self;
67			}
68
69	12		public function has(string $sheet): bool
70			{
71	12		return $this->sheets->contains($sheet);
72			}
73
74	4		public function get(string $sheet): SheetInterface
75			{
76	4		return $this->sheets->get($sheet);
77			}
78			}
79

Baptouuuu / Spreadsheet

Push — master ( 1718fc...f98f5b )

Spreadsheet::replace() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like