artemis
  •  
Options
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
火曜日 17 1
11:03 pm

都元ダイスケ

[maven-release-plugin] copy for tag artemis-0.3.7
[maven-release-plugin] copy for tag artemis-0.3.7
Options
  1. /tags/artemis-0.3.7
金曜日 13 1
2:17 pm

都元ダイスケ

[maven-release-plugin] copy for tag artemis-0.3.6
[maven-release-plugin] copy for tag artemis-0.3.6
Options
  1. /tags/artemis-0.3.6
日曜日 30 10 2011
5:56 pm

都元ダイスケ

[maven-release-plugin] copy for tag artemis-0.3.5
[maven-release-plugin] copy for tag artemis-0.3.5
Options
  1. /tags/artemis-0.3.5
1:40 pm

都元ダイスケ

[maven-release-plugin] copy for tag artemis-0.3.4
[maven-release-plugin] copy for tag artemis-0.3.4
Options
  1. /tags/artemis-0.3.4
12:16 pm

都元ダイスケ

[maven-release-plugin] copy for tag artemis-0.3.3
[maven-release-plugin] copy for tag artemis-0.3.3
Options
  1. /tags/artemis-0.3.3
木曜日 06 10 2011
11:22 am

都元ダイスケ

[maven-release-plugin] copy for tag artemis-0.3.2
[maven-release-plugin] copy for tag artemis-0.3.2
Options
  1. /tags/artemis-0.3.2
  2. /tags/artemis-0.3.2/jiemamy-test-helper
  3. /tags/artemis-0.3.2/jiemamy-diagram
  4. /tags/artemis-0.3.2/jiemamy-core
  5. /tags/artemis-0.3.2/jiemamy-utils
  6. /tags/artemis-0.3.2/jiemamy-artemis-parent
  7. /tags/artemis-0.3.2/jiemamy-diff
  8. /tags/artemis-0.3.2/jiemamy-sql
    • -0
    • +35
    /tags/artemis-0.3.2/jiemamy-core/.project
    • -0
    • +174
    /tags/artemis-0.3.2/jiemamy-core/pom.xml
  11. /tags/artemis-0.3.2/jiemamy-core/src/test
  12. /tags/artemis-0.3.2/jiemamy-core/src
  13. /tags/artemis-0.3.2/jiemamy-core/src/main/java/org/jiemamy
  14. /tags/artemis-0.3.2/jiemamy-core/src/main/java/org
  15. /tags/artemis-0.3.2/jiemamy-core/src/main/java
  16. … 518 more files in changeset.
11:21 am

都元ダイスケ

[maven-release-plugin] rollback
[maven-release-plugin] rollback
Options
  1. /tags/artemis-0.3.2
  2. /tags/artemis-0.3.2/jiemamy-test-helper
  3. /tags/artemis-0.3.2/jiemamy-diagram
  4. /tags/artemis-0.3.2/jiemamy-core
  5. /tags/artemis-0.3.2/jiemamy-utils
  6. /tags/artemis-0.3.2/jiemamy-artemis-parent
  7. /tags/artemis-0.3.2/jiemamy-diff
  8. /tags/artemis-0.3.2/jiemamy-sql
    • -35
    • +0
    /tags/artemis-0.3.2/jiemamy-core/.project
    • -174
    • +0
    /tags/artemis-0.3.2/jiemamy-core/pom.xml
  11. /tags/artemis-0.3.2/jiemamy-core/src/test
  12. /tags/artemis-0.3.2/jiemamy-core/src
  13. /tags/artemis-0.3.2/jiemamy-core/src/main/java/org/jiemamy
  14. /tags/artemis-0.3.2/jiemamy-core/src/main/java/org
  15. /tags/artemis-0.3.2/jiemamy-core/src/main/java
  16. … 518 more files in changeset.
11:11 am

都元ダイスケ

