2

みんな!

私のアプリケーションでは、パッシブ JTable を作成する必要があります。パッシブとは、行の選択が JTable によって直接行われるのではなく、別のコンポーネントによって要求されたときに行われることを意味します。したがって、ユーザーが新しい行に移動しても、テーブルはすぐには反応しませんが、最初にデータセットに目的の新しい行に基づいて内部状態を更新するように要求し、その後、データセットはテーブルを呼び出して実際の選択を行います。そのため、テーブルで新しい行が選択される前にアクションを実行しようとしています。

私が何を望んでいるのかを理解してもらうために、小さなプロトタイプを作成しました。プロトタイプの下に私の質問があります。

import java.awt.BorderLayout;
import java.awt.Dimension;

import javax.swing.DefaultListSelectionModel;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JPanel;
import javax.swing.JScrollPane;
import javax.swing.JTable;
import javax.swing.ListSelectionModel;
import javax.swing.SwingUtilities;
import javax.swing.event.ListSelectionEvent;
import javax.swing.event.ListSelectionListener;
import javax.swing.table.AbstractTableModel;

public class SSCCE extends JPanel
{
    public SSCCE()
    {
        setLayout(new BorderLayout());

        final JLabel selectedRow = new JLabel();

        final Table table = new Table();
        table.getSelectionModel().addListSelectionListener(
            new ListSelectionListener()
            {
                @Override
                public void valueChanged(ListSelectionEvent e)
                {
                    if (!e.getValueIsAdjusting())
                    {
                        selectedRow.setText(
                            "Selected row: " + table.getSelectedRow());
                    }
                }
            }
        );

        new DataSet(table);

        add(new JScrollPane(table), BorderLayout.CENTER);
        add(selectedRow, BorderLayout.PAGE_END);
    }

    private static void createAndShowGUI()
    {
        JFrame frame = new JFrame("Table Test");
        frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        frame.add(new SSCCE());
        frame.pack();
        frame.setLocationRelativeTo(null);
        frame.setVisible(true);
    }

    public static void main(String[] args)
    {
        SwingUtilities.invokeLater(
            new Runnable()
            {
                @Override
                public void run()
                {
                    createAndShowGUI();
                }
            }
        );
    }
}

class DataSet
{
    private final Table _table;
    private int _currentIndex;

    DataSet(Table table)
    {
        _table = table;
        _table.setDataSet(this);
    }

    int getCurrentIndex()
    {
        return _currentIndex;
    }

    void moveTo(int index) throws MovementException
    {
        if (index < 0 || index > 4)
        {
            throw new IndexOutOfBoundsException();
        }
        // Let's suppose there was a problem moving to the 2nd index. Maybe
        // the data set was in edit mode and couldn't persist the changes
        // because of a validation error.
        if (index == 2)
        {
            throw new MovementException();
        }
        _currentIndex = index;
        // Notifies the table that the data was moved so that the table can
        // update its selection model based on the current index of the
        // data set.
        _table.dataMoved();
    }
}

class MovementException extends RuntimeException
{
}

class Table extends JTable
{
    private DataSet _dataSet;
    // When true signals that the data was moved in the data set, so selection
    // is allowed.
    private boolean _dataMoved;
    // Previous selected column.
    private int _oldSelectedColumn;

    Table()
    {
        super(new Model());

        setSelectionMode(ListSelectionModel.SINGLE_SELECTION);
        setCellSelectionEnabled(true);
        getTableHeader().setReorderingAllowed(false);
        setPreferredScrollableViewportSize(new Dimension(500, 170));

        getColumnModel().setSelectionModel(new ColumnSelectionModel());
    }

    void setDataSet(DataSet dataSet)
    {
        _dataSet = dataSet;
    }

    // Called by DataSet#moveTo.
    void dataMoved()
    {
        _dataMoved = true;
        try
        {
            int rowIndex = _dataSet.getCurrentIndex();
            // Select the new row.
            setRowSelectionInterval(rowIndex, rowIndex);
        }
        finally
        {
            _dataMoved = false;
        }
    }

    @Override
    protected ListSelectionModel createDefaultSelectionModel()
    {
        return new RowSelectionModel();
    }

    private class ColumnSelectionModel extends DefaultListSelectionModel
    {
        @Override
        public void setSelectionInterval(int index0, int index1)
        {
            // Save the old selected column to be restored in
            // RowSelectionModel#setSelectionInterval in case of an error.
            _oldSelectedColumn = getSelectedColumn();
            super.setSelectionInterval(index0, index1);
        }
    }

    private class RowSelectionModel extends DefaultListSelectionModel
    {
        @Override
        public void setSelectionInterval(int index0, int index1)
        {
            if (_dataMoved || index1 == _dataSet.getCurrentIndex())
            {
                super.setSelectionInterval(index0, index1);
            }
            else
            {
                try
                {
                    _dataSet.moveTo(index1);
                }
                catch (MovementException ex)
                {
                    // There was a problem in the data set. Restore the old
                    // selected column.
                    setColumnSelectionInterval(
                    _oldSelectedColumn, _oldSelectedColumn);
                    throw ex;
                }
            }
        }
    }

    private static class Model extends AbstractTableModel
    {
        private String[] columnNames =
            {"First Name", "Last Name", "Sport", "# of Years", "Vegetarian"};
        private Object[][] data = {
            {"Kathy", "Smith", "Snowboarding", 5, false},
            {"John", "Doe", "Rowing", 3, true},
            {"Sue", "Black", "Knitting", 2, false},
            {"Jane", "White", "Speed reading", 20, true},
            {"Joe", "Brown", "Pool", 10, false}
        };

