RoyallTheFourth\HtmlDocument\Element\Nav

Complexity

Total Complexity

4

Size/Duplication

Total Lines	25
Duplicated Lines	100 %

Coupling/Cohesion

Components	1
Dependencies	6

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	6	6	1
A	withAttribute()	10	10	2
A	withChild()	4	4	1

<?php

namespace RoyallTheFourth\HtmlDocument\Element;

use RoyallTheFourth\HtmlDocument\Attribute\BooleanAttribute;
use RoyallTheFourth\HtmlDocument\Attribute\StandardAttribute;
use RoyallTheFourth\HtmlDocument\Set\AttributeSet;
use RoyallTheFourth\HtmlDocument\Set\ElementSet;
use RoyallTheFourth\HtmlDocument\Tag\Standard;

/**
 * Class Nav
 * @see https://developer.mozilla.org/en-US/docs/Web/HTML/Element/nav
 */
final class Nav extends AbstractElement implements ParentElementInterface

This class seems to be duplicated in your project.

{
    public function __construct(AttributeSet $attributes = null, ElementSet $children = null)
    {
        $this->attributes = $attributes ?? new AttributeSet();
        $this->children = $children ?? new ElementSet();
        $this->tag = new Standard('nav', $attributes, $children);
    }

    public function withAttribute(string $name, string $value = null): Nav
    {
        if ($value) {

The expression $value of type null|string is loosely compared to true; this is ambiguous if the string can be empty. You might want to explicitly use !== null instead.

            $attribute = new StandardAttribute($name, $value);
        } else {
            $attribute = new BooleanAttribute($name);
        }

        return new Nav($this->attributes->add($attribute), $this->children);
    }

    public function withChild(ElementInterface $element): Nav
    {
        return new Nav($this->attributes, $this->children->add($element));
    }
}