Gestione di servizi multifunzione in rapida crescita

Non sono un esperto di Spring ma dall'esperienza di progettazione / architettura generale mi sembra che troppi metodi per una singola API / servizio. È una specie di odore di design vedere questi molti metodi in un'unica interfaccia.

Dovresti provare a suddividerlo in aree di funzionalità logicamente distinte, ciascuna con un'interfaccia separata. Quindi il codice per implementare ogni area distinta dovrebbe diventare gestibile.

Se hai un codice client difficile da modificare, allora è meglio mantenere la vecchia versione del servizio - basta trasformarlo in una facciata leggera che chiama nelle nuove versioni refactored. I tuoi clienti non dovranno cambiare nulla se non lo desiderano (anche se dovresti chiarire che dovranno passare alle nuove interfacce se vogliono accedere a qualsiasi funzionalità futura).

Dai giusti metodi elencati nella domanda precedente potresti avere almeno quattro domini separati:

• Gestione utenti
• Gestione dei membri
• Registrazioni (registrati, ecc.)
• Nomination (nominateMember ecc.)
• ecc.

Sembra che tu non possa dividere l'interfaccia di servizio in classi / servizi diversi, per mantenere la retrocompatibilità.

Ciò che dovrebbe fare, però, è dividere la funzionalità in classi in base a SRP, e quindi (best-case) avere interfacce per tali classi come dipendenze nel costruttore della classe di servizio o (peggio -case) instatialo tutti nel costruttore.

Ogni metodo nella classe di servizio diventa una singola linea di bontà, e il resto della tua applicazione può essere saggiamente pensato senza modificare l'API pubblica.

Esempio:

public class Service : IUserService
{
    private IUserRepository userRepo;
    private IMemberStuff memStuff;

    public Service(IUserRepository userRepo, IMemberStuff memStuff)
    {
       this.userRepo = userRepo;
       this.memStuff = memStuff;
    }

    public void addUser(...)
    {
        userRepo.addUser(...);
    }

    public void deleteUser(...)
    {
    }

    public void updateUser(...)
    {
    }

    public void addMember(...)
    {
    }

    public void deleteMember(...)
    {
    }

    public void updateMember(...)
    {
    }

    public void registerMember(...)
    {
    }

    public void deactivateMember(...)
    {
    }

    public Member getExpiringMember(...)
    {
    }

    public void nominateMember(...)
    {
    }
}