Java-Gaming.org
Play Revenge of the Titans! The situation is critical. We need fancy commanders to defend Earth, the moon, Mars!
Featured games (78)
games approved by the League of Dukes
Games in Showcase (408)
games submitted by our members
Games in WIP (293)
games currently in development
News: Read the Java Gaming Resources, or peek at the official Java tutorials
 
    Home     Help   Search   Login   Register   
Pages: [1]
  ignore  |  Print  
  Problem with object modifications not sending over network  (Read 452 times)
0 Members and 1 Guest are viewing this topic.
Offline bgilb

Senior Newbie




Java games rock!


« Posted 2013-01-20 01:11:04 »

The variables are not the same for some reason.

Here is where I receive the object:
1  
2  
3  
4  
5  
6  
7  
8  
9  
10  
11  
12  
13  
14  
         if(omsg instanceof ServerPlayerStateMessage){
            ServerPlayerStateMessage msg = (ServerPlayerStateMessage) omsg;
            for(int i=0; i<msg.players.length; i++){
               PlayerModel newModel = msg.players[i];

               PlayerModel oldModel = playerManager.getByID(newModel.clientID);
               
               
               if(oldModel != null){
                  oldModel.setTilePos(newModel.tileX,newModel.tileY);
               }

            }
         }


Here is where I send:
1  
2  
3  
4  
5  
6  
7  
            if(msg instanceof ClientMoveMessage){
               System.out.println("Client move msg");
               
               ClientMoveMessage msg2 = (ClientMoveMessage) msg;
               
               cThread1.getPlayerModel().setTilePos(msg2.tileX, msg2.tileY);
            }

1  
2  
3  
4  
5  
6  
7  
8  
9  
10  
11  
12  
13  
14  
15  
16  
17  
18  
   public void sendPlayerStates(){
      ArrayList<PlayerModel> playerModels = new ArrayList<PlayerModel>();
     
      for(int i=0; i<cThreads.size(); i++){
         ClientThread t = cThreads.get(i);
         playerModels.add(t.getPlayerModel());
      }
     
         
      ServerPlayerStateMessage msg = new ServerPlayerStateMessage();
     
      msg.players = new PlayerModel[playerModels.size()];
     
      msg.players = playerModels.toArray(msg.players);
     
      broadcast(msg);
         
   }


Here is the actual objectoutputstream writing:

1  
2  
3  
4  
5  
6  
7  
8  
9  
10  
11  
12  
13  
14  
15  
   public void sendAll() {
      if(socket.isConnected()){
         Object msg = null;
         while((msg = outQueue.poll()) != null){
            try {
               out.writeUnshared(msg);
               out.flush();
               out.reset();
            } catch (IOException e) {
               // TODO Auto-generated catch block
              e.printStackTrace();
            }
         }
      }
   }


1  
2  
3  
4  
public class ServerPlayerStateMessage implements Serializable {
   
   public PlayerModel[] players;
}


1  
2  
3  
4  
5  
6  
7  
8  
9  
10  
11  
12  
13  
public class PlayerModel implements Serializable {
   
   //Owner of this player model
  public int clientID;
   public int tileX;
   public int tileY;
   
   public void setTilePos(int tileX, int tileY){
      this.tileX = tileX;
      this.tileY = tileY;
   }
   
}
Offline HeroesGraveDev

JGO Wizard


Medals: 65
Projects: 8


Muahahahahahaha...


« Reply #1 - Posted 2013-01-20 02:10:54 »

I bet it's a protocol mistake.

Offline ClickerMonkey

Senior Member


Medals: 11


Game Engineer


« Reply #2 - Posted 2013-01-20 05:29:28 »

Maybe it's a logical problem and oldModel is actually null, you should add asserts or debugging print statements.

Games published by our own members! Check 'em out!
Play the free demo of Revenge of the Titans!
Offline bgilb

Senior Newbie




Java games rock!


« Reply #3 - Posted 2013-01-20 18:15:22 »


okay as I was writing this message I found the error. My objectoutputstream thread for writing had 2 class files because I moved it at some point. I made the modification to reset the stream to the wrong file! Darn
Pages: [1]
  ignore  |  Print  
 
 

Play Revenge of the Titans! The situation is critical. We need fancy commanders to defend Earth, the moon, Mars!
 
Play Revenge of the Titans! The situation is critical. We need fancy commanders to defend Earth, the moon, Mars and Titan!

Add your game by posting it in the WIP section,
or publish it in Showcase.

The first screenshot will be displayed as a thumbnail.

The invasion has landed! On Mars! And you're there to beat 'em!
cubemaster21 (131 views)
2013-05-17 21:29:12

alaslipknot (140 views)
2013-05-16 21:24:48

gouessej (170 views)
2013-05-16 00:53:38

gouessej (164 views)
2013-05-16 00:17:58

theagentd (174 views)
2013-05-15 15:01:13

theagentd (159 views)
2013-05-15 15:00:54

StreetDoggy (203 views)
2013-05-14 15:56:26

kutucuk (227 views)
2013-05-12 17:10:36

kutucuk (227 views)
2013-05-12 15:36:09

UnluckyDevil (230 views)
2013-05-12 05:09:57
Complex number cookbook
by Roquen
2013-04-24 12:47:31

2D Dynamic Lighting
by Oskuro
2013-04-17 16:46:12

2D Dynamic Lighting
by Oskuro
2013-04-17 16:45:57

2D Dynamic Lighting
by Oskuro
2013-04-17 16:23:20

Noise (bandpassed white)
by Roquen
2013-04-05 17:36:01

Noise (bandpassed white)
by Roquen
2013-04-03 16:17:38

Java Data structures
by Roquen
2013-03-29 13:21:12

Topic Request
by kutucuk
2013-03-22 21:42:01
Powered by MySQL Powered by PHP Powered by SMF 1.1.18 | SMF © 2013, Simple Machines | Managed by Enhanced Four Valid XHTML 1.0! Valid CSS!
Page created in 0.153 seconds with 21 queries.