CpeLogger::log_out() - Code Metrics - Inspection of "Add date_default_timezone_set()" - sportarchive/CloudProcessingEngine-SDK - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( d5358f...a9ad96 )

by Nicolas

created 2016-10-18 15:59 UTC

CpeLogger::log_out() C

↳ Parent: Project

Complexity

Conditions	8
Paths	14

Size

Total Lines	52
Code Lines	37

Duplication

Lines	0
Ratio	0 %

Importance

Changes	4
Bugs	1	Features	0

Metric	Value
c	4
b	1
f	0
dl	0
loc	52
rs	6.8493
cc	8
eloc	37
nc	14
nop	4

How to fix Long Method

<?php

namespace SA\CpeSdk;

use SA\CpeSdk;

/**
 * Allow formatted logging on STDOUT
 * Send logs to Syslog for log offloading
 */
class CpeLogger
{
    public $logPath;
    public $printout;

    // Exception
    const LOG_TYPE_ERROR = "LOG_TYPE_ERROR";
    const OPENLOG_ERROR  = "OPENLOG_ERROR";

    // Specify the path where to create the log files
    public function __construct(
        $logPath = null,
        $suffix = null, 
        $printout = false)
    {
        global $argv;
function myFunction($a, $b) {
    // Do something
}

        date_default_timezone_set('UTC')
        
        $this->printout = $printout;

        $this->logPath = "/var/tmp/logs/cpe/";
                
        if ($logPath)
            $this->logPath = $logPath;

        if (!file_exists($this->logPath))
            mkdir($this->logPath, 0755, true);

        $file = basename($argv[0]);
        if ($suffix)
            $file .= "-".$suffix;
        // Append progname to the path
        $this->logPath .= "/".$file.".log";

        $this->log_out(
            "INFO",
            "[CPE SDK] ".basename(__FILE__),
            "Logging to: ".$this->logPath);
    }

    // Log message to syslog and log file
    public function log_out(
        $type,
        $source,
        $message,
        $workflowId = null)
    {
        $log = [
            "time"    => date("Y-m-d H:i:s", time()),
            "source"  => $source,
            "type"    => $type,
            "message" => $message
        ];
    
        if ($workflowId)
            $log["workflowId"] = $workflowId;

        // Open Syslog. Use programe name as key
        if (!openlog (__FILE__, LOG_CONS|LOG_PID, LOG_LOCAL1))
            throw new CpeException("Unable to connect to Syslog!",
                OPENLOG_ERROR);
        
        // Change Syslog priority level
        switch ($type)
        {
        case "INFO":
            $priority = LOG_INFO;
            break;
        case "ERROR":
            $priority = LOG_ERR;
            break;
        case "FATAL":
            $priority = LOG_ALERT;
            break;
        case "WARNING":
            $priority = LOG_WARNING;
            break;
        case "DEBUG":
            $priority = LOG_DEBUG;
            break;
        default:
            throw new CpeException("Unknown log Type!", 
                LOG_TYPE_ERROR);
        }
        
        // Print log in file
        $this->print_to_file($log, $workflowId);
        
        // Encode log message in JSON for better parsing
        $out = json_encode($log);
        // Send to syslog
        syslog($priority, $out);
    }

    // Write log in file
    private function print_to_file($log, $workflowId)
    {
        if (!is_string($log['message']))
            $log['message'] = json_encode($log['message']);
        
        $toPrint = $log['time'] . " [" . $log['type'] . "] [" . $log['source'] . "] ";
        // If there is a workflow ID. We append it.
        if ($workflowId)
            $toPrint .= "[$workflowId] ";
        $toPrint .= $log['message'] . "\n";

        if ($this->printout)
            print $toPrint;
        
        if (file_put_contents(
                $this->logPath,
                $toPrint,
                FILE_APPEND) === false)
            print "ERROR: Can't write into log file!\n";
    }
}

Push — master ( d5358f...a9ad96 )

CpeLogger::log_out() C

Complexity

Size

Duplication

Importance

How to fix Long Method

Long Method

1. Pass all data via parameters

2. Create a class that maintains your state

