App\Babel\Crawl\CrawlerBase

Complexity

Total Complexity

26

Size/Duplication

Total Lines	172
Duplicated Lines	0 %

Importance

Changes

0

10 Methods

Rating	Name	Duplication	Size	Complexity
A	updateProblem()	0	4	1
A	_resetPro()	0	25	1
A	line()	0	6	2
A	procInfo()	0	11	5
A	__construct()	0	2	1
B	_cacheImage()	0	50	11
A	insertProblem()	0	4	1
A	cmp()	0	3	2
A	importCommandLine()	0	2	1
A	getUrl()	0	11	1

<?php

namespace App\Babel\Crawl;

use App\Models\ProblemModel;
use KubAT\PhpSimple\HtmlDomParser;
use Auth;

class CrawlerBase
{
    public $pro=[
        'file'=> 0,
        'pcode'=>'',
        'solved_count'=>'',
        'time_limit'=>'',
        'memory_limit'=>'',
        'title'=>'',
        'OJ'=>'',
        'description'=>'',
        'input'=>'',
        'output'=>'',
        'note'=>'',
        'input_type'=>'',
        'output_type'=>'',
        'contest_id'=>'',
        'index_id'=>'',
        'origin'=>'',
        'source'=>'',
        'sample'=>[],
        'markdown'=>0,
        'tot_score'=>1,
        'partial'=>0,
        'special_compiler'=>null,
    ];

    public $data=null;
    public $command=null;

    /**
     * Initial
     *
     * @return Response

The type App\Babel\Crawl\Response was not found. Did you mean Response? If so, make sure to prefix the type with \.

     */
    public function __construct()
    {
    }

    protected function _resetPro()
    {
        $this->pro=[
            'file'=> 0,
            'pcode'=>'',
            'solved_count'=>'',
            'time_limit'=>'',
            'memory_limit'=>'',
            'title'=>'',
            'OJ'=>'',
            'description'=>'',
            'input'=>'',
            'output'=>'',
            'note'=>'',
            'input_type'=>'',
            'output_type'=>'',
            'contest_id'=>'',
            'index_id'=>'',
            'origin'=>'',
            'source'=>'',
            'sample'=>[],
            'markdown'=>0,
            'tot_score'=>1,
            'partial'=>0,
            'special_compiler'=>null,
        ];
    }

    public function importCommandLine($commandTemp){
        $this->command=$commandTemp;
    }

    public static function cmp($a, $b)
    {
        return ($a[1]>$b[1]) ?-1 : 1;
    }

