AbstractFileStrategy::getAdapters() - Code Metrics - Inspection of "fix issue #131: allow the usage of .bat, .cmd and..." - alchemy-fr/Zippy - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#132)

unknown

created 2021-04-13 14:22 UTC

AbstractFileStrategy::getAdapters() A

↳ Parent: AbstractFileStrategy

Complexity

Conditions	3
Paths	3

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	13
rs	9.8333
c	0
b	0
f	0
cc	3
nc	3
nop	0

<?php

/*
 * This file is part of Zippy.
 *
 * (c) Alchemy <[email protected]>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace Alchemy\Zippy\FileStrategy;

use Alchemy\Zippy\Adapter\AdapterContainer;
use Alchemy\Zippy\Exception\RuntimeException;

abstract class AbstractFileStrategy implements FileStrategyInterface
{
    protected $container;

    public function __construct(AdapterContainer $container)
    {
        $this->container = $container;
    }

    /**
     * {@inheritdoc}
     */
    public function getAdapters()
    {
        $services = array();
        foreach ($this->getServiceNames() as $serviceName) {
            try {
                $services[] = $this->container[$serviceName];
            } catch (RuntimeException $e) {


            }
        }

        return $services;
    }

    /**
     * Returns an array of service names that defines adapters.
     *
     * @return string[]
     */
    abstract protected function getServiceNames();
}


1			<?php
2
3			/*
4			* This file is part of Zippy.
5			*
6			* (c) Alchemy <[email protected]>
7			*
8			* For the full copyright and license information, please view the LICENSE
9			* file that was distributed with this source code.
10			*/
11
12			namespace Alchemy\Zippy\FileStrategy;
13
14			use Alchemy\Zippy\Adapter\AdapterContainer;
15			use Alchemy\Zippy\Exception\RuntimeException;
16
17			abstract class AbstractFileStrategy implements FileStrategyInterface
18			{
19			protected $container;
20
21			public function __construct(AdapterContainer $container)
22			{
23			$this->container = $container;
24			}
25
26			/**
27			* {@inheritdoc}
28			*/
29			public function getAdapters()
30			{
31			$services = array();
32			foreach ($this->getServiceNames() as $serviceName) {
33			try {
34			$services[] = $this->container[$serviceName];
35			} catch (RuntimeException $e) {
			0 ignored issues – show Coding Style Comprehensibility introduced 2016-02-12 10:17 UTC by Report Bug Copy Issue Report Consider adding a comment why this CATCH block is empty. Loading history...
36
37			}
38			}
39
40			return $services;
41			}
42
43			/**
44			* Returns an array of service names that defines adapters.
45			*
46			* @return string[]
47			*/
48			abstract protected function getServiceNames();
49			}
50

alchemy-fr / Zippy

Pull Request — master (#132)

AbstractFileStrategy::getAdapters() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like