T4web\Cron\Log\TextFile

Complexity

Total Complexity

12

Size/Duplication

Total Lines	112
Duplicated Lines	14.29 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	log()	0	20	2
B	getExecutionTime()	0	27	5
A	prepareOutput()	8	8	2
A	prepareError()	8	8	2

<?php

namespace T4web\Cron\Log;

use T4web\Cron\Exception\RuntimeException;
use T4web\Cron\Config;

class TextFile implements LoggerInterface
{
    /**
     * @var string
     */
    private $logDirectory;

    /**
     * @var FileSystem
     */
    private $fileSystem;

    /**
     * TextFile constructor.
     *
     * @param FileSystem $fileSystem
     * @param Config     $config
     */
    public function __construct(FileSystem $fileSystem, Config $config)
    {
        $this->fileSystem = $fileSystem;
        $this->logDirectory = $config->getLogDirectory();
    }

    /**
     * @param string $jobId
     * @param int $startTime
     * @param int $endTime
     * @param bool $isSuccessful
     * @param array $output
     * @param array $error
     */
    public function log($jobId, $startTime, $endTime, $isSuccessful, $output, $error)
    {
        $filename = sprintf("%s/%s.log", $this->logDirectory, $jobId);

        $message = '';
        if ($isSuccessful) {
            $message .= sprintf('[%s] Job processed successful', date('Y-m-d H:i:s')) . PHP_EOL;
        } else {
            $message .= sprintf('[%s] Job fail', date('Y-m-d H:i:s')) . PHP_EOL;
        }

        $message .= '  Start: ' . date('Y-m-d H:i:s', $startTime) . PHP_EOL;
        $message .= '  End: ' . date('Y-m-d H:i:s', $endTime) . PHP_EOL;
        $message .= '  Execution time: ' . $this->getExecutionTime($startTime, $endTime) . PHP_EOL;
        $message .= $this->prepareOutput($output);
        $message .= $this->prepareError($error);
        $message .= PHP_EOL;

        $this->fileSystem->put($filename, $message);
    }

    /**
     * @param int $startTime
     * @param int $endTime
     * @return string
     */
    private function getExecutionTime($startTime, $endTime)
    {
        $executionTime = $endTime - $startTime;
        $days = floor($executionTime / 3600*24);
        $hours = floor(($executionTime - ($days*3600*24)) / 3600);
        $minutes = floor(($executionTime - ($hours*3600)) / 60);
        $secs = floor($executionTime % 60);

        $executionTimeText = '';
        if ($days > 0) {
            $executionTimeText .= "$days days ";
        }

        if ($hours > 0) {
            $executionTimeText .= "$hours hours ";
        }

        if ($minutes > 0) {
            $executionTimeText .= "$minutes mins ";
        }

        if ($secs > 0) {
            $executionTimeText .= "$secs seconds";
        }

        return $executionTimeText;
    }

    /**
     * @param $output
     * @return string|void
     */
    private function prepareOutput($output)

This method seems to be duplicated in your project.

    {
        if (empty($output)) {
            return;
        }

        return '--Output: ' . PHP_EOL . implode('', $output) . PHP_EOL . '--End output.' . PHP_EOL;
    }

    /**
     * @param $error
     * @return string|void
     */
    private function prepareError($error)

This method seems to be duplicated in your project.

    {
        if (empty($error)) {
            return;
        }

        return '--Error: ' . PHP_EOL . implode('', $error) . PHP_EOL . '--End error.' . PHP_EOL;
    }
}