vue3 computed中的代码导致栈溢出？

Question

// index.vue
<custom-calendar :check-date="checkDate" class="calendar-box" />

// CustomCalenda.vaue
<script lang="ts" setup>

const props = defineProps({
  checkDate: {
    type: Array,
    default() {
      return []
    }
  }
})

const minDate = computed(() => {
  debugger
  if (props.checkDate.length) {
    let newArr = props.checkDate.sort((a:any, b:any):number => a.getTime() - b.getTime())
    return new Date(+newArr[0] as number)
  } else {
    return new Date()
  }
})

const maxDate = computed(() => {
  debugger
  if (props.checkDate.length) {
    let newArr = props.checkDate.sort((a:any, b:any):number => b.getTime() - a.getTime()) 
    return new Date(+newArr[0] as number)
  } else {
    return new Date()
  }
})
const curYear = ref<number>(new Date().getFullYear())
const curMonth = ref<number>(new Date().getMonth())
watch(() => maxDate.value, (newVal:Date|null) => {
  debugger
  if (newVal) {
    curYear.value = newVal.getFullYear()
    curMonth.value = newVal.getMonth()
  }
}, {
  immediate: true
})

这段代码控制台报栈溢出

我打了断点，发现代码会在两个computed中无限的执行下去，导致栈溢出，但是我没有找到原因为什么会循环执行？

Answer 1

在Vue中，使用computed属性时，如果依赖的响应式属性发生变化，computed属性会自动重新计算。在你提供的代码中，minDate和maxDate两个computed属性都依赖于props.checkDate数组。如果checkDate数组发生变化，这两个属性都会重新计算，而每个computed属性的计算过程中又都会对checkDate数组进行排序操作，这可能导致它们相互触发重新计算，从而形成无限循环。

可以如下面这样修改看看：

<script lang="ts" setup>
import { ref, computed, watch, onMounted } from 'vue';

const props = defineProps({
  checkDate: {
    type: Array,
    default() {
      return []
    }
  }
});

// 新增一个响应式属性来存储排序后的数组
const sortedCheckDates = ref([]);

const minDate = computed(() => {
  if (sortedCheckDates.value.length) {
    return new Date(sortedCheckDates.value[0].getTime());
  } else {
    return new Date();
  }
});

const maxDate = computed(() => {
  if (sortedCheckDates.value.length) {
    return new Date(sortedCheckDates.value[sortedCheckDates.value.length - 1].getTime());
  } else {
    return new Date();
  }
});

// 监听checkDate的变化，并更新sortedCheckDates
watch(() => props.checkDate, (newVal) => {
  sortedCheckDates.value = newVal.sort((a, b) => a.getTime() - b.getTime());
});

const curYear = ref(new Date().getFullYear());
const curMonth = ref(new Date().getMonth());

watch(() => maxDate.value, (newVal) => {
  if (newVal) {
    curYear.value = newVal.getFullYear();
    curMonth.value = newVal.getMonth();
  }
}, {
  immediate: true
});

onMounted(() => {
  // 初始化sortedCheckDates
  sortedCheckDates.value = props.checkDate.sort((a, b) => a.getTime() - b.getTime());
});
</script>

Answer 2

原因：
computed 中的 props.checkDate.sort 会 track checkDate 的 length 和 索引 属性，并同时 trigger 索引 对应的值，这样有当两个这样的 computed 时，后一个trigger 索引, 会引起前一个 computed 响应，又会触发后一个 computed 响应，从而导致无限循环。解决办法是 slice 一下, 浅拷贝 props.checkDate，而对浅拷贝后的 sort 不会引起这样重复响应

简单解决方法：
对 props.checkDate 排序前先 slice 一下

const minDate = computed(() => {
  if (props.checkDate.length) {
    let newArr = props.checkDate.slice().sort((a, b) => a.getTime() - b.getTime())
    console.log('newArr', JSON.stringify(newArr))
    return new Date(+newArr[0])
  } else {
    return new Date()
  }
})
const maxDate = computed(() => {
  if (props.checkDate.length) {
    let newArr = props.checkDate.slice().sort((a, b) => b.getTime() - a.getTime()) 
    console.log('newArr', JSON.stringify(newArr))
    return new Date(+newArr[0])
  } else {
    return new Date()
  }
})

在线查看：
https://play.vuejs.org/#eNrlVk1P20AQ/SsjX+JQYxe1VQEFJNpSqUgFV...

Answer 3

Array.prototype.toSorted()

Array 实例的 toSorted() 方法是 sort() 方法的复制方法版本。它返回一个新数组，其元素按升序排列。

可以在不改动代码的基础上考虑用这个api 排序试试