Function validateArguments has a Cognitive Complexity of 27 (exceeds 5 allowed). Consider refactoring.

Open

    protected function validateArguments(array $argv, string $caller): array
    {
        $argc = count($argv);

        if (1 == $argc && is_array($argv[0])) {

• Read up
  Read up

The class AbstractPoint has an overall complexity of 58 which is very high. The configured complexity threshold is 50.

Open

abstract class AbstractPoint extends AbstractGeometry implements PointInterface
{
    /**
     * The X coordinate or the longitude.
     */

• Exclude checks
  • Disable engine
  • Disable check

Method validateArguments has 32 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    protected function validateArguments(array $argv, string $caller): array
    {
        $argc = count($argv);

        if (1 == $argc && is_array($argv[0])) {

Consider simplifying this complex logical expression.

Open

            if ((is_numeric($argv[0]) || is_string($argv[0]))
                && (is_numeric($argv[1]) || is_string($argv[1]))
                && (is_numeric($argv[2]) || null === $argv[2] || is_string($argv[2]))
            ) {
                return $argv;

The method validateArguments() has an NPath complexity of 3276. The configured NPath complexity threshold is 200.

Open

    protected function validateArguments(array $argv, string $caller): array
    {
        $argc = count($argv);

        if (1 == $argc && is_array($argv[0])) {

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo {
    function bar() {
        // lots of complicated code
    }
}

The method validateArguments() has a Cyclomatic Complexity of 25. The configured cyclomatic complexity threshold is 10.

Open

    protected function validateArguments(array $argv, string $caller): array
    {
        $argc = count($argv);

        if (1 == $argc && is_array($argv[0])) {

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

// Cyclomatic Complexity = 11
class Foo {
1   public function example() {
2       if ($a == $b) {
3           if ($a1 == $b1) {
                fiddle();
4           } elseif ($a2 == $b2) {
                fiddle();
            } else {
                fiddle();
            }
5       } elseif ($c == $d) {
6           while ($c == $d) {
                fiddle();
            }
7        } elseif ($e == $f) {
8           for ($n = 0; $n < $h; $n++) {
                fiddle();
            }
        } else {
            switch ($z) {
9               case 1:
                    fiddle();
                    break;
10              case 2:
                    fiddle();
                    break;
11              case 3:
                    fiddle();
                    break;
                default:
                    fiddle();
                    break;
            }
        }
    }
}

The method createException has a boolean flag argument $subArray, which is a certain sign of a Single Responsibility Principle violation.

Open

    private function createException(array $argv, string $caller, bool $subArray = false): InvalidValueException

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo {
    public function bar($flag = true) {
    }
}

Identical blocks of code found in 2 locations. Consider refactoring.

Open

            if ((is_numeric($argv[0]) || is_string($argv[0])) && (is_numeric($argv[1]) || is_string($argv[1]))) {

• Read up
  Read up

Identical blocks of code found in 2 locations. Consider refactoring.

Open

            if ((is_numeric($argv[0]) || is_string($argv[0]))
                && (is_numeric($argv[1]) || is_string($argv[1]))

• Read up
  Read up

Avoid variables with short names like $y. Configured minimum length is 3.

Open

    protected float|int $y;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Something {
    private $q = 15; // VIOLATION - Field
    public static function main( array $as ) { // VIOLATION - Formal
        $r = 20 + $this->q; // VIOLATION - Local
        for (int $i = 0; $i < 10; $i++) { // Not a Violation (inside FOR)
            $r += $this->q;
        }
    }
}

Avoid variables with short names like $y. Configured minimum length is 3.

Open

    abstract protected function construct(string $x, string $y, ?int $srid = null): void;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Something {
    private $q = 15; // VIOLATION - Field
    public static function main( array $as ) { // VIOLATION - Formal
        $r = 20 + $this->q; // VIOLATION - Local
        for (int $i = 0; $i < 10; $i++) { // Not a Violation (inside FOR)
            $r += $this->q;
        }
    }
}

Avoid variables with short names like $y. Configured minimum length is 3.

Open

    public function setY(float|int|string $y): static

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Something {
    private $q = 15; // VIOLATION - Field
    public static function main( array $as ) { // VIOLATION - Formal
        $r = 20 + $this->q; // VIOLATION - Local
        for (int $i = 0; $i < 10; $i++) { // Not a Violation (inside FOR)
            $r += $this->q;
        }
    }
}

Avoid variables with short names like $x. Configured minimum length is 3.

Open

    abstract protected function construct(string $x, string $y, ?int $srid = null): void;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Something {
    private $q = 15; // VIOLATION - Field
    public static function main( array $as ) { // VIOLATION - Formal
        $r = 20 + $this->q; // VIOLATION - Local
        for (int $i = 0; $i < 10; $i++) { // Not a Violation (inside FOR)
            $r += $this->q;
        }
    }
}

A file should declare new symbols (classes, functions, constants, etc.) and cause no other side effects, or it should execute logic with side effects, but should not do both. The first symbol is defined on line 50 and the first side effect is on line 33.

Open

<?php

• Exclude checks
  • Disable engine
  • Disable check

Avoid variables with short names like $x. Configured minimum length is 3.

Open

    public function setX(float|int|string $x): static

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Something {
    private $q = 15; // VIOLATION - Field
    public static function main( array $as ) { // VIOLATION - Formal
        $r = 20 + $this->q; // VIOLATION - Local
        for (int $i = 0; $i < 10; $i++) { // Not a Violation (inside FOR)
            $r += $this->q;
        }
    }
}

Avoid variables with short names like $x. Configured minimum length is 3.

Open

    protected float|int $x;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Something {
    private $q = 15; // VIOLATION - Field
    public static function main( array $as ) { // VIOLATION - Formal
        $r = 20 + $this->q; // VIOLATION - Local
        for (int $i = 0; $i < 10; $i++) { // Not a Violation (inside FOR)
            $r += $this->q;
        }
    }
}

Possible parse error: class missing opening or closing brace

Open

abstract class AbstractPoint extends AbstractGeometry implements PointInterface

• Exclude checks
  • Disable engine
  • Disable check

Scope keyword "static" must be followed by a single space

Open

    public function setLongitude(float|int|string $longitude): static

• Exclude checks
  • Disable engine
  • Disable check

Scope keyword "static" must be followed by a single space

Open

    public function setLatitude(float|int|string $latitude): static

• Exclude checks
  • Disable engine
  • Disable check

Possible parse error: class missing opening or closing brace

Open

abstract class AbstractPoint extends AbstractGeometry implements PointInterface

• Exclude checks
  • Disable engine
  • Disable check

Scope keyword "static" must be followed by a single space

Open

    public function setX(float|int|string $x): static

• Exclude checks
  • Disable engine
  • Disable check

Scope keyword "static" must be followed by a single space

Open

    public function setY(float|int|string $y): static

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 129 characters

Open

            throw new InvalidValueException(sprintf(InvalidValueException::OUT_OF_RANGE_LATITUDE, $latitude), $e->getCode(), $e);

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 130 characters

Open

                GeoParserRangeException::SECONDS_OUT_OF_RANGE => sprintf(InvalidValueException::OUT_OF_RANGE_SECOND, $coordinate),

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 131 characters

Open

            throw new InvalidValueException(sprintf(InvalidValueException::OUT_OF_RANGE_LONGITUDE, $longitude), $e->getCode(), $e);

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 133 characters

Open

                GeoParserRangeException::LATITUDE_OUT_OF_RANGE => sprintf(InvalidValueException::OUT_OF_RANGE_LATITUDE, $coordinate),

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 132 characters

Open

            throw new RangeException(sprintf('Coordinate must be comprised between %d and %d, got "%s".', $min, $max, $coordinate));

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 130 characters

Open

                GeoParserRangeException::MINUTES_OUT_OF_RANGE => sprintf(InvalidValueException::OUT_OF_RANGE_MINUTE, $coordinate),

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 124 characters

Open

            throw new InvalidValueException(sprintf('Invalid coordinate value, got "%s".', $coordinate), $e->getCode(), $e);

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 132 characters

Open

            throw new RangeException(sprintf('Coordinate must be comprised between %d and %d, got "%s".', $min, $max, $coordinate));

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 135 characters

Open

                GeoParserRangeException::LONGITUDE_OUT_OF_RANGE => sprintf(InvalidValueException::OUT_OF_RANGE_LONGITUDE, $coordinate),

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 0 spaces, found 4

Open

    }

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 0 spaces, found 4

Open

    public function setLongitude(float|int|string $longitude): static

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 4 spaces, found 8

Open

        try {

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 8 spaces, found 12

Open

            foreach ($argv[0] as $value) {

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 4 spaces, found 8

Open

        if (3 == $argc) {

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 0 spaces, found 4

Open

    private function geoParse(string $coordinate): float|int

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 0 spaces, found 4

Open

    protected float|int $x;

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 0 spaces, found 4

Open

    }

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 0 spaces, found 4

Open

    }

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 4 spaces, found 8

Open

        }

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 8 spaces, found 12

Open

            if (is_numeric($value) || is_string($value)) {

• Exclude checks
  • Disable engine
  • Disable check

Space before opening parenthesis of function call prohibited

Open

            $message = match ($e->getCode()) {

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 0 spaces, found 4

Open

    public function getLatitude(): float|int

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 0 spaces, found 4

Open

    protected float|int $y;

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 0 spaces, found 4

Open

    public function getLongitude(): float|int

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 4 spaces, found 8

Open

        try {

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 4 spaces, found 8

Open

        }

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 4 spaces, found 8

Open

        if (2 == $argc) {

• Exclude checks
  • Disable engine
  • Disable check

Closing brace indented incorrectly; expected 8 spaces, found 4

Open

    }

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 0 spaces, found 4

Open

    public function getX(): float|int

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 4 spaces, found 8

Open

        } catch (RangeException $e) {

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 4 spaces, found 8

Open

        if (1 == $argc && is_array($argv[0])) {

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 8 spaces, found 12

Open

            if ($count < 2 || $count > 3) {

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 0 spaces, found 4

Open

    public function __construct()

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 0 spaces, found 4

Open

    }

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 0 spaces, found 4

Open

    }

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 0 spaces, found 4

Open

    public function toArray(): array

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 4 spaces, found 8

Open

        }

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 0 spaces, found 4

Open

    }

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 8 spaces, found 12

Open

            }

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 4 spaces, found 8

Open

        } catch (GeoParserRangeException $e) {

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 8 spaces, found 16

Open

                default => $e->getMessage(),

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 0 spaces, found 4

Open

    public function setY(float|int|string $y): static

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 8 spaces, found 12

Open

            }

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 0 spaces, found 4

Open

    private function createException(array $argv, string $caller, bool $subArray = false): InvalidValueException

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 4 spaces, found 8

Open

        });

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 4 spaces, found 8

Open

        if ($subArray) {

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 8 spaces, found 12

Open

            }

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 4 spaces, found 8

Open

        }

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 0 spaces, found 4

Open

    }

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 0 spaces, found 4

Open

    }

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 0 spaces, found 4

Open

    }

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 0 spaces, found 4

Open

    }

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 4 spaces, found 8

Open

        try {

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 0 spaces, found 4

Open

    }

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 0 spaces, found 4

Open

    }

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 8 spaces, found 12

Open

            if (is_array($argv[0]) && (is_numeric($argv[1]) || null === $argv[1] || is_string($argv[1]))) {

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 8 spaces, found 12

Open

            if ((is_numeric($argv[0]) || is_string($argv[0])) && (is_numeric($argv[1]) || is_string($argv[1]))) {

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 0 spaces, found 4

Open

    }

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 8 spaces, found 12

Open

            }

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 4 spaces, found 8

Open

        if ($coordinate < $min || $coordinate > $max) {

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 0 spaces, found 4

Open

    public function getY(): float|int

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 0 spaces, found 4

Open

    public function setX(float|int|string $x): static

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 0 spaces, found 4

Open

    }

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 8 spaces, found 12

Open

            }

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 4 spaces, found 8

Open

        }

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 0 spaces, found 4

Open

    protected function validateArguments(array $argv, string $caller): array

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 12 spaces, found 16

Open

                if (is_numeric($value) || is_string($value)) {

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 4 spaces, found 8

Open

        }

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 0 spaces, found 4

Open

    public function setLatitude(float|int|string $latitude): static

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 4 spaces, found 8

Open

        } catch (RangeException $e) {

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 12 spaces, found 16

Open

                }

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 8 spaces, found 12

Open

            }

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 8 spaces, found 12

Open

            if ((is_numeric($argv[0]) || is_string($argv[0]))

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 4 spaces, found 8

Open

        }

• Exclude checks
  • Disable engine
  • Disable check

Closing brace indented incorrectly; expected 4 spaces, found 0

Open

}

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 0 spaces, found 4

Open

    public function getType(): string

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 0 spaces, found 4

Open

    private function checkRange(float|int $coordinate, int $min, int $max): float|int

• Exclude checks
  • Disable engine
  • Disable check