RogueSharp icon indicating copy to clipboard operation
RogueSharp copied to clipboard

DijkstraShortestPath Check method should be used or removed

Open FaronBracy opened this issue 6 years ago • 0 comments

The DijkstraShortestPath class has a Check method that is currently only used by unit tests. https://github.com/FaronBracy/RogueSharp/blob/master/RogueSharp/Algorithms/DijkstraShortestPath.cs#L153

It was originally intended to be a private method and called from the bottom of the constructor.

Investigate if this method is useful or not. If it's not get rid of it. If it is useful consider making it private and make the class use it where appropriate.

FaronBracy avatar Feb 02 '19 17:02 FaronBracy