SendEmail::execute() - Code Metrics - Inspection of "Merge remote-tracking branch 'origin/master'" - shippinno/email - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( d274c4...2117c8 )

by Hirofumi

created 2018-11-06 03:13 UTC

SendEmail::execute() A

↳ Parent: SendEmail

Complexity

Conditions	4
Paths	4

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	10
CRAP Score	4.0119

Importance

Changes

Metric	Value
cc	4
nc	4
nop	3
dl	0
loc	17
ccs	10
cts	11
cp	0.9091
crap	4.0119
rs	9.7
c	0
b	0
f	0

<?php
declare(strict_types=1);

namespace Shippinno\Email;

use Psr\Log\LoggerAwareTrait;
use Psr\Log\LoggerInterface;
use Psr\Log\NullLogger;
use Tanigami\ValueObjects\Web\Email;
use Throwable;

abstract class SendEmail
{
    use LoggerAwareTrait;

    /**
     * @param LoggerInterface|null $logger
     */
    public function __construct(
        LoggerInterface $logger = null
    ) {
        $this->setLogger(is_null($logger) ? new NullLogger : $logger);
    }

    /**
     * @param Email $email
     * @param int $maxAttempts
     * @param SmtpConfiguration|null $smtpConfiguration
     * @return int
     * @throws EmailNotSentException
     * @throws Throwable
     */
    public function execute(Email $email, int $maxAttempts = 1, SmtpConfiguration $smtpConfiguration = null): int
    {
        for ($i = 1; $i <= $maxAttempts; $i++) {
            try {
                return $this->doExecute($email, $smtpConfiguration);
            } catch (EmailNotSentException|Throwable $e) {
                if ($i < $maxAttempts) {
                    continue;
                }
                $this->logger->debug(sprintf('Gave up sending an email after reattempting %d times.', $i), [
                    'subject' => $email->subject(),
                    'exception' => $e,
                ]);
                throw $e;
            }
        }
    }

    /**
     * @param Email $email
     * @param SmtpConfiguration|null $smtpConfiguration
     * @return int
     * @throws EmailNotSentException
     */
    abstract protected function doExecute(Email $email, SmtpConfiguration $smtpConfiguration = null): int;
}


1		<?php
2		declare(strict_types=1);
3
4		namespace Shippinno\Email;
5
6		use Psr\Log\LoggerAwareTrait;
7		use Psr\Log\LoggerInterface;
8		use Psr\Log\NullLogger;
9		use Tanigami\ValueObjects\Web\Email;
10		use Throwable;
11
12		abstract class SendEmail
13		{
14		use LoggerAwareTrait;
15
16		/**
17		* @param LoggerInterface\|null $logger
18		*/
19	4	public function __construct(
20		LoggerInterface $logger = null
21		) {
22	4	$this->setLogger(is_null($logger) ? new NullLogger : $logger);
23	4	}
24
25		/**
26		* @param Email $email
27		* @param int $maxAttempts
28		* @param SmtpConfiguration\|null $smtpConfiguration
29		* @return int
30		* @throws EmailNotSentException
31		* @throws Throwable
32		*/
33	3	public function execute(Email $email, int $maxAttempts = 1, SmtpConfiguration $smtpConfiguration = null): int
34		{
35	3	for ($i = 1; $i <= $maxAttempts; $i++) {
36		try {
37	3	return $this->doExecute($email, $smtpConfiguration);
38	2	} catch (EmailNotSentException\|Throwable $e) {
39	2	if ($i < $maxAttempts) {
40	1	continue;
41		}
42	2	$this->logger->debug(sprintf('Gave up sending an email after reattempting %d times.', $i), [
43	2	'subject' => $email->subject(),
44	2	'exception' => $e,
45		]);
46	2	throw $e;
47		}
48		}
49		}
50
51		/**
52		* @param Email $email
53		* @param SmtpConfiguration\|null $smtpConfiguration
54		* @return int
55		* @throws EmailNotSentException
56		*/
57		abstract protected function doExecute(Email $email, SmtpConfiguration $smtpConfiguration = null): int;
58		}
59

shippinno / email

Push — master ( d274c4...2117c8 )

SendEmail::execute() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like