Collection::getIterator() - Code Metrics - Inspection of "Remove unused code" - mattbit/flat - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( dd1a15...b9b9a3 )

by Matteo

created 2016-12-18 00:52 UTC

Collection::getIterator() A

↳ Parent: Project

Complexity

Conditions	1
Paths	1

Size

Total Lines	4
Code Lines	2

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	2

Importance

Changes

Metric	Value
dl	0
loc	4
rs	10
c	0
b	0
f	0
ccs	0
cts	2
cp	0
cc	1
eloc	2
nc	1
nop	0
crap	2

<?php

namespace Mattbit\Flat;

use Mattbit\Flat\Query\Parser;
use Mattbit\Flat\Query\Matcher;
use Mattbit\Flat\Storage\DocumentStore;
use Mattbit\Flat\Model\DocumentInterface;
use Mattbit\Flat\Query\Expression\ExpressionInterface;
use Traversable;

class Collection implements \IteratorAggregate

{
    /**
     * @var string
     */
    protected $name;

    /**
     * @var DocumentStore
     */
    protected $store;

    /**
     * @var Database
     */
    protected $database;

    /**
     * @var Parser
     */
    protected $parser;

    /**
     * Construct a new collection.
     *
     * @param Database      $database
     * @param DocumentStore $store
     * @param string        $name
     */
    public function __construct(Database $database, DocumentStore $store, $name)
    {
        $this->name = $name;
        $this->store = $store;
        $this->database = $database;
        $this->parser = $database->getParser();
    }

    /**
     * Drop the collection.
     *
     * @return bool
     */
    public function drop()
    {
        return $this->database->dropCollection($this->name);
    }

    /**
     * Remove all the documents from the collection.
     *
     * @return bool
     */
    public function truncate()
    {
        $this->store->truncate();
    }

    /**
     * Insert a new document.
     *
     * @param DocumentInterface $document
     *
     * @return bool
     */
    public function insert(DocumentInterface $document)
    {
        return $this->store->insert($document);
    }

    /**
     * Update existing documents.
     *
     * @param array        $criteria
     * @param DocumentInterface $updated
     * @param bool         $multiple
     *
     * @return int The count of the documents updated.
     */
    public function update($criteria, DocumentInterface $updated, $multiple = false)

    {
        $limit = $multiple ? 1 : null;
        $documents = $this->onMatch($criteria, $limit);

        foreach ($documents as $document) {
            $this->store->update($updated);
        }
    }

    /**
     * Remove documents from the collection.
     *
     * @param mixed $criteria
     * @param bool  $multiple
     *
     * @return int The count of the document deleted.
     */
    public function remove($criteria, $multiple = false)

    {
        $limit = $multiple ? 1 : null;

        $documents = $this->onMatch($criteria, $limit);

        foreach ($documents as $document) {
            $this->store->remove($document->get('_id'));
        }

        return true;
    }

    /**
     * Find documents in the collection.
     *
     * @param array $criteria
     *
     * @return array The array of results.
     */
    public function find($criteria)
    {
        return $this->onMatch($criteria);
    }

    protected function onMatch($criteria, $limit = null)
    {
        $expression = $this->parser->parse($criteria);
        $matcher = $this->newMatcher($expression);

        $documents = $this->store->scan([$matcher, 'match'], $limit);

        return $documents;
    }

    protected function newMatcher(ExpressionInterface $expression)
    {
        return new Matcher($expression);
    }
}


1			<?php
2
3			namespace Mattbit\Flat;
4
5			use Mattbit\Flat\Query\Parser;
6			use Mattbit\Flat\Query\Matcher;
7			use Mattbit\Flat\Storage\DocumentStore;
8			use Mattbit\Flat\Model\DocumentInterface;
9			use Mattbit\Flat\Query\Expression\ExpressionInterface;
10			use Traversable;
11
12			class Collection implements \IteratorAggregate
			0 ignored issues – show Bug introduced 2016-12-18 00:54 UTC by Report Bug Copy Issue Report There is one abstract method `getIterator` in this class; you could implement it, or declare this class as abstract. Loading history...
13			{
14			/**
15			* @var string
16			*/
17			protected $name;
18
19			/**
20			* @var DocumentStore
21			*/
22			protected $store;
23
24			/**
25			* @var Database
26			*/
27			protected $database;
28
29			/**
30			* @var Parser
31			*/
32			protected $parser;
33
34			/**
35			* Construct a new collection.
36			*
37			* @param Database $database
38			* @param DocumentStore $store
39			* @param string $name
40			*/
41	6		public function __construct(Database $database, DocumentStore $store, $name)
42			{
43	6		$this->name = $name;
44	6		$this->store = $store;
45	6		$this->database = $database;
46	6		$this->parser = $database->getParser();
47	6		}
48
49			/**
50			* Drop the collection.
51			*
52			* @return bool
53			*/
54	1		public function drop()
55			{
56	1		return $this->database->dropCollection($this->name);
57			}
58
59			/**
60			* Remove all the documents from the collection.
61			*
62			* @return bool
63			*/
64	1		public function truncate()
65			{
66	1		$this->store->truncate();
67	1		}
68
69			/**
70			* Insert a new document.
71			*
72			* @param DocumentInterface $document
73			*
74			* @return bool
75			*/
76	1		public function insert(DocumentInterface $document)
77			{
78	1		return $this->store->insert($document);
79			}
80
81			/**
82			* Update existing documents.
83			*
84			* @param array $criteria
85			* @param DocumentInterface $updated
86			* @param bool $multiple
87			*
88			* @return int The count of the documents updated.
89			*/
90		View Code Duplication	public function update($criteria, DocumentInterface $updated, $multiple = false)
			0 ignored issues – show Duplication introduced 2016-04-29 06:19 UTC by Report Bug Copy Issue Report This method seems to be duplicated in 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			{
92			$limit = $multiple ? 1 : null;
93			$documents = $this->onMatch($criteria, $limit);
94
95			foreach ($documents as $document) {
96			$this->store->update($updated);
97			}
98			}
99
100			/**
101			* Remove documents from the collection.
102			*
103			* @param mixed $criteria
104			* @param bool $multiple
105			*
106			* @return int The count of the document deleted.
107			*/
108	1	View Code Duplication	public function remove($criteria, $multiple = false)
			0 ignored issues – show Duplication introduced 2016-04-29 06:19 UTC by Report Bug Copy Issue Report This method seems to be duplicated in 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...
109			{
110	1		$limit = $multiple ? 1 : null;
111
112	1		$documents = $this->onMatch($criteria, $limit);
113
114	1		foreach ($documents as $document) {
115	1		$this->store->remove($document->get('_id'));
116	1		}
117
118	1		return true;
119			}
120
121			/**
122			* Find documents in the collection.
123			*
124			* @param array $criteria
125			*
126			* @return array The array of results.
127			*/
128	1		public function find($criteria)
129			{
130	1		return $this->onMatch($criteria);
131			}
132
133	2		protected function onMatch($criteria, $limit = null)
134			{
135	2		$expression = $this->parser->parse($criteria);
136	2		$matcher = $this->newMatcher($expression);
137
138	2		$documents = $this->store->scan([$matcher, 'match'], $limit);
139
140	2		return $documents;
141			}
142
143	2		protected function newMatcher(ExpressionInterface $expression)
144			{
145	2		return new Matcher($expression);
146			}
147			}
148

mattbit / flat

Push — master ( dd1a15...b9b9a3 )

Collection::getIterator() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like