谷本 心 in せろ部屋 このページをアンテナに追加 RSSフィード

2007-02-15

[]色々言ってますが・・・

えーっと、S2JFaceのコミッタにしてもらないでしょうか? (^^ >id:y-komoriさん

あれこれ触っているうちに、自分でやりたくなってきました。


非同期処理関連と、Eclipseプラグイン(ビューワ)あたりを

担当して、やっていきたいと思います。

2007-02-06

[]Re: S2JFaceのいけてないところ

まぁ、書きっぱなしだと何なので。

非同期処理が弱いトコロは、Interceptorで解決できるかな、と思います。


Actionメソッドの実行(invoke)前後に

S2JFaceが行なっている処理(ウィジットバインディングとかExportValueとか)を、

いっそまるごとInterceptor化してしまう。


そうすれば、Actionを作成する開発者は、

メソッドがS2JFaceから呼び出されるのか、

別クラス(別スレッド)から呼び出されるのか、

意識しない実装が出来るわけですよね。


妄想的には、開発者はこんな感じのコードを書くだけで良い。

@EventListener(id = { "menuSearch", "toolSearch" })
public void searchEmployee() {
    List<EmployeeDto> result = (List<EmployeeDto>) windowManager
            .openModal("org/seasar/jface/example/employee/search.xml");
    if (result != null) {
        employees = result;
    }
}

@AsyncEventListener
public void refreshEmployee(List<EmployeeDto> list) {
    employees = list;
}

AsyncEventInterceptorみたいなのを作成して、

既存のS2JFaceのクラスを呼び出して処理すれば良いのではないでしょうか。


[]プラグインを作ってみました。

http://hatena.tanimoto.ninja/s2jface/s2-jface-plugin-0.1.0.zip

こないだスクリーンショットを掲載してたやつです。

ひとまず動くようになったので、公開してみます。

  • .xmlファイルのエディタとして登録されます
  • 画像などのリソースには対応していません
  • XMLがNGな場合の挙動は確認してません
  • ボタンのイベントなんかは、きっと動きません

てか、100行ぐらいのコードで出来たプラグインなんで

まともに動くかどうかも怪しいです (^^;;

2007-02-05

[]S2JFaceのいけてないところ

最近、id:y-komoriさんのS2JFaceを使ってるんですが、

うーん、S2JSFばかりやってた自分から見て、

分かりづらい、びみょーなトコロが、ちらほら。


なので、こんなタイトルで、煽らせてもらいます :)


特にDTOがひどす!

public class EmployeeDto {

    private static final long serialVersionUID = 1L;

    @BindingLabel(column = 0)
    private Integer empno;

    @BindingLabel(column = 1)
    private String ename;

    @BindingLabel(column = 2)
    private String job;

column番号がつくのは、TableViewerのせいでやむをえないかも知れないけど、

XML側でColumにid振ってるんだから、それを参照して欲しいですね。


あとは、@EventListenerおおすぎ

id = メソッド名の時は、力強くアノテーション不要にして構わないでしょう。

# もしくはonXxx(on + ID)で決めウチしてしまう?


  • GUIなのに、非同期処理が弱す!

描画スレッドとは別スレッドで処理を行なって、

その結果を描画したい時とか、ちょいテクニカルな事をしなきゃいけません。


特に、別スレッド

S2Container#getComponentでActionを取得しても

ウィジットがバインドされていないせいで、

Display#asyncExceも、やりにくい。


@AsyncEventListenerなんかのアノテーションで、

スレッドからの処理を受け付けられれば最高!

# さすがに、実現方法は見えませんけど (^^;;

# バインドしている主体をInterceptorにすれば、何とかなり、、、ませんかね。


ライバル達は、この辺り、どうやってるのか、

ちょっと調べてみますかね。

2007-01-18

[]S2JFaceを始めるために、Eclipseプラグインを作ってみる

とりあえず、何にでも手を出すタイプなので、、、。


というわけで、S2JFaceを初めました。

、、、が、画面のXMLを作ろうと思った時に、ふと手が止まりました。


いや、僕、SWTの文法、あんまり覚えてないんですよ (^^;

今のS2JFaceだと、SWTを知ってないと、なかなかスラスラとは書けない。


SWTを知らなくて困るのは、きっと僕だけないと思うので、

まずはプレビューを可能にするようなEclipseプラグインを作り始めました。

f:id:cero-t:20070119022905g:image:w400

おっ、出てんじゃーん。


とりあえず、MultiPageEditorのページ追加する所で

↓こんなソースを書いたら、S2JFaceの画面が出ました。

protected void createPage1() {
	Composite composite = new Composite(getContainer(), SWT.NONE);
	FillLayout layout = new FillLayout();
	composite.setLayout(layout);

	ComponentTreeBuilder builder = new ComponentTreeBuilder();
	Template template = builder.build("test.xml");

	WindowContext context = new WindowContextImpl();
	context.putComponent(WindowContext.SHELL_ID, composite.getShell());

	WindowComponent windowComponent = template.getWindowComponent();
	windowComponent.render(composite, context);

	int index = addPage(composite);
	setPageText(index, "Preview");
}

はい、見ての通り、画面ファイル名 (test.xml) は、ソースに直書きです。


動的にファイルを読み込めるようにしようと思ったら、コケました。

もうちょっとS2JFaceの中身を読んでみます。


追記。

ComponentTreeBuilderをちょっと修正して、

InputStreamの入力を受け付けられるようにしたところ、、、ダメでした。

org.seasar.jface.exception.DuplicateComponentIdException: [EJFC0106]コンポーネントID labelRendererTest は重複して定義されています.

同じ画面(xml)に対してWindowComponent#renderを2回呼ぶと、上記のエラーが発生してしまいます。

render以外に、リフレッシュする方法は、あるのかな?



追記。

スタックトレースです。

org.seasar.jface.exception.DuplicateComponentIdException: [EJFC0106]コンポーネントID labelRendererTest は重複して定義されています.

at org.seasar.jface.impl.WindowContextImpl.putComponent(WindowContextImpl.java:117)

at org.seasar.jface.component.impl.AbstractUIComponent.render(AbstractUIComponent.java:103)

at org.seasar.jface.component.impl.WindowComponent.render(WindowComponent.java:67)

at org.seasar.s2jface.plugin.editors.S2JFaceEditor.createPage1(S2JFaceEditor.java:92)

at org.seasar.s2jface.plugin.editors.S2JFaceEditor.createPages(S2JFaceEditor.java:104)

at org.eclipse.ui.part.MultiPageEditorPart.createPartControl(MultiPageEditorPart.java:276)

at org.eclipse.ui.internal.EditorReference.createPartHelper(EditorReference.java:596)

at org.eclipse.ui.internal.EditorReference.createPart(EditorReference.java:372)

at org.eclipse.ui.internal.WorkbenchPartReference.getPart(WorkbenchPartReference.java:566)

at org.eclipse.ui.internal.EditorReference.getEditor(EditorReference.java:214)

at org.eclipse.ui.internal.WorkbenchPage.busyOpenEditorBatched(WorkbenchPage.java:2595)

at org.eclipse.ui.internal.WorkbenchPage.busyOpenEditor(WorkbenchPage.java:2528)

at org.eclipse.ui.internal.WorkbenchPage.access$10(WorkbenchPage.java:2520)

at org.eclipse.ui.internal.WorkbenchPage$9.run(WorkbenchPage.java:2505)

at org.eclipse.swt.custom.BusyIndicator.showWhile(BusyIndicator.java:67)

at org.eclipse.ui.internal.WorkbenchPage.openEditor(WorkbenchPage.java:2500)

at org.eclipse.ui.internal.WorkbenchPage.openEditor(WorkbenchPage.java:2485)

at org.eclipse.ui.ide.IDE.openEditor(IDE.java:388)

at org.eclipse.ui.ide.IDE.openEditor(IDE.java:350)

at org.eclipse.jdt.internal.ui.javaeditor.EditorUtility.openInEditor(EditorUtility.java:275)

at org.eclipse.jdt.internal.ui.javaeditor.EditorUtility.openInEditor(EditorUtility.java:139)

at org.eclipse.jdt.internal.ui.actions.OpenActionUtil.open(OpenActionUtil.java:49)

at org.eclipse.jdt.ui.actions.OpenAction.run(OpenAction.java:190)

at org.eclipse.jdt.ui.actions.OpenAction.run(OpenAction.java:174)

at org.eclipse.jdt.ui.actions.SelectionDispatchAction.dispatchRun(SelectionDispatchAction.java:267)

at org.eclipse.jdt.ui.actions.SelectionDispatchAction.run(SelectionDispatchAction.java:243)

at org.eclipse.jdt.internal.ui.packageview.PackageExplorerActionGroup.handleOpen(PackageExplorerActionGroup.java:306)

at org.eclipse.jdt.internal.ui.packageview.PackageExplorerPart$4.open(PackageExplorerPart.java:653)

at org.eclipse.jface.viewers.StructuredViewer$2.run(StructuredViewer.java:817)

at org.eclipse.core.runtime.SafeRunner.run(SafeRunner.java:37)

at org.eclipse.core.runtime.Platform.run(Platform.java:843)

at org.eclipse.ui.internal.JFaceUtil$1.run(JFaceUtil.java:44)

at org.eclipse.jface.util.SafeRunnable.run(SafeRunnable.java:149)

at org.eclipse.jface.viewers.StructuredViewer.fireOpen(StructuredViewer.java:815)

at org.eclipse.jface.viewers.StructuredViewer.handleOpen(StructuredViewer.java:1069)

at org.eclipse.jface.viewers.StructuredViewer$6.handleOpen(StructuredViewer.java:1168)

at org.eclipse.jface.util.OpenStrategy.fireOpenEvent(OpenStrategy.java:249)

at org.eclipse.jface.util.OpenStrategy.access$2(OpenStrategy.java:243)

at org.eclipse.jface.util.OpenStrategy$1.handleEvent(OpenStrategy.java:283)

at org.eclipse.swt.widgets.EventTable.sendEvent(EventTable.java:66)

at org.eclipse.swt.widgets.Widget.sendEvent(Widget.java:928)

at org.eclipse.swt.widgets.Display.runDeferredEvents(Display.java:3348)

at org.eclipse.swt.widgets.Display.readAndDispatch(Display.java:2968)

at org.eclipse.ui.internal.Workbench.runEventLoop(Workbench.java:1914)

at org.eclipse.ui.internal.Workbench.runUI(Workbench.java:1878)

at org.eclipse.ui.internal.Workbench.createAndRunWorkbench(Workbench.java:419)

at org.eclipse.ui.PlatformUI.createAndRunWorkbench(PlatformUI.java:149)

at org.eclipse.ui.internal.ide.IDEApplication.run(IDEApplication.java:95)

at org.eclipse.core.internal.runtime.PlatformActivator$1.run(PlatformActivator.java:78)

at org.eclipse.core.runtime.internal.adaptor.EclipseAppLauncher.runApplication(EclipseAppLauncher.java:92)

at org.eclipse.core.runtime.internal.adaptor.EclipseAppLauncher.start(EclipseAppLauncher.java:68)

at org.eclipse.core.runtime.adaptor.EclipseStarter.run(EclipseStarter.java:400)

at org.eclipse.core.runtime.adaptor.EclipseStarter.run(EclipseStarter.java:177)

at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)

at sun.reflect.NativeMethodAccessorImpl.invoke(Unknown Source)

at sun.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source)

at java.lang.reflect.Method.invoke(Unknown Source)

at org.eclipse.core.launcher.Main.invokeFramework(Main.java:336)

at org.eclipse.core.launcher.Main.basicRun(Main.java:280)

at org.eclipse.core.launcher.Main.run(Main.java:977)

at org.eclipse.core.launcher.Main.main(Main.java:952)