        public int getColumnCount()
        {
            return columnNames.length;
        }

        public int getRowCount()
        {
            return data.length;
        }

        public String getColumnName(int col)
        {
            return columnNames[col];
        }

        public Object getValueAt(int row, int col)
        {
            return data[row][col];
        }

        public Class<?> getColumnClass(int c)
        {
            return getValueAt(0, c).getClass();
        }

        public void setValueAt(Object value, int row, int col)
        {
            data[row][col] = value;
            fireTableCellUpdated(row, col);
        }
    }
}
  • このデザインに欠陥はありますか?
  • コントラクトを強制するには、 ColumnSelectionModelクラスとRowSelectionModelクラスのメソッドをさらにオーバーライドする必要がありますか? それともsetSelectionIntervalメソッドだけで十分ですか? この点に関して、今のところ欠陥は見つかりませんでした。
  • ColumnSelectionModelクラスがあると本当にイライラします。その目的は、何か問題が発生した場合にRowSelectionModel#setSelectionIntervalで復元できるように、新しい列が選択される前に古い選択列をキャッチすることだけです。RowSelectionModelクラスだけではできませんでした。別の方法はありますか?

選択モデルを使用しない別のアプローチがあります。あなたはこれを行うことができます:

getColumnModel().setSelectionModel(new ColumnSelectionModel());テーブル コンストラクターの行にコメントを付けます。

メソッドTable#createDefaultSelectionModelメソッドをコメント化します。

Table#dataMovedメソッドを次のように置き換えます。

void dataMoved()
{
    _dataMoved = true;
    try
    {
        int rowIndex = _dataSet.getCurrentIndex();
        changeSelection(rowIndex, getSelectedColumn(), false, false);
    }
    finally
    {
        _dataMoved = false;
    }
}

Table#changeSelectionメソッドをオーバーライドします。

@Override
public void changeSelection(int rowIndex, int columnIndex, boolean toggle, boolean extend)
{
    if (_dataMoved)
    {
        super.changeSelection(rowIndex, columnIndex, toggle, extend);
    }
    else
    {
        if (rowIndex != _dataSet.getCurrentIndex())
        {
            _dataSet.moveTo(rowIndex);
        }
        super.changeSelection(_dataSet.getCurrentIndex(), columnIndex, toggle, extend);
    }
}

しかし、 changeSelectionメソッドのドキュメントには次のように記載されているため、選択モデルよりもはるかに単純ですが、このアプローチは使用しませんでした。

UI によって受信されたキーボードまたはマウス イベントの結果である選択に対するほとんどの変更は、このメソッドを介して伝達されるため、サブクラスによって動作をオーバーライドすることができます。

だから解釈した

ほとんどの変更

すべてが変更されるわけではないため、このメソッドを通過しない選択変更がいくつかある可能性があります。私はこれで正しいですか、それともchangeSelectionアプローチを信頼できますか?

前もって感謝します。

マルコス

4

1 に答える 1

2

あなたのアプローチの主な欠陥は、あなたの(ナビゲーション)モデル - 別名: DataSet - とビューの間の有線の双方向カップリングです。解決策は VetoableSelectionModel のようなものです: 次に、DataSet を vetoablePropertyChangeListener として選択モデルに登録できます。これは、テーブルをサブクラス化せずに構成できる疎結合です。

全体的な配線のいくつかのコード スニペット:

final JTable table = new JTable(new Model());
VetoableListSelectionModel selectionModel = new VetoableListSelectionModel();
table.setSelectionModel(selectionModel);
VetoableChangeListener veto = new VetoableChangeListener() {

    @Override
    public void vetoableChange(PropertyChangeEvent evt)
            throws PropertyVetoException {
        if (2 == (Integer) evt.getNewValue()) throw new PropertyVetoException("", evt);
    }
};
selectionModel.addVetoableChangeListener(veto);
table.getSelectionModel().addListSelectionListener(
    new ListSelectionListener()
    {
        @Override
        public void valueChanged(ListSelectionEvent e)
        {
            if (!e.getValueIsAdjusting())
            {
                selectedRow.setText(
                    "Selected row: " + table.getSelectedRow());
            }
        }
    }
);

スケルトン選択モデル (完全なコードはswingx インキュベーターにあります- 注意: メンテナンスされていません!)

/**
 * Quick impl of a list selection model which respects a veto before
 * changing selection state. The veto is effect in SINGLE_SELECTION mode
 * only.
 */
public class VetoableListSelectionModel extends DefaultListSelectionModel {
    private VetoableChangeSupport vetoableChangeSupport;

    /**
     * Defaults to SINGLE_SELECTION mode.
     *
     */
    public VetoableListSelectionModel() {
        super();
        setSelectionMode(SINGLE_SELECTION);
    }

    @Override
    public void setSelectionInterval(int index0, int index1) {
        if (isVetoable()) {
            try {
                fireVetoableChange(getMinSelectionIndex(), index0);
            } catch (PropertyVetoException e) {
                // vetoed - do nothing
                return;
            }
        }
        super.setSelectionInterval(index0, index1);
    }

    // similar for all methods that change the selection
    ...

    // methods to add/remove listeners and fire the event
    ...
}
于 2013-04-06T13:04:09.840 に答える