拆分成更小的组件便于复用 麻烦老师帮忙看看是否正确

来源:3-5 切换Tab组件

ywang04

2019-07-14

老师 接之前关于属性取舍的问题 我把<li>从<ul>里拆分出来作为一个单独的Tab组件 下面是我的代码 本机上运行没有问题 希望老师帮忙看下有没有更优化的做法? 谢谢!补充:图片需要点击一下才可以看到完整版本。。

http://img.mukewang.com/szimg/5d2b1e5f09060fd414940602.jpg

http://img.mukewang.com/szimg/5d2b1e7e09736cea08260276.jpg

写回答

1回答

张轩

2019-07-15

同学 谢谢你的思考和优化,很不错的完成。之前我们讨论的点是组件应该更抽象,还是更具象。

我觉得作为我本人,如果让组件更抽象的话,可能会考虑一下几个优化的点。

1,Tab 组件中 onTabChange 没有被触发,这个应该加上

2, 我们现在是抽取一个更加抽象,更加和项目无关的组件,所以我们不妨更抽象一些,现在 Tab 内内容自定义程度还是比较低,我在想把要显示的作为它的 children 是否更妙?

<Tab>
   <p>这里面可以是任何 jsx 结构</p>
</Tab>

3,现在既然有单个的Tab,那么肯定是有外围的控制 Tabs,为了更好的重用,我觉得把整体的 Tabs 作为一个整体给别人用,会感觉更容易使用。

<Tabs>
    <Tab>This is Tab one</Tab>
    <Tab>This is Tab 2!</Tab>
    ...
</Tabs>

 

0
3
ywang04
回复
张轩
现在回过头来看老师的回复 已经可以理解了。谢谢
2020-11-01
共3条回复

React16组件化+测试+全流程 实战在线账本项目

轻松上手,从设计图到上线,精通组件化思维和组件测试

713 学习 · 177 问题

查看课程