CLog::mostTimeConsumingDomain() - Code Metrics - Inspection of "Fixed the new tests" - simoneddeland/logger - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( bc35aa...fc2256 )

by Simon

created 2016-09-04 21:38 UTC

CLog::mostTimeConsumingDomain() B

↳ Parent: CLog

Complexity

Conditions	4
Paths	6

Size

Total Lines	22
Code Lines	12

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	15
CRAP Score	4

Importance

Changes	1
Bugs	0	Features	1

Metric	Value
cc	4
eloc	12
c	1
b	0
f	1
nc	6
nop	0
dl	0
loc	22
ccs	15
cts	15
cp	1
crap	4
rs	8.9197

<?php

namespace ultimadark\Logger;

/**
 * Class to log what happens.
 *
 */
class CLog
{

    /**
     * Properties
     *
     */
    private $timestamp = array();
    private $pos = 0;
    private $precision;

    /**
     * Constructor
     *
     * @param int $precision the precision of the rounding when presenting results.
     */
    public function __construct($precision = 4)
    {
        $this->precision = $precision;
    }

    /**
     * Timestamp, log a event with a time.
     *
     * @param string $domain is the module or class.
     * @param string $where a more specific place in the domain.
     * @param string $comment on the timestamp.
     *
     */
    public function timestamp($domain, $where, $comment = null)
    {
        $now = microtime(true);
        $this->timestamp[] = array(
          'domain'  => $domain,
          'where'   => $where,
          'comment' => $comment,
          'when'    => $now,
          'memory'  => memory_get_usage(true),
          'duration'=> 0,
        );

        if ($this->pos > 0) {
            $this->timestamp[$this->pos - 1]['memory-peak'] = memory_get_peak_usage(true);
            $this->timestamp[$this->pos - 1]['duration']    = $now - $this->timestamp[$this->pos - 1]['when'];
        }
    
        $this->pos++;
    }

    /**
     * Print all timestamp to a table.
     *
     * @return string with a html-table to display all timestamps.
     *
     */
    public function timestampAsTable()
    {

        // Set up the table
        $first = $this->timestamp[0]['when'];
        $last = $this->timestamp[count($this->timestamp) - 1]['when'];
        $html = "<table class='logtable'><caption>Timestamps</caption><tr><th>Domain</th><th>Where</th><th>When (sec)</th><th>Duration (sec)</th><th>Percent</th><th>Memory (MB)</th><th>Memory peak (MB)</th><th>Comment</th></tr>";
        $right = ' style="text-align: right;"';
        $total = array('domain' => array(), 'where' => array());

        $totalDuration = $last - $first;

        // Create the main table
        foreach ($this->timestamp as $val) {
            $when     = $val['when'] - $first;
            $duration = round($val['duration'], $this->precision);
            $percent  = round(($when) / $totalDuration * 100);
            $memory   = round($val['memory'] / 1024 / 1024, 2);
            $peak     = isset($val['memory-peak']) ? round($val['memory-peak'] / 1024 / 1024, 2): 0;
            $when     = round($when, $this->precision);
            $html .= "<tr><td>{$val['domain']}</td><td>{$val['where']}</td><td{$right}>{$when}</td><td{$right}>{$duration}</td><td{$right}>{$percent}</td><td{$right}>{$memory}</td><td{$right}>{$peak}</td><td>{$val['comment']}</td></tr>";
            @$total['domain'][$val['domain']] += $val['duration'];
// For example instead of
@mkdir($dir);

// Better use
if (@mkdir($dir) === false) {
    throw new \RuntimeException('The directory '.$dir.' could not be created.');
}
            @$total['where'][$val['where']] += $val['duration'];
// For example instead of
@mkdir($dir);

// Better use
if (@mkdir($dir) === false) {
    throw new \RuntimeException('The directory '.$dir.' could not be created.');
}
        }

        $html .= "</table><br><table class='logtable'><caption>Duration per domain</caption><tr><th>Domain</th><th>Duration</th><th>Percent</th></tr>";
    
    
        // Create the table grouped by domain   
        arsort($total['domain']);
    
        foreach ($total['domain'] as $key => $val) {

            if ($totalDuration != 0) {
                $percent = round($val / $totalDuration * 100, 1);
            } else {
                $percent = 100;
            }

            $roundedDuration = round($val, $this->precision);
            $html .= "<tr><td>{$key}</td><td>{$roundedDuration}</td><td>{$percent}</td></tr>";
        }

        $html .= "</table><br><table class='logtable'><caption>Duration per area</caption><tr><th>Area</th><th>Duration</th><th>Percent</th></tr>";
    
        // Create the table grouped by area
        arsort($total['where']);

        foreach ($total['where'] as $key => $val) {

            if ($totalDuration != 0) {
                $percent = round($val / $totalDuration * 100, 1);
            } else {
                $percent = 100;
            }  

            $roundedDuration = round($val, $this->precision);
            $html .= "<tr><td>{$key}</td><td>{$roundedDuration}</td><td>{$percent}</td></tr>";
        }

        $html .= "</table>";

        return $html;
    }

