PhpProfiler\Inspector\Output\TraceFormatter\Compat\CompatCallTraceFormatter

Complexity

Total Complexity

5

Size/Duplication

Total Lines	28
Duplicated Lines	0 %

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	getInstance()	0	6	2
A	format()	0	7	2
A	__construct()	0	4	1

<?php

/**
 * This file is part of the sj-i/php-profiler package.
 *
 * (c) sji <sji@sj-i.dev>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

declare(strict_types=1);

namespace PhpProfiler\Inspector\Output\TraceFormatter\Compat;

use PhpProfiler\Inspector\Output\TraceFormatter\CallTraceFormatter;
use PhpProfiler\Lib\PhpProcessReader\CallTrace;

final class CompatCallTraceFormatter implements CallTraceFormatter
{
    private CompatCallFrameFormatter $call_frame_formatter;
    private static ?self $cache;

    public static function getInstance(): self
    {
        if (!isset(self::$cache)) {
            self::$cache = new self(new CompatCallFrameFormatter());
        }
        return self::$cache;

The expression return self::cache could return the type null which is incompatible with the type-hinted return PhpProfiler\Inspector\Output\TraceFormatter\Compat\CompatCallTraceFormatter. Consider adding an additional type-check to rule them out.

    }

    public function __construct(
        CompatCallFrameFormatter $call_frame_formatter
    ) {
        $this->call_frame_formatter = $call_frame_formatter;
    }



    public function format(CallTrace $call_trace): string
    {
        $frames = [];
        foreach ($call_trace->call_frames as $call_frame) {
            $frames[] = $this->call_frame_formatter->format($call_frame);
        }
        return join(PHP_EOL, $frames);
    }
}