Changeset da692b9 in network-game for server

Timestamp:

Dec 9, 2012, 8:59:08 PM (12 years ago)

Author:

dportnoy <dmp1488@…>

Branches:

master

Children:

cdb4bec

Parents:

581058c

Message:

Fixed some minor server bugs

File:

• server/server.cpp (modified) (8 diffs)

server/server.cpp

@@ -94 +94 @@
    set_nonblock(sock);
 
-   bool broadcastMessage;
+   bool broadcastResponse;
    while (true) {
 
@@ -104 +104 @@
          cout << "Got a message" << endl;
 
-         broadcastMessage = processMessage(clientMsg, from, vctPlayers, serverMsg);
+         broadcastResponse = processMessage(clientMsg, from, vctPlayers, serverMsg);
 
          cout << "msg: " << serverMsg.buffer << endl;
 
-         if (broadcastMessage)
-         {
+         if (broadcastResponse)
+         {
+            cout << "Should be broadcasting the message" << endl;
+
             vector<Player>::iterator it;
 
@@ -120 +122 @@
          else
          {
+            cout << "Should be sending back the message" << endl;
+
             if ( sendMessage(&serverMsg, sock, &from) < 0 )
                error("sendMessage");
@@ -139 +143 @@
    cout << "MSG contents: " << clientMsg.buffer << endl;
 
+   // maybe we should make a message class and have this be a member
    bool broadcastResponse = false;
 
@@ -186 +191 @@
 
             vctPlayers.push_back(newP);
-            strcpy(serverMsg.buffer, "Login successful. Enjoy chatting with outher players.");
+            strcpy(serverMsg.buffer, "Login successful. Enjoy chatting with other players.");
          }
 
@@ -221 +226 @@
       case MSG_TYPE_CHAT:
       {
+         cout << "Got a chat message" << endl;
+
          Player *p = findPlayerByAddr(vctPlayers, from);
 
@@ -232 +239 @@
 
             stringstream ss;
-            ss << p->name << ": " << clientMsg.buffer << endl;
+            ss << p->name << ": " << clientMsg.buffer;
 
             strcpy(serverMsg.buffer, ss.str().c_str());
@@ -249 +256 @@
          break;
       }
-
-      return broadcastResponse;
-   }
-}
+   }
+
+   return broadcastResponse;
+}
+