    /**
     * Returns load time of the page
     *
     * @return The page load time.
     *
     */
    public function pageLoadTime()
    {
        $first = $this->timestamp[0]['when'];
        $last = $this->timestamp[count($this->timestamp) - 1]['when'];
        $loadtime = round($last - $first, 3);
        return $loadtime;
    }

    /**
     * Get the memory peak.
     *
     * @return The memory peak.
     *
     */
    public function memoryPeak()
    {
        $peak = round(memory_get_peak_usage(true) / 1024 / 1024, 2);
        return $peak;
    }

    /**
     * Get the number of timestamps made so far.
     *
     * @return Number of timestamps.
     *
     */
    public function numberOfTimestamps()
    {
        return count($this->timestamp);
    }

    /**
     * Get the most time consuming domain.
     *
     * @return The most time consuming domain.
     *
     */
    public function mostTimeConsumingDomain()
    {
        $total = array();

        // Gather total duration data of each domain
        foreach ($this->timestamp as $val) {
            $duration = round($val['duration'], $this->precision);
            @$total[$val['domain']] += $duration;
// For example instead of
@mkdir($dir);

// Better use
if (@mkdir($dir) === false) {
    throw new \RuntimeException('The directory '.$dir.' could not be created.');
}
        }

        // Get the key of the most time consuming element
        $mostConsumingTime = 0;
        $mostConsumingDomain = "";
        foreach ($total as $key => $val) {
            if ($val >= $mostConsumingTime) {
                $mostConsumingDomain = $key;
                $mostConsumingTime = $val;
            }
        }

        return $mostConsumingDomain;
    }
}


