Letournel\PathFinder\Core\NodeMap

Complexity

Total Complexity

17

Size/Duplication

Total Lines	98
Duplicated Lines	11.22 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes

0

13 Methods

Rating	Name	Duplication	Size	Complexity
A	__contruct()	0	4	1
A	count()	0	4	1
A	isEmpty()	0	4	1
A	rewind()	0	4	1
A	current()	0	4	1
A	key()	0	4	1
A	next()	0	4	1
A	valid()	0	4	1
A	lookup()	11	11	2
A	lookupFrom()	0	16	3
A	delete()	0	13	2
A	exists()	0	4	1
A	insert()	0	4	1

<?php

namespace Letournel\PathFinder\Core;

class NodeMap implements \Countable, \Iterator
{
    private
        $map;

The visibility should be declared for property $map.

    
    public function __contruct()
    {
        $this->map = array();
    }
    
    public function count()
    {
        return count($this->map);
    }
    
    public function isEmpty()
    {
        return empty($this->map);
    }
    
    public function rewind()
    {
        rewind($this->map);
    }
    
    public function current()
    {
        return current($this->map);
    }
    
    public function key()
    {
        return key($this->map);
    }
    
    public function next()
    {
        return next($this->map);
    }
    
    public function valid()
    {
        return valid($this->map);
    }
    
    public function lookup(Node $node)

This method seems to be duplicated in your project.

    {
        $id = $node->getId();
        
        if(array_key_exists($id, $this->map))
        {
            return $this->map[$id];
        }
        
        return null;
    }
    
    public function lookupFrom(Node $node)
    {
        $path = array();
        while($node instanceof Node)
        {
            array_unshift($path, $node);
            $node = $this->lookup($node);
            
            if(in_array($node, $path))
            {
                break;
            }
        }
        
        return $path;
    }
    
    public function delete(Node $node)
    {
        $id = $node->getId();
        $value = null;
        
        if(array_key_exists($id, $this->map))
        {
            $value = $this->map[$id];
            unset($this->map[$id]);
        }
        
        return $value;
    }
    
    public function exists(Node $node)
    {
        return array_key_exists($node->getId(), $this->map);
    }
    
    public function insert(Node $node, $value = true)
    {
        $this->map[$node->getId()] = $value;
    }
}