WidthCollection::loadWidthsFromCsv() - Code Metrics - Inspection of "Api and styling changes. Auto column sizing added." - nimmneun/onesheet - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#7)

by Stefan

created 2016-08-13 16:55 UTC

WidthCollection::loadWidthsFromCsv() A

↳ Parent: WidthCollection

Complexity

Conditions	3
Paths	3

Size

Total Lines	10
Code Lines	6

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	9
CRAP Score	3

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
c	1
b	0
f	0
dl	0
loc	10
ccs	9
cts	9
cp	1
rs	9.4285
cc	3
eloc	6
nc	3
nop	1
crap	3

<?php

namespace OneSheet\Width;

/**
 * Class WidthCollection
 *
 * @package OneSheet
 */
class WidthCollection
{
    /**
     * Array containing character widths for each font & size.
     *
     * @var array
     */
    private static $widths = array();

    /**
     * Create character width map for each font.
     */
    public function __construct()
    {
        self::loadWidthsFromCsv(dirname(__FILE__) . '/widths.csv');
    }

    /**
     * Dirty way to allow developers to load character widths that
     * are not yet included.
     *
     * @param string $csvPath
     */
    public static function loadWidthsFromCsv($csvPath)
    {
        foreach (file($csvPath) as $line) {
            $widths = explode(',', trim($line));
            if (count(range(33, 126)) + 2 == count($widths)) {
                self::$widths[array_shift($widths)][array_shift($widths)] =
                    array_combine(array_map('chr', range(33, 126)), $widths);
            }
        }
    }

    /**
     * Return character widths for given font name.
     *
     * @param string $fontName
     * @param int    $fontSize
     * @return array
     */
    public function get($fontName, $fontSize)
    {
        if (isset(self::$widths[$fontName][$fontSize])) {
            return self::$widths[$fontName][$fontSize];
        }

        return self::$widths['Calibri'][11];
    }
}


1		<?php
2
3		namespace OneSheet\Width;
4
5		/**
6		* Class WidthCollection
7		*
8		* @package OneSheet
9		*/
10		class WidthCollection
11		{
12		/**
13		* Array containing character widths for each font & size.
14		*
15		* @var array
16		*/
17		private static $widths = array();
18
19		/**
20		* Create character width map for each font.
21		*/
22	5	public function __construct()
23		{
24	5	self::loadWidthsFromCsv(dirname(__FILE__) . '/widths.csv');
25	5	}
26
27		/**
28		* Dirty way to allow developers to load character widths that
29		* are not yet included.
30		*
31		* @param string $csvPath
32		*/
33	5	public static function loadWidthsFromCsv($csvPath)
34		{
35	5	foreach (file($csvPath) as $line) {
36	5	$widths = explode(',', trim($line));
37	5	if (count(range(33, 126)) + 2 == count($widths)) {
38	5	self::$widths[array_shift($widths)][array_shift($widths)] =
39	5	array_combine(array_map('chr', range(33, 126)), $widths);
40	5	}
41	5	}
42	5	}
43
44		/**
45		* Return character widths for given font name.
46		*
47		* @param string $fontName
48		* @param int $fontSize
49		* @return array
50		*/
51	2	public function get($fontName, $fontSize)
52		{
53	2	if (isset(self::$widths[$fontName][$fontSize])) {
54	2	return self::$widths[$fontName][$fontSize];
55		}
56
57		return self::$widths['Calibri'][11];
58		}
59		}
60

nimmneun / onesheet

Pull Request — master (#7)

WidthCollection::loadWidthsFromCsv() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like