表中的SWT Combo性能不佳

时间:2015-02-25 13:55:19

标签: java performance memory-leaks combobox swt

我在表格中遇到组合的性能问题。 当我启动应用程序时,一切都很好。 我可以改变组合的值,表的反应是可以的。 但是当我清理桌子并重新填充它时,似乎有一个灾难性的性能损失。 当我尝试从表中选择一行时,它需要大约1-2秒才能被选中。 当我按下“重置表”-Button 5-10次时,行选择需要5-6秒。

首先,我认为使用后必须处理掉所有东西。但这并没有改变任何事情。 我也尝试过使用文本框。一样的问题。 第一次表现处于可接受的水平。按下重置按钮 - >非常糟糕的表现

以下是此问题的示例:

import org.eclipse.swt.SWT;
import org.eclipse.swt.custom.TableEditor;
import org.eclipse.swt.events.ModifyEvent;
import org.eclipse.swt.events.ModifyListener;
import org.eclipse.swt.events.SelectionAdapter;
import org.eclipse.swt.events.SelectionEvent;
import org.eclipse.swt.events.SelectionListener;
import org.eclipse.swt.graphics.Rectangle;
import org.eclipse.swt.widgets.Button;
import org.eclipse.swt.widgets.Combo;
import org.eclipse.swt.widgets.Control;
import org.eclipse.swt.widgets.Display;
import org.eclipse.swt.widgets.Shell;
import org.eclipse.swt.widgets.Table;
import org.eclipse.swt.widgets.TableColumn;
import org.eclipse.swt.widgets.TableItem;

public class Test {
private static Table table;

public static void main(String[] args) {
    Display display = new Display();
    Shell shell = new Shell(display);
    table = new Table(shell, SWT.BORDER | SWT.V_SCROLL | SWT.H_SCROLL | SWT.FULL_SELECTION );
    table.setHeaderVisible(true);
    TableColumn column = new TableColumn(table, SWT.NONE);
    column.setText("1");
    column.setWidth(70);
    column = new TableColumn(table, SWT.NONE);
    column.setText("2");
    column.setWidth(70);
    column = new TableColumn(table, SWT.NONE);
    column.setText("3");
    column.setWidth(70);
    fillTable();
    Rectangle clientArea = shell.getClientArea();
    table.setBounds(clientArea.x, clientArea.y, 250, 300);
    shell.setSize(400, 400);
    Button button = new Button(shell, SWT.PUSH);
    button.setBounds(10, 320, 150, 25);
    button.setText("Reset Table");
    button.addSelectionListener(new SelectionListener(){

        @Override
        public void widgetDefaultSelected(SelectionEvent e) {
            // TODO Auto-generated method stub
        }

        @Override
        public void widgetSelected(SelectionEvent e) {
            resetTable();
        }

    });
    shell.open();
    while (!shell.isDisposed()) {
        if (!display.readAndDispatch())
            display.sleep();
    }
    display.dispose();
}

public static void addComboToCell(final int intgPEditCell) {
    final TableEditor editor = new TableEditor(table);
    // The editor must have the same size as the cell and must
    // not be any smaller than 50 pixels.
    editor.horizontalAlignment = SWT.LEFT;
    editor.grabHorizontal = true;
    editor.minimumWidth = 50;
    // editing the second column
    table.addSelectionListener(new SelectionAdapter() {
        @Override
        public void widgetSelected(SelectionEvent e) {
            // Clean up any previous editor control
            Control oldEditor = editor.getEditor();
            if (oldEditor != null)
                oldEditor.dispose();

            // Identify the selected row
            TableItem item = (TableItem) e.item;
            if (item == null)
                return;

            // The control that will be the editor must be a child of the
            // Table
            Combo newEditor = new Combo(table, SWT.NONE);
            // SWT.NONE);
            newEditor.setText(item.getText(intgPEditCell));
            newEditor.addModifyListener(new ModifyListener() {
                public void modifyText(ModifyEvent me) {
                    Combo combo = (Combo) editor.getEditor();
                    editor.getItem()
                            .setText(intgPEditCell, combo.getText());
                }
            });
             newEditor.setFocus();
            editor.setEditor(newEditor, item, intgPEditCell);
        }
    });
}

public static void resetTable(){
    TableItem[] tableItems = table.getItems();
    for (int i = 0; i < tableItems.length; i++) {
        tableItems[i].dispose();
    }
    Control[] children = table.getChildren();
    for (int i = 0 ; i < children.length; i++) {
        children[i].dispose();
    }
    table.removeAll();
    fillTable();
}

public static void fillTable(){
    for (int i = 0; i < 50; i++) {
        TableItem item = new TableItem(table, 0);
        item.setText(0, "0 Item " + i);
        item.setText(1, "1 Item " + i);
        item.setText(2, "2 Item " + i);
        addComboToCell(0);
    }
}
}

所以我的问题是如何在重新填充表格后改善性能?! 我忘记了什么或做错了什么? 感谢

1 个答案:

答案 0 :(得分:1)

性能下降是由您添加的TableEditor引起的。每行有一个,所有这些都在监听表选择事件。因此,如果选择了某个项目,则会激活50 TableEditor个,并将它们叠加在一起。除此之外,TableEditor中的resetTable未被删除,因此每次重置按钮的选择都会增加50个。

SWT中TableEditors的conecpt是每列或每个编辑器类型有一个编辑器。在适当的时候,编辑器被移动到正确的位置,用正确的值初始化然后显示。

在您的情况下,addComboToCell ( 0 )调用需要在创建表后的某个位置移动到main方法。