前端问题,假如为一个被很多地方用的基层组件加新功能,你会怎么做 - V2EX
V2EX = way to explore
V2EX 是一个关于分享和探索的地方
Sign Up Now
For Existing Member  Sign In
请不要在回答技术问题时复制粘贴 AI 生成的内容
zhengfan2016

前端问题,假如为一个被很多地方用的基层组件加新功能,你会怎么做

  •  
  •   zhengfan2016 Oct 31, 2024 2268 views
    This topic created in 546 days ago, the information mentioned may be changed or developed.

    假如有这样一个 NavBar 组件,被 50%以上的页面使用,下面 React 简略代码

    interface NavbarProps { BackAction?: ({ navigation, ...rest }) => JSX.Element Content?: ({ title, ...rest }) => JSX.Element Action?: ({ ...rest }) => JSX.Element title?: string } const Navbar = (props:NavbarProps)=>{ const {BackAction = DefaultBackAction, COntent= DefaultContent, Action = DefaultAction, title, } = props || {} return <div className="flex flex-row"> <BackAction {...defaultBackActionProps}> <Content {...{...defaultContentProps,title}}/> <Action {...defaultActionProps}/> </div> } // demo return <Navbar title="title" /> 

    现在你要给这个组件的 title 加一个标题为当前播放音乐主题色的功能,但是只有 1 个页面会用到这个功能

    const color = useMusicThemeColor() 

    请问你选择哪种方案?

    1.增加一个 isMusicTheme=true ,然后把 useMusicThemeColor 放在 NavBar 内部

     <Navbar title="title" isMusicTheme={true} /> 

    2.override Content ,把 useMusicThemeColor 放当前 page

    const color = useMusicThemeColor() ... <Navbar title="title" COntent={({...rest,title})=>{ return <div {...rest} style={{color}}> {title} </div> }} /> 
    11 replies    2024-11-01 17:09:30 +08:00
    chenliangngng
        1
    chenliangngng  
       Oct 31, 2024
    你可以把组件套一层壳,然后加上你要的功能,不当公共组件
    shintendo
        2
    shintendo  
       Oct 31, 2024
    不能让 title 可以接受 JSX.Element 吗
    zhengfan2016
        3
    zhengfan2016  
    OP
       Oct 31, 2024 via Android
    @shintendo 建议您阅读一下代码里 Content 组件,思考为什么 Content 组件会接收 Navbar 传过来的 title
    shintendo
        4
    shintendo  
       Oct 31, 2024
    @zhengfan2016 确实,没仔细看。我选择加个 titleStyle
    llwxi
        5
    llwxi  
       Oct 31, 2024
    先用第二种实现,后续如果有两个组件使用到了,封装一个<MusicThemeColorText>{title}</MusicThemeColorText> 之类的组件?
    zhengfan2016
        6
    zhengfan2016  
    OP
       Oct 31, 2024 via Android
    @llwxi 没错,这就是我想看到的答案。

    但是就我观察大部分前端喜欢用 1 的写法,特别是用 1 的写法来改我的代码,把整个组件改的很臃肿,让我很不爽。
    zb1141920796
        7
    zb1141920796  
       Oct 31, 2024
    图省事就用 1 ,图可扩展性强一点就用 2,
    snowlee
        8
    snowlee  
       Nov 1, 2024
    我会把 title 变成 reactNode, 相传啥就传啥
    lisongeee
        9
    lisongeee  
       Nov 1, 2024
    没看懂你的操作,内部直接用 css 变量不行吗?

    然后直接在目标页面给 body 设置 css 变量,离开后移除此变量
    monologue520
        10
    monologue520  
       Nov 1, 2024
    1.设 flag 控制,如果确定只有一个页面发生变化未尝不可,比较直观简单
    2.增加了组件拓展性,控制当前页的颜色,还有其他的页的 title 默认颜色,可以抽离一个 Title 组件维护 color 属性
    3.使用样式变量,比如 cssinjs 写法中「 const color = useMusicThemeColor()」传递一个 props 给 title 样式,最简单方便
    我个人选 3
    (用不着不爽,沟通一下就可以了)
    iOCZS
        11
    iOCZS  
       Nov 1, 2024
    要是既可以简单传标题,又可以自定义标题内容就好了。vue slot 那套还是比较合适这个想法的。
    About     Help     Advertise     Blog     API     FAQ     Solana     2692 Online   Highest 6679       Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 47ms UTC 02:38 PVG 10:38 LAX 19:38 JFK 22:38
    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