Eclipse Community Forums: Remote Application Platform (RAP) » insight required on changing values in TableViewer with asyncExec

Home » Eclipse Projects » Remote Application Platform (RAP) » insight required on changing values in TableViewer with asyncExec

Show: Today's Messages :: Show Polls :: Message Navigator

insight required on changing values in TableViewer with asyncExec [message #117948]

Fri, 09 January 2009 05:45

Tom H

Messages: 139
Registered: July 2009

Senior Member

Hi,

Sorry for the long mangled code post...

So I'm having a bit of trouble bulding a stocks table that updates in real
time, and uses "tableItem.setBackground( 2, Graphics.getColor(255,0,0) );"
so update the colour of the cell when the value changes

At the moment, I don't seem to be able to make any changes to the current
TableItems at all in the Runnable... It just ignores me ;-(

Any ideas what I am doing wrong in Runnable, or elsewhere?

Thanks,

Tom

package catshelter;

import java.util.List;
import java.util.ArrayList;
import java.util.Random;

import org.eclipse.jface.viewers.IStructuredContentProvider;
import org.eclipse.jface.viewers.ITableLabelProvider;
import org.eclipse.jface.viewers.LabelProvider;
import org.eclipse.jface.viewers.TableViewer;
import org.eclipse.jface.viewers.TableViewerColumn;
import org.eclipse.jface.viewers.Viewer;
import org.eclipse.rwt.graphics.Graphics;
import org.eclipse.rwt.lifecycle.UICallBack;
import org.eclipse.swt.SWT;
import org.eclipse.swt.events.SelectionEvent;
import org.eclipse.swt.events.SelectionListener;
import org.eclipse.swt.graphics.Image;
import org.eclipse.swt.layout.GridData;
import org.eclipse.swt.layout.GridLayout;
import org.eclipse.swt.widgets.Button;
import org.eclipse.swt.widgets.Composite;
import org.eclipse.swt.widgets.Table;
import org.eclipse.swt.widgets.TableItem;
import org.eclipse.ui.PlatformUI;
import org.eclipse.ui.part.ViewPart;

public class WatchlistSecurityView extends ViewPart {

private String[] securities = { "FTSE 100 Cash", "Wall Street Cash",
"FTSE to finish Up", "Germany 30", "Spot FX USD/JPY",
"Spot FX EUR/USD", "Spot Gold ", "US Light Crude", "Barclays Plc",
"Royal Dutch Shell Plc -A", "BNP Paribas", "UBS AG" };

private String[] columns = { "Market", "Period", "Sell", "Buy", "Change",
"% Chg", "Update", "High", "Low" };
TableViewer viewer;

@Override
public void createPartControl(final Composite parent) {

Composite contents = new Composite(parent, SWT.NONE);
contents.setLayout(new GridLayout(1, false));

viewer = new TableViewer(contents, SWT.MULTI | SWT.H_SCROLL
| SWT.V_SCROLL | SWT.FULL_SELECTION);
viewer.getTable().setLayoutData(new GridData(GridData.FILL_BOTH));
viewer.setContentProvider(new SecurityListContentProvider());

SecurityListLabelProvider lp = new SecurityListLabelProvider();
lp.createColumns(viewer);
viewer.setLabelProvider(lp);
viewer.getTable().setLinesVisible(false);
viewer.setInput(getViewSite());
Button b = new Button(contents, SWT.BORDER);
b.setText("Activate!");

final Runnable runnable = new Runnable() {
public void run() {

// while(true){
// if(Event.waitFor()){
System.out.println("executing Runnable");
/*
* MessageDialog.openInformation( parent.getShell(),
* "Treeviewer", "You selected " );
*/
List buffer = new ArrayList();

Random generator = new Random();
for (int i = 0; i < securities.length; i++) {
// TableItem item = new TableItem( viewer.getTable(),
// SWT.NONE);
TableItem item = (TableItem) viewer.getElementAt(i);

item.setText(0, securities[i]);
for (int j = 1; j < columns.length; j++) {
item.setText(j, String.valueOf(generator.nextInt(100)));
}
item.setBackground( Graphics.getColor(0,0,128) );

// TableItem item = new TableItem(viewer.getTable(),
// SWT.NONE);
// item.setText(0,"test");
// viewer.add( new String[] {"item","item","item"} );

// TableItem item = (TableItem)viewer.getElementAt(0);
// item.setBackground( 2, Graphics.getColor(255,0,0) );

// viewer.refresh();
// }
// }
}
}
};

b.addSelectionListener(new SelectionListener() {
public void widgetDefaultSelected(SelectionEvent e) {
}

public void widgetSelected(SelectionEvent e) {
System.out.println("Starting UICallback");
// list.put(inputText.getText(),Status.INCOMPLETE);
UICallBack.activate("notifierId");
/*
* notifierThread = new Thread(createRunnable());
* notifierThread.setDaemon(true); notifierThread.start();
*/
// shell.getDisplay().asyncExec(runnable);
PlatformUI.createDisplay().asyncExec(runnable);
UICallBack.deactivate("notifierId");
}
});

}

public class SecurityListContentProvider implements
IStructuredContentProvider {

@Override
public Object[] getElements(Object inputElement) {
Random generator = new Random();
List rowList = new ArrayList();

for (int i = 0; i < securities.length; i++) {
String[] row = new String[securities.length];
row[0] = securities[i];
for (int j = 1; j < columns.length; j++) {
row[j] = "" + String.valueOf(generator.nextInt(100));
}
rowList.add(row);
}

return rowList.toArray();
}

@Override
public void dispose() {
}

@Override
public void inputChanged(Viewer viewer, Object oldInput, Object
newInput) {
}

}

public class SecurityListLabelProvider extends LabelProvider implements
ITableLabelProvider {

public void createColumns(TableViewer viewer) {
Table table = viewer.getTable();

int[] bounds = { 100, 100, 100 };

for (int i = 0; i < columns.length; i++) {
TableViewerColumn column = new TableViewerColumn(viewer,
SWT.NONE);
column.getColumn().setText(columns[i]);
column.getColumn().setWidth(100);
column.getColumn().setResizable(true);
}
table.setHeaderVisible(true);
table.setLinesVisible(true);
}

@Override
public Image getColumnImage(Object element, int columnIndex) {
Random generator = new Random();
ClassLoader loader = getClass().getClassLoader();

if (columnIndex == 0) {
int myInt = generator.nextInt(100);
if (myInt < 34) {
return Graphics.getImage("icons/state_edit.png", loader);
} else if (myInt > 66) {
return Graphics.getImage("icons/state_closed.png", loader);
} else {
return Graphics.getImage("icons/state_deal.png", loader);
}
}
return null;
}

@Override
public String getColumnText(Object element, int columnIndex) {
String[] str = (String[]) element;
return str[columnIndex];

}

}

@Override
public void setFocus() {
// TODO Auto-generated method stub

}

}

Report message to a moderator

Re: insight required on changing values in TableViewer with asyncExec [message #117962 is a reply to message #117948]

Fri, 09 January 2009 09:00

Ivan Furnadjiev

Messages: 2426
Registered: July 2009
Location: Sofia, Bulgaria

Senior Member

This is a multi-part message in MIME format.
--------------050604090806000709080605
Content-Type: text/plain; charset=ISO-8859-15; format=flowed
Content-Transfer-Encoding: 7bit

Hi Tom,

I've modified a little bit your example to make it work (see attached
file). Hope that this will help.

Best,
Ivan

Tom H wrote:
>
>
> Hi,
>
> Sorry for the long mangled code post...
>
> So I'm having a bit of trouble bulding a stocks table that updates in
> real time, and uses "tableItem.setBackground( 2,
> Graphics.getColor(255,0,0) );" so update the colour of the cell when
> the value changes
>
> At the moment, I don't seem to be able to make any changes to the
> current TableItems at all in the Runnable... It just ignores me ;-(
>
> Any ideas what I am doing wrong in Runnable, or elsewhere?
>
> Thanks,
>
> Tom
>
>
>
> package catshelter;
>
> import java.util.List;
> import java.util.ArrayList;
> import java.util.Random;
>
> import org.eclipse.jface.viewers.IStructuredContentProvider;
> import org.eclipse.jface.viewers.ITableLabelProvider;
> import org.eclipse.jface.viewers.LabelProvider;
> import org.eclipse.jface.viewers.TableViewer;
> import org.eclipse.jface.viewers.TableViewerColumn;
> import org.eclipse.jface.viewers.Viewer;
> import org.eclipse.rwt.graphics.Graphics;
> import org.eclipse.rwt.lifecycle.UICallBack;
> import org.eclipse.swt.SWT;
> import org.eclipse.swt.events.SelectionEvent;
> import org.eclipse.swt.events.SelectionListener;
> import org.eclipse.swt.graphics.Image;
> import org.eclipse.swt.layout.GridData;
> import org.eclipse.swt.layout.GridLayout;
> import org.eclipse.swt.widgets.Button;
> import org.eclipse.swt.widgets.Composite;
> import org.eclipse.swt.widgets.Table;
> import org.eclipse.swt.widgets.TableItem;
> import org.eclipse.ui.PlatformUI;
> import org.eclipse.ui.part.ViewPart;
>
> public class WatchlistSecurityView extends ViewPart {
>
> private String[] securities = { "FTSE 100 Cash", "Wall Street Cash",
> "FTSE to finish Up", "Germany 30", "Spot FX USD/JPY",
> "Spot FX EUR/USD", "Spot Gold ", "US Light Crude",
> "Barclays Plc",
> "Royal Dutch Shell Plc -A", "BNP Paribas", "UBS AG" };
>
> private String[] columns = { "Market", "Period", "Sell", "Buy",
> "Change",
> "% Chg", "Update", "High", "Low" };
> TableViewer viewer;
>
> @Override
> public void createPartControl(final Composite parent) {
>
> Composite contents = new Composite(parent, SWT.NONE);
> contents.setLayout(new GridLayout(1, false));
>
> viewer = new TableViewer(contents, SWT.MULTI | SWT.H_SCROLL
> | SWT.V_SCROLL | SWT.FULL_SELECTION);
> viewer.getTable().setLayoutData(new
> GridData(GridData.FILL_BOTH));
> viewer.setContentProvider(new SecurityListContentProvider());
>
> SecurityListLabelProvider lp = new SecurityListLabelProvider();
> lp.createColumns(viewer);
> viewer.setLabelProvider(lp);
> viewer.getTable().setLinesVisible(false);
> viewer.setInput(getViewSite());
> Button b = new Button(contents, SWT.BORDER);
> b.setText("Activate!");
>
> final Runnable runnable = new Runnable() {
> public void run() {
>
> // while(true){
> // if(Event.waitFor()){
> System.out.println("executing Runnable");
> /*
> * MessageDialog.openInformation( parent.getShell(),
> * "Treeviewer", "You selected " );
> */
> List buffer = new ArrayList();
>
> Random generator = new Random();
> for (int i = 0; i < securities.length; i++) {
> // TableItem item = new TableItem( viewer.getTable(),
> // SWT.NONE);
> TableItem item = (TableItem) viewer.getElementAt(i);
>
> item.setText(0, securities[i]);
> for (int j = 1; j < columns.length; j++) {
> item.setText(j,
> String.valueOf(generator.nextInt(100)));
> }
> item.setBackground( Graphics.getColor(0,0,128) );
>
> // TableItem item = new TableItem(viewer.getTable(),
> // SWT.NONE);
> // item.setText(0,"test");
> // viewer.add( new String[] {"item","item","item"} );
>
> // TableItem item =
> (TableItem)viewer.getElementAt(0);
> // item.setBackground( 2,
> Graphics.getColor(255,0,0) );
>
> // viewer.refresh();
> // }
> // }
> }
> }
> };
>
> b.addSelectionListener(new SelectionListener() {
> public void widgetDefaultSelected(SelectionEvent e) {
> }
>
> public void widgetSelected(SelectionEvent e) {
> System.out.println("Starting UICallback");
> // list.put(inputText.getText(),Status.INCOMPLETE);
> UICallBack.activate("notifierId");
> /*
> * notifierThread = new Thread(createRunnable());
> * notifierThread.setDaemon(true);
> notifierThread.start();
> */
> // shell.getDisplay().asyncExec(runnable);
> PlatformUI.createDisplay().asyncExec(runnable);
> UICallBack.deactivate("notifierId");
> }
> });
>
> }
>
> public class SecurityListContentProvider implements
> IStructuredContentProvider {
>
> @Override
> public Object[] getElements(Object inputElement) {
> Random generator = new Random();
> List rowList = new ArrayList();
>
> for (int i = 0; i < securities.length; i++) {
> String[] row = new String[securities.length];
> row[0] = securities[i];
> for (int j = 1; j < columns.length; j++) {
> row[j] = "" + String.valueOf(generator.nextInt(100));
> }
> rowList.add(row);
> }
>
> return rowList.toArray();
> }
>
> @Override
> public void dispose() {
> }
>
> @Override
> public void inputChanged(Viewer viewer, Object oldInput,
> Object newInput) {
> }
>
> }
>
> public class SecurityListLabelProvider extends LabelProvider
> implements
> ITableLabelProvider {
>
> public void createColumns(TableViewer viewer) {
> Table table = viewer.getTable();
>
> int[] bounds = { 100, 100, 100 };
>
> for (int i = 0; i < columns.length; i++) {
> TableViewerColumn column = new TableViewerColumn(viewer,
> SWT.NONE);
> column.getColumn().setText(columns[i]);
> column.getColumn().setWidth(100);
> column.getColumn().setResizable(true);
> }
> table.setHeaderVisible(true);
> table.setLinesVisible(true);
> }
>
> @Override
> public Image getColumnImage(Object element, int columnIndex) {
> Random generator = new Random();
> ClassLoader loader = getClass().getClassLoader();
>
> if (columnIndex == 0) {
> int myInt = generator.nextInt(100);
> if (myInt < 34) {
> return Graphics.getImage("icons/state_edit.png",
> loader);
> } else if (myInt > 66) {
> return Graphics.getImage("icons/state_closed.png",
> loader);
> } else {
> return Graphics.getImage("icons/state_deal.png",
> loader);
> }
> }
> return null;
> }
>
> @Override
> public String getColumnText(Object element, int columnIndex) {
> String[] str = (String[]) element;
> return str[columnIndex];
>
> }
>
> }
>
> @Override
> public void setFocus() {
> // TODO Auto-generated method stub
>
> }
>
> }
>
>

--------------050604090806000709080605
Content-Type: text/plain;
name="WatchlistSecurityView.java"
Content-Transfer-Encoding: base64
Content-Disposition: inline;
filename="WatchlistSecurityView.java"

aW1wb3J0IGphdmEudXRpbC5BcnJheUxpc3Q7DQppbXBvcnQgamF2YS51dGls Lkxpc3Q7DQpp
bXBvcnQgamF2YS51dGlsLlJhbmRvbTsNCg0KaW1wb3J0IG9yZy5lY2xpcHNl LmpmYWNlLnZp
ZXdlcnMuSVN0cnVjdHVyZWRDb250ZW50UHJvdmlkZXI7DQppbXBvcnQgb3Jn LmVjbGlwc2Uu
amZhY2Uudmlld2Vycy5JVGFibGVMYWJlbFByb3ZpZGVyOw0KaW1wb3J0IG9y Zy5lY2xpcHNl
LmpmYWNlLnZpZXdlcnMuTGFiZWxQcm92aWRlcjsNCmltcG9ydCBvcmcuZWNs aXBzZS5qZmFj
ZS52aWV3ZXJzLlRhYmxlVmlld2VyOw0KaW1wb3J0IG9yZy5lY2xpcHNlLmpm YWNlLnZpZXdl
cnMuVGFibGVWaWV3ZXJDb2x1bW47DQppbXBvcnQgb3JnLmVjbGlwc2UuamZh Y2Uudmlld2Vy
cy5WaWV3ZXI7DQppbXBvcnQgb3JnLmVjbGlwc2Uucnd0LmxpZmVjeWNsZS5V SUNhbGxCYWNr
Ow0KaW1wb3J0IG9yZy5lY2xpcHNlLnN3dC5TV1Q7DQppbXBvcnQgb3JnLmVj bGlwc2Uuc3d0
LmV2ZW50cy5TZWxlY3Rpb25FdmVudDsNCmltcG9ydCBvcmcuZWNsaXBzZS5z d3QuZXZlbnRz
LlNlbGVjdGlvbkxpc3RlbmVyOw0KaW1wb3J0IG9yZy5lY2xpcHNlLnN3dC5n cmFwaGljcy5J
bWFnZTsNCmltcG9ydCBvcmcuZWNsaXBzZS5zd3QubGF5b3V0LkdyaWREYXRh Ow0KaW1wb3J0
IG9yZy5lY2xpcHNlLnN3dC5sYXlvdXQuR3JpZExheW91dDsNCmltcG9ydCBv cmcuZWNsaXBz
ZS5zd3Qud2lkZ2V0cy5CdXR0b247DQppbXBvcnQgb3JnLmVjbGlwc2Uuc3d0 LndpZGdldHMu
Q29tcG9zaXRlOw0KaW1wb3J0IG9yZy5lY2xpcHNlLnN3dC53aWRnZXRzLkRp c3BsYXk7DQpp
bXBvcnQgb3JnLmVjbGlwc2Uuc3d0LndpZGdldHMuVGFibGU7DQppbXBvcnQg b3JnLmVjbGlw
c2Uuc3d0LndpZGdldHMuVGFibGVJdGVtOw0KaW1wb3J0IG9yZy5lY2xpcHNl LnVpLnBhcnQu
Vmlld1BhcnQ7DQoNCnB1YmxpYyBjbGFzcyBXYXRjaGxpc3RTZWN1cml0eVZp ZXcgZXh0ZW5k
cyBWaWV3UGFydCB7DQoNCiAgcHJpdmF0ZSBTdHJpbmdbXSBzZWN1cml0aWVz ID0gew0KICAg
ICJGVFNFIDEwMCBDYXNoIiwNCiAgICAiV2FsbCBTdHJlZXQgQ2FzaCIsDQog ICAgIkZUU0Ug
dG8gZmluaXNoIFVwIiwNCiAgICAiR2VybWFueSAzMCIsDQogICAgIlNwb3Qg RlggVVNEL0pQ
WSIsDQogICAgIlNwb3QgRlggRVVSL1VTRCIsDQogICAgIlNwb3QgR29sZCAg ICAiLA0KICAg
ICJVUyBMaWdodCBDcnVkZSIsDQogICAgIkJhcmNsYXlzIFBsYyIsDQogICAg IlJveWFsIER1
dGNoIFNoZWxsIFBsYyAtQSIsDQogICAgIkJOUCBQYXJpYmFzIiwNCiAgICAi VUJTIEFHIg0K
ICB9Ow0KICBwcml2YXRlIFN0cmluZ1tdIGNvbHVtbnMgPSB7DQogICAgIk1h cmtldCIsDQog
ICAgIlBlcmlvZCIsDQogICAgIlNlbGwiLA0KICAgICJCdXkiLA0KICAgICJD aGFuZ2UiLA0K
ICAgICIlIENoZyIsDQogICAgIlVwZGF0ZSIsDQogICAgIkhpZ2giLA0KICAg ICJMb3ciDQog
IH07DQogIFRhYmxlVmlld2VyIHZpZXdlcjsNCg0KICBAT3ZlcnJpZGUNCiAg cHVibGljIHZv
aWQgY3JlYXRlUGFydENvbnRyb2woIGZpbmFsIENvbXBvc2l0ZSBwYXJlbnQg KSB7DQogICAg
Q29tcG9zaXRlIGNvbnRlbnRzID0gbmV3IENvbXBvc2l0ZSggcGFyZW50LCBT V1QuTk9ORSAp
Ow0KICAgIGNvbnRlbnRzLnNldExheW91dCggbmV3IEdyaWRMYXlvdXQoIDEs IGZhbHNlICkg
KTsNCiAgICB2aWV3ZXIgPSBuZXcgVGFibGVWaWV3ZXIoIGNvbnRlbnRzLCBT V1QuTVVMVEkN
CiAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICB8IFNX VC5IX1NDUk9M
TA0KICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgIHwg U1dULlZfU0NS
T0xMDQogICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg fCBTV1QuRlVM
TF9TRUxFQ1RJT04gKTsNCiAgICBmaW5hbCBUYWJsZSB0YWJsZSA9IHZpZXdl ci5nZXRUYWJs
ZSgpOw0KICAgIHRhYmxlLnNldExheW91dERhdGEoIG5ldyBHcmlkRGF0YSgg R3JpZERhdGEu
RklMTF9CT1RIICkgKTsNCiAgICB2aWV3ZXIuc2V0Q29udGVudFByb3ZpZGVy KCBuZXcgU2Vj
dXJpdHlMaXN0Q29udGVudFByb3ZpZGVyKCkgKTsNCiAgICBTZWN1cml0eUxp c3RMYWJlbFBy
b3ZpZGVyIGxwID0gbmV3IFNlY3VyaXR5TGlzdExhYmVsUHJvdmlkZXIoKTsN CiAgICBscC5j
cmVhdGVDb2x1bW5zKCB2aWV3ZXIgKTsNCiAgICB2aWV3ZXIuc2V0TGFiZWxQ cm92aWRlcigg
bHAgKTsNCiAgICB2aWV3ZXIuc2V0SW5wdXQoIGdldFZpZXdTaXRlKCkgKTsN CiAgICB0YWJs
ZS5zZXRMaW5lc1Zpc2libGUoIGZhbHNlICk7DQogICAgQnV0dG9uIGIgPSBu ZXcgQnV0dG9u
KCBjb250ZW50cywgU1dULkJPUkRFUiApOw0KICAgIGIuc2V0VGV4dCggIkFj dGl2YXRlISIg
KTsNCiAgICBiLmFkZFNlbGVjdGlvbkxpc3RlbmVyKCBuZXcgU2VsZWN0aW9u TGlzdGVuZXIo
KSB7DQoNCiAgICAgIHB1YmxpYyB2b2lkIHdpZGdldERlZmF1bHRTZWxlY3Rl ZCggU2VsZWN0
aW9uRXZlbnQgZSApIHsNCiAgICAgIH0NCg0KICAgICAgcHVibGljIHZvaWQg d2lkZ2V0U2Vs
ZWN0ZWQoIFNlbGVjdGlvbkV2ZW50IGUgKSB7DQogICAgICAgIC8vIGFjdGl2 YXRlIFVJLWNh
bGxiYWNrIG1lY2hhbmlzbQ0KICAgICAgICBTeXN0ZW0ub3V0LnByaW50bG4o ICJTdGFydGlu
ZyBVSUNhbGxiYWNrIiApOw0KICAgICAgICBVSUNhbGxCYWNrLmFjdGl2YXRl KCBXYXRjaGxp
c3RTZWN1cml0eVZpZXcuY2xhc3MuZ2V0TmFtZSgpICk7DQogICAgICAgIC8v IGNyZWF0ZSBh
bmQgc3RhcnQgYmFja2dyb3VuZCB0aHJlYWQgdGhhdCB1cGRhdGVzDQogICAg ICAgIFRocmVh
ZCB0aHJlYWQgPSBuZXcgVGhyZWFkKCBjcmVhdGVSdW5uYWJsZSggdGFibGUg KSApOw0KICAg
ICAgICB0aHJlYWQuc2V0RGFlbW9uKCB0cnVlICk7DQogICAgICAgIHRocmVh ZC5zdGFydCgp
Ow0KICAgICAgfQ0KICAgIH0gKTsNCiAgfQ0KDQogIHByaXZhdGUgUnVubmFi bGUgY3JlYXRl
UnVubmFibGUoIGZpbmFsIFRhYmxlIHRhYmxlICkgew0KICAgIGZpbmFsIERp c3BsYXkgZGlz
cGxheSA9IHRhYmxlLmdldERpc3BsYXkoKTsNCiAgICBSdW5uYWJsZSByZXN1 bHQgPSBuZXcg
UnVubmFibGUoKSB7DQoNCiAgICAgIHB1YmxpYyB2b2lkIHJ1bigpIHsNCiAg ICAgICAgZm9y
KCBpbnQgaSA9IDA7IGkgPD0gMTA7IGkrKyApIHsNCiAgICAgICAgICBmaW5h bCBib29sZWFu
IGRlYWN0aXZhdGUgPSBpID09IDEwOw0KICAgICAgICAgIHRyeSB7DQogICAg ICAgICAgICAv
LyBzaW11bGF0ZSBzb21lIHdvcmsNCiAgICAgICAgICAgIFRocmVhZC5zbGVl cCggNTAwICk7
DQogICAgICAgICAgfSBjYXRjaCggZmluYWwgVGhyb3dhYmxlIHNob3VsZE5v dEhhcHBlbiAp
IHsNCiAgICAgICAgICAgIHNob3VsZE5vdEhhcHBlbi5wcmludFN0YWNrVHJh Y2UoKTsNCiAg
ICAgICAgICB9DQogICAgICAgICAgLy8gcGVyZm9ybSB1cGRhdGUNCiAgICAg ICAgICBkaXNw
bGF5LnN5bmNFeGVjKCBuZXcgUnVubmFibGUoKSB7DQoNCiAgICAgICAgICAg IHB1YmxpYyB2
b2lkIHJ1bigpIHsNCiAgICAgICAgICAgICAgU3lzdGVtLm91dC5wcmludGxu KCAiRXhlY3V0
aW5nIFJ1bm5hYmxlIiApOw0KICAgICAgICAgICAgICBSYW5kb20gZ2VuZXJh dG9yID0gbmV3
IFJhbmRvbSgpOw0KICAgICAgICAgICAgICBmb3IoIGludCBpID0gMDsgaSA8 IHNlY3VyaXRp
ZXMubGVuZ3RoOyBpKysgKSB7DQogICAgICAgICAgICAgICAgVGFibGVJdGVt IGl0ZW0gPSB0
YWJsZS5nZXRJdGVtKCBpICk7DQogICAgICAgICAgICAgICAgZm9yKCBpbnQg aiA9IDE7IGog
PCBjb2x1bW5zLmxlbmd0aDsgaisrICkgew0KICAgICAgICAgICAgICAgICAg aXRlbS5zZXRU
ZXh0KCBqLCBTdHJpbmcudmFsdWVPZiggZ2VuZXJhdG9yLm5leHRJbnQoIDEw MCApICkgKTsN
CiAgICAgICAgICAgICAgICB9DQogICAgICAgICAgICAgIH0NCiAgICAgICAg ICAgICAgaWYo
IGRlYWN0aXZhdGUgKSB7DQogICAgICAgICAgICAgICAgLy8gZGVhY3RpdmF0 ZSBVSS1jYWxs
YmFjayBtZWNoYW5pc20NCiAgICAgICAgICAgICAgICBTeXN0ZW0ub3V0LnBy aW50bG4oICJT
dG9wcGluZyBVSUNhbGxiYWNrIiApOw0KICAgICAgICAgICAgICAgIFN0cmlu ZyBpZCA9IFdh
dGNobGlzdFNlY3VyaXR5Vmlldy5jbGFzcy5nZXROYW1lKCk7DQogICAgICAg ICAgICAgICAg
VUlDYWxsQmFjay5kZWFjdGl2YXRlKCBpZCApOw0KICAgICAgICAgICAgICB9 DQogICAgICAg
ICAgICB9DQogICAgICAgICAgfSApOw0KICAgICAgICB9DQogICAgICB9DQog ICAgfTsNCiAg
ICByZXR1cm4gcmVzdWx0Ow0KICB9DQogIHB1YmxpYyBjbGFzcyBTZWN1cml0 eUxpc3RDb250
ZW50UHJvdmlkZXINCiAgICBpbXBsZW1lbnRzIElTdHJ1Y3R1cmVkQ29udGVu dFByb3ZpZGVy
DQogIHsNCg0KICAgIEBPdmVycmlkZQ0KICAgIHB1YmxpYyBPYmplY3RbXSBn ZXRFbGVtZW50
cyggT2JqZWN0IGlucHV0RWxlbWVudCApIHsNCiAgICAgIFJhbmRvbSBnZW5l cmF0b3IgPSBu
ZXcgUmFuZG9tKCk7DQogICAgICBMaXN0IHJvd0xpc3QgPSBuZXcgQXJyYXlM aXN0KCk7DQog
ICAgICBmb3IoIGludCBpID0gMDsgaSA8IHNlY3VyaXRpZXMubGVuZ3RoOyBp KysgKSB7DQog
ICAgICAgIFN0cmluZ1tdIHJvdyA9IG5ldyBTdHJpbmdbIHNlY3VyaXRpZXMu bGVuZ3RoIF07
DQogICAgICAgIHJvd1sgMCBdID0gc2VjdXJpdGllc1sgaSBdOw0KICAgICAg ICBmb3IoIGlu
dCBqID0gMTsgaiA8IGNvbHVtbnMubGVuZ3RoOyBqKysgKSB7DQogICAgICAg ICAgcm93WyBq
IF0gPSAiIiArIFN0cmluZy52YWx1ZU9mKCBnZW5lcmF0b3IubmV4dEludCgg MTAwICkgKTsN
CiAgICAgICAgfQ0KICAgICAgICByb3dMaXN0LmFkZCggcm93ICk7DQogICAg ICB9DQogICAg
ICByZXR1cm4gcm93TGlzdC50b0FycmF5KCk7DQogICAgfQ0KDQogICAgQE92 ZXJyaWRlDQog
ICAgcHVibGljIHZvaWQgZGlzcG9zZSgpIHsNCiAgICB9DQoNCiAgICBAT3Zl cnJpZGUNCiAg
ICBwdWJsaWMgdm9pZCBpbnB1dENoYW5nZWQoIFZpZXdlciB2aWV3ZXIsIE9i amVjdCBvbGRJ
bnB1dCwgT2JqZWN0IG5ld0lucHV0ICkNCiAgICB7DQogICAgfQ0KICB9DQog IHB1YmxpYyBj
bGFzcyBTZWN1cml0eUxpc3RMYWJlbFByb3ZpZGVyIGV4dGVuZHMgTGFiZWxQ cm92aWRlcg0K
ICAgIGltcGxlbWVudHMgSVRhYmxlTGFiZWxQcm92aWRlcg0KICB7DQoNCiAg ICBwdWJsaWMg
dm9pZCBjcmVhdGVDb2x1bW5zKCBUYWJsZVZpZXdlciB2aWV3ZXIgKSB7DQog ICAgICBUYWJs
ZSB0YWJsZSA9IHZpZXdlci5nZXRUYWJsZSgpOw0KICAgICAgaW50W10gYm91 bmRzID0gew0K
ICAgICAgICAxMDAsIDEwMCwgMTAwDQogICAgICB9Ow0KICAgICAgZm9yKCBp bnQgaSA9IDA7
IGkgPCBjb2x1bW5zLmxlbmd0aDsgaSsrICkgew0KICAgICAgICBUYWJsZVZp ZXdlckNvbHVt
biBjb2x1bW4gPSBuZXcgVGFibGVWaWV3ZXJDb2x1bW4oIHZpZXdlciwgU1dU Lk5PTkUgKTsN
CiAgICAgICAgY29sdW1uLmdldENvbHVtbigpLnNldFRleHQoIGNvbHVtbnNb IGkgXSApOw0K
ICAgICAgICBjb2x1bW4uZ2V0Q29sdW1uKCkuc2V0V2lkdGgoIDEwMCApOw0K ICAgICAgICBj
b2x1bW4uZ2V0Q29sdW1uKCkuc2V0UmVzaXphYmxlKCB0cnVlICk7DQogICAg ICB9DQogICAg
ICB0YWJsZS5zZXRIZWFkZXJWaXNpYmxlKCB0cnVlICk7DQogICAgICB0YWJs ZS5zZXRMaW5l
c1Zpc2libGUoIHRydWUgKTsNCiAgICB9DQoNCiAgICBAT3ZlcnJpZGUNCiAg ICBwdWJsaWMg
SW1hZ2UgZ2V0Q29sdW1uSW1hZ2UoIE9iamVjdCBlbGVtZW50LCBpbnQgY29s dW1uSW5kZXgg
KSB7DQogICAgICBSYW5kb20gZ2VuZXJhdG9yID0gbmV3IFJhbmRvbSgpOw0K ICAgICAgQ2xh
c3NMb2FkZXIgbG9hZGVyID0gZ2V0Q2xhc3MoKS5nZXRDbGFzc0xvYWRlcigp Ow0KICAgICAg
aWYoIGNvbHVtbkluZGV4ID09IDAgKSB7DQogICAgICAgIGludCBteUludCA9 IGdlbmVyYXRv
ci5uZXh0SW50KCAxMDAgKTsNCiAgICAgICAgLy8gaWYoIG15SW50IDwgMzQg KSB7DQogICAg
ICAgIC8vIHJldHVybiBHcmFwaGljcy5nZXRJbWFnZSggImljb25zL3N0YXRl X2VkaXQucG5n
IiwgbG9hZGVyICk7DQogICAgICAgIC8vIH0gZWxzZSBpZiggbXlJbnQgPiA2 NiApIHsNCiAg
ICAgICAgLy8gcmV0dXJuIEdyYXBoaWNzLmdldEltYWdlKCAiaWNvbnMvc3Rh dGVfY2xvc2Vk
LnBuZyIsIGxvYWRlciApOw0KICAgICAgICAvLyB9IGVsc2Ugew0KICAgICAg ICAvLyByZXR1
cm4gR3JhcGhpY3MuZ2V0SW1hZ2UoICJpY29ucy9zdGF0ZV9kZWFsLnBuZyIs IGxvYWRlciAp
Ow0KICAgICAgICAvLyB9DQogICAgICB9DQogICAgICByZXR1cm4gbnVsbDsN CiAgICB9DQoN
CiAgICBAT3ZlcnJpZGUNCiAgICBwdWJsaWMgU3RyaW5nIGdldENvbHVtblRl eHQoIE9iamVj
dCBlbGVtZW50LCBpbnQgY29sdW1uSW5kZXggKSB7DQogICAgICBTdHJpbmdb XSBzdHIgPSAo
IFN0cmluZ1tdICllbGVtZW50Ow0KICAgICAgcmV0dXJuIHN0clsgY29sdW1u SW5kZXggXTsN
CiAgICB9DQogIH0NCg0KICBAT3ZlcnJpZGUNCiAgcHVibGljIHZvaWQgc2V0 Rm9jdXMoKSB7
DQogICAgLy8gVE9ETyBBdXRvLWdlbmVyYXRlZCBtZXRob2Qgc3R1Yg0KICB9 DQp9
--------------050604090806000709080605--

Report message to a moderator

Re: insight required on changing values in TableViewer with asyncExec [message #118167 is a reply to message #117962]

Sat, 10 January 2009 02:59

Tom H

Messages: 139
Registered: July 2009

Senior Member

Hey Ivan,

Thanks for your brilliant effort, I've got it updating just how I need
from your example.

However I am now trying to set a column colour to the initial table, ie I
want to set the buy and sell column to blue and red, but using a
getElements that returns an Array of Arrays of Strings;
String[] row = new String[securities.length];
etc, means that I can't set TableItem properties to it.
Do you have further insight on how to initialise a table, and
setForeground on it?
I've pasted the current code below. So far I can't see how to create a
TableItem, without providing a parent, which the TableItem immediately
gets appended to....?

Many Thanks,

Tom

public Object[] getElements(Object inputElement) {
Random generator = new Random();
List rowList = new ArrayList();
for (int i = 0; i < securities.length; i++) {
String[] row = new String[securities.length];
row[0] = securities[i];
for (int j = 1; j < columns.length; j++) {
row[j] = "" + String.valueOf(generator.nextInt(100));
}
rowList.add(row);
}
return rowList.toArray();
}

Report message to a moderator

Re: insight required on changing values in TableViewer with asyncExec [message #118251 is a reply to message #118167]

Mon, 12 January 2009 09:32

Benjamin Muskalla

Messages: 237
Registered: July 2009

Senior Member

Hi Tom,

have you ever tried to implement a decorating label provider? Like
IColorLabelProvider or ITableColorProvider. I think this is the
recommended way as JFace tries to separate your model from the concrete
widget. Modifying the widgets directly can lead to unexpected results as
JFace replaces the widgets under certain circumstances (like refresh())
and does not preserve the changes you've made.

Greets
Benny

Tom H wrote:
> Hey Ivan,
>
> Thanks for your brilliant effort, I've got it updating just how I need
> from your example.
> However I am now trying to set a column colour to the initial table, ie
> I want to set the buy and sell column to blue and red, but using a
> getElements that returns an Array of Arrays of Strings;
> String[] row = new String[securities.length];
> etc, means that I can't set TableItem properties to it.
> Do you have further insight on how to initialise a table, and
> setForeground on it?
> I've pasted the current code below. So far I can't see how to create a
> TableItem, without providing a parent, which the TableItem immediately
> gets appended to....?
>
> Many Thanks,
>
> Tom
>
> public Object[] getElements(Object inputElement) {
> Random generator = new Random();
> List rowList = new ArrayList();
> for (int i = 0; i < securities.length; i++) {
> String[] row = new String[securities.length];
> row[0] = securities[i];
> for (int j = 1; j < columns.length; j++) {
> row[j] = "" + String.valueOf(generator.nextInt(100));
> }
> rowList.add(row);
> }
> return rowList.toArray();
> }
>

Report message to a moderator