FoundMoreNews::__invoke() - Code Metrics - Inspection of "Adding unit tests" - Chemaclass/stock-ticker - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — master (#4)

by Chema

created 2020-12-16 18:48 UTC

FoundMoreNews::__invoke() A

↳ Parent: FoundMoreNews

Complexity

Conditions	1
Paths	1

Size

Total Lines	8
Code Lines	4

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	5
CRAP Score	1

Importance

Changes

Metric	Value
cc	1
eloc	4
c	0
b	0
f	0
nc	1
nop	1
dl	0
loc	8
rs	10
ccs	5
cts	5
cp	1
crap	1

<?php

declare(strict_types=1);

namespace Chemaclass\StockTicker\Domain\Notifier\Policy\Condition;

use Chemaclass\StockTicker\Domain\Notifier\Policy\PolicyConditionInterface;
use Chemaclass\StockTicker\Domain\ReadModel\Company;

final class FoundMoreNews implements PolicyConditionInterface
{
    public const NEWS = 'NEWS';

    /**
     * @var array<string,string>
     * For example ['TickerSymbol' => 'datetime']
     */
    private static array $cacheOldestDateTimeBySymbol = [];

    public function __construct()
    {
        // TODO: Add as optional arg: the datetime from where to start looking for news.
    }

    public function __invoke(Company $company): bool
    {
        $current = $this->findLatestDateTimeFromNews($company);
        $previous = self::$cacheOldestDateTimeBySymbol[$company->symbol()->toString()] ?? '';

        self::$cacheOldestDateTimeBySymbol[$company->symbol()->toString()] = $current;

        return $current > $previous;
    }

    private function findLatestDateTimeFromNews(Company $company): string
    {
        $reduced = array_reduce(
            (array) $company->info(self::NEWS),
            static function (?array $carry, array $current): array {
                if (null === $carry) {

                    return $current;
                }

                return $carry['datetime'] > $current['datetime']
                    ? $carry
                    : $current;
            }
        );

        return $reduced['datetime'];
    }
}


1		<?php
2
3		declare(strict_types=1);
4
5		namespace Chemaclass\StockTicker\Domain\Notifier\Policy\Condition;
6
7		use Chemaclass\StockTicker\Domain\Notifier\Policy\PolicyConditionInterface;
8		use Chemaclass\StockTicker\Domain\ReadModel\Company;
9
10		final class FoundMoreNews implements PolicyConditionInterface
11		{
12		public const NEWS = 'NEWS';
13
14		/**
15		* @var array<string,string>
16		* For example ['TickerSymbol' => 'datetime']
17		*/
18		private static array $cacheOldestDateTimeBySymbol = [];
19
20	1	public function __construct()
21		{
22		// TODO: Add as optional arg: the datetime from where to start looking for news.
23	1	}
24
25	1	public function __invoke(Company $company): bool
26		{
27	1	$current = $this->findLatestDateTimeFromNews($company);
28	1	$previous = self::$cacheOldestDateTimeBySymbol[$company->symbol()->toString()] ?? '';
29
30	1	self::$cacheOldestDateTimeBySymbol[$company->symbol()->toString()] = $current;
31
32	1	return $current > $previous;
33		}
34
35	1	private function findLatestDateTimeFromNews(Company $company): string
36		{
37	1	$reduced = array_reduce(
38	1	(array) $company->info(self::NEWS),
39	1	static function (?array $carry, array $current): array {
40	1	if (null === $carry) {
		0 ignored issues – show introduced 2020-12-15 16:58 UTC by Report Bug Copy Issue Report The condition `null === $carry` is always `false`. Loading history...
41	1	return $current;
42		}
43
44	1	return $carry['datetime'] > $current['datetime']
45	1	? $carry
46	1	: $current;
47	1	}
48		);
49
50	1	return $reduced['datetime'];
51		}
52		}
53

Chemaclass / stock-ticker

Pull Request — master (#4)

FoundMoreNews::__invoke() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like