[maven-release-plugin] copy for tag artemis-0.3.2
[maven-release-plugin] copy for tag artemis-0.3.2
Options
  1. /tags/artemis-0.3.2
  2. /tags/artemis-0.3.2/jiemamy-test-helper
  3. /tags/artemis-0.3.2/jiemamy-diagram
  4. /tags/artemis-0.3.2/jiemamy-core
  5. /tags/artemis-0.3.2/jiemamy-utils
  6. /tags/artemis-0.3.2/jiemamy-artemis-parent
  7. /tags/artemis-0.3.2/jiemamy-diff
  8. /tags/artemis-0.3.2/jiemamy-sql
    • -0
    • +35
    /tags/artemis-0.3.2/jiemamy-core/.project
    • -0
    • +174
    /tags/artemis-0.3.2/jiemamy-core/pom.xml
  11. /tags/artemis-0.3.2/jiemamy-core/src/test
  12. /tags/artemis-0.3.2/jiemamy-core/src
  13. /tags/artemis-0.3.2/jiemamy-core/src/main/java/org/jiemamy
  14. /tags/artemis-0.3.2/jiemamy-core/src/main/java/org
  15. /tags/artemis-0.3.2/jiemamy-core/src/main/java
  16. … 518 more files in changeset.
11:02 am

都元ダイスケ

[maven-release-plugin] rollback
[maven-release-plugin] rollback
Options
  1. /tags/artemis-0.3.2
10:46 am

都元ダイスケ

[maven-release-plugin] copy for tag artemis-0.3.2
[maven-release-plugin] copy for tag artemis-0.3.2
Options
  1. /tags/artemis-0.3.2
火曜日 27 7 2010
1:55 pm

Kazuki YAMAMOTO

  • More
  • CR-27
  • summarized and closed
1:55 pm

Kazuki YAMAMOTO

DefaultCheckConstraintModel を作りたいから DefaultCheckConstraintModelBuilder を使うという関係なんですね。上記の回答内容で了解です。

DefaultCheckConstraintModel を作りたいから DefaultCheckConstraintModelBuilder を使うという関係なんですね。上記の回答内容で了解です。

1:45 pm

Junichi Kato (j5ik2o)

  • More
  • CR-27
  • finished reviewing
1:07 pm

Keisuke.K

to: yamkazu この内容でどうでしょ？

to: yamkazu
この内容でどうでしょ？

土曜日 24 7 2010
12:10 pm

都元ダイスケ

  • More
  • CR-27
  • finished reviewing
水曜日 21 7 2010
3:43 pm

Keisuke.K

だいちゃんも言ってますが、DefaultCheckConstraintModel を作るのであれば DefaultCheckConstraintModelBuilder を作る時点で CheckConstraintModelBuilder 型で受け取る必要性が感じられないです。 あと実装の問題もあって、どの時点から Abstract Builder の VO 型を決定付けようか、という点。 ...

だいちゃんも言ってますが、DefaultCheckConstraintModel を作るのであれば DefaultCheckConstraintModelBuilder を作る時点で CheckConstraintModelBuilder 型で受け取る必要性が感じられないです。
あと実装の問題もあって、どの時点から Abstract Builder の VO 型を決定付けようか、という点。
現在の実装だと、例えば DefaultForeignKeyConstraintModelBuilder の階層は
ValueObjectBuilder -> ConstraintModelBuilder -> KeyConstraintModelBuilder -> ForeignKeyConstraintModelBuilder -> DefaultForeignKeyConstraintModelBuilder
になっていますが、DefaultPrimaryKeyConstraintModel の階層は
ValueObjectBuilder -> ConstraintModelBuilder -> KeyConstraintModelBuilder -> DefaultPrimaryKeyConstraintModelBuilder
となっています。
ここで、KeyConstraintModelBuilder の時点で VO 型を決定付けると、ForeignKeyConstraintModelBuilder#setXxx() の addConfigurator(new BuilderConfigurator<S>(){}) になっている部分の S が変えられようにも変えられないんですよ。
DefaultPrimaryKeyConstraintModelBuilder の前に PrimaryKeyConstraintModelBuilder を作れと言ってしまえばそれまでなんですが、実装が何もないクラスになります。
中身が空っぽのクラスがざっと増えるのはちょっと、と思ったので作らなかったんですが、どう思います？

日曜日 18 7 2010
11:13 pm

都元ダイスケ

なるほど。えびちゃん、解説願います。 個人的なイメージとしては、「DefaultCheckConstraintModelを作りたい」のだから、DefaultCheckConstraintModelBuilderを作る時点でCheckConstraintModelBuilder型で受け取る必要はあまりないかなぁ、なんて。

