changing values in two columns in a table

[oll3i]

i want to change the values in two columns
one colum is a combobox and the secons column is editable too
i want to get the value of that second column and the value of combobox
and sent that to another application
i have setValueAt(Object value,int row, int col) but it works only for combobox
when i enter some data in the second editable column the value i entered disappears

rows are a list data = new ArrayList<OrderClientProducent>();

thank you

 
import javax.swing.DefaultCellEditor;
import javax.swing.JComboBox;
import javax.swing.JFrame;
import javax.swing.JPanel;
import javax.swing.JScrollPane;
import javax.swing.JTable;
import javax.swing.event.TableModelEvent;
import javax.swing.event.TableModelListener;
import javax.swing.table.AbstractTableModel;
import javax.swing.table.DefaultTableCellRenderer;
import javax.swing.table.TableCellRenderer;
import javax.swing.table.TableColumn;
 
import com.db4o.Db4o;
import com.db4o.ObjectContainer;
import com.db4o.ObjectSet;
 
import javax.naming.*;
import javax.jms.*;
 
import java.awt.BorderLayout;
import java.awt.Component;
import java.awt.Dimension;
import java.awt.GridLayout;
 
import java.util.ArrayList;
import java.util.List;
/** 
 * TableRenderDemo is just like TableDemo, except that it
 * explicitly initializes column sizes and it uses a combo box
 * as an editor for the Sport column.
 */
public class TableClientProducer extends JPanel implements TableModelListener{
    private boolean DEBUG = false;
    private JPanel thePanel;
    List<OrderClientProducent> data;
    private MyTableModel model;
    JComboBox comboBox;
    JTable table;
 
 
 
 
    Context context = null;
    ConnectionFactory factory = null;
    Connection connection = null;
    String factoryName = "ConnectionFactory";
    String destName = "queue1";
    Destination dest = null;
 
    Session session = null;
    MessageProducer sender = null;
    String text = "Message ";
 
 
 
 
    public TableClientProducer() {
        data = new ArrayList<OrderClientProducent>();
        //super(new GridLayout(1,0));
        thePanel = new JPanel(new BorderLayout());
        model = new MyTableModel();
 
 
        table = new JTable(new MyTableModel());
        table.setPreferredScrollableViewportSize(new Dimension(800, 70));
        table.setFillsViewportHeight(true);
        table.getModel().addTableModelListener(this);
        //Create the scroll pane and add the table to it.
        JScrollPane scrollPane = new JScrollPane(table);
 
        //Set up column sizes.
        ////initColumnSizes(table);
 
        //Fiddle with the Sport column's cell editors/renderers.
        setUpStatusColumn(table, table.getColumnModel().getColumn(6));
 
        //Add the scroll pane to this panel.
        thePanel.add(scrollPane, BorderLayout.CENTER);
    }
    public JPanel getPanel() {
        return thePanel;
    }
 
    public void addData(OrderClientProducent ocp){
        data.add(ocp);
        int size = data.size();     
        model.fireTableRowsInserted(size, size);
    }
    public void tableChanged(TableModelEvent e) {
        int row = e.getFirstRow();
        int column = e.getColumn();
        MyTableModel model = (MyTableModel)e.getSource();
        String columnName = model.getColumnName(column);
        Object data = model.getValueAt(row, column);
 
        // Do something with the data...
 
 
 
 
 
    }
    public void setComoboxSelectedIndex(String value) {
        if(value.equals("Nie potwierdzono"))  
             comboBox.setSelectedIndex(0); 
        else if (value.equals("Potwierdzono"))
            comboBox.setSelectedIndex(1); 
        else if (value.equals("Dostarczono"))
            comboBox.setSelectedIndex(2); 
        else if (value.equals("Nie dostarczono"))
            comboBox.setSelectedIndex(3);
        else if (value.equals("Zamkniete"))
            comboBox.setSelectedIndex(4);
        else comboBox.setSelectedIndex(5);
   }
    public int GetIndexRowSelected(){return table.getSelectedRow();} 
    /*
     * This method picks good column sizes.
     * If all column heads are wider than the column's cells'
     * contents, then you can just use column.sizeWidthToFit().
     */
 
