Mike Lipay

I'm sorry, mistyped, I did correct the code:


Date date1 = new GregorianCalendar(2009,0,26).getTime(); Date date2 = new GregorianCalendar(2008,2,20).getTime(); private DefaultTableModel model = new DefaultTableModel(columnNames, serviceTable.length) { public Class<?> getColumnClass(int c) { if (c == clmnMileage || c == clmnInvoice) return Integer.class; if (c == clmnSvcDate) return Date.class; if (c == clmnCost) return Double.class; return String.class; }

I was trying to answer questions that people were asking.

Rob Camick wrote:Well I give up, since you are obviously paying no attention to what I've said in this posting or others.

Why did you last reply include code for a Calendar object? This posting is about using the Date class not the Calendar class. The SSCCE you posted demonstrated your problem and you only need to change one line to fix it. Both Rob's have told you how to fix it.

Yes, I am paying attention, to several different conversations here. I changed GregorianCalendar to Date, after which I started receiving the deprecation error. Then I was told to see why, the API said to use Calendar because Date(int,int,int) was no longer supported. So I started explaining the problems I had using the Calendar class, i.e., the .set(int,int,int) method is returning errors, which is what sent me to GregorianCalendar to begin with.

I am paying attention to everyone here, and trying to answer and test what everyone suggests. And believe me, I am more confused than ever with trying to follow everything at the same time.

Sorry, but a lot of this just doesn't make sense. I see nowhere where I would get the idea that I have to use a mixture of Date and Calendar. From the Calendar API I would assume that this code should work:

Calendar date1 = new Calendar(); date1.set(2009,0,26); Calendar date1 = new Calendar(); date1.set(2009,0,26);

Calendar() Constructs a Calendar with the default time zone and locale.
set(int year, int month, int date) Sets the values for the fields year, month, and date.

It doesn't.


Service.java:113: <identifier> expected date1.set(2009,0,26); ^

Paul Clapham wrote:Okay. So what does that error message actually tell you about the Date class? And what does the API documentation recommend you should do instead?

It tells me that Date is on the way out, what I could find tells me to use Calendar instead. Tried that:

Calendar date1 = new Calendar.getInstance(); date1.set(2009,0,26);

Tossed this error during compile:
C:\Documents and Settings\pmll42\Desktop\Java\Test>java Service.java Service.java:113: <identifier> expected date1.set(2009,0,26); ^ Service.java:113: illegal start of type date1.set(2009,0,26); ^ Service.java:113: illegal start of type date1.set(2009,0,26); ^ Service.java:113: illegal start of type date1.set(2009,0,26); ^ 4 errors C:\Documents and Settings\pmll42\Desktop\Java\Test>

According to the examples on java.sun.com what I coded was correct. Looking elsewhere for solutions is what led me to GregorianCalendar.

Paul Clapham wrote:The Date class isn't deprecated, so I expect your compiler error didn't really say that. Posting the actual error message also helps, whereas posting crude paraphrases of the error message is just distracting.

Here is the message:
C:\Documents and Settings\pmll42\Desktop\Java\Test>javac Service.java Note: Service.java uses or overrides a deprecated API. Note: Recompile with -Xlint:deprecation for details. C:\Documents and Settings\pmll42\Desktop\Java\Test>javac -Xlint:deprecation Serv ice.java Service.java:110: warning: [deprecation] Date(int,int,int) in java.util.Date has been deprecated Date date1 = new Date(2009,0,26); ^ Service.java:111: warning: [deprecation] Date(int,int,int) in java.util.Date has been deprecated Date date2 = new Date(2008,2,20); ^ 2 warnings C:\Documents and Settings\pmll42\Desktop\Java\Test>

I tried using the Date class first, but received a warning from the compiler about Date being deprecated. Is there a way around that other than using GregorianCalendar?

Ok, I didn't want to post the code if editing dates wasn't possible.

Ever other field in the table edits with no problem, and I can't see what I missed for the date.

import java.awt.*; import java.io.*; import java.text.*; import java.util.*; import javax.swing.*; import javax.swing.event.ListSelectionEvent; import javax.swing.event.ListSelectionListener; import javax.swing.table.*; /** * Service Form * @author Mike Lipay * @version 1.0 */ public class Service { public static void main (String[] args) { JFrame MainFrame = new JFrame(); MainFrame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE); MainFrame.pack(); MainFrame.setLocationRelativeTo(null); MainFrame.setVisible(true); Service.Service(MainFrame); } public static void Service (JFrame frame) { displayMenu(frame); } /** * Display main window to the user */ public static void displayMenu(JFrame aFrame) { final JFrame frame = aFrame; EventQueue.invokeLater (new Runnable() { public void run() { ServiceDialog dialog = new ServiceDialog(frame); dialog.setDefaultCloseOperation (JFrame.DISPOSE_ON_CLOSE); dialog.setVisible(true); } }); } } /** * Create the window frame */ class ServiceDialog extends JDialog implements Serializable { /** * Create the main frame which enables the selection of or addition of * vehicles to the Vehicle Identification File (VIF.obj) */ public ServiceDialog(JFrame owner) { super (owner, "Vehicle Specifications", true); // Main window setup setLocationByPlatform(true); setSize(windowWidth, windowHeight); setTitle("Vehicle Service"); GridBagLayout layout = new GridBagLayout(); setLayout(layout); // Add Service Table to panel table = new JTable(model); PopulateTable(); add (new JScrollPane(table), gbcTablField); table.setAutoCreateRowSorter(true); table.setFillsViewportHeight(true); table.setSelectionMode(ListSelectionModel.SINGLE_SELECTION); table.getSelectionModel().addListSelectionListener(new RowListener()); table.getSelectionModel().addListSelectionListener(new ColumnListener()); TableColumn tcol = table.getColumnModel().getColumn(clmnMileage); tcol.setCellRenderer(new CustomTableCellRenderer()); tcol = table.getColumnModel().getColumn(clmnInvoice); tcol.setCellRenderer(new CustomTableCellRenderer()); tcol = table.getColumnModel().getColumn(clmnSvcDate); tcol.setCellRenderer(new CustomTableCellRenderer()); tcol = table.getColumnModel().getColumn(clmnCost); tcol.setCellRenderer(new CustomTableCellRenderer()); } // Table values final String[] columnNames = { "Mileage", "Service Date", "Shop", "Invoice Nbr", "Labor", "Parts", "Cost", }; Date date1 = new GregorianCalendar(2009,0,26).getTime(); Date date2 = new GregorianCalendar(2008,2,20).getTime(); private Object[][] serviceTable = { { 97300, date1, "McElhinney's Service", 4798, "Check alternator and oil pressure", "Battery, Alternator, Serpentine Belt, Oil Pressure Sensor", 707.68}, { 0, date2, "McElhinney's Service", 11112, "Replace engine", "Motor, cables, fluids", 4534.61} }; private JTable table; private void PopulateTable() { for ( int r = 0; r < serviceTable.length; r++ ) for ( int c = 0; c < serviceTable[r].length; c++ ) model.setValueAt(serviceTable[r][c], r, c); } public class CustomTableCellRenderer extends DefaultTableCellRenderer { public Component getTableCellRendererComponent (JTable table, Object obj, boolean isSelected, boolean hasFocus, int row, int col) { setHorizontalAlignment(SwingConstants.RIGHT); if (col == clmnMileage) return super.getTableCellRendererComponent (table, mf.format(obj), isSelected, hasFocus, row, col); if (col == clmnSvcDate) return super.getTableCellRendererComponent (table, df.format(obj), isSelected, hasFocus, row, col); if (col == clmnSvcDate) System.out.println(obj); if (col == clmnCost) return super.getTableCellRendererComponent (table, cf.format(obj), isSelected, hasFocus, row, col); return super.getTableCellRendererComponent (table, obj, isSelected, hasFocus, row, col); } } private DefaultTableModel model = new DefaultTableModel(columnNames, serviceTable.length) { public Class<?> getColumnClass(int c) { if (c == clmnMileage || c == clmnInvoice) return Integer.class; if (c == clmnSvcDate) return GregorianCalendar.class; if (c == clmnCost) return Double.class; return String.class; } }; private class RowListener implements ListSelectionListener { public void valueChanged(ListSelectionEvent e) { if (!e.getValueIsAdjusting()) { int[] selected = table.getSelectedRows(); selRow = selected[0]; } } } private class ColumnListener implements ListSelectionListener { public void valueChanged(ListSelectionEvent e) { if (!e.getValueIsAdjusting()) { int[] selected = table.getSelectedColumns(); selCol = selected[clmnMileage]; } } } // Field formatting final NumberFormat cf = NumberFormat.getCurrencyInstance(); final DateFormat df = DateFormat.getDateInstance(DateFormat.MEDIUM); final DecimalFormat mf = new DecimalFormat ("000,000"); // Main panel fields final int windowWidth = 600; final int windowHeight = 350; private JPanel servicePanel = new JPanel(); private int selRow; private int selCol; final int clmnMileage = 0; final int clmnSvcDate = 1; final int clmnShop = 2; final int clmnInvoice = 3; final int clmnLabor = 4; final int clmnParts = 5; final int clmnCost = 6; // Field locations in panel private GBC gbcTablField = new GBC(0,5,5,1).setFill(GBC.BOTH).setWeight(100, 100); }

Paul, thanks, using the super. worked.

Is there a way to allow a user to edit a date in a JTable? All of the columns I have in my table are open for editing, but when I click on the date column I cannot open it for editing. Is there something special that needs to be done for dates, or are dates not editable?

That's like another issue I'm having, and I've been real hesitant to post it. I searched for answers on how to capture a change in a cell, and all of the responses I've seen mention using setValueAt, no final code is posted and, in most cases, the thread just ends. Well, I tried that:

private DefaultTableModel model = new DefaultTableModel(columnNames, serviceTable.length) { public void setValueAt(Object val, int row, int col) { System.out.println(val.toString()); } };

And, yes, I can capture the changes, but when the table initially displays all of its' contents are blank. Right number of rows and columns, but no data. Further searching showed suggesting adding

fireTableCellUpdated();

Well, that doesn't compile. Changing it to
fireTableCellUpdated(row, col);
compiles, but nothing is different. I figure others got it to work, but I'll be d--ned if I can figure out from the docs I've looked at on how it's done. Guess I'm just dumber than the others here, which makes me hesitant to even ask questions.

I don't know, before the code was:

public Class<?> getColumnClass(int c) { if (c == clmnCost) return Integer.class; return serviceTable[0][c].getClass(); }

When I change it to DefaultTableModel from AbstractTableModel it didn't work. The second change was small, but (at the time of this post) I thought perhaps there was something different between Default and Abastract. The new code worked, maybe I made other changes that messed things up.

public Class<?> getColumnClass(int c) { if (c == 3) return Integer.class; if (c == 6) return Integer.class; return String.class; }

Rob, you did give me an answer when I was working with an AbstractTableModel, but when I switched to the DefaultTableModel it no longer worked. Maybe it was the way I implemented your suggestion, not sure. That is why I asked again for the DTM, sorry if it was inappropriate.

Found my solution:

private DefaultTableModel model = new DefaultTableModel(columnNames, (serviceTable.length + 1)) { public Class<?> getColumnClass(int c) { if (c == 3) return Integer.class; if (c == 6) return Integer.class; return String.class; } };

Using the DefaultTableModel, is there a way to align the contents in a cell? I have the following data (dollar amount, column 6) that I would prefer be right-aligned in the cell.


// Service Table Model private void PopulateTable() { for ( int r = 0; r < serviceTable.length; r++ ) { for ( int c = 0; c < serviceTable[r].length; c++ ) { if (c == 0) model.setValueAt(mf.format(serviceTable[r][c]), r, c); else if (c == 6) model.setValueAt(cf.format(serviceTable[r][c]), r, c); else model.setValueAt(serviceTable[r][c], r, c); } } } // Field formatting final DecimalFormat mf = new DecimalFormat ("000,000"); final NumberFormat cf = NumberFormat.getCurrencyInstance(); // Main panel fields private JPanel servicePanel = new JPanel(); // Table values private Object[][] serviceTable = { { 97300, "1/26/2009", "McElhinney's Service", 4798, "Check alternator and oil pressure", "Battery, Alternator, Serpentine Belt, Oil Pressure Sensor", 707.68}, { 0, "3/20/2008", "McElhinney's Service", 11112, "Replace engine", "Motor, cables, fluids", 4534.61} }; private JTable table; private DefaultTableModel model = new DefaultTableModel(columnNames, serviceTable.length);

Seetharaman Venkatasamy wrote:better to say *array of arrays* insteadof multidimensional . your for loop declaration is wrong .

Sorry, multi-dimensional is how I've always seen nested arrays referred to. I'm guessing "array of arrays" is a java term. I'll try to remember this in the future.