Changeset 4da5aa3 in network-game for client/Client/main.cpp

Timestamp:

Nov 26, 2012, 10:18:25 PM (12 years ago)

Author:

dportnoy <dmp1488@…>

Branches:

master

Children:

73f75c1

Parents:

a4db787

Message:

Created a process message function to handle message received from the server

File:

• client/Client/main.cpp (modified) (6 diffs)

client/Client/main.cpp

@@ -36 +36 @@
 void initWinSock();
 void shutdownWinSock();
+void processMessage(NETWORK_MSG &msg, int &state, chat &chatConsole, string &username);
+
 void error(const char *);
 
@@ -207 +209 @@
                      msgTo.type = MSG_TYPE_LOGIN;
                      username = input;
+
+                     sendMessage(&msgTo, sock, &server);
+                     receiveMessage(&msgFrom, sock, &from);
+                     processMessage(msgFrom, state, chatConsole, username);
+                     cout << "state: " << state << endl;
 
                      break;
@@ -222 +229 @@
                         msgTo.type = MSG_TYPE_CHAT;
 
+                     sendMessage(&msgTo, sock, &server);
+                     receiveMessage(&msgFrom, sock, &from);
+                     processMessage(msgFrom, state, chatConsole, username);
+                     cout << "state: " << state << endl;
+
                      break;
                   }
                   case STATE_LOGOUT:
                   {
-                     cout << "Bug: You're logged out, so you shouldn't be receiving any messages." << endl;
+                     chatConsole.addLine("You're logged out, so you can't send any messages to the server.");
+
+                     cout << "You're logged out, so you can't send any messages to the server." << endl;
                   
                      break;
@@ -237 +251 @@
                   }
                }
-
-               sendMessage(&msgTo, sock, &server);
-
-               receiveMessage(&msgFrom, sock, &from);
-               string response = string(msgFrom.buffer);
-
-               // this whole select block should go in a new function.
-               // Figure out how to pass and return params properly
-               switch(state)
-               {
-                  case STATE_START:
-                  {
-                     chatConsole.addLine(string(msgFrom.buffer));
-
-                     if (response.compare("Player has already logged in.") == 0)
-                     {
-                        cout << "User login failed" << endl;
-                        username.clear();
-                     }
-                     else
-                     {
-                        cout << "User login successful" << endl;
-                        state = STATE_LOGIN;
-                     }
-
-                     break;
-                  }
-                  case STATE_LOGIN:
-                  {
-                     chatConsole.addLine(string(msgFrom.buffer));
-
-                     if (response.compare("You have been successfully logged out. You may quit the game.") == 0)
-                     {
-                        state = STATE_LOGOUT;
-                     }
-                     else
-                     {
-                        cout << "Added new line" << endl;
-                     }
-                     
-                     break;
-                  }
-                  case STATE_LOGOUT:
-                  {
-                     cout << "Bug: You're logged out, so you shouldn't be receiving any messages." << endl;
-                  
-                     break;
-                  }
-                  default:
-                  {
-                     cout << "The state has an invalid value: " << state << endl;
-
-                     break;
-                  }
-               }
             }
          }else {
@@ -364 +323 @@
  
    return 0;
+}
+
+// need to make a function like this that works on windows
+void error(const char *msg)
+{
+   perror(msg);
+   shutdownWinSock();
+   exit(1);
 }
 
@@ -391 +358 @@
 }
 
-// need to make a function like this that works on windows
-void error(const char *msg)
+void processMessage(NETWORK_MSG &msg, int &state, chat &chatConsole, string &username)
 {
-   perror(msg);
-   shutdownWinSock();
-   exit(1);
+   string response = string(msg.buffer);
+
+   // this whole select block should go in a new function.
+   // Figure out how to pass and return params properly
+   switch(state)
+   {
+      case STATE_START:
+      {
+         chatConsole.addLine(response);
+
+         if (response.compare("Player has already logged in.") == 0)
+         {
+            cout << "User login failed" << endl;
+            username.clear();
+         }
+         else
+         {
+            cout << "User login successful" << endl;
+            state = STATE_LOGIN;
+         }
+
+         break;
+      }
+      case STATE_LOGIN:
+      {
+         chatConsole.addLine(response);
+
+         if (response.compare("You have successfully logged out. You may quit the game.") == 0)
+         {
+            cout << "Logged out" << endl;
+            state = STATE_LOGOUT;
+         }
+         else
+         {
+            cout << "Added new line" << endl;
+         }
+                     
+         break;
+      }
+      case STATE_LOGOUT:
+      {
+         cout << "Bug: You're logged out, so you shouldn't be receiving any messages." << endl;
+                  
+         break;
+      }
+      default:
+      {
+         cout << "The state has an invalid value: " << state << endl;
+
+         break;
+      }
+   }
 }