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

• Informador: (Anónimo)
• Propietario: tohosaku
• Tipo: リファクタリング
• Estado: Cerrado
• Componente: (Ninguno)
• Hito: release0.2.10 (cerrado)
• Prioridad: 3
• Gravedad: 3
• Resolución: Accepted
• Fichero: Ninguno

Details Responder

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

• Tipo Update from Bugs to リファクタリング
• Prioridad Update from 5 - Medium to 3

Comentario Responder

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

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

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

• Propietario Update from (Ninguno) to tohosaku
• Resolución Update from Ninguno to Accepted
• Hito Update from (Ninguno) to release0.2.10 (cerrado)

Comentario Responder

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

• Estado Update from Open to Cerrado
• Ticket Close date is changed to 2014-12-23 12:40

Comentario Responder

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

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

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