写 Vue 我建议非必要别用 watch
source link: https://ssshooter.com/vue-watch/
Go to the source link to view the article. You can view the picture content, updated content and better typesetting reading experience. If the link is broken, please click the button below to view the snapshot at that time.
写 Vue 我建议非必要别用 watch
代码大概如下，删除了很多无关内容。
<template>
<div>
<SearchBar @search="handleSearch" />
<Pagination
v-model:page="pagination.page"
:page-size="pagination.pageSize"
:total="pagination.total"
/>
</div>
</template>
<script setup lang="ts">
import { reactive, ref, watch, inject, computed } from 'vue'
import SearchBar from '@/components/SearchBar.vue'
const route = useRoute()
const pagination = reactive({
page: 1,
pageSize: isPublic.value ? 10 : 9,
total: 0,
})
const keyword = ref('')
const fetchList = async () => {
loading.value = true
const res = await connect.get(`/api/${route.params.type}`, {
params: {
pageSize: pagination.pageSize,
page: pagination.page,
name: keyword.value,
},
})
pagination.total = res.total
loading.value = false
}
watch(
() => route.params.type,
async () => {
pagination.page = 1
fetchList()
},
{ immediate: true }
)
watch(
() => pagination.page,
async () => {
fetchList()
}
)
watch(
() => keyword.value,
async () => {
if (pagination.page === 1) fetchList()
else {
pagination.page = 1
}
}
)
const handleSearch = (val: string) => {
keyword.value = val
}
const handleDelete = async (item: MindMapItem) => {
await confirmModal.value?.confirm()
await connect.delete('/api/map/' + item._id)
fetchList()
}
</script>本来只有 2 个 watch，今天新功能加了个关键词搜索，又得多 watch 一个 keyword.value。
于是这里变成了 3 个 watch，而且里面有逻辑，甚至是相互依赖的逻辑。
上面的代码没写完，但是整理一下，最终目标是这样的：
- 请求参数有三个变量：route.params.type、keyword 和 pagination
- route.params.type 改变时需要重置 pagination 和 keyword，然后重新请求
- keyword 改变时需要重制 pagination，然后重新请求
- pagination 改变时需要重新请求
watch 真的好？
如果继续用 watch，因为需要重置 pagination 和 keyword，硬生生把三个 watch 写成了个像是任务委托一样的效果，例如 keyword.value 修改时如果 page 是 1 就直接请求，否则修改 page 再让 page 的 watch 触发请求。
watch(
() => keyword.value,
async () => {
if (pagination.page === 1) fetchList()
else {
pagination.page = 1
}
}
)这么耦合真的好吗？这不好。我劝自己耗子尾汁，好好反思。
得出的结论是：watch 不是好文明，能不用 watch，就别用 watch。
这不是我第一次对 watch 有意见，在工作中我就见过很多复杂组件动则 5 个以上的 watch，有的里面还有复杂逻辑。
重点是啥，还没注释……watch 天然就容易让人不写注释，给人一种“啊，这个值变了，运行下面的逻辑是理所当然的吧。”，那你问问两个月后的自己，是不是真的这样？你自己写的 watch 你自己看得懂吗？一个值变了就触发逻辑，但问题是，它变的原因可多了。
所以 watch 生而在语义上不明确，它只解释了对值的依赖，没有解释依赖的原因。
watchEffect 呢？
上面的例子，假如把 fetchList 写成 watchEffect，其实还是一样的问题，需要在里面额外加 if else 处理重置逻辑。不过逻辑集中在一个 watchEffect 大概还是比分散在 N 个 watch 里好。
总结一下，watch 或者 watchEffect 有其用武之地，但最好满足以下的条件：
- 变动触发点大于 2 个才考虑
watch（只有一个触发机会的话，什么时候用，什么时候跑就好了） - 所有场景全都适用同一个处理逻辑
- 与其他 watch 没耦合
不过如果没有事件机制来触发的话，那就只能 watch 了。
<template>
<div>
<SearchBar @search="handleSearch" />
<Pagination
v-model:page="pagination.page"
@update:page="fetchList"
:page-size="pagination.pageSize"
:total="pagination.total"
/>
</div>
</template>
<script setup lang="ts">
import { reactive, ref, watch, inject, computed } from 'vue'
import SearchBar from '@/components/SearchBar.vue'
const route = useRoute()
const pagination = reactive({
page: 1,
pageSize: isPublic.value ? 10 : 9,
total: 0,
})
const keyword = ref('')
const fetchList = async () => {
// 省略
}
watch(
() => route.params.type,
async () => {
keyword.value = ''
pagination.page = 1
fetchList()
},
{ immediate: true }
)
const handleSearch = (val: string) => {
keyword.value = val
pagination.page = 1
fetchList()
}
const handleDelete = async (item: MindMapItem) => {
await confirmModal.value?.confirm()
await connect.delete('/api/map/' + item._id)
fetchList()
}
</script>修改后，只保留 route.params.type 的 watch，不会发生冲突，另外两个通过事件触发。至于触发事件也不用额外写 @change，直接用 @update:xxx 就可以了。
这样只有易读的重置逻辑，没有 if else！清爽！
Recommend
About Joyk
Aggregate valuable and interesting links.
Joyk means Joy of geeK