1			<?php
2
3			namespace ultimadark\Logger;
4
5			/**
6			* Class to log what happens.
7			*
8			*/
9			class CLog
10			{
11
12			/**
13			* Properties
14			*
15			*/
16			private $timestamp = array();
17			private $pos = 0;
18			private $precision;
19
20			/**
21			* Constructor
22			*
23			* @param int $precision the precision of the rounding when presenting results.
24			*/
25	3		public function __construct($precision = 4)
26			{
27	3		$this->precision = $precision;
28	3		}
29
30			/**
31			* Timestamp, log a event with a time.
32			*
33			* @param string $domain is the module or class.
34			* @param string $where a more specific place in the domain.
35			* @param string $comment on the timestamp.
36			*
37			*/
38	3		public function timestamp($domain, $where, $comment = null)
39			{
40	3		$now = microtime(true);
41	3		$this->timestamp[] = array(
42	3		'domain' => $domain,
43	3		'where' => $where,
44	3		'comment' => $comment,
45	3		'when' => $now,
46	3		'memory' => memory_get_usage(true),
47	3		'duration'=> 0,
48			);
49
50	3		if ($this->pos > 0) {
51	3		$this->timestamp[$this->pos - 1]['memory-peak'] = memory_get_peak_usage(true);
52	3		$this->timestamp[$this->pos - 1]['duration'] = $now - $this->timestamp[$this->pos - 1]['when'];
53	3		}
54
55	3		$this->pos++;
56	3		}
57
58			/**
59			* Print all timestamp to a table.
60			*
61			* @return string with a html-table to display all timestamps.
62			*
63			*/
64	1		public function timestampAsTable()
65			{
66
67			// Set up the table
68	1		$first = $this->timestamp[0]['when'];
69	1		$last = $this->timestamp[count($this->timestamp) - 1]['when'];
70	1		$html = "<table class='logtable'><caption>Timestamps</caption><tr><th>Domain</th><th>Where</th><th>When (sec)</th><th>Duration (sec)</th><th>Percent</th><th>Memory (MB)</th><th>Memory peak (MB)</th><th>Comment</th></tr>";
71	1		$right = ' style="text-align: right;"';
72	1		$total = array('domain' => array(), 'where' => array());
73
74	1		$totalDuration = $last - $first;
75
76			// Create the main table
77	1		foreach ($this->timestamp as $val) {
78	1		$when = $val['when'] - $first;
79	1		$duration = round($val['duration'], $this->precision);
80	1		$percent = round(($when) / $totalDuration * 100);
81	1		$memory = round($val['memory'] / 1024 / 1024, 2);
82	1		$peak = isset($val['memory-peak']) ? round($val['memory-peak'] / 1024 / 1024, 2): 0;
83	1		$when = round($when, $this->precision);
84	1		$html .= "<tr><td>{$val['domain']}</td><td>{$val['where']}</td><td{$right}>{$when}</td><td{$right}>{$duration}</td><td{$right}>{$percent}</td><td{$right}>{$memory}</td><td{$right}>{$peak}</td><td>{$val['comment']}</td></tr>";
85	1		@$total['domain'][$val['domain']] += $val['duration'];
			0 ignored issues – show Security Best Practice introduced 2016-09-04 20:54 UTC by Report Bug Copy Issue Report It seems like you do not handle an error condition here. This can introduce security issues, and is generally not recommended. If you suppress an error, we recommend checking for the error condition explicitly: // For example instead of @mkdir($dir); // Better use if (@mkdir($dir) === false) { throw new \RuntimeException('The directory '.$dir.' could not be created.'); } Loading history...
86	1		@$total['where'][$val['where']] += $val['duration'];
			0 ignored issues – show Security Best Practice introduced 2016-09-04 20:54 UTC by Report Bug Copy Issue Report It seems like you do not handle an error condition here. This can introduce security issues, and is generally not recommended. If you suppress an error, we recommend checking for the error condition explicitly: // For example instead of @mkdir($dir); // Better use if (@mkdir($dir) === false) { throw new \RuntimeException('The directory '.$dir.' could not be created.'); } Loading history...
87	1		}
88
89	1		$html .= "</table><br><table class='logtable'><caption>Duration per domain</caption><tr><th>Domain</th><th>Duration</th><th>Percent</th></tr>";
90
91
92			// Create the table grouped by domain
93	1		arsort($total['domain']);
94
95	1	View Code Duplication	foreach ($total['domain'] as $key => $val) {
			0 ignored issues – show Duplication introduced 2016-09-04 20:54 UTC by Report Bug Copy Issue Report This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
96	1		if ($totalDuration != 0) {
97	1		$percent = round($val / $totalDuration * 100, 1);
98	1		} else {
99			$percent = 100;
100			}
101
102	1		$roundedDuration = round($val, $this->precision);
103	1		$html .= "<tr><td>{$key}</td><td>{$roundedDuration}</td><td>{$percent}</td></tr>";
104	1		}
105
106	1		$html .= "</table><br><table class='logtable'><caption>Duration per area</caption><tr><th>Area</th><th>Duration</th><th>Percent</th></tr>";
107
108			// Create the table grouped by area
109	1		arsort($total['where']);
110
111	1	View Code Duplication	foreach ($total['where'] as $key => $val) {
			0 ignored issues – show Duplication introduced 2016-09-04 20:54 UTC by Report Bug Copy Issue Report This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
112	1		if ($totalDuration != 0) {
113	1		$percent = round($val / $totalDuration * 100, 1);
114	1		} else {
115			$percent = 100;
116			}
117
118	1		$roundedDuration = round($val, $this->precision);
119	1		$html .= "<tr><td>{$key}</td><td>{$roundedDuration}</td><td>{$percent}</td></tr>";
120	1		}
121
122	1		$html .= "</table>";
123
124	1		return $html;
125			}
126
127			/**
128			* Returns load time of the page
129			*
130			* @return The page load time.
131			*
132			*/
133			public function pageLoadTime()
134			{
135			$first = $this->timestamp[0]['when'];
136			$last = $this->timestamp[count($this->timestamp) - 1]['when'];
137			$loadtime = round($last - $first, 3);
138			return $loadtime;
139			}
140
141			/**
142			* Get the memory peak.
143			*
144			* @return The memory peak.
145			*
146			*/
147			public function memoryPeak()
148			{
149			$peak = round(memory_get_peak_usage(true) / 1024 / 1024, 2);
150			return $peak;
151			}
152
153			/**
154			* Get the number of timestamps made so far.
155			*
156			* @return Number of timestamps.
157			*
158			*/
159	1		public function numberOfTimestamps()
160			{
161	1		return count($this->timestamp);
162			}
163
164			/**
165			* Get the most time consuming domain.
166			*
167			* @return The most time consuming domain.
168			*
169			*/
170	1		public function mostTimeConsumingDomain()
171			{
172	1		$total = array();
173
174			// Gather total duration data of each domain
175	1		foreach ($this->timestamp as $val) {
176	1		$duration = round($val['duration'], $this->precision);
177	1		@$total[$val['domain']] += $duration;
			0 ignored issues – show Security Best Practice introduced 2016-09-04 20:54 UTC by Report Bug Copy Issue Report It seems like you do not handle an error condition here. This can introduce security issues, and is generally not recommended. If you suppress an error, we recommend checking for the error condition explicitly: // For example instead of @mkdir($dir); // Better use if (@mkdir($dir) === false) { throw new \RuntimeException('The directory '.$dir.' could not be created.'); } Loading history...
178	1		}
179
180			// Get the key of the most time consuming element
181	1		$mostConsumingTime = 0;
182	1		$mostConsumingDomain = "";
183	1		foreach ($total as $key => $val) {
184	1		if ($val >= $mostConsumingTime) {
185	1		$mostConsumingDomain = $key;
186	1		$mostConsumingTime = $val;
187	1		}
188	1		}
189
190	1		return $mostConsumingDomain;
191			}
192			}
193

simoneddeland / logger

Push — master ( bc35aa...fc2256 )

CLog::mostTimeConsumingDomain() B

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like