drupol\phpartition\Algorithm\Greedy

Complexity

Total Complexity

3

Size/Duplication

Total Lines	33
Duplicated Lines	0 %

Coupling/Cohesion

Dependencies

3

Test Coverage

Coverage

100%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	getFormattedData()	0	9	1
A	getSubsetWeight()	0	9	2

<?php

namespace drupol\phpartition\Algorithm;

use drupol\phpartition\BasePartitionAlgorithm;
use drupol\phpartition\PartitionAlgorithmInterface;
use drupol\phpartition\Subset;

/**
 * Class Greedy.
 *
 * @package drupol\phpartition\Algorithm
 */
class Greedy extends BasePartitionAlgorithm implements PartitionAlgorithmInterface {

  /**
   * The greedy algorithm needs the input data to be sorted (desc).
   *
   * @return array
   */
  public function getFormattedData() {
    $data = parent::getFormattedData();

    usort($data, function($a, $b) {

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

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

Expected 1 space after FUNCTION keyword; 0 found

      return $a->getValue() < $b->getValue();
    });

    return $data;
  }

  /**
   * @param \drupol\phpartition\Subset $subset
   *
   * @return int|mixed
   */
  public function getSubsetWeight(Subset $subset) {
    $sum = 0;

    foreach($subset->getItems() as $item) {

Expected 1 space after FOREACH keyword; 0 found

      $sum += $item->getValue();
    }

    return $sum;
  }

}