    private function _cacheImage($data)
    {
        if(!isset($data["ori"]) || !isset($data["path"]) || !isset($data["baseurl"]) || !isset($data["space_deli"]) || !isset($data["cookie"])){
            throw new Exception("data is not completely exist in cacheImage");

The type App\Babel\Crawl\Exception was not found. Did you mean Exception? If so, make sure to prefix the type with \.

        }
        $ori = $data["ori"];
        $path = $data["path"];
        $baseurl = $data["baseurl"];
        $space_deli = $data["space_deli"];
        $cookie = $data["cookie"];

        $para["path"]=$path;

$para was never initialized. Although not strictly required by PHP, it is generally a good practice to add $para = array(); before regardless.

        $para["base"]=$baseurl;
        $para["trans"]=!$space_deli;
        $para["cookie"]=$cookie;

        if ($space_deli) {
            $reg="/< *im[a]?g[^>]*src *= *[\"\\']?([^\"\\' >]*)[^>]*>/si";
        } else {
            $reg="/< *im[a]?g[^>]*src *= *[\"\\']?([^\"\\'>]*)[^>]*>/si";
        }

        return preg_replace_callback($reg, function($matches) use ($para) {
            global $config;
            $url=trim($matches[1]);
            if (stripos($url, "http://")===false && stripos($url, "https://")===false) {
                if ($para["trans"]) {
                    $url=str_replace(" ", "%20", $url);
                }
                $url=$para["base"].$url;
            }
            $name=basename($url);
            $name="images/".strtr($name, ":", "_");
            $result=str_replace(trim($matches[1]), "online_Judges/spoj/images/".strtr(basename($url), ":", "_"), $matches[0]);
            $ch=curl_init();
            curl_setopt($ch, CURLOPT_URL, $url);

It seems like $ch can also be of type false; however, parameter $ch of curl_setopt() does only seem to accept resource, maybe add an additional type check?

            curl_setopt($ch, CURLOPT_SSL_VERIFYPEER, false);
            curl_setopt($ch, CURLOPT_FOLLOWLOCATION, true);
            if ($para["cookie"]!="") {
                curl_setopt($ch, CURLOPT_COOKIEFILE, $para["cookie"]);
            }
            curl_setopt($ch, CURLOPT_HEADER, 0);
            curl_setopt($ch, CURLOPT_RETURNTRANSFER, true);
            $content=curl_exec($ch);

It seems like $ch can also be of type false; however, parameter $ch of curl_exec() does only seem to accept resource, maybe add an additional type check?

            curl_close($ch);

It seems like $ch can also be of type false; however, parameter $ch of curl_close() does only seem to accept resource, maybe add an additional type check?

            $fp=fopen($name, "wb");
            fwrite($fp, $content);

It seems like $fp can also be of type false; however, parameter $handle of fwrite() does only seem to accept resource, maybe add an additional type check?

            fclose($fp);

It seems like $fp can also be of type false; however, parameter $handle of fclose() does only seem to accept resource, maybe add an additional type check?

            return $result;
        }, $ori);
    }

    public function procInfo($data)
    {
        if(isset($data["path"]))       $path = $data["path"];             else throw new Exception("path is not exist in data");
        if(isset($data["baseurl"]))    $baseurl = $data["baseurl"];       else throw new Exception("baseurl is not exist in data");
        if(isset($data["space_deli"])) $space_deli = $data["space_deli"]; else $space_deli = true;
        if(isset($data["cookie"]))     $cookie = $data["cookie"];         else $cookie = "";

        $this->pro["description"]=$this->_cacheImage($this->pro["description"], $path, $baseurl, $space_deli, $cookie);

The call to App\Babel\Crawl\CrawlerBase::_cacheImage() has too many arguments starting with $path.

        $this->pro["input"]=$this->_cacheImage($this->pro["input"], $path, $baseurl, $space_deli, $cookie);
        $this->pro["output"]=$this->_cacheImage($this->pro["output"], $path, $baseurl, $space_deli, $cookie);
        $this->pro["note"]=$this->_cacheImage($this->pro["note"], $path, $baseurl, $space_deli, $cookie);
    }

    public function getUrl($url)
    {
        $ch=curl_init();
        curl_setopt($ch, CURLOPT_URL, $url);

It seems like $ch can also be of type false; however, parameter $ch of curl_setopt() does only seem to accept resource, maybe add an additional type check?

        curl_setopt($ch, CURLOPT_RETURNTRANSFER, true);
        curl_setopt($ch, CURLOPT_SSL_VERIFYPEER, false);
        curl_setopt($ch, CURLOPT_HEADER, 0);
        curl_setopt($ch, CURLOPT_FOLLOWLOCATION, true);
        $content=curl_exec($ch);

It seems like $ch can also be of type false; however, parameter $ch of curl_exec() does only seem to accept resource, maybe add an additional type check?

        curl_close($ch);

It seems like $ch can also be of type false; however, parameter $ch of curl_close() does only seem to accept resource, maybe add an additional type check?

        return $content;
    }

    public function insertProblem($oid=2)

The parameter $oid is not used and could be removed.

    {
        $problemModel=new ProblemModel();
        return $problemModel->insertProblem($this->pro);
    }

    public function updateProblem($oid=2)

The parameter $oid is not used and could be removed.

    {
        $problemModel=new ProblemModel();
        return $problemModel->updateProblem($this->pro);
    }

    protected function line($line)
    {
        if(is_null($this->command)){
            echo $line;
        }else{
            $this->command->line($line);
        }
    }
}