YmlCatalog::getNewModel() - Code Metrics - Inspection of "Добавил поддержку ActiveDataProvider" - pastuhov/yii2-yml-catalog - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#38)

unknown

created 2017-05-18 12:50 UTC

YmlCatalog::getNewModel() B

↳ Parent: YmlCatalog

Complexity

Conditions	8
Paths	10

Size

Total Lines	18
Code Lines	13

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	72

Importance

Changes

Metric	Value
c	0
b	0
f	0
dl	0
loc	18
ccs	0
cts	0
cp	0
rs	7.7777
cc	8
eloc	13
nc	10
nop	1
crap	72

<?php
namespace pastuhov\ymlcatalog;

use pastuhov\ymlcatalog\models\BaseModel;
use pastuhov\ymlcatalog\models\Category;
use pastuhov\ymlcatalog\models\Currency;
use pastuhov\ymlcatalog\models\LocalDeliveryCost;
use pastuhov\ymlcatalog\models\Shop;
use pastuhov\ymlcatalog\models\SimpleOffer;
use pastuhov\ymlcatalog\models\DeliveryOption;
use Yii;
use pastuhov\FileStream\BaseFileStream;
use yii\base\Exception;
use yii\data\ActiveDataProvider;
use yii\db\ActiveQuery;

/**
 * Yml генератор каталога.
 *
 * @package pastuhov\ymlcatalog
 */
class YmlCatalog
{
    /**
     * @var BaseFileStream
     */
    protected $handle;
    /**
     * @var string
     */
    protected $shopClass;
    /**
     * @var string
     */
    protected $currencyClass;
    /**
     * @var string
     */
    protected $categoryClass;
    /**
     * @var null|string
     */
    protected $localDeliveryCostClass;
    /**
     * @var string
     */
    protected $offerClass;
    /**
     * @var null|string
     */
    protected $date;

    /**
     * @var null|callable
     */
    protected $onValidationError;

    /**
     * @var null|string
     */
    protected $customOfferClass;

    /**
     * @var null|string
     */
    protected $deliveryOptionClass;

    /**
     * @param BaseFileStream $handle
     * @param string $shopClass class name
     * @param string $currencyClass class name
     * @param string $categoryClass class name
     * @param string $localDeliveryCostClass class name
     * @param array $offerClasses
     * @param null|string $date
     * @param null|callable $onValidationError
     * @param null|string $customOfferClass
     */
    public function __construct(
        BaseFileStream $handle,
        $shopClass,
        $currencyClass,
        $categoryClass,
        $localDeliveryCostClass = null,
        Array $offerClasses,
        $date = null,
        $onValidationError = null,
        $customOfferClass = null,
        $deliveryOptionClass = null
    ) {
        $this->handle = $handle;
        $this->shopClass = $shopClass;
        $this->currencyClass = $currencyClass;
        $this->categoryClass = $categoryClass;
        $this->localDeliveryCostClass = $localDeliveryCostClass;
        $this->offerClasses = $offerClasses;

        $this->date = $date;
        $this->onValidationError = $onValidationError;
        $this->customOfferClass = $customOfferClass;
        $this->deliveryOptionClass = $deliveryOptionClass;
    }

    /**
     * @throws Exception
     */
    public function generate()
    {
        $date = $this->getDate();

        $this->write(
            '<?xml version="1.0" encoding="utf-8"?>' . PHP_EOL .
            '<!DOCTYPE yml_catalog SYSTEM "shops.dtd">' . PHP_EOL .
            '<yml_catalog date="' . $date . '">' . PHP_EOL
        );

        $this->writeTag('shop');
        $this->writeModel(new Shop(), new $this->shopClass());
        $this->writeTag('currencies');
        $this->writeEachModel($this->currencyClass);
        $this->writeTag('/currencies');
        $this->writeTag('categories');
        $this->writeEachModel($this->categoryClass);
        $this->writeTag('/categories');
        if($this->deliveryOptionClass) {
''   == false // true
''   == null  // true
'ab' == false // false
'ab' == null  // false

// It is often better to use strict comparison
'' === false // false
'' === null  // false
            $this->writeTag('delivery-options');
            $this->writeModel(new DeliveryOption(), \Yii::createObject($this->deliveryOptionClass));
            $this->writeTag('/delivery-options');
        }
        if($this->localDeliveryCostClass) {
''   == false // true
''   == null  // true
'ab' == false // false
'ab' == null  // false

// It is often better to use strict comparison
'' === false // false
'' === null  // false
            $this->writeModel(new LocalDeliveryCost(), \Yii::createObject($this->localDeliveryCostClass));

        }
        $this->writeTag('offers');
        foreach ($this->offerClasses as $offerClass) {

            $this->writeEachModel($offerClass);
        }
        $this->writeTag('/offers');
        $this->writeTag('/shop');

        $this->write('</yml_catalog>');
    }

