Java: ObjectOutputStream with Serializable problem

Question

I have these three classes:

Command:

package pack;

public abstract class Command impements java.io.Serializable
{
    public abstract void execute();
}

Client:

package pack;

// imports....

public class Client
{
    Socket socket;

    // Constructor...

    public void sendCommand(Command c)
    {
         try
         {
              new ObjectOuputStream(socket.getOutputStream()).writeObject(c);
         } catch (Exception e) {e.printStackTrace()};
    }
}

MyKeyListener:

This keylistener is added to a component in a JFrame.

public class MyKeyListener implements KeyListener
{

     private Client client;

     public MyKeyListener(Client c)
     { client = c; }


     public void keyTyped(....)......; // This method does nothing

     public void keyPressed(KeyEvent e)
     {
          client.sendCommand(new Command() {
               public void execute()
               {
                    new Robot().keyPress(e.getKeyCode());
               }
          });
     }

     // The same for keyRelease()....
}

The problem is: if I run the code and he wants to send a Command. The streams stops writing because "MyKeyListener is not Serializable" ???!! But I never try to send MyKeyListener

I hope that the typos are not actually in your code.

Svante
– Svante

2009-09-11 16:32:17 +00:00
Commented Sep 11, 2009 at 16:32 — Svante
– Svante, Commented Sep 11, 2009 at 16:32

Michael Borgwardt · Accepted Answer · 2009-09-11 16:17:40Z

8

Nested classes in Java actually don't exist at the byte code level - the compiler fakes them by inserting hidden fields, access methods and constructors. In your case, the anonymous subclass of Command probably has a compiler-generated reference to the MyKeyListener instance in which it was created, even if that reference is not used for anything.

To fix this, use a top-level subclass of Command instead of an anonymous class.

answered Sep 11, 2009 at 16:17

Michael Borgwardt

347k81 gold badges491 silver badges726 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

William Louth · Accepted Answer · 2009-09-11 16:16:12Z

0

Hint: Try typing MyKeyListener.this.toString() in the (anonymous) Command class execute() method.

answered Sep 11, 2009 at 16:16

William Louth

2071 silver badge4 bronze badges

Collectives™ on Stack Overflow

Java: ObjectOutputStream with Serializable problem

2 Answers 2

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related