Question

I am trying to write a method that will get the current selections from the table and create a ArrayList from the selections.

Method:

 public void getPlotterSelection() {
   selPrinters = new ArrayList<PrinterProfile>();
   int[] row = table.getSelectionIndices();
   Arrays.sort(row);
   if (row.length > 0) {
     for(int i = row.length-1; i >= 0; i--){
        PrinterProfile pp = new PrinterProfile(aa.get(i).getPrinterName(), aa.get(i).getProfileName());
        selPrinters.add(pp);
     }
  }
}

This is the error I am getting

ERROR: 16:16:49,503 – TcLogger$IC_LogListener.logging:?
org.eclipse.core.runtime – org.eclipse.ui – 0 – Unhandled event loop exception
org.eclipse.swt.SWTException: Widget is disposed
at org.eclipse.swt.SWT.error(SWT.java:3884)
at org.eclipse.swt.SWT.error(SWT.java:3799)
at org.eclipse.swt.SWT.error(SWT.java:3770)
at org.eclipse.swt.widgets.Widget.error(Widget.java:463)
at org.eclipse.swt.widgets.Widget.checkWidget(Widget.java:336)
at org.eclipse.swt.widgets.Table.getSelectionIndices(Table.java:2536)
etc …….

The problem is with this line of code

int[] row = table.getSelectionIndices();

Once again..

I am trying to get the user selected rows in the table and put them in a arraylist.

Edit Adding more code

 //////////////////////////////////////////////////////////////////////////
 //                       createDialogArea()                             //
 //////////////////////////////////////////////////////////////////////////
 protected Control createDialogArea(Composite parent) {
  final Composite area = new Composite(parent, SWT.NONE);
  final GridLayout gridLayout = new GridLayout();
  gridLayout.marginWidth = 15;
  gridLayout.marginHeight = 10;
  area.setLayout(gridLayout);

  GridData gridData = new GridData(SWT.FILL, SWT.FILL, true, true);
  area.setLayoutData(gridData);
  checkingArray();
  createCopyNumber(area);
  createPlotterTable(area);
  return area;
}

public void checkingArray() {
   aa = abd.getPrintersArray();
}

//////////////////////////////////////////////////////////////////////////
//                     createPlotterTable()                             //
//////////////////////////////////////////////////////////////////////////
private void createPlotterTable(Composite parent) {
  Composite composite = new Composite(parent, SWT.BORDER); 
  GridLayout gridLayout = new GridLayout(1, false);
  composite.setLayout(gridLayout);
  GridData gridData = new GridData(SWT.FILL, SWT.FILL, true, true);
  composite.setLayoutData(gridData);

  //gridData = new GridData(SWT.FILL, SWT.FILL, true, true ); 
  table = new Table(composite, SWT.BORDER | SWT.FULL_SELECTION | SWT.MULTI);
  table.setHeaderVisible(true);
  table.setLinesVisible(true);

  TableColumn[] column = new TableColumn[2];
  column[0] = new TableColumn(table, SWT.FILL);
  column[0].setText("Printer Name");
  column[0].setWidth(200);

  column[1] = new TableColumn(table, SWT.FILL);
  column[1].setText("Profile Name");
  column[1].setWidth(200);

  gridData = new GridData();
  gridData.verticalAlignment = GridData.FILL;
  gridData.horizontalAlignment = GridData.FILL;
  gridData.grabExcessHorizontalSpace = true;
  gridData.grabExcessVerticalSpace = true;
  table.setLayoutData(gridData);

  fillTable(table);
  table.setRedraw(true);
} 

private void fillTable(Table table) {
  table.setRedraw(false);

  for(Iterator iterator = abd.getPrintersArray().iterator();iterator.hasNext();){
     PrinterProfile printer = (PrinterProfile) iterator.next();
     TableItem item = new TableItem(table, SWT.FILL);
     int c = 0;
     item.setText(c++, printer.getPrinterName());
     item.setText(c++, printer.getProfileName());
  }
table.setRedraw(true);
} 

 public void getPlotterSelection() {

   selPrinters = new ArrayList<PrinterProfile>();

   int[] row = table.getSelectionIndices();

   Arrays.sort(row);
   if (row.length > 0) {
     for(int i = row.length-1; i >= 0; i--){
        PrinterProfile pp = new PrinterProfile(aa.get(i).getPrinterName(), aa.get(i).getProfileName());
        selPrinters.add(pp);
     }
   }
}  

This is the button that calls the method

 Button okButton = createButton(parent, IDialogConstants.OK_ID, "OK", true);
   okButton.setEnabled(true);
   okButton.addSelectionListener(new SelectionAdapter() {
      public void widgetSelected(SelectionEvent e) {
         getPlotterSelection();
      }
   });

Answer 1

I would suggest you to user Table viewer rather than Table. It makes your life easier. I see that you are showing table in a dialog. I guess your dialog is getting closed/disposed when you click OK before it get to getPlotterSelection() method.