Redislabs\Module\RedisGraph\Result

Complexity

Total Complexity

15

Size/Duplication

Total Lines	73
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

13 Methods

Rating	Name	Duplication	Size	Complexity
A	getRelationshipsDeleted()	0	3	1
A	getRelationshipsCreated()	0	3	1
A	getPropertiesSet()	0	3	1
A	getNodesCreated()	0	3	1
A	getLabelsAdded()	0	3	1
A	createFromResponse()	0	6	2
A	getNodesDeleted()	0	3	1
A	__construct()	0	3	1
A	prettyPrint()	0	7	2
A	getResultSet()	0	3	1
A	getExecutionTime()	0	3	1
A	getCachedExecution()	0	3	1
A	getLabels()	0	3	1

<?php

declare(strict_types=1);

namespace Redislabs\Module\RedisGraph;

use SevenEcks\Tableify\Tableify;

class Result
{
    private array $statistics;

    public function __construct(private ?array $labels, private ?array $resultSet, Statistics $statistics)
    {
        $this->statistics = $statistics->getResultStatistics();
    }

    public static function createFromResponse(array $response): self
    {
        $stats = $response[2] ?? $response[0];
        $resultKeys = isset($response[1]) ? $response[0] : [];
        $resultSet = $response[1] ?? [];
        return new self($resultKeys, $resultSet, new Statistics($stats));
    }

    public function getResultSet(): array
    {
        return $this->resultSet;

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

    }
    public function getLabels(): array
    {
        return $this->labels;

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

    }


    public function getLabelsAdded(): int
    {
        return $this->statistics['LABELS_ADDED'];
    }

    public function getNodesCreated(): int
    {
        return $this->statistics['NODES_CREATED'];
    }

    public function getNodesDeleted(): int
    {
        return $this->statistics['NODES_DELETED'];
    }

    public function getRelationshipsCreated(): int
    {
        return $this->statistics['RELATIONSHIPS_CREATED'];
    }

    public function getRelationshipsDeleted(): int
    {
        return $this->statistics['RELATIONSHIPS_DELETED'];
    }

    public function getExecutionTime(): float
    {
        return $this->statistics['INTERNAL_EXECUTION_TIME'];
    }

    public function getPropertiesSet(): int
    {
        return $this->statistics['PROPERTIES_SET'];
    }
    public function getCachedExecution(): int
    {
        return $this->statistics['CACHED_EXECUTION'];
    }

    public function prettyPrint(): void
    {
        $table = Tableify::new(array_merge([$this->labels], $this->resultSet));

It seems like $this->resultSet can also be of type null; however, parameter $arrays of array_merge() does only seem to accept array, maybe add an additional type check?

        $table = $table->make();
        $tableData = $table->toArray();
        foreach ($tableData as $row) {
            echo $row . "\n";
        }
    }
}