Prime::count() - Code Metrics - Inspection of "Initial commit." - drupol/phpermutations - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 0318a4...4ae196 )

by Pol

created 2016-12-29 00:36 UTC

Prime::count() A

↳ Parent: Prime

Complexity

Conditions	3
Paths	3

Size

Total Lines	11
Code Lines	6

Duplication

Lines	5
Ratio	45.45 %

Code Coverage

Tests	8
CRAP Score	3

Importance

Changes

Metric	Value
dl	5
loc	11
ccs	8
cts	8
cp	1
rs	9.4285
c	0
b	0
f	0
cc	3
eloc	6
nc	3
nop	0
crap	3

<?php

namespace drupol\phpermutations\Iterators;

use drupol\phpermutations\Combinatorics;

/**
 * Class Prime.
 *
 * @package drupol\phpermutations\Iterators
 */
class Prime extends Combinatorics implements \Iterator {

  /**
   * @var int
   */
  protected $max;

  /**
   * @var int
   */
  protected $key;

  /**
   * @var int
   */
  protected $count = 1;

  /**
   * @var int
   */
  protected $current = 2;

  /**
   * Prime constructor.
   */
  public function __construct() {
    $this->setMaxLimit(PHP_INT_MAX);
  }

  /**
   * @inheritdoc
   */
  public function current() {
    return $this->current;
  }

  /**
   * @inheritdoc
   */
  public function next() {
    ++$this->key;
    for($i = $this->key; $i < $this->getMaxLimit(); $i++) {

      if ($this->isPrimeNumber($i)) {
        $this->key = $i;
        $this->current = $i;
        $this->count++;
        break;
      }
    }
  }

  /**
   * @inheritdoc
   */
  public function key() {
    return $this->key;
  }

  /**
   * @inheritdoc
   */
  public function valid() {
    return ($this->current == $this->key());
  }

  /**
   * @inheritdoc
   */
  public function rewind() {
    $this->key = 2;
  }

  /**
   * @return int
   */
  public function count() {
    $this->count = 0;

    for ($j = 2; $j <= $this->getMaxLimit(); $j++) {

      if ($this->isPrimeNumber($j)) {
        $this->count++;
      }
    }

    return $this->count;
  }

  /**
   * @return array
   */
  public function toArray() {
    $data = array();

    for ($this->rewind(); $this->valid(); $this->next()) {
      $data[] = $this->current();
    }

    return $data;
  }


  /**
   * @param $number
   *
   * @return bool
   */
  protected function isPrimeNumber($number) {
    $n = abs($number);

    $i = 2;

    while ($i <= sqrt($n)) {
      if ($n % $i == 0) {
        return FALSE;

      }
      $i++;
    }

    return TRUE;

  }

  /**
   * @param int $max
   */
  public function setMaxLimit($max) {
    $this->max = $max;
  }

  /**
   * @return int
   */
  public function getMaxLimit() {
    return intval($this->max);
  }

}


1			<?php
2
3			namespace drupol\phpermutations\Iterators;
4
5			use drupol\phpermutations\Combinatorics;
6
7			/**
8			* Class Prime.
9			*
10			* @package drupol\phpermutations\Iterators
11			*/
12			class Prime extends Combinatorics implements \Iterator {
13
14			/**
15			* @var int
16			*/
17			protected $max;
18
19			/**
20			* @var int
21			*/
22			protected $key;
23
24			/**
25			* @var int
26			*/
27			protected $count = 1;
28
29			/**
30			* @var int
31			*/
32			protected $current = 2;
33
34			/**
35			* Prime constructor.
36			*/
37	2		public function __construct() {
38	2		$this->setMaxLimit(PHP_INT_MAX);
39	2		}
40
41			/**
42			* @inheritdoc
43			*/
44	2		public function current() {
45	2		return $this->current;
46			}
47
48			/**
49			* @inheritdoc
50			*/
51	2		public function next() {
52	2		++$this->key;
53	2		for($i = $this->key; $i < $this->getMaxLimit(); $i++) {
			0 ignored issues – show Coding Style introduced 2016-12-29 00:39 UTC by Report Bug Copy Issue Report Expected 1 space after FOR keyword; 0 found Loading history...
54	2		if ($this->isPrimeNumber($i)) {
55	2		$this->key = $i;
56	2		$this->current = $i;
57	2		$this->count++;
58	2		break;
59			}
60	2		}
61	2		}
62
63			/**
64			* @inheritdoc
65			*/
66	2		public function key() {
67	2		return $this->key;
68			}
69
70			/**
71			* @inheritdoc
72			*/
73	2		public function valid() {
74	2		return ($this->current == $this->key());
75			}
76
77			/**
78			* @inheritdoc
79			*/
80	2		public function rewind() {
81	2		$this->key = 2;
82	2		}
83
84			/**
85			* @return int
86			*/
87	2		public function count() {
88	2		$this->count = 0;
89
90	2	View Code Duplication	for ($j = 2; $j <= $this->getMaxLimit(); $j++) {
			0 ignored issues – show Duplication introduced 2016-12-29 00:39 UTC by Report Bug Copy Issue Report This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
91	2		if ($this->isPrimeNumber($j)) {
92	2		$this->count++;
93	2		}
94	2		}
95
96	2		return $this->count;
97			}
98
99			/**
100			* @return array
101			*/
102	2		public function toArray() {
103	2		$data = array();
104
105	2		for ($this->rewind(); $this->valid(); $this->next()) {
106	2		$data[] = $this->current();
107	2		}
108
109	2		return $data;
110			}
111
112
113			/**
114			* @param $number
115			*
116			* @return bool
117			*/
118	2		protected function isPrimeNumber($number) {
119	2		$n = abs($number);
			0 ignored issues – show Comprehensibility introduced 2016-12-29 00:39 UTC by Report Bug Copy Issue Report Avoid variables with short names like $n. 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...
120	2		$i = 2;
121
122	2		while ($i <= sqrt($n)) {
123	2		if ($n % $i == 0) {
124	2		return FALSE;
			1 ignored issue – show Coding Style introduced 2016-12-29 00:39 UTC by Report Bug Copy Issue Report `TRUE`, `FALSE` and `NULL` must be lowercase; expected `false`, but found `FALSE`. Loading history...
125			}
126	2		$i++;
127	2		}
128
129	2		return TRUE;
			0 ignored issues – show Coding Style introduced 2016-12-29 00:39 UTC by Report Bug Copy Issue Report `TRUE`, `FALSE` and `NULL` must be lowercase; expected `true`, but found `TRUE`. Loading history...
130			}
131
132			/**
133			* @param int $max
134			*/
135	2		public function setMaxLimit($max) {
136	2		$this->max = $max;
137	2		}
138
139			/**
140			* @return int
141			*/
142	2		public function getMaxLimit() {
143	2		return intval($this->max);
144			}
145
146			}
147

drupol / phpermutations

Push — master ( 0318a4...4ae196 )

Prime::count() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like