WebStream\Log\LoggerFormatter

Complexity

Total Complexity

18

Size/Duplication

Total Lines	116
Duplicated Lines	6.9 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes	5
Bugs	0	Features	0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	6	1
A	getFormattedMessage()	0	15	1
A	compile()	0	4	1
A	compileApplicationName()	0	10	4
B	compileDateTime()	0	22	6
B	compileLogLevel()	8	14	5

<?php
namespace WebStream\Log;

use WebStream\Module\Container;
use WebStream\Module\Utility\LoggerUtils;

/**
 * LoggerFormatterクラス
 * ログフォーマッタ処理を行う
 * @author Ryuichi Tanaka
 * @since 2015/12/06
 * @version 0.7
 */
class LoggerFormatter
{
    use LoggerUtils;

    /**
     * @var Container ログ設定コンテナ
     */
    private $logConfig;

    /**
     * コンストラクタ
     * @param string 設定ファイルパス
     */
    public function __construct(Container $logConfig)
    {
        // $this->loadConfig($configPath);

75% of this comment could be valid code. Did you maybe forget this after debugging?

        $this->logConfig = $logConfig;
        $this->compile();
    }

    /**
     * フォーマット済みメッセージを返却する
     * @param  string メッセージ
     * @param  string ログレベル
     * @return フォーマット済みメッセージ
     */
    public function getFormattedMessage($message, $logLevel)
    {
        $formattedMessage = $this->logConfig->format;

The property format does not exist on object<WebStream\Module\Container>. Since you implemented __set, maybe consider adding a @property annotation.

        // 日付
        $formattedMessage = $this->compileDateTime($formattedMessage);

        // ログレベル
        $formattedMessage = $this->compileLogLevel($formattedMessage, $logLevel);

        // メッセージ
        $formattedMessage = preg_replace('/%m/', $message, $formattedMessage);

        return $formattedMessage . "\n";
    }

    /**
     * 固定の項目を埋め込む
     */
    private function compile()
    {
        $this->logConfig->format = $this->compileApplicationName($this->logConfig->format, $this->logConfig->applicationName);

The property format does not exist on object<WebStream\Module\Container>. Since you implemented __set, maybe consider adding a @property annotation.

The property format does not exist on object<WebStream\Module\Container>. Since you implemented __get, maybe consider adding a @property annotation.

The property applicationName does not exist on object<WebStream\Module\Container>. Since you implemented __get, maybe consider adding a @property annotation.

    }

    /**
     * アプリケーション名項目を埋め込む
     * @param  string メッセージ
     * @param  string アプリケーション名
     * @return 埋め込み済みメッセージ
     */
    private function compileApplicationName($message, $applicationName)
    {
        // アプリケーション名
        if ($applicationName !== null && preg_match('/%([0-9]{0,})c/', $this->logConfig->format, $matches)) {

The property format does not exist on object<WebStream\Module\Container>. Since you implemented __get, maybe consider adding a @property annotation.

            $applicationName = $matches[1] !== null ? str_pad($applicationName, intval($matches[1]), ' ') : $applicationName;
            $message = preg_replace('/%(?:[0-9]{0,})c/', $applicationName, $message);
        }

        return $message;
    }

    /**
     * 日付項目を埋め込む
     * @param  string メッセージ
     * @return 埋め込み済みメッセージ
     */
    private function compileDateTime($message)
    {
        if (preg_match('/%([0-9]{0,})d(?:\{(.+?)\}){1}/', $message, $formatMatches)) {
            $message = preg_replace('/%[0-9]{0,}d/', '%d', $message);
            $now = microtime(true);
            $decimal = "000";
            if (preg_match('/^[0-9]*\\.([0-9]+)$/', $now, $matches) === 1) {
                $decimal = str_pad(substr($matches[1], 0, 3), 3, "0");
            }
            $dateTimeFormat = preg_replace('/(%f)/', $decimal, $formatMatches[2]);
            $dateTime = strftime($dateTimeFormat, $now);
            $dateTime = empty($formatMatches[1]) ? $dateTime : str_pad($dateTime, $formatMatches[1], ' ');
            $message = preg_replace('/%d\{.+?\}/', $dateTime, $message);
        } elseif (preg_match('/%([0-9]{0,})d/', $message, $formatMatches)) {
            $message = preg_replace('/%[0-9]{0,}d/', '%d', $message);
            $dateTime = strftime($this->defaultDateTimeFormatter());
            $dateTime = empty($formatMatches[1]) ? $dateTime : str_pad($dateTime, $formatMatches[1], ' ');
            $message = preg_replace('/%d/', $dateTime, $message);
        }

        return $message;
    }

    /**
     * ログレベル項目を埋め込む
     * @param  string メッセージ
     * @param  string ログレベル
     * @return 埋め込み済みメッセージ
     */
    private function compileLogLevel($message, $logLevel)
    {
        // ログレベル
        if (preg_match('/%([0-9]{0,})L/', $message, $matches)) {

This code seems to be duplicated across your project.

            $upperLevel = strtoupper(empty($matches[1]) ? $logLevel : str_pad($logLevel, $matches[1], ' '));
            $message = preg_replace('/%([0-9]{0,})L/', $upperLevel, $message);
        }
        if (preg_match('/%([0-9]{0,})l/', $message, $matches)) {

This code seems to be duplicated across your project.

            $lowerLevel = empty($matches[1]) ? $logLevel : str_pad($logLevel, $matches[1], ' ');
            $message = preg_replace('/%([0-9]{0,})l/', $lowerLevel, $message);
        }

        return $message;
    }
}