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

• Rapporteur: (Anonyme)
• Propriétaire: tohosaku
• Type: リファクタリング
• État: Atteints
• Composant: (Aucun)
• Jalon: release0.2.10 (atteints)
• Priorité: 3
• Sévérité: 3
• Résolution: Accepted
• Fichier: Aucun

Détails Répondre

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

• Type Update from Bogues to リファクタリング
• Priorité Update from 5 - moyen to 3

Commentaire Répondre

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

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

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

• Propriétaire Update from (Aucun) to tohosaku
• Résolution Update from Aucun to Accepted
• Jalon Update from (Aucun) to release0.2.10 (atteints)

Commentaire Répondre

リファクタリングありがとうございます。類似したコードが多いことを気にしていたのでとてもありがたいです。 特に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

• État Update from Ouvert to Atteints
• Ticket Close date is changed to 2014-12-23 12:40

Commentaire Répondre

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

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

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