Leankoala\HealthFoundation\Check\Files\Content\NumberOfLinesCheck

Complexity

Total Complexity

14

Size/Duplication

Total Lines	97
Duplicated Lines	0 %

Importance

Changes	9
Bugs	0	Features	0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	run()	0	11	3
A	processData()	0	22	4
A	init()	0	10	2
A	getCheckIdentifier()	0	3	1
A	getNumberOfLines()	0	18	4

<?php

namespace Leankoala\HealthFoundation\Check\Files\Content;

use Leankoala\HealthFoundation\Check\BasicCheck;
use Leankoala\HealthFoundation\Check\MetricAwareResult;
use Leankoala\HealthFoundation\Check\Result;

class NumberOfLinesCheck extends BasicCheck
{
    protected $identifier = 'base:files:content:numberOfLines';

    private $files = [];

    private $relation;

    private $limit;

    const RELATION_MAX = 'max';
    const RELATION_MIN = 'min';

    private $pattern = [];

    /**
     * @return MetricAwareResult
     */
    public function run()
    {
        foreach ($this->files as $file) {
            if (!file_exists($file)) {
                return new MetricAwareResult(Result::STATUS_FAIL, 'Unable to get document length because file "' . $file . '" does not exist.');
            }
        }

        $numberLines = $this->getNumberOfLines();

        return $this->processData($numberLines);
    }

    /**
     * @return int
     */
    private function getNumberOfLines()
    {
        $numberOfLines = 0;

        foreach ($this->files as $file) {
            $grep = '';
            if ($this->pattern) {

The expression $this->pattern of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.

                foreach ($this->pattern as $pattern) {
                    $grep .= ' | grep -a  "' . $pattern . '"';
                }
            }

            $command = 'cat ' . $file . $grep . ' | wc -l';
            $output = [];
            exec($command, $output, $return);
            $numberOfLines += (int)$output[0];
        }
        return $numberOfLines;
    }

    /**
     * @param $numberLines
     * @return Result
     */
    private function processData($numberLines)
    {
        if ($this->relation === self::RELATION_MAX) {
            if ($numberLines > $this->limit) {
                $result = new MetricAwareResult(Result::STATUS_FAIL, 'The document contains too many lines (' . $numberLines . '). Expected where ' . $this->limit . ' at the most.');
            } else {
                $result = new MetricAwareResult(Result::STATUS_PASS, 'The document contains ' . $numberLines . ' lines. Expected where ' . $this->limit . ' at the most.');
            }
        } else {
            if ($numberLines < $this->limit) {
                $result = new MetricAwareResult(Result::STATUS_FAIL, 'The document contains too few lines (' . $numberLines . '). Expected where ' . $this->limit . ' at least.');
            } else {
                $result = new MetricAwareResult(Result::STATUS_PASS, 'The document contains ' . $numberLines . ' lines. Expected where ' . $this->limit . ' at least.');
            }
        }

        $result->setMetric($numberLines, 'lines');

        $result->addAttribute('files', $this->files);
        $result->addAttribute('pattern', $this->pattern);

        return $result;
    }

    public function init($files, $limit, $relation = self::RELATION_MAX, $pattern = [])
    {
        if (is_string($files)) {
            $files = [$file];

The variable $file does not exist. Did you maybe mean $files?

        }

        $this->files = $files;
        $this->pattern = (array)$pattern;
        $this->relation = $relation;
        $this->limit = $limit;
    }

    protected function getCheckIdentifier()
    {
        return $this->identifier . '.' . md5(json_encode($this->files) . serialize($this->pattern));
    }
}