DateInjector::tryToGetDateObjFromString() - Code Metrics - Inspection of "Reduce complexity of some functions" - Level-2/Maphper - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 562277...03179d )

by Richard

created 2017-12-31 03:09 UTC

DateInjector::tryToGetDateObjFromString() A

↳ Parent: DateInjector

Complexity

Conditions	4
Paths	4

Size

Total Lines	8
Code Lines	5

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	4
eloc	5
nc	4
nop	1
dl	0
loc	8
rs	9.2
c	0
b	0
f	0

<?php
namespace Maphper\Lib;
//Replaces dates in an object graph with \DateTime instances
class DateInjector {
	private $processCache;

	public function replaceDates($obj, $reset = true) {
		//prevent infinite recursion, only process each object once
		if ($this->checkCache($obj, $reset)) return $obj;

		if ($this->isIterable($obj)) foreach ($obj as &$o) $o = $this->replaceDates($o, false);
		if ($this->isPossiblyDateString($obj)) $obj = $this->tryToGetDateObjFromString($obj);
		return $obj;
	}

    private function tryToGetDateObjFromString($obj) {
        try {
            $date = new \DateTime($obj);
            if ($date->format('Y-m-d H:i:s') == substr($obj, 0, 20) || $date->format('Y-m-d') == substr($obj, 0, 10)) $obj = $date;
        }
        catch (\Exception $e) {	//Doesn't need to do anything as the try/catch is working out whether $obj is a date
        }
        return $obj;
    }

    private function isIterable($obj) {
        return is_array($obj) || (is_object($obj) && ($obj instanceof \Iterator));
    }

    private function isPossiblyDateString($obj) {
        return is_string($obj) && isset($obj[0]) && is_numeric($obj[0]) && strlen($obj) <= 20;
    }

	private function checkCache($obj, $reset) {
		if ($reset) $this->processCache = new \SplObjectStorage();
        if (!is_object($obj)) return false;

		if ($this->processCache->contains($obj)) return $obj;
		else $this->processCache->attach($obj, true);

		return false;
	}
}


1			<?php
2			namespace Maphper\Lib;
3			//Replaces dates in an object graph with \DateTime instances
4			class DateInjector {
5			private $processCache;
6
7			public function replaceDates($obj, $reset = true) {
8			//prevent infinite recursion, only process each object once
9			if ($this->checkCache($obj, $reset)) return $obj;
10
11			if ($this->isIterable($obj)) foreach ($obj as &$o) $o = $this->replaceDates($o, false);
12			if ($this->isPossiblyDateString($obj)) $obj = $this->tryToGetDateObjFromString($obj);
13			return $obj;
14			}
15
16			private function tryToGetDateObjFromString($obj) {
17			try {
18			$date = new \DateTime($obj);
19			if ($date->format('Y-m-d H:i:s') == substr($obj, 0, 20) \|\| $date->format('Y-m-d') == substr($obj, 0, 10)) $obj = $date;
20			}
21			catch (\Exception $e) { //Doesn't need to do anything as the try/catch is working out whether $obj is a date
22			}
23			return $obj;
24			}
25
26			private function isIterable($obj) {
27			return is_array($obj) \|\| (is_object($obj) && ($obj instanceof \Iterator));
28			}
29
30			private function isPossiblyDateString($obj) {
31			return is_string($obj) && isset($obj[0]) && is_numeric($obj[0]) && strlen($obj) <= 20;
32			}
33
34			private function checkCache($obj, $reset) {
35			if ($reset) $this->processCache = new \SplObjectStorage();
36			if (!is_object($obj)) return false;
37
38			if ($this->processCache->contains($obj)) return $obj;
39			else $this->processCache->attach($obj, true);
40
41			return false;
42			}
43			}
44

Level-2 / Maphper

Push — master ( 562277...03179d )

DateInjector::tryToGetDateObjFromString() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like