    public void clearRows(){
        System.out.println(data.size());
        //theTableModel.fireTableRowsDeleted(0,theRows.size()-1);
        //for(int i=0;i<theRows.size();i++){
            ///theRows.remove(i);}
        int numRows = data.size()-1;
        for (int i=numRows;i>=0;i--) {
            data.remove(i);
            model.fireTableRowsDeleted(i,i);
        }    
     }
    public void setUpStatusColumn(JTable table,
                                 TableColumn statusColumn) {
        //Set up the editor for the sport cells.
        comboBox = new JComboBox();
        comboBox.addItem("Nie potwierdzono");
        comboBox.addItem("Potwierdzono");
        comboBox.addItem("Dostarczono");
        comboBox.addItem("Nie dostarczono");
        comboBox.addItem("Zamkniete");
        comboBox.addItem("Anulowane");
 
 
 
        statusColumn.setCellEditor(new DefaultCellEditor(comboBox));
 
        //Set up tool tips for the sport cells.
        DefaultTableCellRenderer renderer =
                new DefaultTableCellRenderer();
        renderer.setToolTipText("Kliknij aby ukazal sie combo box");
        statusColumn.setCellRenderer(renderer);
    }
 
 
    public int getID(int row){
        OrderClientProducent orderclientproducer = (OrderClientProducent) data.get(row);
        return orderclientproducer.getID();}
 
 
      public String getStatus(int row){
          OrderClientProducent orderclientproducer = (OrderClientProducent) data.get(row);
        return orderclientproducer.getStatus();}
 
      public String getIlosc(int row){
          OrderClientProducent orderclientproducer = (OrderClientProducent) data.get(row);
          return orderclientproducer.getQuantity();
      }
 
      public int getNumerZamowienia(int row){
          OrderClientProducent orderclientproducer = (OrderClientProducent) data.get(row);
            return orderclientproducer.getOrderNumber();  
 
      }
 
      public String getDateSent(int row){
          OrderClientProducent orderclientproducer = (OrderClientProducent) data.get(row);
          return orderclientproducer.getDateSent(); 
      }
     public String getDateDeadline(int row)    {
 
         OrderClientProducent orderclientproducer = (OrderClientProducent) data.get(row);
 
         return orderclientproducer.getDateReceived();
     }    
 
 
class MyTableModel extends AbstractTableModel {
        String[] columnNames = {"ID","Nazwa klienta","Ilosc",
                "Numer Zamowienia","Data orzymania zamowienia","Termin dostawy","Status"};
 
 
 
        public int getColumnCount() {
            return columnNames.length;
        }
 
        public int getRowCount() {
            return data.size();
        }
 
        public String getColumnName(int col) {
            return columnNames[col];
        }
 
        public Object getValueAt(int row, int col) {
            OrderClientProducent orderclientproducer = data.get(row);
            if(col==0)return orderclientproducer.getID();
            else if (col==1) return orderclientproducer.getClientName();
            else if (col==2) return orderclientproducer.getQuantity();
            else if(col==3) return orderclientproducer.getOrderNumber();
            else if(col==4) return orderclientproducer.getDateSent();
            else if(col==5){  
 
                return orderclientproducer.getDateReceived();
 
 
 
            }
            else return orderclientproducer.getStatus();
        }
 
 
        /*
         * JTable uses this method to determine the default renderer/
         * editor for each cell.  If we didn't implement this method,
         * then the last column would contain text ("true"/"false"),
         * rather than a check box.
         */
        public Class getColumnClass(int c) {
            return getValueAt(0, c).getClass();
        }
 
