Better way of managing Controllers

src/game/GameController.cpp

@@ -19,8 +19,6 @@ public:
 	virtual void ControllerExit()
 	{
 		cc->gameModel->SetUser(cc->loginWindow->GetUser());
-		delete cc->loginWindow;
-		cc->loginWindow = NULL;
 	}
 };
 
@@ -116,9 +114,14 @@ void GameController::DrawPoints(queue<ui::Point*> & pointQueue)
 	}
 }
 
-void GameController::Tick()
+void GameController::Update()
 {
 	//gameModel->GetSimulation()->update_particles();
+	if(loginWindow && loginWindow->HasExited)
+	{
+		delete loginWindow;
+		loginWindow = NULL;
+	}
 }
 
 void GameController::SetPaused(bool pauseState)

src/game/GameController.h

@@ -31,7 +31,7 @@ public:
 	GameView * GetView();
 	void AdjustBrushSize(int direction);
 	void DrawPoints(queue<ui::Point*> & pointQueue);
-	void Tick();
+	void Update();
 	void SetPaused(bool pauseState);
 	void SetPaused();
 	void SetActiveMenu(Menu * menu);

src/game/GameView.cpp

@@ -414,7 +414,7 @@ void GameView::OnTick(float dt)
 	{
 		c->DrawPoints(pointQueue);
 	}
-	c->Tick();
+	c->Update();
 }
 
 void GameView::OnDraw()

src/login/LoginController.cpp

@@ -8,7 +8,9 @@
 #include "LoginController.h"
 #include "client/User.h"
 
-LoginController::LoginController(ControllerCallback * callback) {
+LoginController::LoginController(ControllerCallback * callback):
+	HasExited(false)
+{
 	// TODO Auto-generated constructor stub
 	loginView = new LoginView();
 	loginModel = new LoginModel();
@@ -38,6 +40,7 @@ void LoginController::Exit()
 	}
 	if(callback)
 		callback->ControllerExit();
+	HasExited = true;
 }
 
 LoginController::~LoginController() {

src/login/LoginController.h

@@ -23,6 +23,7 @@ class LoginController {
 	LoginModel * loginModel;
 	ControllerCallback * callback;
 public:
+	bool HasExited;
 	LoginController(ControllerCallback * callback = NULL);
 	void Login(string username, string password);
 	void Exit();