ImportHelper::convertColumnIndexes() - Code Metrics - Inspection of "Chg: Обновлена сборка js" - shogodev/argilla - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 951ad1...29ba67 )

by Alexey

created 2016-04-25 14:57 UTC

ImportHelper::convertColumnIndexes() B

↳ Parent: Project

Complexity

Conditions	6
Paths	4

Size

Total Lines	19
Code Lines	10

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
c	1
b	0
f	0
dl	0
loc	19
rs	8.8571
cc	6
eloc	10
nc	4
nop	1

<?php
/**
 * @author Alexey Tatarinov <[email protected]>
 * @link https://github.com/shogodev/argilla/
 * @copyright Copyright &copy; 2003-2015 Shogo
 * @license http://argilla.ru/LICENSE
 */
class ImportHelper
{
  private static $urlCache;

  /**
   * @param $folder
   * @param array $extensions
   *
   * @return null
   * @internal param string $extension
   */
  public static function getFileLastModified($folder, $extensions = array('csv'))
  {
    $extensions = is_array($extensions) ? $extensions : array($extensions);
    $files = CFileHelper::findFiles($folder, array(
      'level' => 0,
      'fileTypes' => $extensions,
    ));

    $filesWithDate = array();
    foreach($files as $file)
      $filesWithDate[filemtime($file)] = $file;

    return !empty($filesWithDate) ? $filesWithDate[max(array_keys($filesWithDate))] : null;
  }

  /**
   * @param $folder
   * @param array $extensions
   *
   * @return array
   * @internal param string $extension
   */
  public static function getFiles($folder, $extensions = array('csv'))
  {
    $extensions = is_array($extensions) ? $extensions : array($extensions);

    $files = array_reverse(CFileHelper::findFiles($folder, array(
      'fileTypes' => $extensions
    )));

    asort($files, SORT_NATURAL);

    return $files;
  }

  /**
   * Преобразует номер столбца в индекс колонки exсel таблици A - 0, B - 1
   *
   * @param $letters
   * @param bool $firstZero индексация начинается с 0
   *
   * @return int
   */
  public static function lettersToNumber($letters, $firstZero = true)
  {
    $num = 0;
    $arr = array_reverse(str_split($letters));

    for($i = 0; $i < count($arr); $i++)
for ($i=0; $i<count($array); $i++) { // calls count() on each iteration
}

// Better
for ($i=0, $c=count($array); $i<$c; $i++) { // calls count() just once
}
    {
      $num += (ord(strtolower($arr[$i])) - 96) * (pow(26, $i));
    }

    return $firstZero ? $num - 1 : $num;
  }

  /**
   * Преобразует индекс колонки exсel в номер столбца в 1 - A, 2 - B
   *
   * @param $number
   *
   * @return string
   */
  public static function numberToLetters($number)
  {
    $letter = "";
    if( $number <= 0 )
      return '';

    while($number != 0)
    {
      $p = ($number - 1) % 26;
      $number = intval(($number - $p) / 26);
      $letter = chr(65 + $p).$letter;
    }

    return $letter;
  }

  /**
   * Преобразует дивпазон индексов колонок exсel в массив
   * например 'a-c' в array('A', 'B', 'C');
   *
   * @param $string
   *
   * @return array
   */
  public static function getLettersRange($string)
  {
    list($begin, $end) = explode('-', $string);

    $array = array();

    $current = self::lettersToNumber($begin);
    $end = self::lettersToNumber($end);

    while(1)
    {
      $array[] = self::numberToLetters(($current++) + 1);

      if( $current > $end )
        break;
    }

    return $array;
  }

  public static function getModelWithoutBehaviors($className, $scenario = null)
  {
    /**
     * @var CActiveRecord $model
     */
    $model = new $className(null);
    if( $scenario )
      $model->setScenario($scenario);

    $model->setIsNewRecord(true);
    $model->init();

    return $model;
  }

  /**
   * @param $array
   *
   * @return mixed
   */
  public static function convertColumnIndexes($array)
  {
    foreach($array as $key => $columnIndex)
    {
      if( !empty($columnIndex) && !is_numeric($columnIndex) && !is_array($columnIndex) )
      {
        $array[$key] = self::lettersToNumber($columnIndex);
      }
      else if(is_array($columnIndex))
      {
        $columnIndex[0] = self::lettersToNumber(Arr::get($columnIndex, 0));
        $array[$key] = $columnIndex;
      }
      else
        unset($array[$key]);
    }

    return $array;
  }

