Phpml\Clustering\KMeans\Cluster

Complexity

Total Complexity

19

Size/Duplication

Total Lines	116
Duplicated Lines	5.17 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes

0

12 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	6	1
A	getPoints()	0	9	2
A	getPointsWithLabels()	0	9	2
A	toArray()	0	7	1
A	attach()	0	10	2
A	detach()	0	6	1
A	attachAll()	0	4	1
A	detachAll()	0	4	1
B	updateCentroid()	6	19	5
A	getIterator()	0	4	1
A	count()	0	4	1
A	setCoordinates()	0	4	1

<?php

declare(strict_types=1);

namespace Phpml\Clustering\KMeans;

use Countable;
use IteratorAggregate;
use LogicException;
use SplObjectStorage;

class Cluster extends Point implements IteratorAggregate, Countable
{
    /**
     * @var Space
     */
    protected $space;

    /**
     * @var SplObjectStorage|Point[]
     */
    protected $points;

    public function __construct(Space $space, array $coordinates)
    {
        parent::__construct($coordinates);
        $this->space = $space;
        $this->points = new SplObjectStorage();
    }

    public function getPoints(): array
    {
        $points = [];
        foreach ($this->points as $point) {
            $points[] = $point->toArray();
        }

        return $points;
    }

    public function getPointsWithLabels(): array
    {
        $points = [];
        foreach ($this->points as $point) {
            $points[$point->label] = $point->toArray();
        }

        return $points;
    }

    public function toArray(): array
    {
        return [
            'centroid' => parent::toArray(),
            'points' => $this->getPoints(),
        ];
    }

    public function attach(Point $point): Point
    {
        if ($point instanceof self) {
            throw new LogicException('Cannot attach a cluster to another');
        }

        $this->points->attach($point);

        return $point;
    }

    public function detach(Point $point): Point
    {
        $this->points->detach($point);

        return $point;
    }

    public function attachAll(SplObjectStorage $points): void
    {
        $this->points->addAll($points);
    }

    public function detachAll(SplObjectStorage $points): void
    {
        $this->points->removeAll($points);
    }

    public function updateCentroid(): void
    {
        $count = count($this->points);
        if ($count === 0) {
            return;
        }

        $centroid = $this->space->newPoint(array_fill(0, $this->dimension, 0));

        foreach ($this->points as $point) {
            for ($n = 0; $n < $this->dimension; ++$n) {

This code seems to be duplicated across your project.

                $centroid->coordinates[$n] += $point->coordinates[$n];
            }
        }

        for ($n = 0; $n < $this->dimension; ++$n) {

This code seems to be duplicated across your project.

            $this->coordinates[$n] = $centroid->coordinates[$n] / $count;
        }
    }

    /**
     * @return Point[]|SplObjectStorage
     */
    public function getIterator()
    {
        return $this->points;

The expression $this->points; of type SplObjectStorage|Phpml\Clustering\KMeans\Point[] adds the type Phpml\Clustering\KMeans\Point[] to the return on line 112 which is incompatible with the return type declared by the interface IteratorAggregate::getIterator of type Traversable.

    }

    /**
     * @return mixed
     */
    public function count()
    {
        return count($this->points);
    }

    public function setCoordinates(array $newCoordinates): void
    {
        $this->coordinates = $newCoordinates;
    }
}