PriceFormatter::format() - Code Metrics - jeancsil/flight-spy - Measure and Improve Code Quality continuously with Scrutinizer

PriceFormatter::format() B
last analyzed 2017-04-10 08:24 UTC

↳ Parent: PriceFormatter

Complexity

Conditions	4
Paths	4

Size

Total Lines	23
Code Lines	16

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	20

Importance

Changes	2
Bugs	0	Features	0

Metric	Value
dl	0
loc	23
ccs	0
cts	22
cp	0
rs	8.7972
c	2
b	0
f	0
cc	4
eloc	16
nc	4
nop	2
crap	20

<?php
/**
 * @author Jean Silva <[email protected]>
 * @license MIT
 */
namespace Jeancsil\FlightSpy\Service\Currency;

class PriceFormatter
{
    const CURRENCIES_FILE = 'currencies.json';

    private $currencies;

    public function __construct($resourcesDir)
    {
        $this->initializeCurrenciesFile($resourcesDir);
    }

    /**
     * @param string $price
     * @param string $currencyCode
     * @return string

     */
    public function format($price, $currencyCode)
    {
        foreach ($this->currencies as $currency) {
            if ($currency['Code'] == $currencyCode) {
                if ($currency['SymbolOnLeft'] == 'true') {
                    return $currency['Symbol'] .
                        number_format(
                            $price,
                            2,
                            $currency['DecimalSeparator'],
                            $currency['ThousandsSeparator']
                        );
                }

                return number_format(
                    $price,
                    2,
                    $currency['DecimalSeparator'],
                    $currency['ThousandsSeparator']
                ) . $currency['Symbol'];
            }
        }
    }

    /**
     * @param $resourcesDir
     */
    private function initializeCurrenciesFile($resourcesDir)
    {
        if (!$this->currencies) {
            $currenciesFile = getcwd() . '/' . $resourcesDir . '/' . static::CURRENCIES_FILE;
$a = "a";
$ab = "ab";
$abc = "abc";
            $currenciesArray = json_decode(file_get_contents($currenciesFile), true);
$a = "a";
$ab = "ab";
$abc = "abc";
            $this->currencies = $currenciesArray['ReferenceServiceResponseDto']['Currencies']['CurrencyDto'];
        }
    }
}


1			<?php
2			/**
3			* @author Jean Silva <[email protected]>
4			* @license MIT
5			*/
6			namespace Jeancsil\FlightSpy\Service\Currency;
7
8			class PriceFormatter
9			{
10			const CURRENCIES_FILE = 'currencies.json';
11
12			private $currencies;
13
14			public function __construct($resourcesDir)
15			{
16			$this->initializeCurrenciesFile($resourcesDir);
17			}
18
19			/**
20			* @param string $price
21			* @param string $currencyCode
22			* @return string
			0 ignored issues – show Documentation introduced 2017-01-29 15:15 UTC by Report Bug Copy Issue Report Should the return type not be `string\|null`? This check compares the return type specified in the `@return` annotation of a function or method doc comment with the types returned by the function and raises an issue if they mismatch. Loading history...
23			*/
24			public function format($price, $currencyCode)
25			{
26			foreach ($this->currencies as $currency) {
27			if ($currency['Code'] == $currencyCode) {
28			if ($currency['SymbolOnLeft'] == 'true') {
29			return $currency['Symbol'] .
30			number_format(
31			$price,
32			2,
33			$currency['DecimalSeparator'],
34			$currency['ThousandsSeparator']
35			);
36			}
37
38			return number_format(
39			$price,
40			2,
41			$currency['DecimalSeparator'],
42			$currency['ThousandsSeparator']
43			) . $currency['Symbol'];
44			}
45			}
46			}
47
48			/**
49			* @param $resourcesDir
50			*/
51			private function initializeCurrenciesFile($resourcesDir)
52			{
53			if (!$this->currencies) {
54			$currenciesFile = getcwd() . '/' . $resourcesDir . '/' . static::CURRENCIES_FILE;
			0 ignored issues – show Coding Style introduced 2017-01-29 15:15 UTC by Report Bug Copy Issue Report Equals sign not aligned with surrounding assignments; expected 3 spaces but found 1 space This check looks for multiple assignments in successive lines of code. It will report an issue if the operators are not in a straight line. To visualize $a = "a"; $ab = "ab"; $abc = "abc"; will produce issues in the first and second line, while this second example $a = "a"; $ab = "ab"; $abc = "abc"; will produce no issues. Loading history...
55			$currenciesArray = json_decode(file_get_contents($currenciesFile), true);
			0 ignored issues – show Coding Style introduced 2017-01-29 15:15 UTC by Report Bug Copy Issue Report Equals sign not aligned with surrounding assignments; expected 2 spaces but found 1 space This check looks for multiple assignments in successive lines of code. It will report an issue if the operators are not in a straight line. To visualize $a = "a"; $ab = "ab"; $abc = "abc"; will produce issues in the first and second line, while this second example $a = "a"; $ab = "ab"; $abc = "abc"; will produce no issues. Loading history...
56			$this->currencies = $currenciesArray['ReferenceServiceResponseDto']['Currencies']['CurrencyDto'];
57			}
58			}
59			}
60

jeancsil / flight-spy

PriceFormatter::format() B last analyzed 2017-04-10 08:24 UTC

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like

PriceFormatter::format() B
last analyzed 2017-04-10 08:24 UTC