PhpProfiler\Lib\Process\MemoryMap\ProcessModuleMemoryMap

Complexity

Total Complexity

13

Size/Duplication

Total Lines	65
Duplicated Lines	0 %

Importance

Changes	2
Bugs	0	Features	1

5 Methods

Rating	Name	Duplication	Size	Complexity
A	getBaseAddress()	0	10	3
A	getMemoryAddressFromOffset()	0	10	3
A	__construct()	0	3	1
A	getSortedOffsetToMemoryAreaMap()	0	11	3
A	isInRange()	0	8	3

<?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\Lib\Process\MemoryMap;

/**
 * Class ProcessModuleMemoryMap
 * @package PhpProfiler\Lib\Process\MemoryMap
 */
final class ProcessModuleMemoryMap implements ProcessModuleMemoryMapInterface
{
    /** @var ProcessMemoryArea[] */
    private array $memory_areas;
    private ?int $base_address = null;
    /** @var array<int,int>|null  */
    private ?array $sorted_offset_to_memory_map = null;

    /**
     * ProcessModuleMemoryMap constructor.
     * @param ProcessMemoryArea[] $memory_areas
     */
    public function __construct(array $memory_areas)
    {
        $this->memory_areas = $memory_areas;
    }

    public function getBaseAddress(): int
    {
        if (!isset($this->base_address)) {
            $base_address = PHP_INT_MAX;
            foreach ($this->memory_areas as $memory_area) {
                $base_address = min($base_address, hexdec($memory_area->begin));
            }
            $this->base_address = $base_address;
        }
        return $this->base_address;

The expression return $this->base_address could return the type null which is incompatible with the type-hinted return integer. Consider adding an additional type-check to rule them out.

    }

    public function getMemoryAddressFromOffset(int $offset): int
    {
        $ranges = $this->getSortedOffsetToMemoryAreaMap();
        $file_offset_decided = 0;
        foreach ($ranges as $file_offset => $memory_begin) {
            if ($file_offset <= $offset) {
                $file_offset_decided = $file_offset;
            }
        }
        return $ranges[$file_offset_decided] + ($offset - $file_offset_decided);
    }

    public function isInRange(int $address): bool
    {
        foreach ($this->memory_areas as $memory_area) {
            if ($memory_area->isInRange($address)) {
                return true;
            }
        }
        return false;
    }

    /**
     * @return array<int, int>
     */
    private function getSortedOffsetToMemoryAreaMap(): array
    {
        if (!isset($this->sorted_offset_to_memory_map)) {
            $ranges = [];
            foreach ($this->memory_areas as $memory_area) {
                $ranges[hexdec($memory_area->file_offset)] = hexdec($memory_area->begin);
            }
            ksort($ranges);
            $this->sorted_offset_to_memory_map = $ranges;
        }
        return $this->sorted_offset_to_memory_map;

The expression return $this->sorted_offset_to_memory_map could return the type null which is incompatible with the type-hinted return array. Consider adding an additional type-check to rule them out.

    }
}