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 453 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;
   }
   
}
Online 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!
Legends of Yore - The Casual Retro Roguelike
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!
 
Browse for soundtracks for your game!

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 (139 views)
2013-05-17 21:29:12

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

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

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

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

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

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

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

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

UnluckyDevil (238 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.075 seconds with 20 queries.