ObjectCollection::validateType() - Code Metrics - funivan/collection - Measure and Improve Code Quality continuously with Scrutinizer

ObjectCollection::validateType() A
last analyzed 2016-01-25 09:24 UTC

↳ Parent: ObjectCollection

Complexity

Conditions	3
Paths	3

Size

Total Lines	7
Code Lines	5

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
c	1
b	0
f	0
dl	0
loc	7
rs	9.4285
cc	3
eloc	5
nc	3
nop	1

<?php

  namespace Fiv\Collection;

  abstract class ObjectCollection extends TypedCollection {

    /**
     * Used for validation
     * Return class name
     *
     * @codeCoverageIgnore
     * @return string
     */
    public abstract function objectsClassName();

    /**
     * Clone each item
     */
    public function __clone() {
      $items = array();
      foreach ($this->items as $item) {
        $items[] = clone $item;
      }
      $this->setItems($items);
    }


    public function validateType($item) {
      $itemClass = $this->objectsClassName();
      if (($item instanceof $itemClass) == false) {

        $className = ($item === null) ? null : get_class($item);
        throw new \Exception('Invalid object type. Expect ' . $this->objectsClassName() . ' given ' . $className);
      }
    }

  }


1			<?php
2
3			namespace Fiv\Collection;
4
5			abstract class ObjectCollection extends TypedCollection {
6
7			/**
8			* Used for validation
9			* Return class name
10			*
11			* @codeCoverageIgnore
12			* @return string
13			*/
14			public abstract function objectsClassName();
15
16			/**
17			* Clone each item
18			*/
19			public function __clone() {
20			$items = array();
21			foreach ($this->items as $item) {
22			$items[] = clone $item;
23			}
24			$this->setItems($items);
25			}
26
27
28			public function validateType($item) {
29			$itemClass = $this->objectsClassName();
30			if (($item instanceof $itemClass) == false) {
			0 ignored issues – show Coding Style Best Practice introduced 2016-01-25 09:16 UTC by Report Bug Copy Issue Report It seems like you are loosely comparing two booleans. Considering using the strict comparison `===` instead. When comparing two booleans, it is generally considered safer to use the strict comparison operator. Loading history...
31			$className = ($item === null) ? null : get_class($item);
32			throw new \Exception('Invalid object type. Expect ' . $this->objectsClassName() . ' given ' . $className);
33			}
34			}
35
36			}
37

funivan / collection

ObjectCollection::validateType() A last analyzed 2016-01-25 09:24 UTC

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like

ObjectCollection::validateType() A
last analyzed 2016-01-25 09:24 UTC