tl-nguyen/labyrinth2

TelerikAcademy: High Quality Code - Teamwork

labyrinth2's Issues

StyleCop GameEngine

StyleCop UserInput Console and IUserInput

StyleCop ICell IFactory IGameLogic IResultFactory

StyleCop full pass Entities

StyleCop UI

Must run on all files in UI folder

StyleCop Cell and Factory

Test Entities

Make tests for all classes inside Entities namespace ( folder ).

Test GameLogic

Make tests for GameLogic class ( using mock objects )

Test GameStart

Make tests for Game start class ( 1 test is sufficient )

Test UI

Make tests for all classes inside UI namespace (folder)

StyleCop Commons

Must run on all files in the folder Commons

Test Renderer

Make tests for all classes inside Renderer namespace (folder)

StyleCop Results

Must run on all files in Results

Test Results

Make tests for all classes inside Results namespace (folder)

Test Factory

Finish and check tests for Factory class

Test Cell

Make tests for Cell class

StyleCop GameStart and GameLogic

Style Cop Loggers

Must run on all classes in Loggers folder

Rendering structure

The renderering will have a facade that will show only 2 public methods, and it will instantiate the classes it uses internally from its own Factory, that will have an interface, that it will also take in its constructor.

The main factory will have only 1 method that will instantiate this, and it will choose what factory to give to its interface.

Test Loggers

Make tests for all classes inside Loggers namespace (folder)

StyleCop LabyrinthHandler

Must run on all classes in LabyrinthHandler folder

Remove player class

Fix this so the functionality is only in the Labyrinth, and fix it in the classes that use it.

StyleCop Renderer

Must run on all files in Renderer folder

Test UserInput

Make tests for the UserInput class

Important: Remove the constant LABYRINTH_SIZE, and refactor so the logic works without it

This is in general a bad practice as this is a public constant, in a class that can be replaced, and if this happens the whole program will not work.

Test LabyrinthHandler

Make tests for MoveHandler class inside LabyrinthHandler namespace ( folder ).

Test GameEngine

Make tests for GameEngine class