loic/chessarbiter
1
0
Fork 0
mirror of https://gitlab.com/manzerbredes/chessarbiter.git synced 2025-04-05 17:46:26 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions

Cleaning and debug code

Browse source
This commit is contained in:
Loic Guegan 2023-05-10 06:55:12 +02:00
parent 987bf4b2f2
commit e93ddffa4a
6 changed files with 11 additions and 8 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
src/Board.cpp
View file

@ -21,7 +21,7 @@ bool Board::AddPiece(char p, const std::string &coord) {
} }
bool Board::RemovePiece(const std::string &coord) { bool Board::RemovePiece(const std::string &coord) {
for (char i = 0; i < pieces.size(); i++) { for (char i = 0; i < (char)pieces.size(); i++) {
if (pieces[i].coord == coord) { if (pieces[i].coord == coord) {
pieces.erase(pieces.begin() + i); pieces.erase(pieces.begin() + i);
return (true); return (true);
@ -183,7 +183,7 @@ bool Board::IsMovePossible(const std::string &move) {
char r = src[1] + d2 * side; char r = src[1] + d2 * side;
// Perform empty square checks // Perform empty square checks
while (f != dst[0], r != dst[1]) { while (f != dst[0] && r != dst[1]) {
if (!IsEmpty(f + std::string() + r)) { if (!IsEmpty(f + std::string() + r)) {
return (false); return (false);
} }

1
src/ChessArbiter.cpp
View file

@ -480,7 +480,6 @@ std::string ChessArbiter::ParseSAN(const std::string &SANMove) {
piece = SANMove[0]; piece = SANMove[0];
char c1 = (SANMove.size() >= 2) ? SANMove[1] : '?'; char c1 = (SANMove.size() >= 2) ? SANMove[1] : '?';
char c2 = (SANMove.size() >= 3) ? SANMove[2] : '?'; char c2 = (SANMove.size() >= 3) ? SANMove[2] : '?';
char c3 = (SANMove.size() >= 4) ? SANMove[3] : '?';
if (c1 == 'x') { if (c1 == 'x') {
dst = SANMove.substr(2, 2); dst = SANMove.substr(2, 2);
} else if (c2 == 'x') { } else if (c2 == 'x') {

6
src/Fen.cpp
View file

@ -28,7 +28,7 @@ std::string FENParser::normalize_rank(const std::string &fen_rank) {
} }
char FENParser::NextToken(const std::string &fen, char loc) { char FENParser::NextToken(const std::string &fen, char loc) {
while (loc < fen.size() && IS_BLANK(fen[loc])) { while (loc < (char)fen.size() && IS_BLANK(fen[loc])) {
loc++; loc++;
} }
return (loc); return (loc);
@ -36,7 +36,7 @@ char FENParser::NextToken(const std::string &fen, char loc) {
char FENParser::NextRank(const std::string &fen, char loc) { char FENParser::NextRank(const std::string &fen, char loc) {
loc++; loc++;
while (loc < fen.size() && fen[loc] != '/' && fen[loc] != ' ') { while (loc < (char)fen.size() && fen[loc] != '/' && fen[loc] != ' ') {
loc++; loc++;
} }
return (loc); return (loc);
@ -190,7 +190,7 @@ FEN FENParser::Parse(const std::string &fen) {
// Parse move counter // Parse move counter
loc = NextToken(fen, loc); loc = NextToken(fen, loc);
std::string move; std::string move;
while (loc < fen.size() && !IS_BLANK(fen[loc])) { while (loc < (char)fen.size() && !IS_BLANK(fen[loc])) {
if (!IS_DIGIT(fen[loc])) { if (!IS_DIGIT(fen[loc])) {
throw InvalidFEN(); throw InvalidFEN();
} }

2
src/Fen.hpp
View file

@ -6,7 +6,7 @@
(c == '0' || c == '1' || c == '2' || c == '3' || c == '4' || c == '5' || \ (c == '0' || c == '1' || c == '2' || c == '3' || c == '4' || c == '5' || \
c == '6' || c == '7' || c == '8' || c == '9') c == '6' || c == '7' || c == '8' || c == '9')
#define IS_BLANK(c) (c == ' ' || c == '\n' || c == '\t' || c == '\r') #define IS_BLANK(c) (c == ' ' || c == '\n' || c == '\t' || c == '\r')
#define CHECK_LOC() {if(loc>=fen.size()){throw InvalidFEN();}} #define CHECK_LOC() {if(loc>=(char)fen.size()){throw InvalidFEN();}}
namespace chessarbiter { namespace chessarbiter {

2
src/Piece.hpp
View file

@ -16,9 +16,9 @@ namespace chessarbiter {
*/ */
class Piece { class Piece {
public: public:
char piece;
bool isBlack; bool isBlack;
std::string coord; std::string coord;
char piece;
Piece(char c, const std::string &coord); Piece(char c, const std::string &coord);
/// @brief Get all possible moves according to the type of piece and its position /// @brief Get all possible moves according to the type of piece and its position
std::vector<std::string> GetMoves(); std::vector<std::string> GetMoves();

4
tests/chessarbiter.cpp
View file

@ -529,4 +529,8 @@ TEST_CASE("Specific bugs found on a game", "[BugFixes]") {
a.Setup("rnb1k1nr/pppp1ppp/4P3/8/1b3B2/1Nq5/PPP1PPPP/R2KNB1R b kq - 16 12"); a.Setup("rnb1k1nr/pppp1ppp/4P3/8/1b3B2/1Nq5/PPP1PPPP/R2KNB1R b kq - 16 12");
a.Play("c3e1"); a.Play("c3e1");
CHECK(a.GetSAN()=="Qxe1#"); CHECK(a.GetSAN()=="Qxe1#");
// Bug 8 Not a bug but just check if bishop cannot jump above pieces
a.Setup("rnbqkbnr/p1pp1ppp/8/1p2p3/4P3/8/PPPP1PPP/RNBQKBNR w KQkq - 0 1");
CHECK(!a.Play("f1a6"));
} }