        /*
         * Don't need to implement this method unless your table's
         * editable.
         */
        public boolean isCellEditable(int row, int col) {
 
            if (col == 5 || col==6 ) {
                return true;
            } else {
                return false;
            }
        }
        public void setValueAt(Object value,int row, int col) {
 
            OrderClientProducent orderclientproducer = data.get(row);
            int id=orderclientproducer.getID();
            int n=orderclientproducer.getOrderNumber();
            String q=orderclientproducer.getQuantity();
            String d_s=orderclientproducer.getDateSent();
 
            String d_r=getDateDeadline(row);
            String s=orderclientproducer.getStatus();
            ObjectContainer db = Db4o.openFile("ordersProducer.yap"); 
            try{
                ObjectSet result = db.get(new OrderClientProducent(id,"Client",q,n,d_s,d_r,s));    
                OrderClientProducent ocp = (OrderClientProducent)result.next();
                ocp.setStatus(value);
                db.set(ocp);
            }
            finally {db.close();}
            orderclientproducer.setStatus(value);       
            fireTableCellUpdated(row, col);  
 
 
 
 
        try {
              // create the JNDI initial context.
              context = new InitialContext();
 
              // look up the ConnectionFactory
              factory = (ConnectionFactory) context.lookup(factoryName);
 
              // look up the Destination
              dest = (Destination) context.lookup(destName);
 
              // create the connection
              connection = factory.createConnection();
 
              // create the session
              session = connection.createSession(
                  false, Session.AUTO_ACKNOWLEDGE);
 
              // create the sender
              sender = session.createProducer(dest);
 
              // start the connection, to enable message sends
              connection.start();
 
  System.out.println("Sender sending to client!");
 
  TextMessage message = session.createTextMessage();
  message.setText(id+":"+q+":"+n+":"+d_r);
  sender.send(message);
  System.out.println("Sent: " + message.getText());
 
 
 
 
 
          } catch (JMSException exception) {
              exception.printStackTrace();
          } catch (NamingException exception) {
              exception.printStackTrace();
          } catch (NumberFormatException exception){
               //the string could not be converted to Int.
               exception.printStackTrace();
               System.out.println("Nie wybrano zadnego wiersza!");
 
 
          } finally {
              // close the context
              if (context != null) {
                  try {
                      context.close();
                  } catch (NamingException exception) {
                      exception.printStackTrace();
                  }
              }
 
              // close the connection
              if (connection != null) {
                  try {
                      connection.close();
                  } catch (JMSException exception) {
                      exception.printStackTrace();
                  }
              }
          }//finally 
 
 
 
        }}}
 

[blazedaces]

First of all, please don't post a lot of code for people to fish through. Not only is this against the posting guidelines, but think about it, how many people will see all that code and really want to read through it to find your problem. Help people help you.

You could have at least showed where it's "supposed" to be setting the value.

Is it right here?:

            orderclientproducer.setStatus(value);       
            fireTableCellUpdated(row, col);  
 

If so, when you do "fireTableCellUpdated(row, col);" I don't see you're passing any "value" to it. Could that be your problem? If not, show exactly where you think you're changing both table values.

Thank you and good luck.
Hope this was helpful,
-blazed

[oll3i]

normally i wd do it data[row][col]=value but my data is a list of OrderClientProducent and in that OrderClientProducent i need to change the dateReceived and status if somebody enteres the date received or changes the combobox

[oll3i]

whne i try to change the value in one of these columns i get

