Greedy::getFormattedData() - Code Metrics - Inspection of "Initial commit." - drupol/phpartition - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 8c892e...6a431f )

by Pol

created 2016-12-17 21:42 UTC

Greedy::getFormattedData() A

↳ Parent: Greedy

Complexity

Conditions	1
Paths	1

Size

Total Lines	9
Code Lines	5

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	2

Importance

Changes

Metric	Value
dl	0
loc	9
ccs	0
cts	0
cp	0
rs	9.6666
c	0
b	0
f	0
cc	1
eloc	5
nc	1
nop	0
crap	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) {

      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) {

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

    return $sum;
  }

}

1			<?php
2
3			namespace drupol\phpartition\Algorithm;
4
5			use drupol\phpartition\BasePartitionAlgorithm;
6			use drupol\phpartition\PartitionAlgorithmInterface;
7			use drupol\phpartition\Subset;
8
9			/**
10			* Class Greedy.
11			*
12			* @package drupol\phpartition\Algorithm
13			*/
14			class Greedy extends BasePartitionAlgorithm implements PartitionAlgorithmInterface {
15
16			/**
17			* The greedy algorithm needs the input data to be sorted (desc).
18			*
19			* @return array
20			*/
21			public function getFormattedData() {
22			$data = parent::getFormattedData();
23
24			usort($data, function($a, $b) {
			0 ignored issues – show Comprehensibility introduced 2016-12-17 21:54 UTC by Report Bug Copy Issue Report Avoid variables with short names like $a. Configured minimum length is 3. Short variable names may make your code harder to understand. Variable names should be self-descriptive. This check looks for variable names who are shorter than a configured minimum. Loading history... Comprehensibility introduced 2016-12-17 21:54 UTC by Report Bug Copy Issue Report Avoid variables with short names like $b. Configured minimum length is 3. Short variable names may make your code harder to understand. Variable names should be self-descriptive. This check looks for variable names who are shorter than a configured minimum. Loading history... Coding Style introduced 2016-12-17 21:54 UTC by Report Bug Copy Issue Report Expected 1 space after FUNCTION keyword; 0 found Loading history...
25			return $a->getValue() < $b->getValue();
26			});
27
28			return $data;
29			}
30
31			/**
32			* @param \drupol\phpartition\Subset $subset
33			*
34			* @return int\|mixed
35			*/
36			public function getSubsetWeight(Subset $subset) {
37			$sum = 0;
38
39			foreach($subset->getItems() as $item) {
			0 ignored issues – show Coding Style introduced 2016-12-17 21:54 UTC by Report Bug Copy Issue Report Expected 1 space after FOREACH keyword; 0 found Loading history...
40			$sum += $item->getValue();
41			}
42
43			return $sum;
44			}
45
46			}

drupol / phpartition

Push — master ( 8c892e...6a431f )

Greedy::getFormattedData() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like