ErrorReporter::reportError() - Code Metrics - Inspection of "Version 1" - DaveLiddament/sarb - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — master (#69)

by Dave

created 2020-08-28 20:28 UTC

ErrorReporter::reportError() A

↳ Parent: ErrorReporter

Complexity

Conditions	5
Paths	4

Size

Total Lines	21
Code Lines	14

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	5
eloc	14
c	1
b	0
f	0
nc	4
nop	2
dl	0
loc	21
rs	9.4888

<?php

declare(strict_types=1);

namespace DaveLiddament\StaticAnalysisResultsBaseliner\Framework\Command\internal;

use DaveLiddament\StaticAnalysisResultsBaseliner\Domain\BaseLiner\BaseLineImportException;
use DaveLiddament\StaticAnalysisResultsBaseliner\Domain\Common\SarbException;
use Symfony\Component\Console\Output\ConsoleOutputInterface;
use Symfony\Component\Console\Output\OutputInterface;
use Throwable;

class ErrorReporter
{
    public static function reportError(OutputInterface $output, Throwable $throwable): int
    {
        try {
            throw $throwable;
        } catch (InvalidConfigException $e) {
            self::writeToStdError($output, "<error>{$e->getMessage()}</error>");

            return 2;
        } catch (BaseLineImportException $e) {
            self::writeToStdError($output, "<error>{$e->getMessage()}</error>");

            return 3;
        } catch (SarbException $e) {
            self::writeToStdError($output, "<error>Something went wrong: {$e->getMessage()}");

            return 4;
        } catch (Throwable $e) {
            // This should never happen. All exceptions should extend SarbException
            self::writeToStdError($output, "<error>Unexpected critical error: {$e->getMessage()}</error>");

            return 5;
        }
    }

    public static function writeToStdError(OutputInterface $output, string $message): void
    {
        $errorOutput = $output instanceof ConsoleOutputInterface ? $output->getErrorOutput() : $output;
        $errorOutput->writeln($message);
    }
}


1			<?php
2
3			declare(strict_types=1);
4
5			namespace DaveLiddament\StaticAnalysisResultsBaseliner\Framework\Command\internal;
6
7			use DaveLiddament\StaticAnalysisResultsBaseliner\Domain\BaseLiner\BaseLineImportException;
8			use DaveLiddament\StaticAnalysisResultsBaseliner\Domain\Common\SarbException;
9			use Symfony\Component\Console\Output\ConsoleOutputInterface;
10			use Symfony\Component\Console\Output\OutputInterface;
11			use Throwable;
12
13			class ErrorReporter
14			{
15			public static function reportError(OutputInterface $output, Throwable $throwable): int
16			{
17			try {
18			throw $throwable;
19			} catch (InvalidConfigException $e) {
20			self::writeToStdError($output, "<error>{$e->getMessage()}</error>");
21
22			return 2;
23			} catch (BaseLineImportException $e) {
24			self::writeToStdError($output, "<error>{$e->getMessage()}</error>");
25
26			return 3;
27			} catch (SarbException $e) {
28			self::writeToStdError($output, "<error>Something went wrong: {$e->getMessage()}");
29
30			return 4;
31			} catch (Throwable $e) {
32			// This should never happen. All exceptions should extend SarbException
33			self::writeToStdError($output, "<error>Unexpected critical error: {$e->getMessage()}</error>");
34
35			return 5;
36			}
37			}
38
39			public static function writeToStdError(OutputInterface $output, string $message): void
40			{
41			$errorOutput = $output instanceof ConsoleOutputInterface ? $output->getErrorOutput() : $output;
42			$errorOutput->writeln($message);
43			}
44			}
45

DaveLiddament / sarb

Pull Request — master (#69)

ErrorReporter::reportError() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like