package org.drooms.impl.logic;

import java.util.Collection;
import java.util.Deque;
import java.util.HashSet;
import java.util.List;
import java.util.Map;
import java.util.Set;

import org.drooms.api.Edge;
import org.drooms.api.Node;
import org.drooms.api.Player;
import org.drooms.api.Playground;

import edu.uci.ics.jung.algorithms.shortestpath.ShortestPathUtils;
import edu.uci.ics.jung.algorithms.shortestpath.UnweightedShortestPath;
import edu.uci.ics.jung.graph.Graph;
import edu.uci.ics.jung.graph.UndirectedSparseGraph;

public class PathTracker<P extends Playground<N, E>, N extends Node, E extends Edge<N>> {

    private final P playground;
    private Graph<N, E> currentGraph;

    public PathTracker(final P playground, final Collection<Player> players) {
        this.playground = playground;
    }

    private Graph<N, E> cloneGraph(final Graph<N, E> src,
            final Collection<N> removeNodes) {
        final Graph<N, E> clone = new UndirectedSparseGraph<>();
        for (final N v : src.getVertices()) {
            clone.addVertex(v);
        }
        for (final E e : src.getEdges()) {
            clone.addEdge(e, src.getIncidentVertices(e));
        }
        for (final N node : removeNodes) {
            clone.removeVertex(node);
        }
        return clone;
    }

    public List<E> getPath(final N start, final N end) {
        return ShortestPathUtils
                .getPath(this.currentGraph, new UnweightedShortestPath<N, E>(
                        this.currentGraph), start, end);
    }

    public P getPlayground() {
        return this.playground;
    }

    public void movePlayers(final Map<Player, Deque<N>> newPositions) {
        final Set<N> unavailable = new HashSet<N>();
        for (final Map.Entry<Player, Deque<N>> entry : newPositions.entrySet()) {
            unavailable.addAll(entry.getValue());
        }
        final Graph<N, E> graphWithoutPlayers = this.playground.getGraph();
        this.currentGraph = this.cloneGraph(graphWithoutPlayers, unavailable);
    }

}

@@ -1604,22 +1604,91 @@
  %7D%0A%0A    
-public
+// FIXME Strategies should never be able to call this method.%0A    protected
  void mo

Make it harder for strategies to call movePlayers()