Stratadox\CardGame\Match\Command\Block

Complexity

Total Complexity

6

Size/Duplication

Total Lines	50
Duplicated Lines	0 %

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	withDefender()	0	3	1
A	attacker()	0	3	1
A	__construct()	0	10	1
A	go()	0	7	1
A	in()	0	3	1
A	as()	0	3	1

<?php declare(strict_types=1);

namespace Stratadox\CardGame\Match\Command;

use Stratadox\CardGame\Match\Command\BlockTheAttacker;
use Stratadox\CardGame\Match\MatchId;

final class Block
{
    /** @var int */
    private $attacker;
    /** @var int|null */
    private $defender;
    /** @var int|null */
    private $player;
    /** @var MatchId|null */
    private $match;

    public function __construct(
        int $attacker,
        ?int $defender,
        ?int $player,
        ?MatchId $match
    ) {
        $this->attacker = $attacker;
        $this->defender = $defender;
        $this->player = $player;
        $this->match = $match;
    }

    public static function attacker(int $attacker): self
    {
        return new self($attacker, null, null, null);
    }

    public function withDefender(int $defender): self
    {
        return new self($this->attacker, $defender, $this->player, $this->match);
    }

    public function as(int $player): self
    {
        return new self($this->attacker, $this->defender, $player, $this->match);
    }

    public function in(MatchId $match): self
    {
        return new self($this->attacker, $this->defender, $this->player, $match);
    }

    public function go(): BlockTheAttacker
    {
        return BlockTheAttacker::number(
            $this->attacker,
            $this->defender,

It seems like $this->defender can also be of type null; however, parameter $defender of Stratadox\CardGame\Match\Command\BlockTheAttacker::number() does only seem to accept integer, maybe add an additional type check?

            $this->player,

It seems like $this->player can also be of type null; however, parameter $player of Stratadox\CardGame\Match\Command\BlockTheAttacker::number() does only seem to accept integer, maybe add an additional type check?

            $this->match

It seems like $this->match can also be of type null; however, parameter $match of Stratadox\CardGame\Match\Command\BlockTheAttacker::number() does only seem to accept Stratadox\CardGame\Match\MatchId, maybe add an additional type check?

        );
    }
}