CdbStopwordAnalyzer::createCdbByLanguage() - Code Metrics - Inspection of "Add SanitizerFactory" - onoi/tesa - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 9401a4...2cc7a2 )

by mw

created 2016-08-02 14:22 UTC

CdbStopwordAnalyzer::createCdbByLanguage() B

↳ Parent: CdbStopwordAnalyzer

Complexity

Conditions	4
Paths	4

Size

Total Lines	27
Code Lines	14

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	1

Metric	Value
c	1
b	0
f	1
dl	0
loc	27
rs	8.5806
cc	4
eloc	14
nc	4
nop	2

<?php

namespace Onoi\Tesa\StopwordAnalyzer;

use Cdb\Reader;
use Cdb\Writer;
use Exception;
use RuntimeException;

/**
 * @license GNU GPL v2+
 * @since 0.1
 *
 * @author mwjames
 */
class CdbStopwordAnalyzer implements StopwordAnalyzer {

	/**
	 * Any change to the content of its data files should be reflected in a
	 * version change (the version number does not necessarily correlate with
	 * the library version)
	 */
	const VERSION = '0.1.cdb';

	/**
	 * @var Cdb
	 */
	private $cdb;

	/**
	 * @since 0.1
	 *
	 * @param string $target
	 */
	public function __construct( $target ) {
		try {
			$this->cdb = Reader::open( $target );

		} catch( Exception $e ) {
			// Do nothing
		}
	}

	/**
	 * @since 0.1
	 *
	 * @return boolean
	 */
	public function isAvailable() {
		return $this->cdb !== null;
	}

	/**
	 * @since 0.1
	 *
	 * @param string $language
/**
 * @param array $germany
 * @param array $island
 * @param array $italy
 */
function finale($germany, $island) {
    return "2:1";
}
	 *
	 * @return string
	 */
	public static function getLocation() {
		return str_replace( array( '\\', '/' ), DIRECTORY_SEPARATOR, __DIR__ . '/data/' );
	}

	/**
	 * @since 0.1
	 *
	 * @param string $language
	 *
	 * @return string
	 */
	public static function getTargetByLanguage( $language ) {
		return self::getLocation() . 'cdb/' . strtolower( $language )  . '.cdb';
	}

	/**
	 * @since 0.1
	 *
	 * @param string $word
	 *
	 * @return boolean
	 */
	public function isStopWord( $word ) {

		if ( $this->cdb !== null && $this->cdb->get( $word ) !== false ) {
			return true;
		}

		return false;
	}

	/**
	 * @since 0.1
	 *
	 * @param string $location
	 * @param string $language
	 *
	 * @return boolean
	 */
	public static function createCdbByLanguage( $location, $language ) {

		$language = strtolower( $language );
		$source = $location . $language . '.json';

		if ( !file_exists( $source ) ) {
			throw new RuntimeException( "{$source} is not available." );
		}

		$contents = json_decode( file_get_contents( $source ), true );

		if ( !isset( $contents['list'] ) ) {
			throw new RuntimeException( "JSON is missing the `list` index." );
		}

		$writer = Writer::open(
			self::getTargetByLanguage( $language )
		);

		foreach ( $contents['list'] as $words ) {
			$writer->set( trim( $words ), true );
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
		}

		$writer->close();

		return true;
	}

}


1			<?php
2
3			namespace Onoi\Tesa\StopwordAnalyzer;
4
5			use Cdb\Reader;
6			use Cdb\Writer;
7			use Exception;
8			use RuntimeException;
9
10			/**
11			* @license GNU GPL v2+
12			* @since 0.1
13			*
14			* @author mwjames
15			*/
16			class CdbStopwordAnalyzer implements StopwordAnalyzer {
17
18			/**
19			* Any change to the content of its data files should be reflected in a
20			* version change (the version number does not necessarily correlate with
21			* the library version)
22			*/
23			const VERSION = '0.1.cdb';
24
25			/**
26			* @var Cdb
27			*/
28			private $cdb;
29
30			/**
31			* @since 0.1
32			*
33			* @param string $target
34			*/
35			public function __construct( $target ) {
36			try {
37			$this->cdb = Reader::open( $target );
			0 ignored issues – show Documentation Bug introduced 2016-08-02 14:35 UTC by Report Bug Copy Issue Report It seems like `\Cdb\Reader::open($target)` of type `object<Cdb\Reader\DBA>` or `object<Cdb\Reader\PHP>` is incompatible with the declared type `object<Onoi\Tesa\StopwordAnalyzer\Cdb>` of property `$cdb`. Our type inference engine has found an assignment to a property that is incompatible with the declared type of that property. Either this assignment is in error or the assigned type should be added to the documentation/type hint for that property.. Loading history...
38			} catch( Exception $e ) {
39			// Do nothing
40			}
41			}
42
43			/**
44			* @since 0.1
45			*
46			* @return boolean
47			*/
48			public function isAvailable() {
49			return $this->cdb !== null;
50			}
51
52			/**
53			* @since 0.1
54			*
55			* @param string $language
			0 ignored issues – show Bug introduced 2016-08-02 14:35 UTC by Report Bug Copy Issue Report There is no parameter named `$language`. Was it maybe removed? This check looks for PHPDoc comments describing methods or function parameters that do not exist on the corresponding method or function. Consider the following example. The parameter `$italy` is not defined by the method `finale(...)`. /** * @param array $germany * @param array $island * @param array $italy */ function finale($germany, $island) { return "2:1"; } The most likely cause is that the parameter was removed, but the annotation was not. Loading history...
56			*
57			* @return string
58			*/
59			public static function getLocation() {
60			return str_replace( array( '\\', '/' ), DIRECTORY_SEPARATOR, __DIR__ . '/data/' );
61			}
62
63			/**
64			* @since 0.1
65			*
66			* @param string $language
67			*
68			* @return string
69			*/
70			public static function getTargetByLanguage( $language ) {
71			return self::getLocation() . 'cdb/' . strtolower( $language ) . '.cdb';
72			}
73
74			/**
75			* @since 0.1
76			*
77			* @param string $word
78			*
79			* @return boolean
80			*/
81			public function isStopWord( $word ) {
82
83			if ( $this->cdb !== null && $this->cdb->get( $word ) !== false ) {
84			return true;
85			}
86
87			return false;
88			}
89
90			/**
91			* @since 0.1
92			*
93			* @param string $location
94			* @param string $language
95			*
96			* @return boolean
97			*/
98			public static function createCdbByLanguage( $location, $language ) {
99
100			$language = strtolower( $language );
101			$source = $location . $language . '.json';
102
103			if ( !file_exists( $source ) ) {
104			throw new RuntimeException( "{$source} is not available." );
105			}
106
107			$contents = json_decode( file_get_contents( $source ), true );
108
109			if ( !isset( $contents['list'] ) ) {
110			throw new RuntimeException( "JSON is missing the `list` index." );
111			}
112
113			$writer = Writer::open(
114			self::getTargetByLanguage( $language )
115			);
116
117			foreach ( $contents['list'] as $words ) {
118			$writer->set( trim( $words ), true );
			0 ignored issues – show Documentation introduced 2016-08-02 14:35 UTC by Report Bug Copy Issue Report `true` is of type `boolean`, but the function expects a `string`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
119			}
120
121			$writer->close();
122
123			return true;
124			}
125
126			}
127

onoi / tesa

Push — master ( 9401a4...2cc7a2 )

CdbStopwordAnalyzer::createCdbByLanguage() B

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like