Changeset 458 in tmcsimulator for trunk/src/tmcsim/client/cadclientgui/screens/UnitStatus.java

Timestamp:

07/19/2019 08:14:58 AM (7 years ago)

Author:

jdalbey

Message:

UnitStatus?.java in CADClient. Put refreshTable method body in Swing Event Dispatch Thread to try to solve random null pointer errors. See #87.

File:

• trunk/src/tmcsim/client/cadclientgui/screens/UnitStatus.java (modified) (2 diffs)

trunk/src/tmcsim/client/cadclientgui/screens/UnitStatus.java

@@ -214 +214 @@
 
         unitStatusTable.setModel(new DefaultTableModel());
+        
         ((DefaultTableModel) unitStatusTable.getModel())
                 .setColumnIdentifiers(TableHeaders.UNIT_STATUS_HEADERS);
@@ -1057 +1058 @@
         if (unitStatusTable.getTableHeader().getResizingColumn() == null)
         {//only update info if resize not in progress
-            try
-            {
-                int index = unitStatusTable.getSelectedRow();
-                List<? extends SortKey> keys = unitStatusTable.getRowSorter().getSortKeys();
-                int[] columnWidths = new int[20];
-                for (int i = 0; i < unitStatusTable.getColumnCount(); i++)
-                {
-                    columnWidths[i] = unitStatusTable.getColumnModel().getColumn(i).getWidth();
-                }
-
-                unitStatusTable.setModel(ScreenManager.theCoordinator.getCadDataTable(CADDataEnums.TABLE.UNIT_STATUS));
-
-                for (int i = 0; i < unitStatusTable.getColumnCount(); i++)
-                {
-                    unitStatusTable.getColumnModel().getColumn(i).setPreferredWidth(columnWidths[i]);
-                }
-                unitStatusTable.getRowSorter().setSortKeys(keys);
-                unitStatusTable.getSelectionModel().setSelectionInterval(index, index);
-            } catch (RemoteException ex)
-            {
-                System.out.println("RemoteException in refreshTable method of UnitStatus.java");
-                System.out.println(ex.getMessage());
-            }
+               
+                // Perhaps this needs to be done in Swing's Event Dispatch Thread.
+                // See https://stackoverflow.com/questions/1732503/java-lang-arrayindexoutofboundsexception-0-0-attempting-to-populate-jtable
+                SwingUtilities.invokeLater(new Runnable(){public void run(){
+                    int index = unitStatusTable.getSelectedRow();
+                    List<? extends SortKey> keys = unitStatusTable.getRowSorter().getSortKeys();
+                    // Save column widths prior to updating table
+                    int[] columnWidths = new int[20];
+                    for (int i = 0; i < unitStatusTable.getColumnCount(); i++)
+                    {
+                        columnWidths[i] = unitStatusTable.getColumnModel().getColumn(i).getWidth();
+                    }
+                    //Update the model here
+                    try
+                    {
+                        // Update the table with current Coordinator data
+                        unitStatusTable.setModel(ScreenManager.theCoordinator.getCadDataTable(CADDataEnums.TABLE.UNIT_STATUS));
+                    } catch (RemoteException ex)
+                    {
+                        System.out.println("RemoteException in refreshTable method of UnitStatus.java");
+                        System.out.println(ex.getMessage());
+                    }
+                    // Set preferred column widths
+                    for (int i = 0; i < unitStatusTable.getColumnCount(); i++)
+                    {
+                        unitStatusTable.getColumnModel().getColumn(i).setPreferredWidth(columnWidths[i]);
+                    }
+                    unitStatusTable.getRowSorter().setSortKeys(keys);
+                    unitStatusTable.getSelectionModel().setSelectionInterval(index, index);
+                }});
+
             revalidate();
             repaint();