GraphConstructor::getCommitQueryWithMerge() - Code Metrics - Inspection of "scrutinizer config fixed" - mkorkmaz/redislabs-redisgraph-php - Measure and Improve Code Quality continuously with Scrutinizer

Test Failed

Push — master ( 8960d3...0d17c6 )

by Mehmet

created 2021-06-29 11:06 UTC

GraphConstructor::getCommitQueryWithMerge() A

↳ Parent: GraphConstructor

Complexity

Conditions	3
Paths	4

Size

Total Lines	10
Code Lines	6

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	3
eloc	6
c	0
b	0
f	0
nc	4
nop	0
dl	0
loc	10
rs	10

<?php

declare(strict_types=1);

namespace Redislabs\Module\RedisGraph;

use Redislabs\Module\RedisGraph\Interfaces\QueryInterface;

class GraphConstructor
{
    private string $name;
    private array $nodes = [];
    private array $edges = [];

    public function __construct(string $name)
    {
        $this->name = $name;
    }

    public function addNode(Node $node): void
    {
        $this->nodes[] = $node;
    }

    public function addEdge(Edge $edge): void
    {
        $this->edges[] = $edge;
    }

    public function getCommitQuery(): QueryInterface
    {
        $query = 'CREATE ';
        foreach ($this->nodes as $index => $node) {
            $query .= $node->toString() . ', ';
        }
        $edgeCount = count($this->edges);
        foreach ($this->edges as $index => $edge) {
            $query .= $edge->toString();
            if ($index < $edgeCount - 1) {
                $query .= ', ';
            }
        }
        return new Query($this->name, $query);
    }

    public function getCommitQueryWithMerge(): QueryInterface
    {
        $query = '';
        foreach ($this->nodes as $index => $node) {
            $query .= 'MERGE ' . $node->toString() . ' ';
        }
        foreach ($this->edges as $index => $edge) {
            $query .= 'MERGE ' . $edge->toString() . ' ';
        }
        return new Query($this->name, trim($query));
    }
}


1			<?php
2
3			declare(strict_types=1);
4
5			namespace Redislabs\Module\RedisGraph;
6
7			use Redislabs\Module\RedisGraph\Interfaces\QueryInterface;
8
9			class GraphConstructor
10			{
11			private string $name;
12			private array $nodes = [];
13			private array $edges = [];
14
15			public function __construct(string $name)
16			{
17			$this->name = $name;
18			}
19
20			public function addNode(Node $node): void
21			{
22			$this->nodes[] = $node;
23			}
24
25			public function addEdge(Edge $edge): void
26			{
27			$this->edges[] = $edge;
28			}
29
30			public function getCommitQuery(): QueryInterface
31			{
32			$query = 'CREATE ';
33			foreach ($this->nodes as $index => $node) {
34			$query .= $node->toString() . ', ';
35			}
36			$edgeCount = count($this->edges);
37			foreach ($this->edges as $index => $edge) {
38			$query .= $edge->toString();
39			if ($index < $edgeCount - 1) {
40			$query .= ', ';
41			}
42			}
43			return new Query($this->name, $query);
44			}
45
46			public function getCommitQueryWithMerge(): QueryInterface
47			{
48			$query = '';
49			foreach ($this->nodes as $index => $node) {
50			$query .= 'MERGE ' . $node->toString() . ' ';
51			}
52			foreach ($this->edges as $index => $edge) {
53			$query .= 'MERGE ' . $edge->toString() . ' ';
54			}
55			return new Query($this->name, trim($query));
56			}
57			}
58

mkorkmaz / redislabs-redisgraph-php

Push — master ( 8960d3...0d17c6 )

GraphConstructor::getCommitQueryWithMerge() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like