Merge pull request #177 from domaframework/kotlin-1.0.6

Kotlin 1.0.6対応

Author: nakamura-to

docs/sources/kotlin-support.rst

@@ -5,7 +5,7 @@ Kotlin サポート
 .. contents:: 目次
    :depth: 3
 
-Doma は `Kotlin <https://kotlinlang.org/>`_ を実験的にサポートしています。
+Doma は `Kotlin <https://kotlinlang.org/>`_ 1.0.6を実験的にサポートしています。
 
 Kotlin利用のベストプラクティス
 ================================
@@ -20,7 +20,6 @@ Kotlin利用のベストプラクティス
 * コンストラクタは1つだけ定義する
 * コンストラクタ以外でプロパティを定義しない
 * コンストラクタで定義するプロパティには `val` を使用する
-* 継承は使わない
 
 .. code-block:: java
 
@@ -52,7 +51,6 @@ Kotlin利用のベストプラクティス
 * コンストラクタは1つだけ定義する
 * コンストラクタ以外でプロパティを定義しない
 * コンストラクタで定義するプロパティには `val` を使用する
-* 継承は使わない
 
 .. code-block:: java
 
@@ -62,15 +60,14 @@ Kotlin利用のベストプラクティス
 Daoインタフェース
 -------------------
 
-* SQLファイルとマッピングする場合は `@ParameterName` を使ってメソッドのパラメータに名前をつける
 * 更新処理の戻り値の型は `org.seasar.doma.jdbc.Result` や `org.seasar.doma.jdbc.BatchResult` を使う
 
 .. code-block:: java
 
   @Dao(config = AppConfig::class)
   interface PersonDao {
     @Select
-    fun selectById(@ParameterName("id") id: Int): Person
+    fun selectById(id: Int): Person
     @Insert
     fun insert(person: Person): Result<Person>
   }
@@ -87,10 +84,9 @@ Daoインタフェース
 kaptによるビルド
 -------------------
 
-Kotlin で記述されたクラスやインタフェースに対して注釈処理をするには `kapt <http://blog.jetbrains.com/kotlin/2015/06/better-annotation-processing-supporting-stubs-in-kapt/>`_ を実行する必要があります。
-2016年5月現在、Kotlin 1.0.2のkaptは機能が不足しておりかつ動作が不安定です。また、ドキュメントがありません。
-Daoインタフェースで `@ParameterName` を用いて明示的に名前をつけることを推奨するのも kapt の不具合に由来します。
-不安定な挙動を避けるため、Gradleでビルドする際、常に `clean build` を実行することを推奨します。
+Kotlinで記述されたクラスやインタフェースに対して注釈処理をするには `kapt <https://blog.jetbrains.com/kotlin/2016/12/kotlin-1-0-6-is-here/>`_ を実行する必要があります。
+kaptは実験的な位置付けにありドキュメントがありません。
+Gradleでビルドする際は、確実な注釈処理が行われるように常に `clean build` を実行することを推奨します。
 
 .. code-block:: sh
 
@@ -101,10 +97,8 @@ Eclispeを利用する場合設定を適切に行えばJavaの注釈処理は自
 JavaとKotlinの混在
 -------------------------
 
-kaptの不具合を避けるため、Domaに関するコードの全てもしくは一部をJavaで書くことは検討に値します。
-特に、DaoについてはJavaで書いても良いかもしれません。
-JavaとKotlinで記述量にほとんど差がなく、 `@ParameterName` を使う必要性を避けられるためです。
-Domaの利用において、JavaとKotlinの混在は特に問題ありません。
+kaptの不確実な挙動を避けるため、Domaに関するコードの全てもしくは一部をJavaで書くことは検討に値します。
+Domaの利用において、JavaとKotlinの混在は問題ありません。
 
 サンプルプロジェクト
 =====================

src/main/java/org/seasar/doma/internal/apt/Options.java

