LosMiddleware\LosLog\AbstractLogger

Complexity

Total Complexity

7

Size/Duplication

Total Lines	43
Duplicated Lines	44.19 %

Coupling/Cohesion

Components	0
Dependencies	4

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	generateFileLogger()	0	8	1
B	validateLogFile()	19	19	6

<?php

namespace LosMiddleware\LosLog;

use Zend\Log\Logger;
use Zend\Log\PsrLoggerAdapter;
use Zend\Log\Writer\Stream;

abstract class AbstractLogger
{
    /**
     * @param string $logFile
     * @param string $logDir
     * @return PsrLoggerAdapter
     * @throws Exception\InvalidArgumentException
     */
    public static function generateFileLogger($logFile, $logDir)
    {
        $fileName = static::validateLogFile($logFile, $logDir);
        $zendLogLogger = new Logger();
        $zendLogLogger->addWriter(new Stream($fileName));

        return new PsrLoggerAdapter($zendLogLogger);
    }

    /**
     * @param string $logFile
     * @param string $logDir
     * @return string
     * @throws Exception\InvalidArgumentException
     */
    public static function validateLogFile($logFile, $logDir)

This method seems to be duplicated in your project.

    {
        // Is logFile a stream url?
        if (strpos($logFile, '://') !== false) {
            return $logFile;
        }

        if (! file_exists($logDir) || ! is_writable($logDir)) {
            throw new Exception\InvalidArgumentException("Log dir {$logDir} must exist and be writable!");
        }

        $fileName = $logDir.DIRECTORY_SEPARATOR.$logFile;

        if (file_exists($fileName) && ! is_writable($fileName)) {
            throw new Exception\InvalidArgumentException("Log file {$fileName} must be writable!");
        }

        return $fileName;
    }
}