ASubject::detach() - Code Metrics - Inspection of "Merge branch 'develop'" - alecrabbit/php-console-spinner - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( de3d61...be839c )

by Alec

created 2023-05-08 11:23 UTC

ASubject::detach() A

↳ Parent: ASubject

Complexity

Conditions	2
Paths	2

Size

Total Lines	4
Code Lines	2

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

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

<?php

declare(strict_types=1);

namespace AlecRabbit\Spinner\Core\A;

use AlecRabbit\Spinner\Contract\IObserver;
use AlecRabbit\Spinner\Contract\ISubject;
use AlecRabbit\Spinner\Exception\InvalidArgumentException;

abstract class ASubject implements ISubject
{
    public function __construct(
        protected ?IObserver $observer = null,
    ) {
    }

    public function notify(): void
    {
        $this->observer?->update($this);
    }

    public function attach(IObserver $observer): void
    {
        if ($this->observer !== null) {
            throw new InvalidArgumentException('Observer is already attached.');
        }

        $this->assertNotSelf($observer);

        $this->observer = $observer;
    }

    protected function assertNotSelf(object $obj): void
    {
        if ($obj === $this) {
            throw new InvalidArgumentException('Object can not be self.');
        }
    }

    public function detach(IObserver $observer): void
    {
        if ($this->observer === $observer) {
            $this->observer = null;
        }
    }
}


1			<?php
2
3			declare(strict_types=1);
4
5			namespace AlecRabbit\Spinner\Core\A;
6
7			use AlecRabbit\Spinner\Contract\IObserver;
8			use AlecRabbit\Spinner\Contract\ISubject;
9			use AlecRabbit\Spinner\Exception\InvalidArgumentException;
10
11			abstract class ASubject implements ISubject
12			{
13			public function __construct(
14			protected ?IObserver $observer = null,
15			) {
16			}
17
18			public function notify(): void
19			{
20			$this->observer?->update($this);
21			}
22
23			public function attach(IObserver $observer): void
24			{
25			if ($this->observer !== null) {
26			throw new InvalidArgumentException('Observer is already attached.');
27			}
28
29			$this->assertNotSelf($observer);
30
31			$this->observer = $observer;
32			}
33
34			protected function assertNotSelf(object $obj): void
35			{
36			if ($obj === $this) {
37			throw new InvalidArgumentException('Object can not be self.');
38			}
39			}
40
41			public function detach(IObserver $observer): void
42			{
43			if ($this->observer === $observer) {
44			$this->observer = null;
45			}
46			}
47			}
48

alecrabbit / php-console-spinner

Push — master ( de3d61...be839c )

ASubject::detach() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like