Azine\EmailBundle\Tests\FindInFileUtil

Complexity

Total Complexity

22

Size/Duplication

Total Lines	114
Duplicated Lines	0 %

Importance

Changes	2
Bugs	0	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
B	scandir2()	0	16	8
B	findTxtt()	0	40	9
A	strpos2()	0	7	2
A	find()	0	18	3

<?php

namespace Azine\EmailBundle\Tests;

//Author		: de77
//Website		: www.de77.com
//License		: MIT (http://en.wikipedia.org/wiki/MIT_License)
//Class desc	: http://de77.com/php-class-fast-find-text-string-in-files-recursively

//------------------------------------------------------------------------------
//          If you like this class- please leave a comment on my site, thanks!
//------------------------------------------------------------------------------

class FindInFileUtil
{
    private $text;
    private $textlen;
    private $results;

    public $caseSensitive = true;
    public $error;
    public $excludeMode = true;
    public $formats = array('.jpg', '.gif', '.avi');

    /**
     * @param string $dir
     * @param string $text
     *
     * @return array of files with the search-text
     */
    public function find($dir, $text)
    {
        $this->textlen = strlen($text);
        $this->text = $text;

        if ($this->textlen > 4096) {
            $this->error = 'You cannot search for such long text. Limit is 4096 Bytes';

            return false;

The expression return false returns the type false which is incompatible with the documented return type array.

        }

        $this->results = array();
        if ('/' != substr($dir, -1, 1)) {
            $dir .= '/';
        }
        $this->scandir2($dir);

        return $this->results;
    }

    /**
     * @param string $file
     */
    private function findTxtt($file)
    {
        $ext = strrchr($file, '.');

        if (true == $this->excludeMode) {

It seems like you are loosely comparing two booleans. Considering using the strict comparison === instead.

            if (in_array($ext, $this->formats)) {
                return false;
            }
        } else {
            if (!in_array($ext, $this->formats)) {
                return false;
            }
        }

        if (filesize($file) < 40960) {
            $data = file_get_contents($file);
            if ($this->strpos2($data, $this->text)) {
                return true;
            }

            return false;
        }
        $f = fopen($file, 'r');
        $currentPos = 0;
        $step = 40960;
        while (!feof($f)) {

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

            $data = fread($f, $step);

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

            if (!$data) {
                break;
            } elseif ($this->strpos2($data, $this->text)) {
                fclose($f);

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

                return true;
            }
            $currentPos = $currentPos + $step - $this->textlen;
            fseek($f, $currentPos, SEEK_SET);

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

        }
        fclose($f);

        return false;
    }

    /**
     * @param string $haystack
     * @param string $keyword
     */
    private function strpos2($haystack, $keyword)
    {
        if (!$this->caseSensitive) {
            return false !== mb_stripos($haystack, $keyword);
        }

        return false !== mb_strpos($haystack, $keyword);
    }

    /**
     * @param string $dir
     */
    private function scandir2($dir)
    {
        if (is_dir($dir)) {
            if ($dh = opendir($dir)) {
                while (false !== ($file = readdir($dh))) {
                    if ('.' != $file and '..' != $file) {
                        if (is_dir($dir.$file)) {
                            $this->scandir2($dir.$file.'/');
                        } else {
                            if ($this->findTxtt($dir.$file)) {
                                $this->results[] = $dir.$file;
                            }
                        }
                    }
                }
                closedir($dh);
            }
        }
    }
}