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

• 報告者: (匿名)
• 担当者: tohosaku
• チケットの種類: リファクタリング
• 状況: 完了
• コンポーネント: (未割り当て)
• マイルストーン: release0.2.10 (完了済み)
• 優先度: 3
• 重要度: 3
• 解決法: 受領
• ファイル: なし

詳細 返信

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

チケットの履歴 (4 件中 3 件表示)

2014-12-20 14:49 更新者: None

• 新しいチケット "editpart などで ジェネリックを使用。" が作成されました

2014-12-20 14:55 更新者: tohosaku

• チケットの種類 が バグ から リファクタリング に更新されました
• 優先度 が 5 - 中 から 3 に更新されました

コメント 返信

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

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

2014-12-21 16:08 更新者: nakag

• 担当者 が (未割り当て) から tohosaku に更新されました
• 解決法 が なし から 受領 に更新されました
• マイルストーン が (未割り当て) から release0.2.10 (完了済み) に更新されました

コメント 返信

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

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

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

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

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

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

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

2014-12-23 12:40 更新者: tohosaku

• 状況 が オープン から 完了 に更新されました
• チケット完了時刻 が 2014-12-23 12:40 に更新されました

コメント 返信

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

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

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