web-dev-qa-db-ja.com

Moshiでサブクラス化するカスタムコンバーター

Userクラスがあります。そして2つのサブクラス。親子。 {"user": "..."}を使用してサーバーからjsonを取得し、user.typeに応じてそれを親または子に変換する必要があります

私が理解しているように、この方法でカスタムコンバーターを追加する必要があります。

        Moshi moshi = new Moshi.Builder()
            .add(new UserAdapter())
            .build();

これがUserAdapterの私の実装です。私はそれがダミーであることを知っていますが、それでもこのように機能していません:

public class UserAdapter {

@FromJson
User fromJson(String userJson) {
    Moshi moshi = new Moshi.Builder().build();
    try {
        JSONObject jsonObject = new JSONObject(userJson);
        String accountType = jsonObject.getString("type");

        switch (accountType) {
            case "Child":
                JsonAdapter<Child> childJsonAdapter = moshi.adapter(Child.class);
                return childJsonAdapter.fromJson(userJson);
            case "Parent":
                JsonAdapter<Parent> parentJsonAdapter = moshi.adapter(Parent.class);
                return parentJsonAdapter.fromJson(userJson);

        }
    } catch (JSONException | IOException e) {
        e.printStackTrace();
    }

    return null;
}

@ToJson
String toJson(User user) {
    Moshi moshi = new Moshi.Builder().build();
    JsonAdapter<User> jsonAdapter = moshi.adapter(User.class);
    String toJson = jsonAdapter.toJson(user);
    return toJson;
}

まず、このコードで次の例外が発生します。

com.squareup.moshi.JsonDataException: Expected a string but was BEGIN_OBJECT at path $.user

次に、より良い方法があると思います。アドバイスをお願いします。

更新。ここにエラーのスタックトレースがあります:

 com.squareup.moshi.JsonDataException: Expected a name but was BEGIN_OBJECT at path $.user
 at com.squareup.moshi.JsonReader.nextName(JsonReader.Java:782)
 at com.squareup.moshi.ClassJsonAdapter.fromJson(ClassJsonAdapter.Java:141)
 at com.squareup.moshi.JsonAdapter$1.fromJson(JsonAdapter.Java:68)
 at com.squareup.moshi.JsonAdapter.fromJson(JsonAdapter.Java:33)
 at retrofit.MoshiResponseBodyConverter.convert(MoshiResponseBodyConverter.Java:33)
 at retrofit.MoshiResponseBodyConverter.convert(MoshiResponseBodyConverter.Java:23)
 at retrofit.OkHttpCall.parseResponse(OkHttpCall.Java:148)
 at retrofit.OkHttpCall.execute(OkHttpCall.Java:116)
 at retrofit.RxJavaCallAdapterFactory$CallOnSubscribe.call(RxJavaCallAdapterFactory.Java:111)
 at retrofit.RxJavaCallAdapterFactory$CallOnSubscribe.call(RxJavaCallAdapterFactory.Java:88)
 at rx.Observable$2.call(Observable.Java:162)
 at rx.Observable$2.call(Observable.Java:154)
 at rx.Observable$2.call(Observable.Java:162)
 at rx.Observable$2.call(Observable.Java:154)
 at rx.Observable.unsafeSubscribe(Observable.Java:7710)
 at rx.internal.operators.OperatorSubscribeOn$1$1.call(OperatorSubscribeOn.Java:62)
 at rx.internal.schedulers.ScheduledAction.run(ScheduledAction.Java:55)
 at Java.util.concurrent.Executors$RunnableAdapter.call(Executors.Java:422)
 at Java.util.concurrent.FutureTask.run(FutureTask.Java:237)
 at Java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.access$201(ScheduledThreadPoolExecutor.Java:152)
 at Java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.run(ScheduledThreadPoolExecutor.Java:265)
 at Java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.Java:1112)
 at Java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.Java:587)
 at Java.lang.Thread.run(Thread.Java:818)
18
Defuera

これは私があなたのJSONデータのカスタムデシリアライゼーションのためにあなたが従うべき例のように思えます: https://github.com/square/moshi#another-example

JSON構造に対応する中間クラスを使用し、Moshiは自動的にそれを膨張させます。次に、膨張したデータを使用して、特殊なユーザークラスを構築できます。例えば:

// Intermediate class with JSON structure
class UserJson {
  // Common JSON fields
  public String type;
  public String name;
  // Parent JSON fields
  public String occupation;
  public Long salary;
  // Child JSON fields
  public String favorite_toy;
  public Integer grade;
}

abstract class User {
  public String type;
  public String name;
}

final class Parent extends User {
  public String occupation;
  public Long salary;
}

final class Child extends User {
  public String favoriteToy;
  public Integer grade;
}

今、アダプター:

