[Seasar-user:7033] Re: [Teeda]Gridの幅調整スクリプトが出力されない

2007年 4月 3日 (火) 16:08:08 JST

　Asarimaです。

　ありがとうございます。助かります。

　ただ、S2.4.9+Teeda 1.0.4からS2.4.11+SNAPSHOTに変更するとTeedaの
起動時にエラーが発生するようになってしまい、確認することができません
でした。1.0.5以降の仕様面で何か対応する必要があるようですね。

　SNAPSHOTのソースを拝見しましたところ、ComponentAttributesMapを変更
されているようでしたので、現プロジェクト（1.0.4環境）において優先的に
参照されるように新しいComponentAttributesMap＋InternalConstants（必要
だったので）の２クラスを追加して確認したところ、問題なく動作しました。

　そういうわけで、1.0.4ではＯＫですがSNAPSHOTは確認できないです。
　申し訳ありません。

>大谷です。
>
>調べた結果、Asarimaさんのフィードバックでおそらく問題ないです。
>ありがとうございます。
>
>この対応を盛り込んだSNAPSHOTを作成してみました。
>お手数ですが、下記URLのjarで一旦動かしてみていただけないでしょうか。
>（Seasar2は2.4.11が必要です。）
>宜しくお願いします。
>
>
>http://maven.seasar.org/maven2-snapshot/org/seasar/teeda/teeda-ajax/1.0.7-
>SNAPSHOT/teeda-ajax-1.0.7-20070403.051906-1.jar
>
>http://maven.seasar.org/maven2-snapshot/org/seasar/teeda/teeda-core/1.0.7-
>SNAPSHOT/teeda-core-1.0.7-20070403.051906-1.jar
>
>http://maven.seasar.org/maven2-snapshot/org/seasar/teeda/teeda-extension/1.0.
>7-SNAPSHOT/teeda-extension-1.0.7-20070403.051906-1.jar
>
>
>07/04/03 に Shinpei Ohtani<[E-MAIL ADDRESS DELETED]> さんは書きました:
>> 大谷です。
>> 返答おそくなり、すいません。
>>
>> こちらでも確認してみます。
>> 宜しくお願いします。
>>
>> 07/04/03 に Asarima<[E-MAIL ADDRESS DELETED]> さんは書きました:
>> > Asarimaです。
>> >
>> > 下記の件、どなたかお分かりになる方はいらっしゃいませんか。
>> >
>> > 同一ページのSerializedViewからUIBodyに復元されるAttributesMapが同一
>> > インスタンスではないか、それがUIBodyのAttributesMapの分離性を欠いてい
>> > ないでしょうか。
>> >
>> > 複数のGridをもつ同一ページに対し、複数クライアントから同時にリクエス
>> > トを処理したとき、UIBodyのencodeBegin～encodeEndまでの処理が重なると、
>> > RenderListenerが保持するListenerの数がおかしくなることはデバッグで確認
>> > しています。
>> >
>> > 例：３つのGridを持つページへの２つの同時リクエストを処理するスレッドＡ，
>> > Ｂが、UIBodyのメソッドを以下の順番で呼び出したとすると、先のリクエ
>> > ストに対してスクリプトが５つ、後に対して１つ出力される。
>> > （本来は３つずつ出力されなければならない）
>> >
>> > （Ａ）encodeBegin
>> > （Ａ）addRenderListener → RenderListenerの数: 1
>> > （Ｂ）encodeBegin
>> > （Ａ）addRenderListener → RenderListenerの数: 2
>> > （Ｂ）addRenderListener → RenderListenerの数: 3
>> > （Ａ）addRenderListener → RenderListenerの数: 4
>> > （Ｂ）addRenderListener → RenderListenerの数: 5
>> > （Ａ）encodeEnd         → Ａ出力 → RenderListenerの数: 0
>> > （Ｂ）addRenderListener → RenderListenerの数: 1
>> > （Ｂ）encodeEnd         → Ｂ出力
>> >
>> > それで、AttributeMapのシャローコピーを復元するようにしたら問題が回避
>> > されることを確認したのですが、この変更で他の問題が発生しないでしょうか。
>> >
>> > 以上です、よろしくお願いいたします。
>> >
>> > >Asarimaです。
>> > >
>> > >　弊社の開発メンバーが以下の記事の通り困っています。
>> > >　私のほうで何とか原因を突き止め、ソースを修正することで対処できました
>> > >（とはいえ、まだ確認中ですが）のでお知らせします。
>> > >　しかし本当にこの変更でよいのかが心配です。このような変更に問題がない
>> > >か、ご意見を頂戴したく思います。よろしくお願いします。
>> > >
>> > >■[Teeda] Gridの幅を調整するするスクリプトが出力されない？
>> > >http://d.hatena.ne.jp/kazita/20070319/1174287422
>> > >
>> > >【不具合の内容】
>> > >
>> > >　Teeda 1.0.4を使用しています。
>> > >　複数のGridを表示する画面があるとします。
>> > >　複数のクライアントからその画面に対して同時にリクエストを実行すると、
>> > >HTMLの最後に出力されるGridの幅調整用のスクリプトが、一部出力されなか
>> > >ったり、余分に出力される不具合が発生します。
>> > >
>> > >【原因の発見と対処】
>> > >
>> > >　Eclipseのデバッグモードを活用しました。
>> > >
>> > >１．THtmlGridRendererのencodeHtmlGridBeginメソッドに入ったところで
>> > >ブレークポイントを仕掛け、２つのクライアントから同時にリクエストを
>> > >実行します。２つのスレッドがブレークポイントで止まるので、交互に再開
>> > >していくと、不具合が確実に再現しました。
>> > >
>> > >２．この状態でTBodyRendererのaddRendererListenerメソッドにブレーク
>> > >ポイントを仕掛けると、２つのスレッドで、２つ目のGridからlistenersが
>> > >同じインスタンスを指しました。
>> > >
>> > >３．listenersを保持している各UIBodyのAttributesの本体である
>> > >UIComponentBaseのフィールドattributesMapが、２つのスレッドで同じ
>> > >インスタンスを指しました。
>> > >
>> > >４．UIComponentBaseのrestoreAttributeMapメソッドで、２つのスレッドで
>> > >同じstateインスタンスの参照がComponentAttributeMapのコンストラクタに
>> > >渡されました。
>> > >
>> > >５．TeedaStateManagerImplのrestoreComponentStateFromServerメソッド
>> > >で、２つのスレッドで同じSerializedViewインスタンスのstateがViewRootに
>> > >復元されます。
>> > >
>> > >６．私は４．に着目し、stateを復元するときにstateのシャローコピーを
>> > >渡せばAttributesが同じインスタンスを指さなくなると考え、UIComponentBase
>> > >のソースを以下のように変更しました。
>> > >
>> > >---
>> > >private void restoreAttributeMap(Object state) {
>> > >    if (state != null) {
>> > >//↓
>> > >//        attributesMap = new ComponentAttributesMap(this, (Map) state);
>> > >          HashMap map = new HashMap();
>> > >          map.putAll((Map)state);
>> > >          attributesMap = new ComponentAttributesMap(this, map);
>> > >//↑
>> > >    } else {
>> > >        clearAttributeMap();
>> > >    }
>> > >}
>> > >---
>> > >
>> > >　これでこの問題は発生しなくなりました。
>> > >
>> > >【考察】
>> > >
>> > >　複数のクライアントから同じ画面に対する要求が同時に起こった場合、コン
>> > >ポーネントツリーにSerializedViewの状態が復元されるようですが、復元される
>> > >オブジェクトのうちAttributeMapについては参照がコピーされます。
>> > >　これでBODYコンポーネントのAttributeMapに対してRenderListenerが同時に
>> > >登録されると、後から登録した方で上書きされるため、上記の不具合が起こると
>> > >考えます。これはGridに限らないとも思われます。
>> > >
>> > >　以上です。よろしくお願いいたします。