1			<?php
2
3			namespace SA\CpeSdk;
4
5			use SA\CpeSdk;
6
7			/**
8			* Allow formatted logging on STDOUT
9			* Send logs to Syslog for log offloading
10			*/
11			class CpeLogger
12			{
13			public $logPath;
14			public $printout;
15
16			// Exception
17			const LOG_TYPE_ERROR = "LOG_TYPE_ERROR";
18			const OPENLOG_ERROR = "OPENLOG_ERROR";
19
20			// Specify the path where to create the log files
21			public function __construct(
22			$logPath = null,
23			$suffix = null,
24			$printout = false)
25			{
26			global $argv;
			0 ignored issues – show Compatibility Best Practice introduced 2015-12-06 23:28 UTC by Report Bug Copy Issue Report Use of `global` functionality is not recommended; it makes your code harder to test, and less reusable. Instead of relying on `global` state, we recommend one of these alternatives: 1. Pass all data via parameters function myFunction($a, $b) { // Do something } 2. Create a class that maintains your state class MyClass { private $a; private $b; public function __construct($a, $b) { $this->a = $a; $this->b = $b; } public function myFunction() { // Do something } } Loading history...
27
28			date_default_timezone_set('UTC')
29
30			$this->printout = $printout;
			0 ignored issues – show Bug introduced 2016-10-18 16:07 UTC by Report Bug Copy Issue Report This code did not parse for me. Apparently, there is an error somewhere around this line: Syntax error, unexpected T_VARIABLE Loading history...
31			$this->logPath = "/var/tmp/logs/cpe/";
32
33			if ($logPath)
34			$this->logPath = $logPath;
35
36			if (!file_exists($this->logPath))
37			mkdir($this->logPath, 0755, true);
38
39			$file = basename($argv[0]);
40			if ($suffix)
41			$file .= "-".$suffix;
42			// Append progname to the path
43			$this->logPath .= "/".$file.".log";
44
45			$this->log_out(
46			"INFO",
47			"[CPE SDK] ".basename(__FILE__),
48			"Logging to: ".$this->logPath);
49			}
50
51			// Log message to syslog and log file
52			public function log_out(
53			$type,
54			$source,
55			$message,
56			$workflowId = null)
57			{
58			$log = [
59			"time" => date("Y-m-d H:i:s", time()),
60			"source" => $source,
61			"type" => $type,
62			"message" => $message
63			];
64
65			if ($workflowId)
66			$log["workflowId"] = $workflowId;
67
68			// Open Syslog. Use programe name as key
69			if (!openlog (__FILE__, LOG_CONS\|LOG_PID, LOG_LOCAL1))
70			throw new CpeException("Unable to connect to Syslog!",
71			OPENLOG_ERROR);
72
73			// Change Syslog priority level
74			switch ($type)
75			{
76			case "INFO":
77			$priority = LOG_INFO;
78			break;
79			case "ERROR":
80			$priority = LOG_ERR;
81			break;
82			case "FATAL":
83			$priority = LOG_ALERT;
84			break;
85			case "WARNING":
86			$priority = LOG_WARNING;
87			break;
88			case "DEBUG":
89			$priority = LOG_DEBUG;
90			break;
91			default:
92			throw new CpeException("Unknown log Type!",
93			LOG_TYPE_ERROR);
94			}
95
96			// Print log in file
97			$this->print_to_file($log, $workflowId);
98
99			// Encode log message in JSON for better parsing
100			$out = json_encode($log);
101			// Send to syslog
102			syslog($priority, $out);
103			}
104
105			// Write log in file
106			private function print_to_file($log, $workflowId)
107			{
108			if (!is_string($log['message']))
109			$log['message'] = json_encode($log['message']);
110
111			$toPrint = $log['time'] . " [" . $log['type'] . "] [" . $log['source'] . "] ";
112			// If there is a workflow ID. We append it.
113			if ($workflowId)
114			$toPrint .= "[$workflowId] ";
115			$toPrint .= $log['message'] . "\n";
116
117			if ($this->printout)
118			print $toPrint;
119
120			if (file_put_contents(
121			$this->logPath,
122			$toPrint,
123			FILE_APPEND) === false)
124			print "ERROR: Can't write into log file!\n";
125			}
126			}

sportarchive / CloudProcessingEngine-SDK

Push — master ( d5358f...a9ad96 )

CpeLogger::log_out() C

Complexity

Size

Duplication

Importance

How to fix Long Method

Long Method

1. Pass all data via parameters

2. Create a class that maintains your state

Duplication Side-by-Side

Filter issues like