AbstractTable::findOneById() - Code Metrics - Inspection of "Add storeAndLoad method to AbstractTable class" - vonbraunlabs/slimx - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 818a10...38ca42 )

by Diogo Oliveira de

created 2019-06-21 12:18 UTC

AbstractTable::findOneById() A

↳ Parent: AbstractTable

Complexity

Conditions	3
Paths	3

Size

Total Lines	18
Code Lines	11

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	3
eloc	11
nc	3
nop	1
dl	0
loc	18
rs	9.9
c	0
b	0
f	0

<?php

namespace SlimX\Models\Tables;

use Psr\Log\LoggerInterface;
filter:
    dependency_paths: ["lib/*"]
use RedBeanPHP\OODBBean;
use RedBeanPHP\R;

/**
 * Table abstraction
 */
abstract class AbstractTable
{
    protected $tableName;
    protected $logger;
    protected $cache;

    /**
     * Default controller.
     *
     * @param LoggerInterface $logger Logger to be used by the class.
     */
    public function __construct(LoggerInterface $logger)
    {
        $this->logger = $logger;
        $this->cache = [];
    }

    /**
     * Find one row, by ID.
     *
     * @param int $id Row ID.
     *
     * @return ?OODBBean Row, if found, NULL otherwise.
     */
    public function findOneById(int $id): ?OODBBean
    {
        if (!isset($this->cache[$id])) {
            $this->logger->debug(
                get_class($this) . '::findOneById cache miss ' . $id
            );
            $bean = R::findOne($this->tableName, 'id = ? AND active = ?', [$id, 1]);
            if (count($this->cache) < 1024) {
                $this->cache[$id] = $bean;
            }
        } else {
            $this->logger->debug(
                get_class($this) . '::findOneById cache hit ' . $id
            );
            $bean = $this->cache[$id];
        }

        return $bean;
    }

    /**
     * Store (insert or update) and load bean.
     *
     * @param OODBBean $bean Bean to be stored.
     *
     * @return OODBBean Bean freshly loaded from database.
     */
    public function storeAndLoad(OODBBean $bean): OODBBean
    {
        return R::load($this->tableName, (int) R::store($bean));
    }
}


1			<?php
2
3			namespace SlimX\Models\Tables;
4
5			use Psr\Log\LoggerInterface;
			0 ignored issues – show Bug introduced 2018-06-15 19:24 UTC by Report Bug Copy Issue Report The type `Psr\Log\LoggerInterface` was not found. Maybe you did not declare it correctly or list all dependencies? The issue could also be caused by a filter entry in the build configuration. If the path has been excluded in your configuration, e.g. `excluded_paths: ["lib/"]`, you can move it to the dependency path list as follows: filter: dependency_paths: ["lib/"] For further information see https://scrutinizer-ci.com/docs/tools/php/php-scrutinizer/#list-dependency-paths Loading history...
6			use RedBeanPHP\OODBBean;
7			use RedBeanPHP\R;
8
9			/**
10			* Table abstraction
11			*/
12			abstract class AbstractTable
13			{
14			protected $tableName;
15			protected $logger;
16			protected $cache;
17
18			/**
19			* Default controller.
20			*
21			* @param LoggerInterface $logger Logger to be used by the class.
22			*/
23			public function __construct(LoggerInterface $logger)
24			{
25			$this->logger = $logger;
26			$this->cache = [];
27			}
28
29			/**
30			* Find one row, by ID.
31			*
32			* @param int $id Row ID.
33			*
34			* @return ?OODBBean Row, if found, NULL otherwise.
35			*/
36			public function findOneById(int $id): ?OODBBean
37			{
38			if (!isset($this->cache[$id])) {
39			$this->logger->debug(
40			get_class($this) . '::findOneById cache miss ' . $id
41			);
42			$bean = R::findOne($this->tableName, 'id = ? AND active = ?', [$id, 1]);
43			if (count($this->cache) < 1024) {
44			$this->cache[$id] = $bean;
45			}
46			} else {
47			$this->logger->debug(
48			get_class($this) . '::findOneById cache hit ' . $id
49			);
50			$bean = $this->cache[$id];
51			}
52
53			return $bean;
54			}
55
56			/**
57			* Store (insert or update) and load bean.
58			*
59			* @param OODBBean $bean Bean to be stored.
60			*
61			* @return OODBBean Bean freshly loaded from database.
62			*/
63			public function storeAndLoad(OODBBean $bean): OODBBean
64			{
65			return R::load($this->tableName, (int) R::store($bean));
66			}
67			}
68

vonbraunlabs / slimx

Push — master ( 818a10...38ca42 )

AbstractTable::findOneById() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like