Mirage SqlManagerImpl#findEntity に指定する可変引数の順番が明示的でない

• Reporter: bcohimei
• Owner: takezoe
• Type: Bugs
• Status: Open [Owner assigned]
• Component: Mirage
• MileStone: (None)
• Priority: 5 - Medium
• Severity: 5 - Medium
• Resolution: None
• File: None

Details

エンティティクラスに複数のプライマリーキーフィールドがあったとします。

 public class SomeEntity {
     @PrimaryKey(persistent=true)
     private String key1;
 
     @PrimaryKey(persistent=true)
     private String key2;
 
     private String value;
     // コンストラクタ、getter/setterは省略
 }

このとき、SqlManagerImpl#findEntityの第２引数以後に指定する引数の順番が、必ずしもエンティティクラスでの宣言順でよいとは限りません。つまり、key1に"1"、key2に"2"を指定して検索したい場合でも・・・

     SomeEntity entity = findEntity(SomeEntity.class, "1", "2");

とすべきか

     SomeEntity entity = findEntity(SomeEntity.class, "2", "1");

とすべきなのかは、実際に実行してみてMirageが出力するデバッグ文をみないと判断できません。

おそらく、java.lang.Class#getMethodsやjava.lang.Class#getFieldsが返す配列の要素の並び方に依存しているのではないでしょうか？これらのメソッドが返す配列の要素の順番は不定であるとされています。

以下は提案ですが、insertEntity や deleteEntity と使用法が同様になるように

     public <T> T findEntity(T param)

とするか、あるいは

     public <T> T findEntity(Class<T> clazz, Map<String, Object> param)

として、WHEREの条件とパラメーターを明示的に関連付けて指定できるようにすべきかと思います。

Ticket History (2/2 Histories)

2010-07-30 14:17 Updated by: bcohimei

• New Ticket "Mirage SqlManagerImpl#findEntity に指定する可変引数の順番が明示的でない" created

2010-07-30 14:59 Updated by: bcohimei

• Details Updated