fix build error with more readable code

not related with the change but resulting in build error
bitcoinsearch · Nov 14, 2024 · cd08ea6 · cd08ea6
1 parent f4b3274
commit cd08ea6
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 5 deletions.
diff --git a/src/context/SearchQueryContext.tsx b/src/context/SearchQueryContext.tsx
@@ -80,7 +80,17 @@ export const SearchQueryProvider = ({
   }, [rawSearchQuery]);
 
   const page = useMemo(() => {
-    return pageQuery ? parseInt(pageQuery) - 1 ?? 0 : 0;
+    // Handle empty or invalid input
+    if (!pageQuery) {
+      return 0;
+    }
+
+    // Convert to number and validate
+    const parsedPage = Number(pageQuery);
+    const isValidPage = !isNaN(parsedPage) && parsedPage > 0;
+
+    // Convert from 1-based to 0-based index, or default to 0
+    return isValidPage ? parsedPage - 1 : 0;
   }, [pageQuery]);
 
   const resultsPerPage = sizeQuery

diff --git a/src/pages/index.tsx b/src/pages/index.tsx
@@ -55,10 +55,10 @@ export const getServerSideProps: GetServerSideProps = async (
     };
   }
 
-  const page = pageQuery ? parseInt(pageQuery) - 1 ?? 0 : 0;
-  const size = sizeQuery
-    ? parseInt(sizeQuery) ?? defaultParam[URLSearchParamsKeyword.SIZE]
-    : defaultParam[URLSearchParamsKeyword.SIZE];
+  const defaultPage = 0;
+  const page = pageQuery ? parseInt(pageQuery) - 1 : defaultPage;
+  const defaultSize = defaultParam[URLSearchParamsKeyword.SIZE];
+  const size = sizeQuery ? parseInt(sizeQuery) || defaultSize : defaultSize;
 
   const options = {
     queryString,