  /**
   * Оборачивает путь в слеши, исключаея дублирование
   *
   * @param $path
   * @param string $wrapper
   *
   * @return string
   */
  public static function wrapInSlash($path, $wrapper = DIRECTORY_SEPARATOR)
  {
    return self::wrapInSlashBegin(self::wrapInSlashEnd($path, $wrapper, false), $wrapper);
  }

  /**
   * Оборачивает начало путь в слеш, исключаея дублирование
   *
   * @param $path
   * @param string $wrapper
   * @param bool $clearEnd
   *
   * @return string
   */
  public static function wrapInSlashBegin($path, $wrapper = DIRECTORY_SEPARATOR, $clearEnd = true)

  {
    $path = preg_replace("/(^\\{$wrapper})/", '', $path);

    if( $clearEnd )
      preg_replace("/(\\{$wrapper})$/", '', $path);

    return $wrapper.$path;
  }

  /**
   * Оборачивает конец путь в слеш, исключаея дублирование
   * @param $path
   * @param string $wrapper
   * @param bool $clearBegin
   *
   * @return string
   */
  public static function wrapInSlashEnd($path, $wrapper = DIRECTORY_SEPARATOR, $clearBegin = true)

  {
    $path = preg_replace("/(\\{$wrapper}$)/", '', $path);

    if( $clearBegin )
      preg_replace("/(^\\{$wrapper})/", '', $path);

    return $path.$wrapper;
  }

  /**
   * @param string $table
   * @param $url
   * @param bool|false $warmUpCache - прогреть
   *
   * @return mixed|string
   */
  public static function createUniqueUrl($table, $url, $warmUpCache = false)
  {
    if( !isset(self::$urlCache[$table]) )
    {
      self::$urlCache[$table] = array();

      if( $warmUpCache )
      {
        foreach(self::getUrls($table) as $itemUrl)
          self::$urlCache[$table][$itemUrl] = $itemUrl;
      }
    }

    if( !$warmUpCache )
    {
      if( $itemUrl = Arr::get(self::getUrls($table, $url), 0 ) )
        self::$urlCache[$table][$itemUrl] = $itemUrl;
    }

    $uniqueUrl = Utils::translite(trim($url));
    $suffix = 1;
    while( isset(self::$urlCache[$table][$uniqueUrl]) )
    {
      $uniqueUrl = $url.'_'.$suffix++;

      if( !$warmUpCache )
      {
        if( $checkUrl = Arr::get(self::getUrls($table, $uniqueUrl), 0 ) )
          self::$urlCache[$table][$checkUrl] = $checkUrl;
      }
    }

    if( $warmUpCache )
      self::$urlCache[$table][$uniqueUrl] = $uniqueUrl;

    return $uniqueUrl;
  }

  /**
   * @param $table|null "null" clears all
   */
  public static function clearUrlCache($table)
  {
    if( is_null($table) )
      self::$urlCache = null;
    else
      self::$urlCache[$table] = null;
  }

  private static function getUrls($table, $url = null)
  {
    $criteria = new CDbCriteria();
    $criteria->select = 'url';

    if( isset($url) )
      $criteria->compare('url', $url);

    $command = Yii::app()->db->schema->commandBuilder->createFindCommand($table, $criteria);

    return $command->queryColumn();
  }
}

