App\Auth\AccountManager\FileBaseAccountManager

Complexity

Total Complexity

8

Size/Duplication

Total Lines	52
Duplicated Lines	30.77 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	isExistAccountId()	0	4	1
A	isEqualPassword()	8	8	2
A	getAccountById()	0	15	2
A	getRole()	8	8	2

<?php

namespace App\Auth\AccountManager;

use App\Auth\AccountManagerInterface;
use App\Auth\Model\Account;

class FileBaseAccountManager implements AccountManagerInterface
{
    private $accounts = [];

    public function __construct(array $settings = [])
    {
        $this->accounts = $settings['accounts'];
    }

    public function isExistAccountId($accountId)
    {
        return isset($this->accounts[$accountId]);
    }

    public function isEqualPassword($accountId, $password)

This method seems to be duplicated in your project.

    {
        if ($this->isExistAccountId($accountId) === false) {
            return false;
        }

        return $this->accounts[$accountId]['password'] === $password;
    }

    /**
     * @param string $accountId
     * @return Account|null
     */
    public function getAccountById($accountId)
    {
        if(isset($this->accounts[$accountId]) === false) {
            return null;
        }

        $user = $this->accounts[$accountId];

        $account = new Account();
        $account->setAccountId($accountId);
        $account->setPassword($user['password']);
        $account->setName($user['name']);

        return $account;
    }

    public function getRole($accountId)

This method seems to be duplicated in your project.

    {
        if ($this->isExistAccountId($accountId) === false) {
            return null;
        }

        return $this->accounts[$accountId]['role'];
    }
}