Un cycle de dépendances


***************************
APPLICATION FAILED TO START
***************************

Description:

The dependencies of some of the beans in the application context form a cycle:

┌─────┐
|  panierService defined in URL [jar:nested: /livre/code/ch2/erreurs/cycle/target/cycle-0.0.1-SNAPSHOT.jar/!BOOT-INF/classes/!/fr/librairie/PanierService.class]
↑     ↓
|  stockService defined in URL [jar:nested: /livre/code/ch2/erreurs/cycle/target/cycle-0.0.1-SNAPSHOT.jar/!BOOT-INF/classes/!/fr/librairie/StockService.class]
└─────┘

Action:

Relying upon circular references is discouraged and they are prohibited by default. Update your application to remove the dependency cycle between beans. As a last resort, it may be possible to break the cycle automatically by setting spring.main.allow-circular-references to true.

Pour créer PanierService, il faut d'abord un StockService ; pour créer celui-ci, il faut d'abord un PanierService : avec l'injection par constructeur, aucun des deux ne peut être construit en premier. Spring le détecte et s'arrête (le schéma du rapport dessine le cycle). La bonne correction n'est pas technique : un cycle trahit des responsabilités mal réparties. Le stock n'a pas à connaître le panier ; c'est le panier qui réserve auprès du stock :


@Service
public class PanierService {

    private final StockService stock;

    public PanierService(StockService stock) {
        this.stock = stock;
    }

    public String ajouter(String isbn, int quantite) {
        return stock.reserver(isbn, quantite) ? "ajouté" : "stock insuffisant";
    }
}


/** Le stock ne connaît plus le panier : le cycle est rompu. */
@Service
public class StockService {

    private final Map<String, Integer> stock = new ConcurrentHashMap<>(Map.of("9782956300014", 12));

    public synchronized boolean reserver(String isbn, int quantite) {
        int dispo = stock.getOrDefault(isbn, 0);
        if (dispo < quantite) {
            return false;
        }
        stock.put(isbn, dispo - quantite);
        return true;
    }
}

synchronized rend la réservation indivisible : deux requêtes simultanées ne peuvent pas réserver le même dernier exemplaire.

Énoncé

Corrigé