Options::parse() - Code Metrics - Inspection of "debug" - injitools/cms-Inji - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( cb2bed...021a0b )

by Alexey

created 2016-10-10 05:25 UTC

Options::parse() F

↳ Parent: Options

Complexity

Conditions	20
Paths	486

Size

Total Lines	66
Code Lines	49

Duplication

Lines	0
Ratio	0 %

Importance

Changes	2
Bugs	1	Features	0

Metric	Value
cc	20
eloc	49
c	2
b	1
f	0
nc	486
nop	0
dl	0
loc	66
rs	3.9132

How to fix Long Method Complexity

<?php

/**
 * Item images parser
 *
 * @author Alexey Krupskiy <[email protected]>
 * @link http://inji.ru/
 * @copyright 2016 Alexey Krupskiy
 * @license https://github.com/injitools/cms-Inji/blob/master/LICENSE
 */

namespace Exchange1c\Parser\Item;

class Options extends \Migrations\Parser {

  static $options;
class A {
    var $property;
}

  public function parse() {
    if (!Options::$options) {
      Options::$options = \Ecommerce\Item\Option::getList();
    }
    $options = [];
    $modelName = 'Ecommerce\Item\Option';
$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}
    foreach ($this->data['ЗначенияСвойства'] as $opt) {
      $optionId = \App::$cur->migrations->ids['parseIds']['Ecommerce\Item\Option'][$opt['Ид']]->object_id;
      if ($optionId && !isset(Options::$options[$optionId])) {
        Options::$options = \Ecommerce\Item\Option::getList();
      }
      if (isset(Options::$options[$optionId]) && Options::$options[$optionId]->type == 'select') {
        if (empty($options[$optionId])) {
          $options[$optionId] = [];
        } else {
          if (!Options::$options[$optionId]->advance) {
            Options::$options[$optionId]->advance = ['multi' => true];
            Options::$options[$optionId]->save();
          }
        }
        $options[$optionId][] = \App::$cur->migrations->ids['parseIds']['Ecommerce\Item\Option\Item'][$opt['Значение']]->object_id;
      } else {
        $options[$optionId] = $opt['Значение'];
      }
    }
    $itemParams = \Ecommerce\Item\Param::getList(['where' => ['item_id', $this->model->id]]);
    foreach ($itemParams as $itemParam) {
      if ($itemParam->item_option_id && !isset(Options::$options[$itemParam->item_option_id])) {
        Options::$options = \Ecommerce\Item\Option::getList();
      }
      if (isset(Options::$options[$itemParam->item_option_id]) && Options::$options[$itemParam->item_option_id]->type == 'select') {
        if (empty($options[$itemParam->item_option_id]) || !in_array($itemParam->value, $options[$itemParam->item_option_id])) {
          $itemParam->delete();
        } else {
          unset($options[$itemParam->item_option_id][array_search($itemParam->value, $options[$itemParam->item_option_id])]);
        }
      } else {
        if (empty($options[$itemParam->item_option_id])) {
          $itemParam->delete();
        } else {
          $itemParam->value = $options[$itemParam->item_option_id];
          $itemParam->save();
          unset($options[$itemParam->item_option_id]);
        }
      }
    }
    foreach ($options as $optionId => $values) {
      if (is_array($values)) {
        foreach ($values as $value) {
          $itemParam = new \Ecommerce\Item\Param([
              'item_option_id' => $optionId,
              'item_id' => $this->model->id,
              'value' => $value
          ]);
          $itemParam->save();
        }
      } else {
        $itemParam = new \Ecommerce\Item\Param([
            'item_option_id' => $optionId,
            'item_id' => $this->model->id,
            'value' => $values
        ]);
        $itemParam->save();
      }
    }
  }

}