class UserAdapter {
  // Note that you pass in a `UserJson` object here
  @FromJson User fromJson(UserJson userJson) {
    switch (userJson.type) {
    case "Parent":
      final Parent parent = new Parent();
      parent.type = userJson.type;
      parent.name = userJson.name;
      parent.occupation = userJson.occupation;
      parent.salary = userJson.salary;
      return parent;
    case "Child":
      final Child child = new Child();
      child.type = userJson.type;
      child.name = userJson.name;
      child.favoriteToy = userJson.favorite_toy;
      child.grade = userJson.grade;
      return child;
    default:
      return null;
    }
  }

  // Note that you return a `UserJson` object here.
  @ToJson UserJson toJson(User user) {
    final UserJson json = new UserJson();
    if (user instanceof Parent) {
      json.type = "Parent";
      json.occupation = ((Parent) user).occupation;
      json.salary = ((Parent) user).salary;
    } else {
      json.type = "Child";
      json.favorite_toy = ((Child) user).favoriteToy;
      json.grade = ((Child) user).grade;
    }
    json.name = user.name;
    return json;
  }
}

これはもっとすっきりしていて、MoshiがJSONからオブジェクトを作成したり、オブジェクトからJSONを作成したりできるようになっていると思います。昔ながらのJSONObjectをいじる必要はありません。

テストする:

Child child = new Child();
child.type = "Child";
child.name = "Foo";
child.favoriteToy = "Java";
child.grade = 2;
Moshi moshi = new Moshi.Builder().add(new UserAdapter()).build();
try {
  // Serialize
  JsonAdapter<User> adapter = moshi.adapter(User.class);
  String json = adapter.toJson(child);
  System.out.println(json);
  // Output is: {"favorite_toy":"Java","grade":2,"name":"Foo","type":"Child"}

  // Deserialize
  // Note the cast to `Child`, since this adapter returns `User` otherwise.
  Child child2 = (Child) adapter.fromJson(json);
  System.out.println(child2.name);
  // Output is: Foo
} catch (IOException e) {
  e.printStackTrace();
}
13
savanto

あなたはおそらく次のように解析を実装しようとしました https://github.com/square/moshi#custom-type-adapters

@FromJsonメソッドの引数としてStringが使用されているため、魔法のようにマッピングヘルパークラスまたはStringに解析でき、手動で解析する必要があります。実際には、マッピングヘルパークラスまたはマップを使用できます。

したがって、あなたの例外Expected a string but was BEGIN_OBJECT at path $.userは、Moshiがそのユーザーを文字列として取得しようとしたために発生しました(これはアダプターで暗黙的に指定されているため)。これは単なるオブジェクトです。

ポリモーフィズムの場合、クラスが非常に大きくなる可能性があり、コードに依存または記憶/コメントする必要があるため、ヘルパークラスに対してすべての可能なフィールドを解析するのは好きではありません。

あなたはそれをマップとして扱うことができます-それは未知のタイプのデフォルトモデルです-そしてそれをjsonに変換するので、あなたのケースでは次のようになります:

    @FromJson
    User fromJson(Map<String, String> map) {
        Moshi moshi = new Moshi.Builder().build();
        String userJson = moshi.adapter(Map.class).toJson(map);
        try {
            JSONObject jsonObject = new JSONObject(userJson);
            String accountType = jsonObject.getString("type");

            switch (accountType) {
                case "Child":
                    JsonAdapter<Child> childJsonAdapter = moshi.adapter(Child.class);
                    return childJsonAdapter.fromJson(userJson);
                case "Parent":
                    JsonAdapter<Parent> parentJsonAdapter = moshi.adapter(Parent.class);
                    return parentJsonAdapter.fromJson(userJson);

            }
        } catch (JSONException | IOException e) {
            e.printStackTrace();
        }

        return null;
    }

もちろん、直接マップを処理することもできます。「タイプ」文字列を取得し、マップの残りの部分を選択したクラスに解析します。そうすれば、JSONObjectを使用する必要がまったくなくなり、Androidに依存しないという優れた利点と、解析のテストが容易になります。

    @FromJson
    User fromJson(Map<String, String> map) {
        Moshi moshi = new Moshi.Builder().build();
        try {
            String userJson = moshi.adapter(Map.class).toJson(map);
            switch (map.get("type")) {
                case "Child":
                    JsonAdapter<Child> childJsonAdapter = moshi.adapter(Child.class);
                    return childJsonAdapter.fromJson(userJson);
                case "Parent":
                    JsonAdapter<Parent> parentJsonAdapter = moshi.adapter(Parent.class);
                    return parentJsonAdapter.fromJson(userJson);

            }
        } catch (IOException e) {
            e.printStackTrace();
        }

        return null;
    }
3
michalbrz

これを行うには、PolymorphicJsonAdapterFactoryを使用するはるかに優れた方法があります。参照 https://proandroiddev.com/moshi-polymorphic-adapter-is-d25deebbd7c5

1
Luke