NirjharLo\Cgss\Lib\Analysis\Lib\Design

Complexity

Total Complexity

11

Size/Duplication

Total Lines	85
Duplicated Lines	0 %

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	analyze_js()	0	32	5
B	analyze_css()	0	40	6

<?php
namespace NirjharLo\Cgss\Lib\Analysis\Lib;

if ( ! defined( 'ABSPATH' ) ) exit;


/**
 * Download and analyze .css or .js files, based on an url.
 *
 * 2 properties:
 * @property array $css_url Input CSS urls
 * @property array $js_url Input JS urls
 */

class Design {


	public $css_url;
	public $js_url;


	//analyze those urls and their content
	public function analyze_js() {

		$num = 0;
		$size = 0;
		$compress_num = 0;
		$compress_size = 0;

		if ( $this->js_url ) {

The expression $this->js_url of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.

			$num = count($this->js_url);

			//get files and check their bodies
			foreach ( $this->js_url as $val ) {
				$body = @file_get_contents( $val, FILE_USE_INCLUDE_PATH );

NirjharLo\Cgss\Lib\Analysis\Lib\FILE_USE_INCLUDE_PATH of type integer is incompatible with the type boolean expected by parameter $use_include_path of file_get_contents().

				if ($body) {
					$pre_size = mb_strlen( $body, '8bit' );
					$size += round( ( $pre_size / 1024 ), 0 );
					$pure_body = filter_var( $body, FILTER_SANITIZE_STRING, FILTER_FLAG_STRIP_LOW );
					$pure_size = mb_strlen( $pure_body, '8bit' );
					$percent_comp = 1 - ( $pure_size / $pre_size );
					if ( $percent_comp > 0.05 ) {
						$compress_num += 1;
						$compress_size += round( ( ( $pre_size - $pure_size ) / 1024 ), 0 );
					}
				}
			}
		}
		return array(
					'count' => $num,
					'size' => $size,
					'compress_num' => $compress_num,
					'compress_size' => $compress_size,
				);
	}


	//analyze those urls and their content
	public function analyze_css() {

		$num = 0;
		$size = 0;
		$import = 0;
		$media = 0;
		$compress_num = 0;
		$compress_size = 0;

		if ( $this->css_url ) {

The expression $this->css_url of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.

			$num = count($this->css_url);

			//get files and check their bodies
			foreach ( $this->css_url as $val ) {
				$body = @file_get_contents( $val, FILE_USE_INCLUDE_PATH );

NirjharLo\Cgss\Lib\Analysis\Lib\FILE_USE_INCLUDE_PATH of type integer is incompatible with the type boolean expected by parameter $use_include_path of file_get_contents().

				if ($body) {
					$pre_size = mb_strlen( $body, '8bit' );
					$size += round( ( $pre_size / 1024 ), 0 );
					if ( strpos( $val, '.css' ) !== false ) {
						$import += substr_count( $body, '@import' );
						$media += substr_count( $body, '@media' );
					}
					$pure_body = filter_var( $body, FILTER_SANITIZE_STRING, FILTER_FLAG_STRIP_LOW );
					$pure_size = mb_strlen( $pure_body, '8bit' );
					$percent_comp = 1 - ( $pure_size / $pre_size );
					if ( $percent_comp > 0.05 ) {
						$compress_num += 1;
						$compress_size += round( ( ( $pre_size - $pure_size ) / 1024 ), 0 );
					}
				}
			}
		}
		return array(
					'count' => $num,
					'size' => $size,
					'import' => $import,
					'media' => $media,
					'compress_num' => $compress_num,
					'compress_size' => $compress_size,
				);
	}
} ?>

It is not recommended to use PHP's closing tag ?> in files other than templates.