1			<?php
2
3			/**
4			* Item images parser
5			*
6			* @author Alexey Krupskiy <[email protected]>
7			* @link http://inji.ru/
8			* @copyright 2016 Alexey Krupskiy
9			* @license https://github.com/injitools/cms-Inji/blob/master/LICENSE
10			*/
11
12			namespace Exchange1c\Parser\Item;
13
14			class Options extends \Migrations\Parser {
15
16			static $options;
			0 ignored issues – show Coding Style introduced 2016-10-05 05:28 UTC by Report Bug Copy Issue Report The visibility should be declared for property `$options`. The PSR-2 coding standard requires that all properties in a class have their visibility explicitly declared. If you declare a property using class A { var $property; } the property is implicitly global. To learn more about the PSR-2, please see the PHP-FIG site on the PSR-2. Loading history...
17
18			public function parse() {
19			if (!Options::$options) {
20			Options::$options = \Ecommerce\Item\Option::getList();
21			}
22			$options = [];
23			$modelName = 'Ecommerce\Item\Option';
			0 ignored issues – show Unused Code introduced 2016-10-05 05:28 UTC by Report Bug Copy Issue Report `$modelName` is not used, you could remove the assignment. This check looks for variable assignements that are either overwritten by other assignments or where the variable is not used subsequently. $myVar = 'Value'; $higher = false; if (rand(1, 6) > 3) { $higher = true; } else { $higher = false; } Both the `$myVar` assignment in line 1 and the `$higher` assignment in line 2 are dead. The first because `$myVar` is never used and the second because `$higher` is always overwritten for every possible time line. Loading history...
24			foreach ($this->data['ЗначенияСвойства'] as $opt) {
25			$optionId = \App::$cur->migrations->ids['parseIds']['Ecommerce\Item\Option'][$opt['Ид']]->object_id;
26			if ($optionId && !isset(Options::$options[$optionId])) {
27			Options::$options = \Ecommerce\Item\Option::getList();
28			}
29			if (isset(Options::$options[$optionId]) && Options::$options[$optionId]->type == 'select') {
30			if (empty($options[$optionId])) {
31			$options[$optionId] = [];
32			} else {
33			if (!Options::$options[$optionId]->advance) {
34			Options::$options[$optionId]->advance = ['multi' => true];
35			Options::$options[$optionId]->save();
36			}
37			}
38			$options[$optionId][] = \App::$cur->migrations->ids['parseIds']['Ecommerce\Item\Option\Item'][$opt['Значение']]->object_id;
39			} else {
40			$options[$optionId] = $opt['Значение'];
41			}
42			}
43			$itemParams = \Ecommerce\Item\Param::getList(['where' => ['item_id', $this->model->id]]);
44			foreach ($itemParams as $itemParam) {
45			if ($itemParam->item_option_id && !isset(Options::$options[$itemParam->item_option_id])) {
46			Options::$options = \Ecommerce\Item\Option::getList();
47			}
48			if (isset(Options::$options[$itemParam->item_option_id]) && Options::$options[$itemParam->item_option_id]->type == 'select') {
49			if (empty($options[$itemParam->item_option_id]) \|\| !in_array($itemParam->value, $options[$itemParam->item_option_id])) {
50			$itemParam->delete();
51			} else {
52			unset($options[$itemParam->item_option_id][array_search($itemParam->value, $options[$itemParam->item_option_id])]);
53			}
54			} else {
55			if (empty($options[$itemParam->item_option_id])) {
56			$itemParam->delete();
57			} else {
58			$itemParam->value = $options[$itemParam->item_option_id];
59			$itemParam->save();
60			unset($options[$itemParam->item_option_id]);
61			}
62			}
63			}
64			foreach ($options as $optionId => $values) {
65			if (is_array($values)) {
66			foreach ($values as $value) {
67			$itemParam = new \Ecommerce\Item\Param([
68			'item_option_id' => $optionId,
69			'item_id' => $this->model->id,
70			'value' => $value
71			]);
72			$itemParam->save();
73			}
74			} else {
75			$itemParam = new \Ecommerce\Item\Param([
76			'item_option_id' => $optionId,
77			'item_id' => $this->model->id,
78			'value' => $values
79			]);
80			$itemParam->save();
81			}
82			}
83			}
84
85			}
86

injitools / cms-Inji

Push — master ( cb2bed...021a0b )

Options::parse() F

Complexity

Size

Duplication

Importance

How to fix Long Method Complexity

Long Method

Duplication Side-by-Side

Filter issues like