Fonction d'achat de produit avec gestion de stock en TypeScript

Question

Voici la solution que j'ai mise en place. Je me demande si c'est pratique. Aussi, pourtant ça fonctionne bien.
function buyProduct(listProducts: StockedProduct[], id: number, quantity: number): StockedProduct[] | null {
let isAvailable: boolean = false;
listProducts.map((product) => {
if (product.id === id && product.quantity >= quantity) {
product.quantity -= quantity;
isAvailable = true;
}
});
return isAvailable ? listProducts : null;
}

Louis-Nicolas Leuillet · Accepted Answer

Alors, ta solution fonctionne, oui, mais il y a un souci avec l'utilisation de map() ici. En fait, map() est conçu pour transformer un tableau et retourner un nouveau tableau. Toi, tu l'utilises juste pour parcourir et modifier en place, donc forEach() serait plus adapté sémantiquement.
Autre point : tu parcours tout le tableau même après avoir trouvé le produit. C'est pas dramatique sur une petite liste, mais c'est pas optimal. Une boucle for avec un break ou un find() serait plus efficace.
Voici une version un peu plus propre :
function buyProduct(
listProducts: StockedProduct[],
id: number,
quantity: number
): StockedProduct[] | null {
const product = listProducts.find(p => p.id === id);
if (product && product.quantity >= quantity) {
product.quantity -= quantity;
return listProducts;
}
return null;
}
Avec find() , tu récupères directement le produit concerné, tu vérifies la quantité, et tu modifies. Plus lisible et plus direct. Mais bon, ta solution marche, c'est déjà bien !

Fonction d'achat de produit avec gestion de stock en TypeScript

Formation TypeScript