某一线前端小组长的 Code Review 分享(第二弹!)

☀️ 前言

大家平时网上搜索能搜出一堆Code Review 实践、Code Review 流程、如何如何 Code Review 等等。好像实际的 review 记录并不多,恰好本人有幸担任前端小组长一前端职,平时摸鱼就去偷偷看看大家又写了什么优秀代码。所以今天就由小弟我,给大家分享一下,小厂一线前端平时写的代码

作者能力有限,里面的评论不一定十分正确,仅做记录,抛砖引玉,以供学习参考~

上一篇文章变量的指针其含义是指该变量的收获了挺多同学的点赞和评论,所以今天继续给大家带来第二波分享,希前端和后端望能给大家带来思考和收获~

本文是该系列的第二篇
上一篇在这里:juejin.cn/post变量与函数/705257…

Talk is cheap. Sho前端开发w me t变量的定义he code!

项目背景:

  • react 16.8+
  • antd@4

使用原生 js 获取 dom 节点,没有使用一个变量来维护 id 值

某一线前端小组长的 Code Review 分享(第二弹!)

方便后续维护

思维固化

某一线前端小组长的 Code Review 分享(第二弹!)

这位同学可能忘记format函数的入参是可变量以改的了

逻辑不够抽象

某一线前端小组长的 Code Review 分享(第二弹!)

简单问题复杂化,这一堆 && 是不是看的头大。。

文件名与实际内容不符

某一线前端小组长的 Code Review 分享(第二弹!)

命名不规范

某一线前端小组长的 Code Review 分享(第二弹!)

又是小驼峰又是前端电视剧全大写、又是大写下前端学什么横线。。

state 的粒度太细

某一线前端小组长的 Code Review 分享(第二弹!)

避免变量一个组件里面有n多行的useState

避免直接修改对象值,React 要求数据不可变

某一线前端小组长的 Code Review 分享(第二弹!)

官网解释: zh-hans.reactjs.org/tutorial/t前端工程师u…

不熟悉 Antd 组件的 api

这里想修改Popover组件内部的样式,想当然的就在外面套多一层div

某一线前端小组长的 Code Review 分享(第二弹!)

ant-design popover api: overlayClassName

Table 的滚动高度没有自适应

某一线前端小组长的 Code Review 分享(第二弹!)

如果用户的显示器大小不一,列表可能展示不全或者只占了一小部分的面积
修改为:scroll={{ y: 'calc(100vh - 表头与顶部距离px)' }}

写在最后

本篇文章就分享到这里啦,整体内容还是稍微零散,对前端工程师于没有基础的变量值同学可能不太友好,不过如果大家看完觉得有所收获,可以留言点赞评论!后续我会继续抽空记录,争取出一系列文章~ /撒花

发表回复

提供最优质的资源集合

立即查看 了解详情