Inbounder\Parsers\AbstractParser

Complexity

Total Complexity

6

Size/Duplication

Total Lines	69
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	__set()	0	4	1
A	__get()	0	6	2
	parse()	0	1	?
A	request()	0	4	1
A	input()	0	4	1
A	toArray()	0	4	1

<?php

namespace Inbounder\Parsers;

use Illuminate\Contracts\Support\Arrayable;
use Illuminate\Http\Request;
use Inbounder\Parsers\Contracts\ParserInterface;

abstract class AbstractParser implements ParserInterface, Arrayable
{
    /**
     * Request.
     */
    protected $request;

    /**
     * Attributes.
     */
    protected $attributes;


    public function __set($name, $value)
    {
        $this->attributes[$name] = $value;
    }

    /**
     * Getter.
     */
    public function __get($name)
    {
        if (array_key_exists($name, $this->attributes)) {
            return $this->attributes[$name];
        }
    }

    /**
     * Parse the request and return itself.
     *
     * @return ParserInterface
     */
    abstract public function parse() : ParserInterface;

    /**
     * Set the request that will be parsed.
     *
     * @param Request $request
     *
     * @return void
     */
    public function request(Request $request)

You have injected the Request via parameter $request. This is generally not recommended as there might be multiple instances during a request cycle (f.e. when using sub-requests). Instead, it is recommended to inject the RequestStack and retrieve the current request each time you need it via getCurrentRequest().

    {
        $this->request = $request;
    }

    /**
     * Retrieve an input from the request.
     *
     * @param string $name
     *
     * @return mixed
     */
    public function input($name)
    {
        return $this->request->input($name);
    }

    /**
     * Get the instance as an array.
     *
     * @return array
     */
    public function toArray()
    {
        return $this->attributes;
    }
}