这是一个创建于 183 天前的主题，其中的信息可能已经有所发展或是发生改变。
如题，我自己的页面，被注释的是我习惯的写法。
感觉我的写法偏向解构流(只 set 接口返回的 data ，剩下的解构出来用)，它的写法偏向 setState 流(url ，image 什么都要 set)😂😂
imgur.com/a/vFaFJBu
感觉我的写法偏向解构流(只 set 接口返回的 data ，剩下的解构出来用)，它的写法偏向 setState 流(url ，image 什么都要 set)😂😂
imgur.com/a/vFaFJBu
17 条回复 • 2024-06-04 18:36:34 +08:00
 |
1
wusheng0 183 天前
看团队有没有规范吧，没有的话可以做个共识。
不需要共识就按自己喜好，不用顺着谁。 |
 |
2
bfdh 183 天前
写新代码，按自己风格；
改旧代码，按原有风格。 |
 |
3
murmur 183 天前
你们的 axios 不封装的么，第一层 data 还需要开发者自己取啊
|
 |
4
X0V0X 183 天前 via Android
好原始的写法
|
 |
5
iugo 183 天前
如果是 React, state 储存对象还是基本数据类型要看具体场景需要. 从代码中看, 如果只用到了 url, 没必要将整个 data 作为 state.
如果 state 储存对象, 需要遵循 immutable 避免. 不过我更建议先将 API 从页面中剥离. |
 |
6
zhengfan2016 OP @murmur 有封装的 appaxios ，主要第一层里除了 data 还有 code 等其他数据，有时会用到
|
|
7
zhengfan2016 OP @X0V0X 其实是有 hook 的，用 react-use ，ahook 之类代码量少很多，但感觉队友更喜欢用 useState 和 useEffect
|
 |
8
Xu3Xan89YsA7oP64 183 天前
不就是有没有用库 hook 的区别吗
你想让别人跟你用相同的库方法，那你就有义务去给人家做好上手指引，例如一份规范之类的、或者你在 code review 中不厌其烦地给对方提供链接和说明。否则就是怎么原始怎么来，毕竟最原始的人人都能看懂 |
 |
9
lyxxxh2 183 天前
我以为是(async await) vs then..
和队友商量呗,不可能商量都商量不了吧。 说服不了,你也可以让下,总比两种风格好。 |
 |
10
xiaoming1992 183 天前 via Android
用 swr ，别用 setState
|
 |
11
bojackhorseman 183 天前
提个不相关的建议，可以 .finally(() => { setLoading(false) }) 避免写两遍
 |
|
12
bojackhorseman 183 天前
@zhengfan2016 #6 像这种还要维护 loading 的，推荐用 react-query https://tanstack.com/query/latest/docs/framework/react/overview#enough-talk-show-me-some-code-already
|
 |
13
szdubinbin 183 天前
风不风格另说，data.data.xxx 记得可选链兜底
 |
|
14
zhengfan2016 OP @bojackhorseman @xiaoming1992 这个之前和项目老大讨论过了，老大的意思是引入会增加复杂度。不用 swr ，react-query 这些库。我目前的办法是去 swr 源码借鉴了点代码，自己做了个简单的有 cache 功能的低仿 swr 的 hook 。
|
 |
15
ccraohng 182 天前
挺答辩的
|
 |
16
weixind 182 天前
@zhengfan2016 你们项目老大思路有点问题。明显是要使用社区解决方法。尽量少用自己写的内容。自己写复杂度才会更高。社区起码代码质量、文档质量要比团队内部写要好很多。
|
 |
17
ivvei 182 天前
各写各的。
|
Select Language