fix: review

src/packages/calendarcard/calendarcard.taro.tsx

@@ -1,4 +1,4 @@
-import React, { ReactNode, useEffect, useState } from 'react'
+import React, { ReactNode, useCallback, useEffect, useState } from 'react'
 import classNames from 'classnames'
 import { View } from '@tarojs/components'
 import { ArrowLeft, ArrowRight, DoubleLeft, DoubleRight } from './icon.taro'
@@ -124,27 +124,30 @@ export const CalendarCard = React.forwardRef<
     }
   }
 
-  const getDays = (month: CalendarCardMonth) => {
-    const y = month.year
-    const m = month.month
-    const days = [
-      ...getPrevMonthDays(y, m, firstDayOfWeek),
-      ...getCurrentMonthDays(y, m),
-    ] as CalendarCardDay[]
-    const size = days.length
-    const yearOfNextMonth = month.month === 12 ? month.year + 1 : month.year
-    const monthOfNextMonth = month.month === 12 ? 1 : month.month + 1
-    // 补全 6 行 7 列视图
-    for (let i = 1; i <= 42 - size; i++) {
-      days.push({
-        type: 'next',
-        year: yearOfNextMonth,
-        month: monthOfNextMonth,
-        date: i,
-      })
-    }
-    return days
-  }
+  const getDays = useCallback(
+    (month: CalendarCardMonth) => {
+      const y = month.year
+      const m = month.month
+      const days = [
+        ...getPrevMonthDays(y, m, firstDayOfWeek),
+        ...getCurrentMonthDays(y, m),
+      ] as CalendarCardDay[]
+      const size = days.length
+      const yearOfNextMonth = month.month === 12 ? month.year + 1 : month.year
+      const monthOfNextMonth = month.month === 12 ? 1 : month.month + 1
+      // 补全 6 行 7 列视图
+      for (let i = 1; i <= 42 - size; i++) {
+        days.push({
+          type: 'next',
+          year: yearOfNextMonth,
+          month: monthOfNextMonth,
+          date: i,
+        })
+      }
+      return days
+    },
+    [firstDayOfWeek]
+  )
 
   useEffect(() => {
     const newDays = getDays(month)

src/packages/calendarcard/calendarcard.tsx

@@ -1,4 +1,4 @@
-import React, { useState, useEffect, ReactNode } from 'react'
+import React, { ReactNode, useEffect, useState, useCallback } from 'react'
 import classNames from 'classnames'
 import { ArrowLeft, ArrowRight, DoubleLeft, DoubleRight } from './icon'
 import { BasicComponent, ComponentDefaults } from '@/utils/typings'
@@ -34,6 +34,7 @@ export interface CalendarCardProps extends BasicComponent {
   onPageChange: (data: CalendarCardMonth) => void
   onChange: (value: CalendarCardValue) => void
 }
+
 const defaultProps = {
   ...ComponentDefaults,
   type: 'single',
@@ -122,27 +123,30 @@ export const CalendarCard = React.forwardRef<
     }
   }
 
-  const getDays = (month: CalendarCardMonth) => {
-    const y = month.year
-    const m = month.month
-    const days = [
-      ...getPrevMonthDays(y, m, firstDayOfWeek),
-      ...getCurrentMonthDays(y, m),
-    ] as CalendarCardDay[]
-    const size = days.length
-    const yearOfNextMonth = month.month === 12 ? month.year + 1 : month.year
-    const monthOfNextMonth = month.month === 12 ? 1 : month.month + 1
-    // 补全 6 行 7 列视图
-    for (let i = 1; i <= 42 - size; i++) {
-      days.push({
-        type: 'next',
-        year: yearOfNextMonth,
-        month: monthOfNextMonth,
-        date: i,
-      })
-    }
-    return days
-  }
+  const getDays = useCallback(
+    (month: CalendarCardMonth) => {
+      const y = month.year
+      const m = month.month
+      const days = [
+        ...getPrevMonthDays(y, m, firstDayOfWeek),
+        ...getCurrentMonthDays(y, m),
+      ] as CalendarCardDay[]
+      const size = days.length
+      const yearOfNextMonth = month.month === 12 ? month.year + 1 : month.year
+      const monthOfNextMonth = month.month === 12 ? 1 : month.month + 1
+      // 补全 6 行 7 列视图
+      for (let i = 1; i <= 42 - size; i++) {
+        days.push({
+          type: 'next',
+          year: yearOfNextMonth,
+          month: monthOfNextMonth,
+          date: i,
+        })
+      }
+      return days
+    },
+    [firstDayOfWeek]
+  )
 
   useEffect(() => {
     const newDays = getDays(month)