Se corrige bug que hacia que se cuelque la aplicacion cuando no se estaba conectado

[z.facultad/75.42/plaqui.git] / Server / src / controlclient.cpp

diff --git a/Server/src/controlclient.cpp b/Server/src/controlclient.cpp
index afdd073f3b61bac933e484b4e8018d60e30d76e5..f33fff5f3197989621377d30f6cc3d8d20006494 100644 (file)
--- a/Server/src/controlclient.cpp
+++ b/Server/src/controlclient.cpp
@@ -44,21 +44,27 @@ ControlClient::~ControlClient(void) {
 }
 
 ControlClient::ControlClient(string host, int port):
 }
 
 ControlClient::ControlClient(string host, int port):

-               Connection(sockbuf::sock_stream) {
+               Connection(sockbuf::sock_stream), host(host), port(port) {

 #ifdef DEBUG
 #ifdef DEBUG

-       cerr << __FILE__ << ": host" << host
+       cerr << __FILE__ << ": host = " << host

                << " | port = " << port << endl;
 #endif // DEBUG
                << " | port = " << port << endl;
 #endif // DEBUG

-       socket->connect(host.c_str(), port);
-       if (!socket->is_open()) {
-               throw ios::failure("Can't connect!");
-       }

 }
 
 void ControlClient::real_run(void) {
 #ifdef DEBUG
        cerr << __FILE__ << ": real_run." << endl;
 #endif // DEBUG
 }
 
 void ControlClient::real_run(void) {
 #ifdef DEBUG
        cerr << __FILE__ << ": real_run." << endl;
 #endif // DEBUG

+       try {
+               socket->connect(host.c_str(), port);
+       } catch (const sockerr& e) {
+               // Poner una señal de error específica?
+               error_received(1);
+               finish();
+               return;
+       }
+       // TODO sacar a la mierda?
+       connected();

        while (!stop) {
                HTTPResponse response;
                try {
        while (!stop) {
                HTTPResponse response;
                try {


@@ -74,18 +80,31 @@ void ControlClient::real_run(void) {
                        cerr << __FILE__ << " : real_run() ERROR nro: " << e << endl;
 #endif // DEBUG
                        // TODO - pasar como parametro codigo de error o algo.
                        cerr << __FILE__ << " : real_run() ERROR nro: " << e << endl;
 #endif // DEBUG
                        // TODO - pasar como parametro codigo de error o algo.

-                       error_received();
+                       error_received(e);

                        continue;
                }
                switch (response.status_code) {
                        case HTTPMessage::OK:
                                ok_received();
                        continue;
                }
                switch (response.status_code) {
                        case HTTPMessage::OK:
                                ok_received();

+                               break;

                        default:
                        default:

-                               error_received();
+                               error_received(response.status_code);
+                               break;

                }
        }
 }
 
                }
        }
 }
 

+void ControlClient::send(const Command& command) {
+       socket << command << flush;
+#ifdef DEBUG
+       cerr << __FILE__ << ": send() Enviado!" << endl;
+#endif // DEBUG
+}
+
+ControlClient::SignalConnected& ControlClient::signal_connected(void) {
+       return connected;
+}
+

 ControlClient::SignalOKReceived& ControlClient::signal_ok_received(void) {
        return ok_received;
 }
 ControlClient::SignalOKReceived& ControlClient::signal_ok_received(void) {
        return ok_received;
 }


@@ -94,13 +113,6 @@ ControlClient::SignalErrorReceived& ControlClient::signal_error_received(void) {
        return error_received;
 }
 
        return error_received;
 }
 

-void ControlClient::send(const Command& command) {
-       socket << command << flush;
-#ifdef DEBUG
-       cerr << __FILE__ << ": send() Enviado!" << endl;
-#endif // DEBUG
-}
-

 } // namespace Server
 
 } // namespace PlaQui
 } // namespace Server
 
 } // namespace PlaQui