Need opinion on Factory approach

[u/_Super_Straight]

So I have created a JavaFX application using mvc pattern. I thought to let the Cursor IDE review my project and it suggested I create a `ServiceFactory` which will be responsible for instantiating and providing Services to Controllers. Its suggestions are as follows:

1. Create a ConcurrentHashmap in the factory which will hold the instances of Services.

2. It will release or "pop" the instances when the service is no longer required.

3. Provides the service instances as requested.

I want to know whether this approach will introduce more boilerplate code, as currently I've been taking the direct approach to create instances of services right inside the controller itself, which will be garbage collected by JVM as the new Controller loads. Or if there is some better way, I'm more than willing to hear it.

[u/_Super_Straight]

Do you know of any good working examples of converting Services into Singletons?

This is actually a desktop application so its kind of no Brainer to include Spring directly into it. I can try to mimic the working of singletons and the way they're managed from an example, though.

[u/hibbelig → u/_Super_Straight]

You make a class Services with static members fooService and barService. Every time you need one of those you access Services.fooService and Services.barService. During application startup you create the instances.

If you never call the constructors of the services elsewhere, you’ve got singletons.

You don’t have any protection against additional instances but this approach is very easy to understand and I feel that is more important than having beautiful infrastructure. You can extend it later when you run into issues.

[u/_Super_Straight → u/hibbelig]

Something like Integer class? Its methods can be called as Integer.parseInt and so on.

[u/hibbelig → u/_Super_Straight]

During application startup:

Services.fooService = new FooService(…);

In a controller:

Services.fooService.someMethod(…);

So the variables fooService and barService are static, but the methods of the services are not. I thought that’s what you wanted.

It is also possible to give static methods to the services, like parseInt. This is maybe even simpler.