added option of injecting a custom parser,

added example of injecting custom parser,
some cleanup, refactoring and optimization

Author: grgk

composer.json

@@ -14,7 +14,8 @@
     "psr/simple-cache": "^1.0"
   },
   "require-dev": {
-    "phpunit/phpunit": "^7"
+    "phpunit/phpunit": "^7",
+    "ext-json": "*"
   },
   "autoload": {
     "psr-4": {

examples/analyze_file_with_custom_parser.php

@@ -0,0 +1,29 @@
+<?php
+/**
+ * File analyze example
+ *
+ * Executes file seo analyze on local html file using custom parser.
+ */
+
+require_once(__DIR__ . '/../vendor/autoload.php');
+
+use SeoAnalyzer\Analyzer;
+use SeoAnalyzer\Factor;
+use SeoAnalyzer\HttpClient\Exception\HttpException;
+use SeoAnalyzer\Page;
+use SeoAnalyzer\Parser\ExampleCustomParser;
+
+try {
+    $page = new Page('https://www.msn.com/pl-pl');
+    $parser = new ExampleCustomParser();
+    $page->setParser($parser);
+    $analyzer = new Analyzer($page);
+    $analyzer->metrics = $page->setMetrics([Factor::ALTS]);
+    $results = $analyzer->analyze();
+} catch (HttpException $e) {
+    echo "Error loading page: " . $e->getMessage();
+} catch (ReflectionException $e) {
+    echo "Error loading metric file: " . $e->getMessage();
+}
+
+print_r($results);

src/Metric/MetricFactory.php

@@ -2,12 +2,15 @@
 
 namespace SeoAnalyzer\Metric;
 
+use ReflectionException;
+
 class MetricFactory
 {
     /**
      * @param string $key
      * @param null $inputData
      * @return mixed
+     * @throws ReflectionException
      */
     public static function get(string $key, $inputData = null)
     {
@@ -24,6 +27,6 @@ public static function get(string $key, $inputData = null)
             }
             return $metric;
         }
-        throw new \ReflectionException('Metric class ' . $class .' not exists');
+        throw new ReflectionException('Metric class ' . $class .' not exists');
     }
 }

src/Metric/Page/SizeMetric.php

@@ -7,6 +7,8 @@
 use SeoAnalyzer\HttpClient\Exception\HttpException;
 use SeoAnalyzer\Metric\MetricFactory;
 use ReflectionException;
+use SeoAnalyzer\Parser\Parser;
+use SeoAnalyzer\Parser\ParserInterface;
 
 class Page
 {
@@ -52,19 +54,33 @@ class Page
      */
     public $client;
 
+    /**
+     * @var ParserInterface
+     */
+    public $parser;
+
     /**
      * Page constructor.
      *
      * @param string|null $url
      * @param string|null $locale
      * @param ClientInterface|null $client
+     * @param ParserInterface|null $parser
      */
-    public function __construct(string $url = null, string $locale = null, ClientInterface $client = null)
-    {
+    public function __construct(
+        string $url = null,
+        string $locale = null,
+        ClientInterface $client = null,
+        ParserInterface $parser = null
+    ) {
         $this->client = $client;
         if (empty($client)) {
             $this->client = new Client();
         }
+        $this->parser = $parser;
+        if (empty($parser)) {
+            $this->parser = new Parser();
+        }
         if (!empty($url)) {
             $this->url = $this->setUpUrl($url);
             $this->getContent();
@@ -74,6 +90,26 @@ public function __construct(string $url = null, string $locale = null, ClientInt
         }
     }
 
+    /**
+     * Sets custom Http Client.
+     *
+     * @param ClientInterface $client
+     */
+    public function setClient(ClientInterface $client): void
+    {
+        $this->client = $client;
+    }
+
+    /**
+     * Sets custom Html Parser.
+     *
+     * @param ParserInterface $parser
+     */
+    public function setParser(ParserInterface $parser): void
+    {
+        $this->parser = $parser;
+    }
+
     /**
      * Verifies URL and sets up some basic metrics.
      *
@@ -180,21 +216,23 @@ protected function getHttpClientOptions()
      */
     public function parse()
     {
-        $parser = new Parser($this->content);
+        if (empty($this->content)) {
+            $this->getContent();
+        }
+        $this->parser->setContent($this->content);
         $this->setFactors([
-            Factor::META_META => $parser->getMeta(),
-            Factor::HEADERS => $parser->getHeaders(),
-            Factor::META_TITLE => $parser->getTitle(),
-            Factor::TEXT => $parser->getText(),
-            Factor::ALTS => $parser->getAlts()
+            Factor::META_META => $this->parser->getMeta(),
+            Factor::HEADERS => $this->parser->getHeaders(),
+            Factor::META_TITLE => $this->parser->getTitle(),
+            Factor::TEXT => $this->parser->getText(),
+            Factor::ALTS => $this->parser->getAlts()
         ]);
     }
 
     /**
      * Returns page metrics.
      *
      * @return array
-     * @throws HttpException
      * @throws ReflectionException
      */
     public function getMetrics(): array
@@ -208,7 +246,6 @@ public function getMetrics(): array
      * @param array $config
      * @return array
      * @throws ReflectionException
-     * @throws HttpException
      */
     public function setMetrics(array $config)
     {
@@ -218,9 +255,6 @@ public function setMetrics(array $config)
 
     private function initializeFactors()
     {
-        if (empty($this->content)) {
-            $this->getContent();
-        }
         if (empty($this->dom)) {
             $this->parse();
         }
@@ -335,7 +369,7 @@ public function getMetricsConfig()
      *
      * @param array $config Metrics config
      * @return array
-     * @throws \ReflectionException
+     * @throws ReflectionException
      */
     public function setUpMetrics(array $config)
     {