Spatie\Regex\ReplaceResult

Complexity

Total Complexity

7

Size/Duplication

Total Lines	61
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	2

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	8	1
B	for()	0	24	4
A	result()	0	4	1
A	count()	0	4	1

<?php

namespace Spatie\Regex;

class ReplaceResult extends RegexResult
{
    /** @var string|array */
    protected $pattern;

    /** @var string|array */
    protected $replacement;

    /** @var string|array */
    protected $subject;

    /** @var string|array */
    protected $result;

    /** @var int */
    protected $count;

    public function __construct($pattern, $replacement, $subject, $result, int $count)
    {
        $this->pattern = $pattern;
        $this->replacement = $replacement;
        $this->subject = $subject;
        $this->result = $result;
        $this->count = $count;
    }

    public static function for($pattern, $replacement, $subject, $limit)

Possible parse error: non-abstract method defined as abstract

It is generally advisable to only define one property per statement.

    {
        $count = 0;

        try {
            if (is_callable($replacement)) {

It is generally advisable to only define one property per statement.

The visibility should be declared for property $replacement.

                $replacement = function (array $matches) use ($pattern, $subject, $replacement) {
                    return $replacement(new MatchResult($pattern, $subject, true, $matches));
                };

                $result = preg_replace_callback($pattern, $replacement, $subject, $limit, $count);

It is generally advisable to only define one property per statement.

The visibility should be declared for property $result.

            } else {
                $result = preg_replace($pattern, $replacement, $subject, $limit, $count);

It is generally advisable to only define one property per statement.

The visibility should be declared for property $result.

            }
        } catch (Exception $exception) {

The class Spatie\Regex\Exception does not exist. Did you forget a USE statement, or did you not list all dependencies?

It is generally advisable to only define one property per statement.

The visibility should be declared for property $exception.

            throw RegexFailed::replace($pattern, $subject, $exception->getMessage());
        }

        if ($result === null) {

It is generally advisable to only define one property per statement.

The visibility should be declared for property $result.

            throw RegexFailed::replace($pattern, $subject, static::lastPregError());
        }

        return new static($pattern, $replacement, $subject, $result, $count);

It is generally advisable to only define one property per statement.

The visibility should be declared for property $pattern.

    }

    public function result()
    {
        return $this->result;
    }

    public function count()
    {
        return $this->count;
    }
}