    /**
     * @return null|string
     */
    protected function getDate()
    {
        $date = $this->date;

        if ($date === null) {
            $date = Yii::$app->formatter->asDatetime(new \DateTime(), 'php:Y-m-d H:i');
        }

        return $date;
    }

    /**
     * @param string $string
     * @throws \Exception
     */
    protected function write($string)
    {
        $this->handle->write($string);
    }

    /**
     * @param string $string tag name
     */
    protected function writeTag($string)
    {
        $this->write('<' . $string . '>' . PHP_EOL);
    }

    /**
     * @param BaseModel $model
     * @param $valuesModel
     * @throws Exception
     */
    protected function writeModel(BaseModel $model, $valuesModel)
    {
        if (method_exists($valuesModel, 'getParams')) {
            $model->setParams($valuesModel->getParams());
        }
        if (method_exists($valuesModel, 'getPictures')) {
            $model->setPictures($valuesModel->getPictures());
        }
        if(method_exists($valuesModel, 'getDeliveryOptions')) {
            $model->setDeliveryOptions($valuesModel->getDeliveryOptions());
        }

        if($model->loadModel($valuesModel, $this->onValidationError)) {
/**
 * @return array|string
 */
function returnsDifferentValues($x) {
    if ($x) {
        return 'foo';
    }

    return array();
}

$x = returnsDifferentValues($y);
if (is_array($x)) {
    // $x is an array.
}
            $string = $model->getYml();
            $this->write($string);
        }
    }

    /**
     * @param string|array $modelClass class name or yii configuration array. You can also set params:
     *      `findParams`:   array of additional find params;
     *      `query`:        ActiveQuery object to generate yml use already created object;
     *      `dataProvider`: ActiveDataProvider or true to generate yml with pagination;
     */
    protected function writeEachModel($modelClass)
    {
        /**
         * @var mixed
         */
        $findParams = [];

        /**
         * @var ActiveQuery
         */
        $query = null;

        /**
         * @var ActiveDataProvider
         */
        $dataProvider = null;

        if (is_array($modelClass)) {
            foreach (['findParams', 'query', 'dataProvider'] as $name) {
                if (array_key_exists($name, $modelClass)) {
                    $$name = $modelClass[$name];
                    unset($modelClass[$name]);
                }
            }
        }

        /**
         * @var BaseFindYmlInterface $class
         */
        $class = \Yii::createObject($modelClass);

        /**
         * @var ReaderInterface
         */
        $reader = ReaderFactory::build(
            $class,
            $dataProvider,
            $query,
            $findParams
        );

        $newModel = $this->getNewModel($class);

        foreach ($reader as $models) {
            foreach ($models as $model) {
                $this->writeModel($newModel, $model);
            }
            $this->gc();
        }
    }

    /**
     * @param $modelClass
     * @return Category|Currency|SimpleOffer
     * @throws Exception
     */
    protected function getNewModel($modelClass)
    {
        $obj = is_object($modelClass) ? $modelClass : \Yii::createObject($modelClass);

        if ($obj instanceof CurrencyInterface) {
            $model = new Currency();
        } elseif ($obj instanceof CategoryInterface) {
            $model = new Category();
        } elseif ($obj instanceof CustomOfferInterface && $this->customOfferClass !== null && class_exists($this->customOfferClass)) {
            $model = \Yii::createObject($this->customOfferClass);
        } elseif ($obj instanceof SimpleOfferInterface) {
            $model = new SimpleOffer();
        } else {
            throw new Exception('Model ' . get_class($obj) . ' has unknown interface');
        }

        return $model;
    }
    
