Changeset ad5d122 in network-game

Timestamp:

Jan 29, 2013, 7:19:59 PM (12 years ago)

Author:

dportnoy <dmp1488@…>

Branches:

master

Children:

5806dc2

Parents:

80b3f94

Message:

Fixed a bug with Player serialization

Files:

• client/Client/main.cpp (modified) (1 diff)
• common/Player.cpp (modified) (2 diffs)

client/Client/main.cpp

@@ -303 +303 @@
       }
       else if(ev.type == ALLEGRO_EVENT_MOUSE_BUTTON_UP) {
-         mapPlayers[curPlayerId].pos.x = ev.mouse.x;
-         mapPlayers[curPlayerId].pos.y = ev.mouse.y;
-
-         // send the server a MSG_TYPE_PLAYER_MOVE message
-         msgTo.type = MSG_TYPE_PLAYER_MOVE;
-
-         ostringstream oss;
-         oss << ev.mouse.x;
-         oss << ev.mouse.y;
-
-         memcpy(msgTo.buffer, oss.str().c_str(), oss.str().length());
-         sendMessage(&msgTo, sock, &server);
+         if(wndCurrent == wndMain) {
+            msgTo.type = MSG_TYPE_PLAYER_MOVE;
+
+            memcpy(msgTo.buffer, &curPlayerId, 4);
+            memcpy(msgTo.buffer+4, &ev.mouse.x, 4);
+            memcpy(msgTo.buffer+8, &ev.mouse.y, 4);
+
+            sendMessage(&msgTo, sock, &server);
+         }
       }
 

common/Player.cpp

@@ -53 +53 @@
    memcpy(buffer, &this->id, 4);
    strcpy(buffer+4, this->name.c_str());
-   memcpy(buffer+4+this->name.length(), &this->pos.x, 4);
-   memcpy(buffer+8+this->name.length(), &this->pos.y, 4);
+   memcpy(buffer+5+this->name.length(), &this->pos.x, 4);
+   memcpy(buffer+9+this->name.length(), &this->pos.y, 4);
 }
 
@@ -63 +63 @@
    memcpy(&this->id, buffer, 4);
    strcpy(test, buffer+4);
-   memcpy(&this->pos.x, buffer+4+strlen(test), 4);
-   memcpy(&this->pos.y, buffer+8+strlen(test), 4);
+   memcpy(&this->pos.x, buffer+5+strlen(test), 4);
+   memcpy(&this->pos.y, buffer+9+strlen(test), 4);
 
    cout << "id: " << this->id << endl;