和队友代码风格有差异,顺着队友的风格写还是? - V2EX
V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
zhengfan2016
V2EX    职场话题

和队友代码风格有差异,顺着队友的风格写还是?

  •  
  •   zhengfan2016 2024 年 6 月 3 日 via Android 2622 次点击
    这是一个创建于 588 天前的主题,其中的信息可能已经有所发展或是发生改变。
    如题,我自己的页面,被注释的是我习惯的写法。

    感觉我的写法偏向解构流(只 set 接口返回的 data ,剩下的解构出来用),它的写法偏向 setState 流(url ,image 什么都要 set)

    imgur.com/a/vFaFJBu
    17 条回复    2024-06-04 18:36:34 +08:00
    wusheng0
        1
    wusheng0  
       2024 年 6 月 3 日
    看团队有没有规范吧,没有的话可以做个共识。
    不需要共识就按自己喜好,不用顺着谁。
    bfdh
        2
    bfdh  
       2024 年 6 月 3 日
    写新代码,按自己风格;
    改旧代码,按原有风格。
    murmur
        3
    murmur  
       2024 年 6 月 3 日
    你们的 axios 不封装的么,第一层 data 还需要开发者自己取啊
    X0V0X
        4
    X0V0X  
       2024 年 6 月 3 日 via Android
    好原始的写法
    iugo
        5
    iugo  
       2024 年 6 月 3 日
    如果是 React, state 储存对象还是基本数据类型要看具体场景需要. 从代码中看, 如果只用到了 url, 没必要将整个 data 作为 state.

    如果 state 储存对象, 需要遵循 immutable 避免.

    不过我更建议先将 API 从页面中剥离.
    zhengfan2016
        6
    zhengfan2016  
    OP
       2024 年 6 月 3 日 via Android
    @murmur 有封装的 appaxios ,主要第一层里除了 data 还有 code 等其他数据,有时会用到
    zhengfan2016
        7
    zhengfan2016  
    OP
       2024 年 6 月 3 日 via Android
    @X0V0X 其实是有 hook 的,用 react-use ,ahook 之类代码量少很多,但感觉队友更喜欢用 useState 和 useEffect
    Xu3Xan89YsA7oP64
        8
    Xu3Xan89YsA7oP64  
       2024 年 6 月 3 日
    不就是有没有用库 hook 的区别吗
    你想让别人跟你用相同的库方法,那你就有义务去给人家做好上手指引,例如一份规范之类的、或者你在 code review 中不厌其烦地给对方提供链接和说明。否则就是怎么原始怎么来,毕竟最原始的人人都能看懂
    lyxxxh2
        9
    lyxxxh2  
       2024 年 6 月 3 日
    我以为是(async await) vs then..
    和队友商量呗,不可能商量都商量不了吧。
    说服不了,你也可以让下,总比两种风格好。
    xiaoming1992
        10
    xiaoming1992  
       2024 年 6 月 3 日 via Android
    用 swr ,别用 setState
    dumbass
        11
    dumbass  
       2024 年 6 月 3 日
    提个不相关的建议,可以 .finally(() => { setLoading(false) }) 避免写两遍
    dumbass
        12
    dumbass  
       2024 年 6 月 3 日
    szdubinbin
        13
    szdubinbin  
       2024 年 6 月 3 日
    风不风格另说,data.data.xxx 记得可选链兜底
    zhengfan2016
        14
    zhengfan2016  
    OP
       2024 年 6 月 3 日 via Android
    @bojackhorseman @xiaoming1992 这个之前和项目老大讨论过了,老大的意思是引入会增加复杂度。不用 swr ,react-query 这些库。我目前的办法是去 swr 源码借鉴了点代码,自己做了个简单的有 cache 功能的低仿 swr 的 hook 。
    ccraohng
        15
    ccraohng  
       2024 年 6 月 4 日
    挺答辩的
    weixind
        16
    weixind  
       2024 年 6 月 4 日
    @zhengfan2016 你们项目老大思路有点问题。明显是要使用社区解决方法。尽量少用自己写的内容。自己写复杂度才会更高。社区起码代码质量、文档质量要比团队内部写要好很多。
    ivvei
        17
    ivvei  
       2024 年 6 月 4 日
    各写各的。
    关于     帮助文档     自助推广系统     博客     API     FAQ     Solana     3431 人在线   最高记录 6679       Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 30ms UTC 10:46 PVG 18:46 LAX 02:46 JFK 05:46
    Do have faith in what you're doing.
    ubao msn snddm index pchome yahoo rakuten mypaper meadowduck bidyahoo youbao zxmzxm asda bnvcg cvbfg dfscv mmhjk xxddc yybgb zznbn ccubao uaitu acv GXCV ET GDG YH FG BCVB FJFH CBRE CBC GDG ET54 WRWR RWER WREW WRWER RWER SDG EW SF DSFSF fbbs ubao fhd dfg ewr dg df ewwr ewwr et ruyut utut dfg fgd gdfgt etg dfgt dfgd ert4 gd fgg wr 235 wer3 we vsdf sdf gdf ert xcv sdf rwer hfd dfg cvb rwf afb dfh jgh bmn lgh rty gfds cxv xcv xcs vdas fdf fgd cv sdf tert sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf shasha9178 shasha9178 shasha9178 shasha9178 shasha9178 liflif2 liflif2 liflif2 liflif2 liflif2 liblib3 liblib3 liblib3 liblib3 liblib3 zhazha444 zhazha444 zhazha444 zhazha444 zhazha444 dende5 dende denden denden2 denden21 fenfen9 fenf619 fen619 fenfe9 fe619 sdf sdf sdf sdf sdf zhazh90 zhazh0 zhaa50 zha90 zh590 zho zhoz zhozh zhozho zhozho2 lislis lls95 lili95 lils5 liss9 sdf0ty987 sdft876 sdft9876 sdf09876 sd0t9876 sdf0ty98 sdf0976 sdf0ty986 sdf0ty96 sdf0t76 sdf0876 df0ty98 sf0t876 sd0ty76 sdy76 sdf76 sdf0t76 sdf0ty9 sdf0ty98 sdf0ty987 sdf0ty98 sdf6676 sdf876 sd876 sd876 sdf6 sdf6 sdf9876 sdf0t sdf06 sdf0ty9776 sdf0ty9776 sdf0ty76 sdf8876 sdf0t sd6 sdf06 s688876 sd688 sdf86