ByJG\RestServer\Writer\StdoutWriter

Complexity

Total Complexity

8

Size/Duplication

Total Lines	43
Duplicated Lines	0 %

Test Coverage

Coverage

76.19%

Importance

Changes	1
Bugs	0	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	flush()	0	6	1
A	echo()	0	3	1
A	responseCode()	0	4	1
A	header()	0	18	5

<?php

namespace ByJG\RestServer\Writer;

class StdoutWriter implements WriterInterface
{
    protected $headerList = [];
    protected $data = '';
    protected $statusCode = 0;

    public function header($header, $replace = true)
    {
        if (preg_match("~^HTTP/~", $header) === 1) {
            array_unshift($this->headerList, $header);
            return;
        }

        if ($replace) {
            $headerParts = explode(':', $header);
            for ($i=0; $i<count($this->headerList); $i++) {

It seems like you are calling the size function count() as part of the test condition. You might want to compute the size beforehand, and not on each iteration.

                if (preg_match("~^" . $headerParts[0] . "~", $this->headerList[$i]) === 1) {
                    $this->headerList[$i] = $header;
                    return;
                }
            }
        }

        $this->headerList[] = $header;
    }

    public function responseCode($responseCode, $description)
    {
        $this->header("HTTP/1.1 $responseCode $description");
        $this->statusCode = $responseCode;
    }

    public function echo($data)
    {
        $this->data .= $data;
    }

    public function flush()
    {
        echo implode("\r\n", $this->headerList);
        echo "\r\n";
        echo "\r\n";
        echo $this->data;
    }
}