Sign upLogin

tomasz154/php-streams

View on GitHub
Star

Showing 3 of 3 total issues

The class Stream has 14 public methods. Consider refactoring Stream to keep number of public methods under 10.
Open

class Stream
{
    private $stream;

    protected function __construct(StreamInterface $stream)
Severity: Minor
Found in src/Stream.php by phpmd

TooManyPublicMethods

Since: 0.1

A class with too many public methods is probably a good suspect for refactoring, in order to reduce its complexity and find a way to have more fine grained objects.

By default it ignores methods starting with 'get' or 'set'.

Example

Source https://phpmd.org/rules/codesize.html#toomanypublicmethods

The class Stream has a coupling between objects value of 16. Consider to reduce the number of dependencies under 13.
Open

class Stream
{
    private $stream;

    protected function __construct(StreamInterface $stream)
Severity: Minor
Found in src/Stream.php by phpmd

CouplingBetweenObjects

Since: 1.1.0

A class with too many dependencies has negative impacts on several quality aspects of a class. This includes quality criteria like stability, maintainability and understandability

Example

class Foo {
    /**
     * @var \foo\bar\X
     */
    private $x = null;

    /**
     * @var \foo\bar\Y
     */
    private $y = null;

    /**
     * @var \foo\bar\Z
     */
    private $z = null;

    public function setFoo(\Foo $foo) {}
    public function setBar(\Bar $bar) {}
    public function setBaz(\Baz $baz) {}

    /**
     * @return \SplObjectStorage
     * @throws \OutOfRangeException
     * @throws \InvalidArgumentException
     * @throws \ErrorException
     */
    public function process(\Iterator $it) {}

    // ...
}

Source https://phpmd.org/rules/design.html#couplingbetweenobjects

Missing class import via use statement (line '9', column '33').
Open

        parent::__construct(new \ArrayIterator($array));
Severity: Minor
Found in src/Stream/ArrayStream.php by phpmd

MissingImport

Since: 2.7.0

Importing all external classes in a file through use statements makes them clearly visible.

Example

function make() {
    return new \stdClass();
}

Source http://phpmd.org/rules/cleancode.html#MissingImport

Severity
Category
Status
Source
Language