sfod / quoridor Goto Github PK

It's error prone to change image when it's size is duplicated in code.

Add macro BOOST_ENABLE_ASSERT_HANDLER

First player pawn is set at the node 0:6

It's simpler to keep all images in one file (e.g. quoridor.png) and load images with one imageset file (e.g. quoridor.imageset).

Event system is built on Boost Signals now. But Qt provides its own signal mechanism, so it would be logical to rewrite event system.

Resource file paths are hard-coded now. That's just wrong.

Pawn is in node 1:5.
Trying to set horizontal wall in 1:5.

In this situation method Board::add_wall must ignore wall and return -1.

Many temporary walls are created and destroyed during setting new wall.

The first object file is used to build quoridor binary.
The second object file is used to build test binaries.

Mouse cursor in the dragging window is calculating in two methods:

• DraggedWindow::handle_stop_drag_
• GameState::handle_pawn_dropped_

And handle_stop_drag_ fires event to which handle_pawn_dropped_ is subscribed.

Sometimes pawn jumps over another pawn by diagonal despite free field behind other pawn.

2-player game starts independently of number of player parameter.

Not all exceptions are handled now.

At this moment adding new element to the board requires a lot of time and effort.

Make wall stock as abstract walls holder.
When player clicked with left mouse button on this stock new wall is sticking to mouse cursor.
Another left mouse button click sets wall, right mouse button click cancels operation.

Use smart pointers

Probably this happens because BoardGraph is undirected graph but each edge is added twice as if graph was directed.

Game is playable only when one player is human and others players are bots.