
1 wusheng0 2024 年 6 月 3 日 看团队有没有规范吧,没有的话可以做个共识。 不需要共识就按自己喜好,不用顺着谁。 |
2 bfdh 2024 年 6 月 3 日 写新代码,按自己风格; 改旧代码,按原有风格。 |
3 murmur 2024 年 6 月 3 日 你们的 axios 不封装的么,第一层 data 还需要开发者自己取啊 |
4 X0V0X 2024 年 6 月 3 日 via Android 好原始的写法 |
5 iugo 2024 年 6 月 3 日 如果是 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 2024 年 6 月 3 日 不就是有没有用库 hook 的区别吗 你想让别人跟你用相同的库方法,那你就有义务去给人家做好上手指引,例如一份规范之类的、或者你在 code review 中不厌其烦地给对方提供链接和说明。否则就是怎么原始怎么来,毕竟最原始的人人都能看懂 |
9 lyxxxh2 2024 年 6 月 3 日 我以为是(async await) vs then.. 和队友商量呗,不可能商量都商量不了吧。 说服不了,你也可以让下,总比两种风格好。 |
10 xiaoming1992 2024 年 6 月 3 日 via Android 用 swr ,别用 setState |
11 dumbass 2024 年 6 月 3 日 |
12 dumbass 2024 年 6 月 3 日 @zhengfan2016 #6 像这种还要维护 loading 的,推荐用 react-query https://tanstack.com/query/latest/docs/framework/react/overview#enough-talk-show-me-some-code-already |
13 szdubinbin 2024 年 6 月 3 日 风不风格另说,data.data.xxx 记得可选链兜底 ![]() |
14 zhengfan2016 OP @bojackhorseman @xiaoming1992 这个之前和项目老大讨论过了,老大的意思是引入会增加复杂度。不用 swr ,react-query 这些库。我目前的办法是去 swr 源码借鉴了点代码,自己做了个简单的有 cache 功能的低仿 swr 的 hook 。 |
15 ccraohng 2024 年 6 月 4 日 挺答辩的 |
16 weixind 2024 年 6 月 4 日 @zhengfan2016 你们项目老大思路有点问题。明显是要使用社区解决方法。尽量少用自己写的内容。自己写复杂度才会更高。社区起码代码质量、文档质量要比团队内部写要好很多。 |
17 ivvei 2024 年 6 月 4 日 各写各的。 |