Print error message when the class constructor fails

nicolarevelant · Oct 21, 2023 · 3a2047c · 3a2047c
1 parent 51b4de5
commit 3a2047c
Show file tree

Hide file tree

Showing 4 changed files with 23 additions and 4 deletions.
diff --git a/src/ChessboardGrid/ChessboardGrid.cpp b/src/ChessboardGrid/ChessboardGrid.cpp
@@ -10,8 +10,12 @@ ChessboardGrid::~ChessboardGrid() = default;
 
 ChessboardGrid::ChessboardGrid(const ImageProviderCB &images, const ColorProviderCB &colors,
 							   wxWindow *parent, wxWindowID winId, const wxPoint &pos) : ChessboardGrid() {
-	if (!Create(images, colors, parent, winId, pos))
+	if (!Create(images, colors, parent, winId, pos)) {
+#ifdef DEBUG
+		std::cerr << "Cannot create ChessboardGrid" << std::endl;
+#endif
 		throw std::exception();
+	}
 }
 
 bool ChessboardGrid::Create(const ImageProviderCB &images, const ColorProviderCB &colors,

diff --git a/src/ChessboardSquare/ChessboardSquare.cpp b/src/ChessboardSquare/ChessboardSquare.cpp
@@ -8,8 +8,12 @@ ChessboardSquare::ChessboardSquare() {
 }
 
 ChessboardSquare::ChessboardSquare(int size, wxWindow *parent, wxWindowID windowId) : ChessboardSquare() {
-	if (!Create(size, parent, windowId))
+	if (!Create(size, parent, windowId)) {
+#ifdef DEBUG
+		std::cerr << "Cannot create ChessboardSquare" << std::endl;
+#endif
 		throw std::exception();
+	}
 }
 
 bool ChessboardSquare::Create(int size, wxWindow *parent, wxWindowID windowId) {

diff --git a/src/Frame/Frame.cpp b/src/Frame/Frame.cpp
@@ -10,8 +10,12 @@
 Frame::Frame() = default;
 
 Frame::Frame(wxWindow *parent, const std::string &theme) : Frame() {
-	if (!Create(parent, theme))
+	if (!Create(parent, theme)) {
+#ifdef DEBUG
+		std::cerr << "Cannot create Frame" << std::endl;
+#endif
 		throw std::exception();
+	}
 }
 
 bool Frame::Create(wxWindow *parent, const std::string &theme) {

diff --git a/src/MatchManager/MatchManager.cpp b/src/MatchManager/MatchManager.cpp
@@ -4,8 +4,12 @@
 #include "MatchManager.h"
 
 MatchManager::MatchManager(ChessboardGrid *chessboard) {
-	if (chessboard == nullptr)
+	if (chessboard == nullptr) {
+#ifdef DEBUG
+		std::cerr << "Cannot create MatchManager: chessboard == nullptr" << std::endl;
+#endif
 		throw std::exception();
+	}
 
 	chessboard->Bind(wxEVT_LEFT_UP, &MatchManager::onChessboardSquareClick, this);
 	chessboard->Bind(wxEVT_MENU, &MatchManager::onThreadFinish, this, THREAD_ID);
@@ -150,8 +154,11 @@ void MatchManager::makePCMove() {
 
 void MatchManager::onThreadFinish(wxCommandEvent &evt) {
 	if (!algorithmThread) {
+#ifdef DEBUG
 		std::cerr << "Unwanted thread finished" << std::endl;
+#endif
 		delete static_cast<GameUtils::Move *>(evt.GetClientData());
+		exit(1);
 	}
 	algorithmThread = nullptr;