[Spring] Should I have a custom Objectmapper as a static attribute in my service class?

Currently have an Objectmapper, from the JSON jackson serialization api, from my configuration class used in my Service class. Using lombok, realized the @AllArgsConstructor doesn't create a constructor for static attributes. It makes sense to have it be static in the conventional sense, since it should be shared between service instances. It is making it hard to do unit tests. As best practice, should I be having the objectmapper attribute in my service class be static or not? I'm thinking I can forgo making it static since I'm getting a universal Objectmapper from the spring context anyway. Would appreciate the help.
10 Replies
JavaBot
JavaBot3mo ago
This post has been reserved for your question.
Hey @Ghostyjangle! Please use /close or the Close Post button above when your problem is solved. Please remember to follow the help guidelines. This post will be automatically closed after 300 minutes of inactivity.
TIP: Narrow down your issue to simple and precise questions to maximize the chance that others will reply in here.
dan1st
dan1st3mo ago
Why should a constructor initialize static fields via a parameter? Then they would be overwritten every time you create a new instance from the docs of ObjectMapper
Mapper instances are fully thread-safe provided that ALL configuration of the instance occurs before ANY read or write calls.
https://fasterxml.github.io/jackson-databind/javadoc/2.7/com/fasterxml/jackson/databind/ObjectMapper.html If you want to share the same configuration for all instances and never change its configuration afterwards, you can make the ObjectMapper static and you probably want to initialize it in a static initializer block or similar
public class YourClass {
private static final ObjectMapper mapper;
static {
mapper = ...;
//initialize the ObjectMapper here and configure it
}
}
public class YourClass {
private static final ObjectMapper mapper;
static {
mapper = ...;
//initialize the ObjectMapper here and configure it
}
}
or
public class YourClass {
private static final ObjectMapper mapper = createObjectMapper();
private static ObjectMapper createObjectMapper() {
ObjectMapper created = ...;
//configure it
return created;
}
}
public class YourClass {
private static final ObjectMapper mapper = createObjectMapper();
private static ObjectMapper createObjectMapper() {
ObjectMapper created = ...;
//configure it
return created;
}
}
Ghostyjangle
GhostyjangleOP3mo ago
oh sorry I forgot to mention that I want to have spring autowire the objectmapper instance to my service class. Or is this not the way I should do it?
dan1st
dan1st3mo ago
in that case, don't make it static But isn't it normally the case that service classes are singletons?
Ghostyjangle
GhostyjangleOP3mo ago
I see it as being such that I can just change the objectmapper for some reason, I can just change it in the designated spring config class fundamentally its weird for objmapper to not be static from what you mentioned, but other than that, are there any implications that I'm missing by not making it static?
dan1st
dan1st3mo ago
not really except that it isn't shared though it could still be shared if you have the same Spring context you'd just have multiple references to the same ObjectMapper you can't use it from static methods but it's really not a problem For Spring specifically, it's fine to make it non-static
Ghostyjangle
GhostyjangleOP3mo ago
ah right, I'll keep a note on that then, thanks for the help.
JavaBot
JavaBot3mo ago
If you are finished with your post, please close it. If you are not, please ignore this message. Note that you will not be able to send further messages here after this post have been closed but you will be able to create new posts.
Ghostyjangle
GhostyjangleOP3mo ago
I'll close the post
JavaBot
JavaBot3mo ago
Post Closed
This post has been closed by <@294662716684107787>.
Want results from more Discord servers?
Add your server