    /**
     * Performs PHP memory garbage collection.
     */
    protected function gc()
    {
        if (!gc_enabled()) {
            gc_enable();
        }
        gc_collect_cycles();
    }
}


1		<?php
2		namespace pastuhov\ymlcatalog;
3
4		use pastuhov\ymlcatalog\models\BaseModel;
5		use pastuhov\ymlcatalog\models\Category;
6		use pastuhov\ymlcatalog\models\Currency;
7		use pastuhov\ymlcatalog\models\LocalDeliveryCost;
8		use pastuhov\ymlcatalog\models\Shop;
9		use pastuhov\ymlcatalog\models\SimpleOffer;
10		use pastuhov\ymlcatalog\models\DeliveryOption;
11		use Yii;
12		use pastuhov\FileStream\BaseFileStream;
13		use yii\base\Exception;
14		use yii\data\ActiveDataProvider;
15		use yii\db\ActiveQuery;
16
17		/**
18		* Yml генератор каталога.
19		*
20		* @package pastuhov\ymlcatalog
21		*/
22		class YmlCatalog
23		{
24		/**
25		* @var BaseFileStream
26		*/
27		protected $handle;
28		/**
29		* @var string
30		*/
31		protected $shopClass;
32		/**
33		* @var string
34		*/
35		protected $currencyClass;
36		/**
37		* @var string
38		*/
39		protected $categoryClass;
40		/**
41		* @var null\|string
42		*/
43		protected $localDeliveryCostClass;
44		/**
45		* @var string
46		*/
47		protected $offerClass;
48		/**
49		* @var null\|string
50		*/
51		protected $date;
52
53		/**
54		* @var null\|callable
55		*/
56		protected $onValidationError;
57
58		/**
59		* @var null\|string
60		*/
61		protected $customOfferClass;
62
63		/**
64		* @var null\|string
65		*/
66		protected $deliveryOptionClass;
67
68		/**
69		* @param BaseFileStream $handle
70		* @param string $shopClass class name
71	9	* @param string $currencyClass class name
72		* @param string $categoryClass class name
73		* @param string $localDeliveryCostClass class name
74	2	* @param array $offerClasses
75		* @param null\|string $date
76		* @param null\|callable $onValidationError
77		* @param null\|string $customOfferClass
78		*/
79		public function __construct(
80		BaseFileStream $handle,
81		$shopClass,
82	9	$currencyClass,
83	9	$categoryClass,
84	9	$localDeliveryCostClass = null,
85	9	Array $offerClasses,
86	9	$date = null,
87	9	$onValidationError = null,
88	9	$customOfferClass = null,
89	9	$deliveryOptionClass = null
90	9	) {
91	9	$this->handle = $handle;
92		$this->shopClass = $shopClass;
93		$this->currencyClass = $currencyClass;
94		$this->categoryClass = $categoryClass;
95		$this->localDeliveryCostClass = $localDeliveryCostClass;
96	9	$this->offerClasses = $offerClasses;
		0 ignored issues – show Bug introduced 2016-01-15 12:20 UTC by Report Bug Copy Issue Report The property `offerClasses` does not seem to exist. Did you mean `offerClass`? An attempt at access to an undefined property has been detected. This may either be a typographical error or the property has been renamed but there are still references to its old name. If you really want to allow access to undefined properties, you can define magic methods to allow access. See the php core documentation on Overloading. Loading history...
97		$this->date = $date;
98	9	$this->onValidationError = $onValidationError;
99		$this->customOfferClass = $customOfferClass;
100	9	$this->deliveryOptionClass = $deliveryOptionClass;
101	9	}
102	9
103	9	/**
104	6	* @throws Exception
105		*/
106	9	public function generate()
107	9	{
108	9	$date = $this->getDate();
109	9
110	9	$this->write(
111	9	'<?xml version="1.0" encoding="utf-8"?>' . PHP_EOL .
112	9	'<!DOCTYPE yml_catalog SYSTEM "shops.dtd">' . PHP_EOL .
113	9	'<yml_catalog date="' . $date . '">' . PHP_EOL
114	9	);
115	9
116	9	$this->writeTag('shop');
117	9	$this->writeModel(new Shop(), new $this->shopClass());
118	4	$this->writeTag('currencies');
119	6	$this->writeEachModel($this->currencyClass);
120	8	$this->writeTag('/currencies');
121		$this->writeTag('categories');
122	6	$this->writeEachModel($this->categoryClass);
123	6	$this->writeTag('/categories');
124		if($this->deliveryOptionClass) {
		0 ignored issues – show Bug Best Practice introduced 2016-08-23 05:49 UTC by Report Bug Copy Issue Report The expression `$this->deliveryOptionClass` of type `null\|string` is loosely compared to `true`; this is ambiguous if the string can be empty. You might want to explicitly use `!== null` instead. In PHP, under loose comparison (like `==`, or `!=`, or `switch` conditions), values of different types might be equal. For `string` values, the empty string `''` is a special case, in particular the following results might be unexpected: '' == false // true '' == null // true 'ab' == false // false 'ab' == null // false // It is often better to use strict comparison '' === false // false '' === null // false Loading history...
125		$this->writeTag('delivery-options');
126		$this->writeModel(new DeliveryOption(), \Yii::createObject($this->deliveryOptionClass));
127		$this->writeTag('/delivery-options');
128	9	}
129		if($this->localDeliveryCostClass) {
		0 ignored issues – show Bug Best Practice introduced 2016-08-23 05:49 UTC by Report Bug Copy Issue Report The expression `$this->localDeliveryCostClass` of type `null\|string` is loosely compared to `true`; this is ambiguous if the string can be empty. You might want to explicitly use `!== null` instead. In PHP, under loose comparison (like `==`, or `!=`, or `switch` conditions), values of different types might be equal. For `string` values, the empty string `''` is a special case, in particular the following results might be unexpected: '' == false // true '' == null // true 'ab' == false // false 'ab' == null // false // It is often better to use strict comparison '' === false // false '' === null // false Loading history...
130	9	$this->writeModel(new LocalDeliveryCost(), \Yii::createObject($this->localDeliveryCostClass));
		0 ignored issues – show Deprecated Code introduced 2016-08-23 06:53 UTC by Report Bug Copy Issue Report The class `pastuhov\ymlcatalog\models\LocalDeliveryCost` has been deprecated. This class, trait or interface has been deprecated. Loading history...
131		}
132	9	$this->writeTag('offers');
133	3	foreach ($this->offerClasses as $offerClass) {
		0 ignored issues – show Bug introduced 2016-01-15 12:20 UTC by Report Bug Copy Issue Report The property `offerClasses` does not seem to exist. Did you mean `offerClass`? An attempt at access to an undefined property has been detected. This may either be a typographical error or the property has been renamed but there are still references to its old name. If you really want to allow access to undefined properties, you can define magic methods to allow access. See the php core documentation on Overloading. Loading history...
134	2	$this->writeEachModel($offerClass);
135		}
136	9	$this->writeTag('/offers');
137		$this->writeTag('/shop');
138
139		$this->write('</yml_catalog>');
140		}
141
142		/**
143	9	* @return null\|string
144	6	*/
145	9	protected function getDate()
146	9	{
147		$date = $this->date;
148
149		if ($date === null) {
150		$date = Yii::$app->formatter->asDatetime(new \DateTime(), 'php:Y-m-d H:i');
151	9	}
152	6
153	9	return $date;
154	9	}
155
156		/**
157		* @param string $string
158		* @throws \Exception
159		*/
160		protected function write($string)
161	9	{
162		$this->handle->write($string);
163	9	}
164	9
165	9	/**
166	9	* @param string $string tag name
167		*/
168	9	protected function writeTag($string)
169	6	{
170		$this->write('<' . $string . '>' . PHP_EOL);
171	9	}
172	9
173	6	/**
174	4	* @param BaseModel $model
175	9	* @param $valuesModel
176	6	* @throws Exception
177	4	*/
178		protected function writeModel(BaseModel $model, $valuesModel)
179	9	{
180	9	if (method_exists($valuesModel, 'getParams')) {
181	9	$model->setParams($valuesModel->getParams());
182	6	}
183	3	if (method_exists($valuesModel, 'getPictures')) {
184		$model->setPictures($valuesModel->getPictures());
185	6	}
186	9	if(method_exists($valuesModel, 'getDeliveryOptions')) {
187		$model->setDeliveryOptions($valuesModel->getDeliveryOptions());
188	9	}
189
190	9	if($model->loadModel($valuesModel, $this->onValidationError)) {
		0 ignored issues – show Bug introduced 2016-09-15 06:16 UTC by Report Bug Copy Issue Report It seems like `$this->onValidationError` can also be of type `callable`; however, `pastuhov\ymlcatalog\models\BaseModel::loadModel()` does only seem to accept `null`, maybe add an additional type check? If a method or function can return multiple different values and unless you are sure that you only can receive a single value in this context, we recommend to add an additional type check: /** * @return array\|string */ function returnsDifferentValues($x) { if ($x) { return 'foo'; } return array(); } $x = returnsDifferentValues($y); if (is_array($x)) { // $x is an array. } If this a common case that PHP Analyzer should handle natively, please let us know by opening an issue. Loading history...
191		$string = $model->getYml();
192		$this->write($string);
193		}
194		}
195	9
196		/**
197	9	* @param string\|array $modelClass class name or yii configuration array. You can also set params:
198	9	* `findParams`: array of additional find params;
199		* `query`: ActiveQuery object to generate yml use already created object;
200	9	* `dataProvider`: ActiveDataProvider or true to generate yml with pagination;
201		*/
202		protected function writeEachModel($modelClass)
203	9	{
204		/**
205	9	* @var mixed
206	9	*/
207	9	$findParams = [];
208	6
209	6	/**
210	9	* @var ActiveQuery
211		*/
212		$query = null;
213
214		/**
215		* @var ActiveDataProvider
216		*/
217	9	$dataProvider = null;
218
219	9	if (is_array($modelClass)) {
220		foreach (['findParams', 'query', 'dataProvider'] as $name) {
221	9	if (array_key_exists($name, $modelClass)) {
222	9	$$name = $modelClass[$name];
223	6	unset($modelClass[$name]);
224	9	}
225	9	}
226	3	}
227	3
228	6	/**
229	6	* @var BaseFindYmlInterface $class
230	4	*/
231		$class = \Yii::createObject($modelClass);
232
233		/**
234	9	* @var ReaderInterface
235		*/
236		$reader = ReaderFactory::build(
237		$class,
238		$dataProvider,
239		$query,
240		$findParams
241		);
242
243		$newModel = $this->getNewModel($class);
244
245		foreach ($reader as $models) {
246		foreach ($models as $model) {
247		$this->writeModel($newModel, $model);
248		}
249		$this->gc();
250		}
251		}
252
253		/**
254		* @param $modelClass
255		* @return Category\|Currency\|SimpleOffer
256		* @throws Exception
257		*/
258		protected function getNewModel($modelClass)
259		{
260		$obj = is_object($modelClass) ? $modelClass : \Yii::createObject($modelClass);
261
262		if ($obj instanceof CurrencyInterface) {
263		$model = new Currency();
264		} elseif ($obj instanceof CategoryInterface) {
265		$model = new Category();
266		} elseif ($obj instanceof CustomOfferInterface && $this->customOfferClass !== null && class_exists($this->customOfferClass)) {
267		$model = \Yii::createObject($this->customOfferClass);
268		} elseif ($obj instanceof SimpleOfferInterface) {
269		$model = new SimpleOffer();
270		} else {
271		throw new Exception('Model ' . get_class($obj) . ' has unknown interface');
272		}
273
274		return $model;
275		}
276
277		/**
278		* Performs PHP memory garbage collection.
279		*/
280		protected function gc()
281		{
282		if (!gc_enabled()) {
283		gc_enable();
284		}
285		gc_collect_cycles();
286		}
287		}
288

pastuhov / yii2-yml-catalog

Pull Request — master (#38)

YmlCatalog::getNewModel() B

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like