なるほど。えびちゃん、解説願います。

個人的なイメージとしては、「DefaultCheckConstraintModelを作りたい」のだから、DefaultCheckConstraintModelBuilderを作る時点でCheckConstraintModelBuilder型で受け取る必要はあまりないかなぁ、なんて。

7:19 pm

Kazuki YAMAMOTO

  • More
  • CR-27
  • started review
[CORE-178]org.jiemamy.model.attribute.constraintのBuilderの継承関係について
[CORE-178]org.jiemamy.model.attribute.constraintのBuilderの継承関係について
火曜日 13 7 2010
11:50 pm

都元ダイスケ

  • More
  • CR-16
  • summarized and closed
金曜日 09 7 2010
9:35 am

Seiji Tonouchi

  • More
  • CR-16
  • finished reviewing
火曜日 06 7 2010
10:39 pm

都元ダイスケ

エラーの方針であれば、可能ならばタイプセーフに、そもそも違った型を渡せないようにすべきかと思います。 タイプセーフ化が構造上不可能な時にのみ、やむを得ず IllegalArgumentException を選択すべきですね。

エラーの方針であれば、可能ならばタイプセーフに、そもそも違った型を渡せないようにすべきかと思います。
タイプセーフ化が構造上不可能な時にのみ、やむを得ず IllegalArgumentException を選択すべきですね。

10:38 pm

regtan

  • More
  • CR-16
  • finished reviewing
10:33 pm

regtan

たしかに便利だとおもうけどもDefaultForeignKeyConstraintModelBuilder#apply() であれば名前的にForeignKeyConstraintModel以外のものがわたって来たらIlleagalArgument扱いにした方がコードとしては混同しなくなっていいとも思う。

たしかに便利だとおもうけどもDefaultForeignKeyConstraintModelBuilder#apply() であれば名前的にForeignKeyConstraintModel以外のものがわたって来たらIlleagalArgument扱いにした方がコードとしては混同しなくなっていいとも思う。

木曜日 27 5 2010
1:05 pm

都元ダイスケ

アリだと思う？ナシだと思う？＞all

アリだと思う？ナシだと思う？＞all

水曜日 19 5 2010
6:11 pm

都元ダイスケ

完全未知のVOが渡ってきたら、build() と全く同じになるんすねー。 なるほど。悪くないと思ったけど…。この仕様アリかなー、ナシかなー。判断不能ｗ

完全未知のVOが渡ってきたら、build() と全く同じになるんすねー。
なるほど。悪くないと思ったけど…。この仕様アリかなー、ナシかなー。判断不能ｗ

6:09 pm

Keisuke.K

CheckConstraintModel の階層上、DefaultFOreignKeyConstraintModelBuilder#apply() で受け付けられるのは ConstraintModel の内容までなので、name, logicalName, description がビルダにセットされますね。

CheckConstraintModel の階層上、DefaultFOreignKeyConstraintModelBuilder#apply() で受け付けられるのは ConstraintModel の内容までなので、name, logicalName, description がビルダにセットされますね。

6:02 pm

都元ダイスケ

これって、DefaultForeignKeyConstraintModelBuilder#apply() の引数にForeignKeyConstraintModel以外のVOをつっこんだらどうなりますか？ 例えば CheckConstraintModel とか。

これって、DefaultForeignKeyConstraintModelBuilder#apply() の引数にForeignKeyConstraintModel以外のVOをつっこんだらどうなりますか？ 例えば CheckConstraintModel とか。

6:01 pm

都元ダイスケ

  • More
  • CR-16
  • started review
[CORE-179] apply() の vo を T ではなく ValueObject に変更。それに伴う各ビルダの修正。
[CORE-179] apply() の vo を T ではなく ValueObject に変更。それに伴う各ビルダの修正。
月曜日 17 5 2010
11:25 pm

都元ダイスケ

  • More
  • CR-12
  • summarized and closed
水曜日 05 5 2010
2:14 pm

都元ダイスケ

ビルダーパターンか。ありかもしれないですね。試しに書いてみますね。

ビルダーパターンか。ありかもしれないですね。試しに書いてみますね。

Earlier activity No later activity