android - 添加工厂时出现Moshi ArrayOutOfBoundsException

Question

打开应用程序时，有时会发现与Moshi相关的此异常:

Caused by java.lang.ArrayIndexOutOfBoundsException: length=33; index=33
       at java.util.ArrayList.add(ArrayList.java:468)
       at com.squareup.moshi.Moshi$Builder.add(Moshi.java:231)

我们在BaseApplication中初始化存储库，有时会在初始化Moshi时导致上述崩溃。我在应用程序报告中发现了此错误，但无法重现。让我们跳到我们所拥有的，看看您是否对它有任何了解。

该工厂用于创建Moshi实例，添加KotlinJsonAdapterFactory时会导致崩溃:

object MyMoshiConverterFactory {

    fun create(setup: (Moshi.Builder.() -> Unit)? = null): Converter.Factory {
        val moshi = MoshiUtil.createMoshi()
        setup?.let { moshi.it() }
        moshi.add(KotlinJsonAdapterFactory()) // Here is the crash!
        return MoshiConverterFactory.create(moshi.build())
    }
}

在这里，我们有一个我们拥有所有使用的转换器的类。它确实有很多转换器，但是为了简单起见，我删除了其中的一些转换器:

object MoshiUtil {

    private val lazyMoshi by lazy {
        Moshi.Builder().apply {
            add(DateAdapter())
            add(DefaultOnDataMismatchAdapter.newFactory(FeedItem::class.java, null))
            add(SkipListNullValuesAdapter.createFactory(Element::class.java))
            add(SkipListNullValuesAdapter.createFactory(Post::class.java))
            add(SkipListNullValuesAdapter.createFactory(MetadataItem::class.java))
            add(GeoGeometry::class.java, GeometryAdapter())
        }
    }

    fun createMoshi() = lazyMoshi
}

最后，在BaseApplication中，我们有类似以下内容:

class BaseApplication {

   @Override
    public void onCreate() {
        super.onCreate();

        val myService = getMyService(applicationContext)

    }

    private fun getMyService(appContext: Context): MyService {
       val converterFactory = MyMoshiConverterFactory.create()

       return Retrofit.Builder().baseUrl(baseUrl).apply {
                addConverterFactory(converterFactory)
                client(okHttpClientBuilder.build())
            }.build().create(MyService::class.java)
        }
    }
 }

那么，您是否看到任何可能导致它的东西？当应用程序中的多个位置同时创建MoshiUtils对象时，您是否认为这可能是启动时发生的并发问题？期待收到您的来信，谢谢!

Answer 1

Moshi.Builder是可变的，并且不是线程安全的，因此，您有时会遇到的此错误是竞争情况。您应该在该基础.build()实例上调用MoshiUtil以获得不可变的Moshi实例，然后使MoshiUtil.createMoshi的返回值为moshi.newBuilder()(创建已经像现有Moshi.Builder实例一样配置的Moshi)，如下所示:

object MoshiUtil {
    private val baseMoshi: Moshi = Moshi.Builder().apply {
        // ...
    }.build()

    fun createMoshi(): Moshi.Builder = baseMoshi.newBuilder()
}

由于现在每个调用 createMoshi的人都拥有自己的 Moshi.Builder实例，因此不再存在任何并发问题。