Exception in thread "AWT-EventQueue-0" java.lang.IllegalStateException
at com.db4o.foundation.Iterable4Adaptor.next(Unknown Source)
at com.db4o.internal.query.result.StatefulQueryResult .next(Unknown Sourc
e)
at com.db4o.internal.query.ObjectSetFacade.next(Unkno wn Source)
at TableClientProducer$MyTableModel.setValueAt(tablec lientproducer.java:
278)
at javax.swing.JTable.setValueAt(JTable.java:2676)
at javax.swing.JTable.editingStopped(JTable.java:4675 )
at javax.swing.AbstractCellEditor.fireEditingStopped( AbstractCellEditor.
java:125)
at javax.swing.DefaultCellEditor$EditorDelegate.stopC ellEditing(DefaultC
ellEditor.java:330)
at javax.swing.DefaultCellEditor$3.stopCellEditing(De faultCellEditor.jav
a:140)
at javax.swing.DefaultCellEditor.stopCellEditing(Defa ultCellEditor.java:
215)
at javax.swing.DefaultCellEditor$EditorDelegate.actio nPerformed(DefaultC
ellEditor.java:347)
at javax.swing.JComboBox.fireActionEvent(JComboBox.ja va:1242)
at javax.swing.JComboBox.setSelectedItem(JComboBox.ja va:569)
at javax.swing.JComboBox.setSelectedIndex(JComboBox.j ava:605)
at javax.swing.plaf.basic.BasicComboPopup$Handler.mou seReleased(BasicCom
boPopup.java:814)
at java.awt.AWTEventMulticaster.mouseReleased(AWTEven tMulticaster.java:2
73)
at java.awt.Component.processMouseEvent(Component.jav a:6038)
at javax.swing.JComponent.processMouseEvent(JComponen t.java:3260)
at javax.swing.plaf.basic.BasicComboPopup$1.processMo useEvent(BasicCombo
Popup.java:480)
at java.awt.Component.processEvent(Component.java:580 3)
at java.awt.Container.processEvent(Container.java:205 8)
at java.awt.Component.dispatchEventImpl(Component.jav a:4410)
at java.awt.Container.dispatchEventImpl(Container.jav a:2116)
at java.awt.Component.dispatchEvent(Component.java:42 40)
at java.awt.LightweightDispatcher.retargetMouseEvent( Container.java:4322
)
at java.awt.LightweightDispatcher.processMouseEvent(C ontainer.java:3986)

at java.awt.LightweightDispatcher.dispatchEvent(Conta iner.java:3916)
at java.awt.Container.dispatchEventImpl(Container.jav a:2102)
at java.awt.Window.dispatchEventImpl(Window.java:2429 )
at java.awt.Component.dispatchEvent(Component.java:42 40)
at java.awt.EventQueue.dispatchEvent(EventQueue.java: 599)
at java.awt.EventDispatchThread.pumpOneEventForFilter s(EventDispatchThre
ad.java:273)
at java.awt.EventDispatchThread.pumpEventsForFilter(E ventDispatchThread.
java:183)
at java.awt.EventDispatchThread.pumpEventsForHierarch y(EventDispatchThre
ad.java:173)
at java.awt.EventDispatchThread.pumpEvents(EventDispa tchThread.java:168)

at java.awt.EventDispatchThread.pumpEvents(EventDispa tchThread.java:160)

at java.awt.EventDispatchThread.run(EventDispatchThre ad.java:121)

[blazedaces]

whne i try to change the value in one of these columns i get

TableClientProducer$MyTableModel.setValueAt(tableclientproducer.java:
278)

As far as I can tell this is the only line you should worry about because it's the one you have control over. Line 278 is blank in the code above, it could be the line before, which happens sometimes, but more likely, I have to ask is the code you posted exactly how it looks and was at the time that error was produced? If not redo it and post the line this error code says it's at.

Also, your table changed event, what is it supposed to do? It doesn't seem to change anything and you still don't seem to pass value somehow so how would the event know what value to change that row,col to? Perhaps make an instance variable and make it equal to value?

Good luck,
-blazed

Edit: I didn't read the "// do something with the data" part... So, can I ask what that part of the code looks like? It would help...

-blazed

[oll3i]

on line 278 i have
OrderClientProducent ocp = (OrderClientProducent)result.next();
i get an object from db4o to set its status and delievery deadline

[blazedaces]

on line 278 i have
OrderClientProducent ocp = (OrderClientProducent)result.next();
i get an object from db4o to set its status and delievery deadline

Could the problem be that it's not casting correctly? I don't know, result is an OrderClientProducent object already, why do you need to cast it? I might be confused.

Try printing out result.next()'s contents before you do this, see if something is wrong with result...

Just trying to figure out where the problem lies.

Unfortunately, I'm not an expert and am not familiar with db4o, so I won't be of much help with specifics there...

-blazed

[oll3i]

i made it :) it changes the values in both columns it was db4o throwing the exception stilll have to solve the second part with db4o
thank YOU