Initx\Querabilis\Driver\HasDefaultRemoveAndElement

Complexity

Total Complexity

4

Size/Duplication

Total Lines	19
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	1

3 Methods

Rating	Name	Duplication	Size	Complexity
A	element()	0	3	1
A	returnOrThrowNoSuchElement()	0	7	2
A	remove()	0	3	1

<?php declare(strict_types=1);

namespace Initx\Querabilis\Driver;

use Initx\Querabilis\Envelope;
use Initx\Querabilis\Exception\NoSuchElementException;

trait HasDefaultRemoveAndElement
{
    public function remove(): Envelope
    {
        return $this->returnOrThrowNoSuchElement($this->poll());

It seems like poll() must be provided by classes using this trait. How about adding it as abstract method to this trait?

    }

    public function element(): Envelope
    {
        return $this->returnOrThrowNoSuchElement($this->peek());

It seems like peek() must be provided by classes using this trait. How about adding it as abstract method to this trait?

    }

    private function returnOrThrowNoSuchElement(?Envelope $envelope = null): Envelope
    {
        if (!$envelope) {
            throw new NoSuchElementException();
        }

        return $envelope;
    }
}