SequenceRepository::generate() - Code Metrics - Appsco/component-share - Measure and Improve Code Quality continuously with Scrutinizer

SequenceRepository::generate() A
last analyzed 2019-01-14 14:03 UTC

↳ Parent: SequenceRepository

Complexity

Conditions	3
Paths	2

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	12
rs	9.8666
c	0
b	0
f	0
cc	3
nc	2
nop	1

<?php

namespace BWC\Share\Data\Sequence;

use Doctrine\DBAL\Connection;

abstract class SequenceRepository implements SequenceRepositoryInterface
{
    const MAX_UNIQUE_LOOPS = 20;

    /** @var \Doctrine\DBAL\Connection */
    private $_con;

    /** @var \BWC\Share\Data\Sequence\SequenceGeneratorInterface */
    private $_generator;



    function __construct(Connection $connection, SequenceGeneratorInterface $generator) {

        $this->_con = $connection;
        $this->_generator = $generator;
    }



    abstract function getTableName();



    /**
     * @param string $name
     * @return string
     */
    function generate($name = 'default') {

        $result = null;
        for ($i=0; $i<self::MAX_UNIQUE_LOOPS; $i++) {
            $value = $this->_generator->generate();
            try {
                $this->_con->insert($this->getTableName(), array('name'=>$name, 'value'=>$value));
                $result = $value;
                break;
            } catch (\Exception $ex) { }

        }
        return $result;
    }


    /**
     * @param string $value
     * @param string $name
     * @return bool
     */
    function exists($value, $name = 'default') {

        $tbl = $this->getTableName();
        $count = $this->_con->fetchColumn("SELECT count(*) FROM `$tbl` WHERE name=:name AND value=:value",
            array(':name'=>$name, ':value'=>$value)
        );
        return $count ? true : false;
    }

}

1			<?php
2
3			namespace BWC\Share\Data\Sequence;
4
5			use Doctrine\DBAL\Connection;
6
7			abstract class SequenceRepository implements SequenceRepositoryInterface
8			{
9			const MAX_UNIQUE_LOOPS = 20;
10
11			/** @var \Doctrine\DBAL\Connection */
12			private $_con;
13
14			/** @var \BWC\Share\Data\Sequence\SequenceGeneratorInterface */
15			private $_generator;
16
17
18
19			function __construct(Connection $connection, SequenceGeneratorInterface $generator) {
			0 ignored issues – show Best Practice introduced 2016-06-07 08:08 UTC by Report Bug Copy Issue Report It is generally recommended to explicitly declare the visibility for methods. Adding explicit visibility (`private`, `protected`, or `public`) is generally recommend to communicate to other developers how, and from where this method is intended to be used. Loading history...
20			$this->_con = $connection;
21			$this->_generator = $generator;
22			}
23
24
25
26			abstract function getTableName();
			0 ignored issues – show Best Practice introduced 2016-06-07 08:08 UTC by Report Bug Copy Issue Report It is generally recommended to explicitly declare the visibility for methods. Adding explicit visibility (`private`, `protected`, or `public`) is generally recommend to communicate to other developers how, and from where this method is intended to be used. Loading history...
27
28
29			/**
30			* @param string $name
31			* @return string
32			*/
33			function generate($name = 'default') {
			0 ignored issues – show Best Practice introduced 2016-06-07 08:08 UTC by Report Bug Copy Issue Report It is generally recommended to explicitly declare the visibility for methods. Adding explicit visibility (`private`, `protected`, or `public`) is generally recommend to communicate to other developers how, and from where this method is intended to be used. Loading history...
34			$result = null;
35			for ($i=0; $i<self::MAX_UNIQUE_LOOPS; $i++) {
36			$value = $this->_generator->generate();
37			try {
38			$this->_con->insert($this->getTableName(), array('name'=>$name, 'value'=>$value));
39			$result = $value;
40			break;
41			} catch (\Exception $ex) { }
			0 ignored issues – show Coding Style Comprehensibility introduced 2016-06-07 08:08 UTC by Report Bug Copy Issue Report Consider adding a comment why this CATCH block is empty. Loading history...
42			}
43			return $result;
44			}
45
46
47			/**
48			* @param string $value
49			* @param string $name
50			* @return bool
51			*/
52			function exists($value, $name = 'default') {
			0 ignored issues – show Best Practice introduced 2016-06-07 08:08 UTC by Report Bug Copy Issue Report It is generally recommended to explicitly declare the visibility for methods. Adding explicit visibility (`private`, `protected`, or `public`) is generally recommend to communicate to other developers how, and from where this method is intended to be used. Loading history...
53			$tbl = $this->getTableName();
54			$count = $this->_con->fetchColumn("SELECT count(*) FROM `$tbl` WHERE name=:name AND value=:value",
55			array(':name'=>$name, ':value'=>$value)
56			);
57			return $count ? true : false;
58			}
59
60			}

Appsco / component-share

SequenceRepository::generate() A last analyzed 2019-01-14 14:03 UTC

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like

SequenceRepository::generate() A
last analyzed 2019-01-14 14:03 UTC