Haruskah saya mendeklarasikan ObjectMapper Jackson sebagai bidang statis?

The Jackson perpustakaan ObjectMapperkelas tampaknya thread aman .

Apakah ini berarti bahwa saya harus menyatakan bidang saya ObjectMapperstatis seperti ini

class Me {
    private static final ObjectMapper mapper = new ObjectMapper();
}

alih-alih sebagai bidang tingkat instance seperti ini?

class Me {
    private final ObjectMapper mapper = new ObjectMapper();
}

Ya, itu aman dan direkomendasikan.

Satu-satunya peringatan dari halaman yang Anda referensikan adalah Anda tidak dapat memodifikasi konfigurasi mapper setelah dibagikan; tetapi Anda tidak mengubah konfigurasi sehingga tidak masalah. Jika Anda memang perlu mengubah konfigurasi, Anda akan melakukannya dari blok statis dan itu akan baik-baik saja.

EDIT : (2013/10)

Dengan 2.0 dan di atas, di atas dapat ditambah dengan mencatat bahwa ada cara yang lebih baik: penggunaan ObjectWriterdan ObjectReaderobjek, yang dapat dibangun oleh ObjectMapper. Mereka sepenuhnya tidak dapat diubah, thread-safe, yang berarti bahwa bahkan secara teori tidak mungkin menyebabkan masalah keamanan thread (yang dapat terjadi dengan ObjectMapperjika kode mencoba mengkonfigurasi ulang instance).

Meskipun ObjectMapper aman untuk di-thread, saya sangat tidak menyarankan untuk mendeklarasikannya sebagai variabel statis, terutama dalam aplikasi multithreaded. Bukan karena itu adalah praktik yang buruk, tetapi karena Anda menghadapi risiko besar mengalami kebuntuan. Saya menceritakannya dari pengalaman saya sendiri. Saya membuat aplikasi dengan 4 utas identik yang mendapatkan dan memproses data JSON dari layanan web. Aplikasi saya sering mengulur-ulur pada perintah berikut, menurut thread dump:

Map aPage = mapper.readValue(reader, Map.class);

Selain itu, kinerjanya juga tidak bagus. Ketika saya mengganti variabel statis dengan variabel berbasis instance, mengulur-ulur menghilang dan kinerja empat kali lipat. Yaitu 2,4 juta dokumen JSON diproses dalam 40 menit.56sec., Bukan 2.5 jam sebelumnya.

Meskipun aman untuk mendeklarasikan ObjectMapper statis dalam hal keamanan utas, Anda harus menyadari bahwa membangun variabel Objek statis di Jawa dianggap praktik buruk. Untuk detail lebih lanjut, lihat Mengapa variabel statis dianggap jahat? (dan jika Anda mau, jawabanku )

Singkatnya, statika harus dihindari karena menyulitkan untuk menulis tes unit ringkas. Misalnya, dengan ObjectMapper final statis, Anda tidak dapat menukar keluar serialisasi JSON untuk kode dummy atau larangan.

Selain itu, final statis mencegah Anda mengkonfigurasi ulang ObjectMapper saat runtime. Anda mungkin tidak membayangkan alasan untuk itu sekarang, tetapi jika Anda mengunci diri Anda sendiri ke dalam pola final statis, tidak akan merusak classloader yang akan membuat Anda menginisialisasi ulang itu.

Dalam kasus ObjectMapper baik-baik saja, tetapi secara umum itu adalah praktik yang buruk dan tidak ada keuntungan dibandingkan menggunakan pola tunggal atau inversi-of-control untuk mengelola objek berumur panjang Anda.

Trik yang saya pelajari dari PR ini jika Anda tidak ingin mendefinisikannya sebagai variabel final statis tetapi ingin menyimpan sedikit overhead dan menjamin thread aman.

private static final ThreadLocal<ObjectMapper> om = new ThreadLocal<ObjectMapper>() {
    @Override
    protected ObjectMapper initialValue() {
        ObjectMapper objectMapper = new ObjectMapper();
        objectMapper.configure(DeserializationFeature.FAIL_ON_UNKNOWN_PROPERTIES, false);
        return objectMapper;
    }
};

public static ObjectMapper getObjectMapper() {
    return om.get();
}

kredit kepada penulis.

com.fasterxml.jackson.databind.type.TypeFactory._hashMapSuperInterfaceChain (HierarchicType)

com.fasterxml.jackson.databind.type.TypeFactory._findSuperInterfaceChain(Type, Class)
  com.fasterxml.jackson.databind.type.TypeFactory._findSuperTypeChain(Class, Class)
     com.fasterxml.jackson.databind.type.TypeFactory.findTypeParameters(Class, Class, TypeBindings)
        com.fasterxml.jackson.databind.type.TypeFactory.findTypeParameters(JavaType, Class)
           com.fasterxml.jackson.databind.type.TypeFactory._fromParamType(ParameterizedType, TypeBindings)
              com.fasterxml.jackson.databind.type.TypeFactory._constructType(Type, TypeBindings)
                 com.fasterxml.jackson.databind.type.TypeFactory.constructType(TypeReference)
                    com.fasterxml.jackson.databind.ObjectMapper.convertValue(Object, TypeReference)

Metode _hashMapSuperInterfaceChain di kelas com.fasterxml.jackson.databind.type.TypeFactory disinkronkan. Saya melihat pertengkaran pada saat yang sama pada beban tinggi.

Mungkin alasan lain untuk menghindari ObjectMapper statis