From 82df61d6058985d6c43cdcdb56188201882e96c8 Mon Sep 17 00:00:00 2001 From: bender Date: Wed, 11 Mar 2026 19:40:52 +0000 Subject: [PATCH] Switch to version 1: modified src/hooks/useProduct.ts --- src/hooks/useProduct.ts | 59 ++++++++++++++++++++++++++++------------- 1 file changed, 40 insertions(+), 19 deletions(-) diff --git a/src/hooks/useProduct.ts b/src/hooks/useProduct.ts index 342835f..3407f3a 100644 --- a/src/hooks/useProduct.ts +++ b/src/hooks/useProduct.ts @@ -1,24 +1,45 @@ -'use client'; +"use client"; -import { useState, useEffect } from 'react'; -import { Product, fetchProductList } from '@/lib/api/product'; +import { useEffect, useState } from "react"; +import { Product, fetchProduct } from "@/lib/api/product"; -export function useProduct(id: string) { - const [product, setProduct] = useState(null); - const [loading, setLoading] = useState(true); +export function useProduct(productId: string) { + const [product, setProduct] = useState(null); + const [isLoading, setIsLoading] = useState(true); + const [error, setError] = useState(null); - useEffect(() => { - const loadProduct = async () => { - try { - const products = await fetchProductList(); - const found = products.find(p => p.id === id); - setProduct(found || null); - } finally { - setLoading(false); - } - }; - loadProduct(); - }, [id]); + useEffect(() => { + let isMounted = true; - return { product, loading }; + async function loadProduct() { + if (!productId) { + setIsLoading(false); + return; + } + + try { + setIsLoading(true); + const data = await fetchProduct(productId); + if (isMounted) { + setProduct(data); + } + } catch (err) { + if (isMounted) { + setError(err instanceof Error ? err : new Error("Failed to fetch product")); + } + } finally { + if (isMounted) { + setIsLoading(false); + } + } + } + + loadProduct(); + + return () => { + isMounted = false; + }; + }, [productId]); + + return { product, isLoading, error }; }