Felix\Scraper\Str

Complexity

Total Complexity

5

Size/Duplication

Total Lines	46
Duplicated Lines	0 %

Coupling/Cohesion

Components	2
Dependencies	0

Test Coverage

Coverage

100%

Importance

Changes	5
Bugs	0	Features	1

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	3	1
A	clean()	0	7	2
A	set()	0	9	1
A	get()	0	4	1

<?php

namespace Felix\Scraper;

class Str

Since you have declared the constructor as private, maybe you should also declare the class as final.

{
    private $str = null;
    private static $instance = null;
    
    private function __construct($str = '') {
       $this->set($str);
    }
    
    public static function clean($str = '') {
        if (! isset(static::$instance)) {

Since $instance is declared private, accessing it with static will lead to errors in possible sub-classes; consider using self, or increasing the visibility of $instance to at least protected.

            self::$instance = new Str($str);
        }
        
        return self::$instance; 
    }

    /**
     * Limpiar la cadena y almacenar.
     * 
     * @param $str string Cadena de caracteres a limpiar.
     * 
     * @return void
     */
    private function set($str)
    {
        $str = html_entity_decode($str);
		$str = str_replace("\xc2\xa0", "", $str);
        $str = trim($str);
        $str = preg_replace("!\s+!", " ", $str);

        $this->str = $str;
    }

	/**
	 * Limpiar una cadena de caracteres.
	 * 
	 * @param string $str Cadena de caracteres a limpiar. 

There is no parameter named $str. Was it maybe removed?

	 * 
	 * @return string
	 */
	public function get()
	{
		return $this->str;
	}
}