1		<?php
2		/**
3		* @author Alexey Tatarinov <[email protected]>
4		* @link https://github.com/shogodev/argilla/
5		* @copyright Copyright © 2003-2015 Shogo
6		* @license http://argilla.ru/LICENSE
7		*/
8		class ImportHelper
9		{
10		private static $urlCache;
11
12		/**
13		* @param $folder
14		* @param array $extensions
15		*
16		* @return null
17		* @internal param string $extension
18		*/
19		public static function getFileLastModified($folder, $extensions = array('csv'))
20		{
21		$extensions = is_array($extensions) ? $extensions : array($extensions);
22		$files = CFileHelper::findFiles($folder, array(
23		'level' => 0,
24		'fileTypes' => $extensions,
25		));
26
27		$filesWithDate = array();
28		foreach($files as $file)
29		$filesWithDate[filemtime($file)] = $file;
30
31		return !empty($filesWithDate) ? $filesWithDate[max(array_keys($filesWithDate))] : null;
32		}
33
34		/**
35		* @param $folder
36		* @param array $extensions
37		*
38		* @return array
39		* @internal param string $extension
40		*/
41		public static function getFiles($folder, $extensions = array('csv'))
42		{
43		$extensions = is_array($extensions) ? $extensions : array($extensions);
44
45		$files = array_reverse(CFileHelper::findFiles($folder, array(
46		'fileTypes' => $extensions
47		)));
48
49		asort($files, SORT_NATURAL);
50
51		return $files;
52		}
53
54		/**
55		* Преобразует номер столбца в индекс колонки exсel таблици A - 0, B - 1
56		*
57		* @param $letters
58		* @param bool $firstZero индексация начинается с 0
59		*
60		* @return int
61		*/
62		public static function lettersToNumber($letters, $firstZero = true)
63		{
64		$num = 0;
65		$arr = array_reverse(str_split($letters));
66
67		for($i = 0; $i < count($arr); $i++)
		0 ignored issues – show Performance Best Practice introduced 2016-02-05 10:19 UTC by Report Bug Copy Issue Report It seems like you are calling the size function `count()` as part of the test condition. You might want to compute the size beforehand, and not on each iteration. If the size of the collection does not change during the iteration, it is generally a good practice to compute it beforehand, and not on each iteration: for ($i=0; $i<count($array); $i++) { // calls count() on each iteration } // Better for ($i=0, $c=count($array); $i<$c; $i++) { // calls count() just once } Loading history... Performance Best Practice introduced 2016-02-05 15:15 UTC by Report Bug Copy Issue Report Consider avoiding function calls on each iteration of the `for` loop. If you have a function call in the test part of a `for` loop, this function is executed on each iteration. Often such a function, can be moved to the initialization part and be cached. // count() is called on each iteration for ($i=0; $i < count($collection); $i++) { } // count() is only called once for ($i=0, $c=count($collection); $i<$c; $i++) { } Loading history...
68		{
69		$num += (ord(strtolower($arr[$i])) - 96) * (pow(26, $i));
70		}
71
72		return $firstZero ? $num - 1 : $num;
73		}
74
75		/**
76		* Преобразует индекс колонки exсel в номер столбца в 1 - A, 2 - B
77		*
78		* @param $number
79		*
80		* @return string
81		*/
82		public static function numberToLetters($number)
83		{
84		$letter = "";
85		if( $number <= 0 )
86		return '';
87
88		while($number != 0)
89		{
90		$p = ($number - 1) % 26;
91		$number = intval(($number - $p) / 26);
92		$letter = chr(65 + $p).$letter;
93		}
94
95		return $letter;
96		}
97
98		/**
99		* Преобразует дивпазон индексов колонок exсel в массив
100		* например 'a-c' в array('A', 'B', 'C');
101		*
102		* @param $string
103		*
104		* @return array
105		*/
106		public static function getLettersRange($string)
107		{
108		list($begin, $end) = explode('-', $string);
109
110		$array = array();
111
112		$current = self::lettersToNumber($begin);
113		$end = self::lettersToNumber($end);
114
115		while(1)
116		{
117		$array[] = self::numberToLetters(($current++) + 1);
118
119		if( $current > $end )
120		break;
121		}
122
123		return $array;
124		}
125
126		public static function getModelWithoutBehaviors($className, $scenario = null)
127		{
128		/**
129		* @var CActiveRecord $model
130		*/
131		$model = new $className(null);
132		if( $scenario )
133		$model->setScenario($scenario);
134
135		$model->setIsNewRecord(true);
136		$model->init();
137
138		return $model;
139		}
140
141		/**
142		* @param $array
143		*
144		* @return mixed
145		*/
146		public static function convertColumnIndexes($array)
147		{
148		foreach($array as $key => $columnIndex)
149		{
150		if( !empty($columnIndex) && !is_numeric($columnIndex) && !is_array($columnIndex) )
151		{
152		$array[$key] = self::lettersToNumber($columnIndex);
153		}
154		else if(is_array($columnIndex))
155		{
156		$columnIndex[0] = self::lettersToNumber(Arr::get($columnIndex, 0));
157		$array[$key] = $columnIndex;
158		}
159		else
160		unset($array[$key]);
161		}
162
163		return $array;
164		}
165
166		/**
167		* Оборачивает путь в слеши, исключаея дублирование
168		*
169		* @param $path
170		* @param string $wrapper
171		*
172		* @return string
173		*/
174		public static function wrapInSlash($path, $wrapper = DIRECTORY_SEPARATOR)
175		{
176		return self::wrapInSlashBegin(self::wrapInSlashEnd($path, $wrapper, false), $wrapper);
177		}
178
179		/**
180		* Оборачивает начало путь в слеш, исключаея дублирование
181		*
182		* @param $path
183		* @param string $wrapper
184		* @param bool $clearEnd
185		*
186		* @return string
187		*/
188	View Code Duplication	public static function wrapInSlashBegin($path, $wrapper = DIRECTORY_SEPARATOR, $clearEnd = true)
		0 ignored issues – show Duplication introduced 2016-02-05 10: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...
189		{
190		$path = preg_replace("/(^\\{$wrapper})/", '', $path);
191
192		if( $clearEnd )
193		preg_replace("/(\\{$wrapper})$/", '', $path);
194
195		return $wrapper.$path;
196		}
197
198		/**
199		* Оборачивает конец путь в слеш, исключаея дублирование
200		* @param $path
201		* @param string $wrapper
202		* @param bool $clearBegin
203		*
204		* @return string
205		*/
206	View Code Duplication	public static function wrapInSlashEnd($path, $wrapper = DIRECTORY_SEPARATOR, $clearBegin = true)
		0 ignored issues – show Duplication introduced 2016-02-05 10: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...
207		{
208		$path = preg_replace("/(\\{$wrapper}$)/", '', $path);
209
210		if( $clearBegin )
211		preg_replace("/(^\\{$wrapper})/", '', $path);
212
213		return $path.$wrapper;
214		}
215
216		/**
217		* @param string $table
218		* @param $url
219		* @param bool\|false $warmUpCache - прогреть
220		*
221		* @return mixed\|string
222		*/
223		public static function createUniqueUrl($table, $url, $warmUpCache = false)
224		{
225		if( !isset(self::$urlCache[$table]) )
226		{
227		self::$urlCache[$table] = array();
228
229		if( $warmUpCache )
230		{
231		foreach(self::getUrls($table) as $itemUrl)
232		self::$urlCache[$table][$itemUrl] = $itemUrl;
233		}
234		}
235
236		if( !$warmUpCache )
237		{
238		if( $itemUrl = Arr::get(self::getUrls($table, $url), 0 ) )
239		self::$urlCache[$table][$itemUrl] = $itemUrl;
240		}
241
242		$uniqueUrl = Utils::translite(trim($url));
243		$suffix = 1;
244		while( isset(self::$urlCache[$table][$uniqueUrl]) )
245		{
246		$uniqueUrl = $url.'_'.$suffix++;
247
248		if( !$warmUpCache )
249		{
250		if( $checkUrl = Arr::get(self::getUrls($table, $uniqueUrl), 0 ) )
251		self::$urlCache[$table][$checkUrl] = $checkUrl;
252		}
253		}
254
255		if( $warmUpCache )
256		self::$urlCache[$table][$uniqueUrl] = $uniqueUrl;
257
258		return $uniqueUrl;
259		}
260
261		/**
262		* @param $table\|null "null" clears all
263		*/
264		public static function clearUrlCache($table)
265		{
266		if( is_null($table) )
267		self::$urlCache = null;
268		else
269		self::$urlCache[$table] = null;
270		}
271
272		private static function getUrls($table, $url = null)
273		{
274		$criteria = new CDbCriteria();
275		$criteria->select = 'url';
276
277		if( isset($url) )
278		$criteria->compare('url', $url);
279
280		$command = Yii::app()->db->schema->commandBuilder->createFindCommand($table, $criteria);
281
282		return $command->queryColumn();
283		}
284		}

shogodev / argilla

GitHub Access Token became invalid

Push — master ( 951ad1...29ba67 )

ImportHelper::convertColumnIndexes() B

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like