editpart などで ジェネリックを使用。

• 回報者: (匿名)
• 負責人: tohosaku
• 類型: リファクタリング
• 狀態: 關閉
• 元件: (無)
• 里程碑: release0.2.10 (closed)
• 優先權: 3
• 嚴重程度: 3
• 處理結果: Accepted
• 檔案: 無

細節 回覆

editpart などで、model と controler が 1:1 の関係にあるクラスがたくさんある場所では、ジェネリックを使えばキャストの削減やメソッドの共通化に役立つ(かもしれない)

Ticket History (3/4 Histories)

2014-12-20 14:49 Updated by: None

• New Ticket "editpart などで ジェネリックを使用。" created

2014-12-20 14:55 Updated by: tohosaku

• 類型 Update from 問題回報 to リファクタリング
• 優先權 Update from 5 - 中 to 3

評語 回覆

tohosaku です。すいません。ログアウトしたままで投稿してしまいました。

あわせて一部、EditPart にからむところをジェネリック化したブランチを push してみました。

2014-12-21 16:08 Updated by: nakag

• 負責人 Update from (無) to tohosaku
• 處理結果 Update from 無 to Accepted
• 里程碑 Update from (無) to release0.2.10 (closed)

評語 回覆

リファクタリングありがとうございます。類似したコードが多いことを気にしていたのでとてもありがたいです。 特にEditPartFactoryのVisitorパターン化はかなり良さげですね。他のコードにも流用できそうな気がして大変勉強になりました。

以下の点を見直して頂いた後でdevelopにmergeしていただければ助かります。

• 新規作成のソースコードの頭にApacheライセンスとauthorをつける形式で統一願います（IVisitorとAbstractEntityModelEditPartだけ？）。別途全プロジェクトにテンプレート設定しておきます。

• DiagramEditPartのimport文から未使用のRulerProviderを削除願います（すみません私が埋め込んだかも）。

• TMDEditPartVisitorのpartフィールドはprivateでも良いかもという点ご確認願います。

githubみたいにsf.jpもコミットに対してコメント付けられたら良かったのですが。。。

お手数をお掛けいたしますがよろしくお願いいたします。

2014-12-23 12:40 Updated by: tohosaku

• 狀態 Update from 開啟 to 關閉
• Ticket Close date is changed to 2014-12-23 12:40

評語 回覆

受理、ありがとうございます。

修正してコミットしました。結構、前に書いたコードでリポジトリの肥やしになっていたので統合できてうれしいです。

テンプレートはありがたいです。よろしくお願いします。