@@ -47,8 +47,6 @@ public final class Options {
 
     public static final String VERSION_VALIDATION = "doma.version.validation";
 
-    public static final String KAPT_ANNOTATIONS = "kapt.annotations";
-
     public static boolean isTestEnabled(ProcessingEnvironment env) {
         String test = env.getOptions().get(Options.TEST);
         return Boolean.valueOf(test).booleanValue();
@@ -116,10 +114,6 @@ public static boolean getVersionValidation(ProcessingEnvironment env) {
         return v != null ? Boolean.valueOf(v).booleanValue() : true;
     }
 
-    public static boolean isKaptEnabled(ProcessingEnvironment env) {
-        return env.getOptions().containsKey(KAPT_ANNOTATIONS);
-    }
-
     protected static class Constants {
 
         public static final String DEFAULT_DAO_SUFFIX = "Impl";

src/main/java/org/seasar/doma/internal/apt/meta/EmbeddableMetaFactory.java

@@ -53,7 +53,6 @@
 import org.seasar.doma.internal.apt.AptException;
 import org.seasar.doma.internal.apt.AptIllegalStateException;
 import org.seasar.doma.internal.apt.Notifier;
-import org.seasar.doma.internal.apt.Options;
 import org.seasar.doma.internal.apt.mirror.EmbeddableMirror;
 import org.seasar.doma.internal.apt.util.ElementUtil;
 import org.seasar.doma.internal.apt.util.TypeMirrorUtil;
@@ -247,21 +246,8 @@ protected void doConstructor(TypeElement embeddableElement,
         EmbeddableConstructorMeta constructorMeta = getConstructorMeta(
                 embeddableElement, embeddableMeta);
         if (constructorMeta == null) {
-            if (Options.isKaptEnabled(env)) {
-                constructorMeta = getConstructorMetaForKotlin(
-                        embeddableElement, embeddableMeta);
-                if (constructorMeta == null) {
-                    throw new AptException(
-                            Message.DOMA4292,
-                            env,
-                            embeddableElement,
-                            new Object[] { embeddableElement.getQualifiedName() });
-                }
-            } else {
-                throw new AptException(Message.DOMA4293, env,
-                        embeddableElement,
-                        new Object[] { embeddableElement.getQualifiedName() });
-            }
+            throw new AptException(Message.DOMA4293, env, embeddableElement,
+                    new Object[] { embeddableElement.getQualifiedName() });
         }
         if (constructorMeta.getConstructorElement().getModifiers()
                 .contains(Modifier.PRIVATE)) {
@@ -271,35 +257,6 @@ protected void doConstructor(TypeElement embeddableElement,
         embeddableMeta.setConstructorMeta(constructorMeta);
     }
 
-    protected EmbeddableConstructorMeta getConstructorMetaForKotlin(
-            TypeElement embeddableElement, EmbeddableMeta embeddableMeta) {
-        for (ExecutableElement constructor : ElementFilter
-                .constructorsIn(embeddableElement.getEnclosedElements())) {
-            if (embeddableMeta.getEmbeddablePropertyMetas().size() == constructor
-                    .getParameters().size()) {
-                Iterator<EmbeddablePropertyMeta> propIt = embeddableMeta
-                        .getEmbeddablePropertyMetas().iterator();
-                Iterator<? extends VariableElement> paramIt = constructor
-                        .getParameters().iterator();
-                int index = 0;
-                for (; propIt.hasNext() && paramIt.hasNext();) {
-                    TypeMirror prop = propIt.next().getType();
-                    TypeMirror param = paramIt.next().asType();
-                    if (!TypeMirrorUtil.isSameType(prop, param, env)) {
-                        throw new AptException(Message.DOMA4308, env,
-                                embeddableElement, new Object[] { prop, param,
-                                        index,
-                                        embeddableElement.getQualifiedName() });
-                    }
-                    index++;
-                }
-                return new EmbeddableConstructorMeta(constructor,
-                        embeddableMeta.getEmbeddablePropertyMetas());
-            }
-        }
-        return null;
-    }
-
     protected EmbeddableConstructorMeta getConstructorMeta(
             TypeElement embeddapleElement, EmbeddableMeta embeddableMeta) {
         Map<String, EmbeddablePropertyMeta> propertyMetaMap = new HashMap<String, EmbeddablePropertyMeta>();

src/main/java/org/seasar/doma/internal/apt/meta/EntityMetaFactory.java

@@ -53,7 +53,6 @@
 import org.seasar.doma.internal.apt.AptException;
 import org.seasar.doma.internal.apt.AptIllegalStateException;
 import org.seasar.doma.internal.apt.Notifier;
-import org.seasar.doma.internal.apt.Options;
 import org.seasar.doma.internal.apt.mirror.EntityMirror;
 import org.seasar.doma.internal.apt.mirror.TableMirror;
 import org.seasar.doma.internal.apt.util.AnnotationValueUtil;
@@ -630,20 +629,8 @@ protected void doConstructor(TypeElement classElement, EntityMeta entityMeta) {
             EntityConstructorMeta constructorMeta = getConstructorMeta(
                     classElement, entityMeta);
             if (constructorMeta == null) {
-                if (Options.isKaptEnabled(env)) {
-                    constructorMeta = getConstructorMetaForKotlin(classElement,
-                            entityMeta);
-                    if (constructorMeta == null) {
-                        throw new AptException(
-                                Message.DOMA4282,
-                                env,
-                                classElement,
-                                new Object[] { classElement.getQualifiedName() });
-                    }
-                } else {
-                    throw new AptException(Message.DOMA4281, env, classElement,
-                            new Object[] { classElement.getQualifiedName() });
-                }
+                throw new AptException(Message.DOMA4281, env, classElement,
+                        new Object[] { classElement.getQualifiedName() });
             }
             if (constructorMeta.getConstructorElement().getModifiers()
                     .contains(Modifier.PRIVATE)) {
@@ -662,35 +649,6 @@ protected void doConstructor(TypeElement classElement, EntityMeta entityMeta) {
         }
     }
 
-    protected EntityConstructorMeta getConstructorMetaForKotlin(
-            TypeElement classElement, EntityMeta entityMeta) {
-        for (ExecutableElement constructor : ElementFilter
-                .constructorsIn(classElement.getEnclosedElements())) {
-            if (entityMeta.getAllPropertyMetas().size() == constructor
-                    .getParameters().size()) {
-                Iterator<EntityPropertyMeta> propIt = entityMeta
-                        .getAllPropertyMetas().iterator();
-                Iterator<? extends VariableElement> paramIt = constructor
-                        .getParameters().iterator();
-                int index = 0;
-                for (; propIt.hasNext() && paramIt.hasNext();) {
-                    TypeMirror prop = propIt.next().getType();
-                    TypeMirror param = paramIt.next().asType();
-                    if (!TypeMirrorUtil.isSameType(prop, param, env)) {
-                        throw new AptException(Message.DOMA4307, env,
-                                classElement,
-                                new Object[] { prop, param, index,
-                                        classElement.getQualifiedName() });
-                    }
-                    index++;
-                }
-                return new EntityConstructorMeta(constructor,
-                        entityMeta.getAllPropertyMetas());
-            }
-        }
-        return null;
-    }
-
     protected EntityConstructorMeta getConstructorMeta(
             TypeElement classElement, EntityMeta entityMeta) {
         Map<String, EntityPropertyMeta> entityPropertyMetaMap = new HashMap<String, EntityPropertyMeta>();

src/main/java/org/seasar/doma/message/Message.java

@@ -425,15 +425,13 @@ public enum Message implements MessageResource {
     DOMA4279("型[{0}]がローカルクラスもしくは無名クラスです。DomainConverterの第1型引数に指定された型もしくはその型を囲む型はトップレベルもしくはメンバでなければいけません。"),
     DOMA4280("型[{0}]の単純名に\"$\"または\"__\"が含まれています。DomainConverterの第1型引数に指定された型もしくはその型を囲む型はこれらの文字列を単純名に含んではいけません。"),
     DOMA4281("イミュータブルなエンティティクラスではコンストラクタのパラメータの数と型と名前は永続対象フィールドに一致しなければいけません。 at {0}"),
-    DOMA4282("Kotlinでイミュータブルなエンティティクラスを定義する場合、すべてのプロパティをコンストラクタで宣言しなければいけません。 at {0}"),
     DOMA4283("@Embeddableはクラス以外には注釈できません。 at {0}"),
     DOMA4285("エンベッダブルクラスには型パラメータを定義できません。 at {0}"),
     DOMA4286("エンベッダブルクラスのフィールドには@OriginalStatesを注釈できません。 at {0}.{1}"),
     DOMA4288("アノテーション[{0}]とアノテーション[{1}]が競合しています。これらは同じフィールドに注釈できません。 at {2}.{3}"),
     DOMA4289("エンベッダブルクラスのフィールドには@Idを注釈できません。 at {0}.{1}"),
     DOMA4290("エンベッダブルクラスのフィールドには@Versionを注釈できません。 at {0}.{1}"),
     DOMA4291("エンベッダブルクラスのフィールドには@GeneratedValueを注釈できません。 at {0}.{1}"),
-    DOMA4292("Kotlinでエンベッダブルクラスを定義する場合、すべてのプロパティをコンストラクタで宣言しなければいけません。 at {0}"),
     DOMA4293("エンベッダブルクラスではコンストラクタのパラメータの数と型と名前は永続対象フィールドに一致しなければいけません。 at {0}"),
     DOMA4294("エンベッダブルクラスには非privateなコンストラクタが必要です。 at {0}"),
     DOMA4295("クラス[{0}]の原型は永続プロパティに使用できません。 at {1}.{2}"),
@@ -448,8 +446,6 @@ public enum Message implements MessageResource {
     DOMA4304("@Versionは@Embeddableが注釈された型のプロパティに注釈できません。 at {0}.{1}"),
     DOMA4305("@OriginalStatesを使用する場合、エンティティクラスのプロパティに@Embeddableが注釈された型を使うことはサポートされていません。 at {0}"),
     DOMA4306("@Columnは@Embeddableが注釈された型のプロパティに注釈できません。 at {0}.{1}"),
-    DOMA4307("永続プロパティの型[{0}]とコンストラクタパラメータの型[{1}]が[{2}]番目で一致しません。Kotlinでエンベッダブルクラスを定義する場合、永続プロパティとコンストラクタのパラメータの宣言順序は一致しなければいけません。継承している場合、コンストラクタパラメータの宣言順序は親クラスに定義されたプロパティ順でなければいけません。 at {3}"),
-    DOMA4308("永続プロパティの型[{0}]とコンストラクタパラメータの型[{1}]が[{2}]番目で一致しません。Kotlinでエンティティクラスを定義する場合、永続プロパティとコンストラクタのパラメータの宣言順序は一致しなければいけません。継承している場合、コンストラクタパラメータの宣言順序は親クラスに定義されたプロパティ順でなければいけません。 at {3}"),
     DOMA4309("ファイルパス[{0}]の大文字小文字がファイルシステム上のパスと一致しません。ファイルシステム上のパスは\"{1}\"です。"),
     DOMA4315("型[{0}]がpublicかつstaticでありません。エンティティクラスもしくはエンティティクラスを囲む型はpublicかつstaticでなければいけません。"),
     DOMA4316("型[{0}]がローカルクラスもしくは無名クラスです。エンティティクラスもしくはエンティティクラスを囲む型はトップレベルもしくはメンバでなければいけません。"),