What's wrong in this code?

Question

I'm saving JTable content with this code:

@override
public void editingStopped(ChangeEvent ce) {

    PreparedStatement pstmt = null;
    try {
        int row = getEditingRow();
        int column = getEditingColumn();
        DefaultStyledDocument doc = (DefaultStyledDocument) getCellEditor().getCellEditorValue();

        doc.setDocumentFilter(null);
        ByteArrayOutputStream bos = new ByteArrayOutputStream();
        ObjectOutputStream oos = new ObjectOutputStream(bos);
        oos.writeObject((DefaultStyledDocument) doc);
        oos.flush();

        byte[] data = bos.toByteArray();

        oos.close();
        bos.close();

        String query = "update BOX_ROWS "
                    + "set COLUMN1= ? "
                    + "where BOX_ID=" + ID
                    + " and INDEX=" + row;
        pstmt = ReseachAssistantUI.conn.prepareStatement(query);
        pstmt.setObject(1, data);

        pstmt.executeUpdate();
        doc.setDocumentFilter(new MyDocumentFilter());

    } catch (SQLException ex) {
        Logger.getLogger(MyTable.class.getName()).log(Level.SEVERE, ex.getMessage(), ex);
        JOptionPane.showMessageDialog(null, "MyTable - " + ex.getMessage());
    } catch (IOException ex) {
        Logger.getLogger(MyTable.class.getName()).log(Level.SEVERE, ex.getMessage(), ex);
        JOptionPane.showMessageDialog(null, "MyTable - " + ex.getMessage());
    } finally {
        DBUtil.closePreparedStatement(pstmt, MyTable.class.getName());
    }

    super.editingStopped(ce);
}

and it runs fine on Windows. However when I run my app on Mac OS X the following message appears:

MyTable - com.apple.laf.AquaComboBoxUI

The table pop up editor does have 2 comboboxes on a tool bar, but I can't see what they have to do with this exception. Does anyone know why does it throw this exception? Is there something wrong with the way I serialize the object?

@MarkoTopolnik When Editting is stopped (When I close the table editor). — Igor, Nov 24 '12 at 11:13
@AndrewThompson Thanks for the suggestion, but it would be too long. — Igor, Nov 24 '12 at 11:15
The first time you introduce an exception is when you refer to it as "this exception". It would be expected to precede this reference with the introduction to the actual exception that happens. An exception would consist of its class name and its stacktrace. — Marko Topolnik, Nov 24 '12 at 11:16
@MarkoTopolnik This is all I got. I'm on a PC and can't get it tested again on a mac right away. I set the logger to print to a file the nest time I get a chance to test the app. :) — Igor, Nov 24 '12 at 11:19
-1 for refusing to provide an SSCCE (for the umpteenth time, isn't it?) — kleopatra, Nov 24 '12 at 11:22
@kleopatra this is much shorter and clearer than a SSCCE would be for this particular problem. — Igor, Nov 24 '12 at 11:23
_clearer_ as can be easily seen by the wildly guessing comments ;-) Voted to close, as is, it's not answerable. — kleopatra, Nov 24 '12 at 11:28
@kleopatra The ComboBox itself uses 5 different classes for font editing. How would that be clearer? — Igor, Nov 24 '12 at 13:29
one last time: read the reference given by @AndrewThompson ... **NOW** (if you had you wouldn't mumble _5 different classes for whatever_) — kleopatra, Nov 24 '12 at 13:45

score 4 · Answer 1 · edited May 23 '17 at 11:43

4

It looks like you're trying to persist the contents of a changed cell after editing concludes but before the model is updated; this answer outlines the normal sequence of events. As you have found, the actual editor component may vary by platform. Instead of overriding JTable#editingStopped(), override TableModel#setValueAt(), where you know the row, column and type of the updated cell. Examples may be found here and here. This related example illustrates using a JComboBox as a CellEditor.

edited May 23 '17 at 11:43

Community

1
1

answered Nov 24 '12 at 13:09

trashgod

203,806
29
246
1,045

Thanks, I'll give this a try a set as the answer if it works. – Igor Nov 24 '12 at 15:45
I finnally got a chance to test it and unfotunately it doesn't work. Here are the first lines of the stacktrace: `java.io.NotSerializableException: com.apple.laf.AquaComboBoxUI at java.io.ObjectOutputStream.writeObject0(ObjectOutputStream.java:1180)` ... I don't get why the combobox is serialized. I just need the document serialized. – Igor Nov 28 '12 at 09:31
Serialize? Why not just write the model's data? – trashgod Nov 28 '12 at 12:12
Because I want to save the text styles too. But, how is the combobox serialized here instead of just the textpane document? – Igor Nov 28 '12 at 12:59
All concrete `javax.swing.text.Document` classes are `Serializable`. – trashgod Nov 28 '12 at 14:50
Why the exception then? Seems to throw it on some versions of _MAC OS X_ only. – Igor Nov 28 '12 at 15:53
Edit your question to include an [sscce](http://sscce.org/) that reproduces the problem on your machine, and I'll try it on mine. – trashgod Nov 28 '12 at 16:12
I tried to create a SSCCE, but it has at least 500 lines. That doesn't seem like a Short SCCE at all and I'm not sure it OK to post 500 lines here. – Igor Nov 29 '12 at 15:59

What's wrong in this code?

1 Answers1

Linked