0

タイプモジュールのArrayListがあり、各モジュールには割り当てのarraylistがあります。次の TableModel を作成しましたが、テーブルで何かを選択すると問題が発生し、ArrayLists が範囲外になります。ここに私のテーブルモデルがあります:

public class AssignmentsTableModel extends AbstractTableModel {
private ArrayList<Module> modules;
private static final String[] COLUMN_NAMES = {"Module Identifier", "Module Name", "Assignment Title", "Author", "Date Set", "Date Due", "Weighting"};

private int moduleID;
private int assignmentID;
private Module module;
private int totalNumberAssignments;

public AssignmentsTableModel(ArrayList<Module> modules) {
    moduleID = 0;
    assignmentID = 0;
    this.modules = modules;
    module = modules.get(moduleID);
    totalNumberAssignments = getRowCount();
}

public AssignmentsTableModel(Module module) {
    modules = new ArrayList<Module>();
    modules.add(module);
}

@Override
public int getRowCount() {
    int rowCount = 0;
    for(Module mod : modules){
        rowCount += mod.getAssignments().size();
    }
    return rowCount;
}

@Override
public int getColumnCount() {
    return COLUMN_NAMES.length;  //To change body of implemented methods use File | Settings | File Templates.
}

@Override
public Object getValueAt(int row, int column) {
    if(column%getColumnCount() == 0 && getRowCount() <= totalNumberAssignments){
        if(isAtLastAssignment(row, moduleID)){
            assignmentID = 0;
            moduleID += 1;
        }
    }

    module = modules.get(moduleID);
    ArrayList<Assignment> assignments = module.getAssignments();

    Assignment assignment = assignments.get(assignmentID);

    switch (column){
        case 0: return module.getIdentifier();
        case 1: return module.getTitle();
        case 2: return assignment.getTitle();
        case 3: return assignment.getAuthor();
        case 4: return assignment.getSet();
        case 5: return assignment.getDue();
        case 6: return assignment.getWeighting();
        default: return null;
    }


}

public String getColumnName(int columnIndex) {
    return COLUMN_NAMES[columnIndex];
}

public boolean isAtLastAssignment(int row, int moduleID){
    boolean atLast = false;
    ArrayList<Assignment> assignments = modules.get(moduleID).getAssignments();
    if(moduleID == 0){
        if(row == assignments.size()){
            atLast = true;
        }
    } else {
        int assignmentsSize = assignments.size() + getCurrentRowNumber(moduleID);
        if(row == assignmentsSize){
            atLast = true;
        }
    }

    return atLast;
}

public int getCurrentRowNumber(int moduleID){
    int rowNumber = 0;

    for(int i = 0; i < moduleID; i++){
        rowNumber = modules.get(i).getAssignments().size()-1;
    }

    return rowNumber;
}
}

ご覧のとおり、moduleID をグローバル変数として保存します。これは、最初の起動時には問題ありませんが、何かが選択されると、最後の値が使用されます。これが起こらないようにするには、他に何ができますか?

4

2 に答える 2

5

TableModel メソッドgetValueAt()は、セルの値を取得してレンダリングできるようにするたびに、テーブル レンダラーによって呼び出されます。テーブルの行をクリックしてテーブルに行を選択済みとしてマークさせることとは (直接的または確実に) 関連していません。

つまり、メソッドを呼び出した結果として、テーブル モデルの状態を変化させたくありませんgetValueAt()。

getValueAt()メソッドで行を配列へのインデックスとして使用するのが最も簡単です。ただし、モジュールごとに可変数の行があるようです。その場合、正しいインデックスをアルゴリズムで生成できない場合は、コンストラクターでデータを一度「フラット化」して、ルックアップ コードを簡素化できます。

つまり、テーブル モデルには以下が含まれます。

private List<RowData> rowData

行データを含む内部クラス:

private static class RowData
{
  private Moddule;
  private Assignment assignment;

  ...
}

そしてあなたのコンストラクターで：

public AssignmentsTableModel(List<Module> modules)
{
  this.rowData = new ArrayList<Module>();

  for (Module module : modules)
  {
    for (Assignment assignment : module.getAssignments())
    {
      rowData.add(new RowData(module, assignment));
    }
  }
}

getValueAt()これにより、メソッドが簡素化されます。

@Override
public Object getValueAt(int row, int column) {
    final RowData row = this.rowData.get(row);

    switch (column){
        case 0: return row.getModule().getIdentifier();
        case 1: return row.getModule().getTitle();
        case 2: return row.getAssignment().getTitle();
        case 3: return row.getAssignment().Author();
        case 4: return row.getAssignment().getSet();
        case 5: return row.getAssignment().getDue();
        case 6: return row.getAssignment().getWeighting();
        default: return null;
    }
}
于 2012-04-02T00:33:06.727 に答える
1

  • 選択は基本的には関係ありませTableModelん。おそらく、ここに投稿していない残りのコードまたはメソッドに別の問題があります。

  • AbstractTableModel基本的な成功のために、データ、行、または列をから（およびその逆）に追加/変更/削除するために必要なModel別Viewの方法がありません。

  • DefaultTableModelを使用します。おそらく、使用する理由はありません。AbstractTableModel

  • 基本的なAbstractTableModelに必要なメソッドを比較したり、別の配列やハッシュの型ではなくAPIで直接使用Object[][]または実装したりできます。Vector<Vector<Object>>

  • JTableの仕組みを読んでください

編集

@ディーンは書いた：

that I can't see another way around but when an update in the table happens

setXxxMethodsそれが（fireXxxXxxメソッドが内部にある）存在する理由です。OracleチュートリアルでJTableは、それについて何度も言及しています。

于 2012-04-02T00:00:28.800 に答える