Question

我正在从事一个工作项目，我遇到了一个不寻常的问题。我有一个JTable，用户可以填充数据。在我的实际代码中有一个＆＃34;添加行＆＃34;允许用户填写一些GUI的按钮，该信息生成行。

这个的另一个重要特征是克隆行的按钮。由于添加行的过程可能非常耗时（有许多字段需要填写）如果用户只需要添加一个不同的单元格的新行，那么他可以使用按钮克隆单元格。

此克隆按钮按预期工作，但有一个相当奇怪的问题。一旦克隆了一行，我注意到当我尝试更改已克隆的任何细胞的内容时，会出现意外结果。例如，如果我将单元格的内容更改为＆＃34; Ryan＆＃34;那么其他细胞也可能会突然改变，如果我在更换一个细胞后点击一个细胞，我点击的细胞会自行改变。我很确定这个问题与克隆方法有关，我真的不知道要修复。

我创建了一个可验证的程序，因此您可以自己发送文本并查看我正在谈论的内容。只需使用克隆按钮几次，然后尝试更改单个单元格的内容并在其他单元格中查看结果。

我真的需要解决这个问题，但我不知道该做什么，而且非常感谢帮助。

主类

package jtabletest;

public class JTableTestMain 

{

    public static void main(String args[]){

        JTableTest jTest = new JTableTest();
        jTest.createGUI();

    }

}

JTable Class

package jtabletest;

import java.awt.BorderLayout;
import java.awt.Dimension;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.util.Vector;

import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JPanel;
import javax.swing.JTable;
import javax.swing.table.DefaultTableModel;

public class JTableTest 

{

    protected static DefaultTableModel dtm = new DefaultTableModel();
    public static JTable tbl;


    public void createGUI(){

        final JFrame frame = new JFrame("JTable Test");
        JPanel mainPanel = new JPanel(new BorderLayout());
        JPanel panelNorth = new JPanel(new BorderLayout());
        JPanel panelSouth = new JPanel(new BorderLayout());
        JPanel buttonPanel = new JPanel();

        JButton cloneButton = new JButton("Clone");
        cloneButton.setPreferredSize(new Dimension(150,40));
        buttonPanel.add(cloneButton);

        JButton printButton = new JButton("Print");
        printButton.setPreferredSize(new Dimension(150,40));
        buttonPanel.add(printButton);

        tbl = new JTable();

        String header[] = new String[]{
                "Employee", "Pay-Rate", "Hours Worked"};


        dtm.setColumnIdentifiers(header);

        tbl.setModel(dtm);

        tbl.setAutoResizeMode(JTable.AUTO_RESIZE_OFF);

        for(int i = 0; i < header.length; i++){
            tbl.getColumnModel().getColumn(i).setPreferredWidth(200);
        }

        dtm.addRow(new Object[]{"Pete","$10.00","40"});
        dtm.addRow(new Object[]{"Bob","12.50","42"});
        dtm.addRow(new Object[]{"Jamar","$7.25,25"});

        cloneButton.addActionListener(new ActionListener(){

            public void actionPerformed(ActionEvent ae){

                int[] selectedRows = tbl.getSelectedRows();

                if(selectedRows.length>0){

                    @SuppressWarnings("rawtypes")
                    Vector data = dtm.getDataVector();

                    int insertPoint = selectedRows[selectedRows.length-1]+1;

                    for(int i = 0; i < selectedRows.length; i++){
                        @SuppressWarnings("rawtypes")
                        Vector targetRow = (Vector)data.elementAt(selectedRows[i]);

                        dtm.insertRow(insertPoint, targetRow);
                        insertPoint++;

                    }

                    dtm.fireTableDataChanged();

                }

            }

        });

        printButton.addActionListener(new ActionListener(){

            public void actionPerformed(ActionEvent ae){

                if(null != tbl.getCellEditor()){
                    tbl.getCellEditor().stopCellEditing();
                }

                for(int i = 0; i < tbl.getRowCount(); i++){
                    System.out.println(tbl.getValueAt(i, 0));
                    System.out.println(tbl.getValueAt(i, 1));
                    System.out.println(tbl.getValueAt(i, 2));
                }

            }

        });

        panelNorth.add(tbl,BorderLayout.NORTH);
        panelNorth.setPreferredSize(new Dimension(500,500));
        panelSouth.add(buttonPanel,BorderLayout.NORTH);
        mainPanel.add(panelNorth,BorderLayout.NORTH);
        mainPanel.add(panelSouth,BorderLayout.SOUTH);
        frame.add(mainPanel);
        frame.setVisible(true);
        frame.setSize(1900,600);
        frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);



    }

}

Answer 1

听起来您正在重复使用相同的引用，而不是在克隆方法上复制新对象。我建议做以下事情。

1）首先创建一个新的Vector，然后看看是否能解决这个问题。

for(int i = 0; i < selectedRows.length; i++){
  @SuppressWarnings("rawtypes")
  Vector targetRow = (Vector)data.elementAt(selectedRows[i]);
  Vector newVector = new Vector();
  for (int t = 0; t < targetRow.size(); t++) {
     newVector.add(targetRow.get(t));
  }
  dtm.insertRow(insertPoint, newVector);
  insertPoint++;
}

并查看这是否可以解决您的问题。如果确实如此，你就完成了。如果它没有

2）创建一个类似于上面的新Vector，对于Vector中任何基于Class的对象，重新创建它们，因为当前你正在处理指针。

我很难说＃1是否会解决您的问题因为我不知道来自表格的Vector的内容，如果它是原始的你可能是安全的，否则你可能需要做解决方案＃2。

Answer 2

你的问题在这一行：

Vector targetRow = (Vector)data.elementAt(selectedRows[i]);

您没有创建副本，而是在创建新引用时添加

dtm.insertRow(insertPoint, targetRow)

您要添加的行实际上是相同的，而不是先前选择的行的副本。

您必须使用类似

的内容

Vector aux = (Vector)data.elementAt(selectedRows[i]);
Vector targetRow = aux.clone();

让它发挥作用。

Answer 3

克隆是这里的关键字。您没有克隆数据。您只是将引用从一个Vector复制到另一个Vector。因此，由于每一行共享相同的引用，因此该值出现在两行中。

所以你需要实际克隆每个元素。

代码如下：

  Vector targetRow = (Vector)data.elementAt(selectedRows[i]);
  Vector clonedRow = new Vector(targetRow.size());

  for (Object object: targetRow)
  {
     clonedRow.addElement( object.clone() );
  }

注意，我之前从未使用过clone（），因此您可以使用：

  Vector targetRow = (Vector)data.elementAt(selectedRows[i]);
  Vector clonedRow = targetRow.clone();

但我不确定它是否只是克隆Vector而不是Vector中的元素。

此外，您永远不会调用firstTableDataChanged（）方法。这是DefaultTableModle在调用insertRow（...）方法时触发相应方法的工作。

编辑：

是的，使用克隆确实有效，但您需要克隆Vector而不是Vector中的每个项目：

//dtm.insertRow(insertPoint, targetRow);
dtm.insertRow(insertPoint, (Vector)targetRow.clone());

或

dtm.insertRow(insertPoint, new Vector(targetRow));

克隆后编辑JTable单元

3 个答案: