Demv\JSend\JSendResponseTrait

Complexity

Total Complexity

5

Size/Duplication

Total Lines	62
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	5

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	respond()	0	8	1
A	getStatusCode()	0	4	1
A	withStatus()	0	13	2
A	getReasonPhrase()	0	4	1

<?php

namespace Demv\JSend;

use Dgame\Ensurance\Exception\EnsuranceException;
use GuzzleHttp\Psr7\MessageTrait;
use function Dgame\Ensurance\ensure;

/**
 * Trait JSendResponseTrait
 * @package Demv\JSend
 */
trait JSendResponseTrait
{
    use MessageTrait;

    /**
     * @var string
     */
    private $message;
    /**
     * @var int|null
     */
    private $code;

    /**
     * @param int|null $code
     */
    final public function respond(int $code = null): void
    {
        $code = $code ?? $this->getStatusCode();
        ensure($code)->isInt()->isBetween(100, 511);
        header('Content-Type: application/json; charset="UTF-8"', true, $code);
        print json_encode($this);
        exit;
    }

    /**
     * @return int
     */
    final public function getStatusCode()
    {
        return $this->code ?? $this->getDefaultHttpStatusCode();

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

    }

    /**
     * @param        $code
     * @param string $reasonPhrase
     *
     * @return JSend|JSendInterface
     * @throws EnsuranceException
     */
    public function withStatus($code, $reasonPhrase = '')
    {
        $decoded           = $this->asArray();

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

        $decoded['status'] = (string) Status::fromStatusCode($code ?? $this->getDefaultHttpStatusCode());

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

        $jsend       = JSend::from($decoded)->into();
        $jsend->code = $code;

Accessing code on the interface Demv\JSend\JSendInterface suggest that you code against a concrete implementation. How about adding an instanceof check?

        if (!empty($reasonPhrase)) {
            $jsend->message = $reasonPhrase;

Accessing message on the interface Demv\JSend\JSendInterface suggest that you code against a concrete implementation. How about adding an instanceof check?

        }

        return $jsend;
    }

    /**
     * @return string
     */
    public function getReasonPhrase()
    {
        return $this->message;
    }
}