Del\Collection\User

Complexity

Total Complexity

10

Size/Duplication

Total Lines	68
Duplicated Lines	36.76 %

Coupling/Cohesion

Components	1
Dependencies	2

Test Coverage

Coverage

96.3%

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	append()	0	4	1
A	current()	0	4	1
A	findKey()	12	12	3
A	findById()	13	13	3
A	update()	0	11	2

<?php

namespace Del\Collection;

use Del\Entity\User as UserEntity;
use Doctrine\Common\Collections\ArrayCollection;
use LogicException;

class User extends ArrayCollection
{
    /**
     * @param UserEntity $user
     * @return $this
     */
    public function update(UserEntity $user)
    {
        $key = $this->findKey($user);
        if($key) {

            $this->offsetSet($key,$user);
            return $this;

        }
        throw new LogicException('User was not in the collection.');
    }

    /**
     * @param UserEntity $user
     */
    public function append(UserEntity $user)
    {
        $this->add($user);
    }

    /**
     * @return UserEntity|null
     */
    public function current()
    {
        return parent::current();
    }

    /**
     * @param UserEntity $user
     * @return bool|int
     */
    public function findKey(UserEntity $user)

This method seems to be duplicated in your project.

    {
        $it = $this->getIterator();
        $it->rewind();
        while($it->valid()) {
            if($it->current()->getId() == $user->getId()) {
                return $it->key();
            }
            $it->next();
        }
        return false;
    }



    public function findById($id)

This method seems to be duplicated in your project.

    {
        $it = $this->getIterator();
        $it->rewind();
        while($it->valid()) {
            if($it->current()->getId() == $id) {
                return $it->current();
            }
            $it->next();
        }

        return false;
    }

}