如何写好业务代码?
在前端工作中有很多业务性代码,如果书写不规范,那么对后期的维护将是非常致命的。
判断配置化
业务场景
后端数据库中经常会一个字段具备几个不同的状态,比如:
status: 2// 各个字段对应的含义0: 出生1: 儿童2: 少年3: 中年4: 老年
- 这样不同的数字代表的含义,需要在前端展示。
- 需要根据不同的状态,前端去做不同的处理
方法一(switch)(bad)
下面这段代码就是常见的无限if/else或者switch场景
// 业务代码switch (status) { case 0: // do something return '出生'; case 1: // do something return '儿童'; ... ... default: return '';}
这样做是不好的,因为如果后端再加了一个字段,比如
5: 已死亡
那么前端需要重新修改switch函数,这样需要去修改对应的业务函数,无疑破坏了业务代码的完整性。
方法二(写成配置文件)(better)
// cfg.jsexport const cfg = new Map([ [0, 出生], [1, 儿童], [2, 少年], ... ... ]);// 业务代码cfg.get(status)
但是这样仅仅是显示相关的状态,如果涉及到状态的判断,那这样的处理方式就有些问题了,比如需要根据具体的状态去做对应的判断
switch (status) { case 0: // do something return ; ... ...}
像上面的情况,又变成了第一种情况,显然这种配置化的方式并不能满足。如果将对应的操作,与配置分割,则代码更加不易维护。
方法三(升级配置文件,处理代码集中)(better)
将状态处理集中起来,如果能将状态展示和对应的状态封装起来,那么就会让后期代码维护显得十分集中。
// cfg.jsconst status = new Map([ [0, 出生], [1, 儿童], [2, 少年], ... ... ]);const checkIsBirth = (status, callback) => { if(status === 0) { callback && callback() }}export default { status, checkIsBirth}// 在具体使用中import Person from './cfg.js'const a = 1;Person.status.get(a);Person.checkIsBirth(a, () => { console.log('Person is in Birth state');})
这样处理,如果以后status发生变化,只需要修改checkIsBirth中的判断逻辑就可以,只需要改动一处。
代码配置化
在使用react编写代码的过程中,经常用到这样的情况,根据情况判断是否展示对应的组件。
方法一(流水线工作)(bad)
function Business({status, bug}) { return ( <Fragment> { status === 0 ? ( <div>123</div> ) : null } { bug === 1 ? ( <div>456</div> ) : null } </Fragment> );}
这样的写法如果仅仅只有一个其实还好,如果有很多个,在代码中会造成代码非常冗长,使假想一个页面中如果有很多这样的,代码看起来非常ugly。所以建议将代码分割开来,形成一个个小的组件,将对应的代码封装起来,将会使代码可读性提高一些。
方法二(组件粒度细化)(better)
function Business() { return ( <Fragment> <One isShow={status === 0} /> <One isShow={bug === 1} /> </Fragment> );}// Onefunction One({isShow}) { return ( <Fragment> { isShow === true ? ( <div>123</div> ) : null } </Fragment> );}// Twofunction Two({isShow}) { return ( <Fragment> { isShow === true ? ( <div>456</div> ) : null } </Fragment> );}
如果经常这么写是不是会觉得很烦,可以将通用的逻辑抽象为通用的组件。
方法三(高阶组件)(better)
其实可以观望一下decorator以后的用法,暂时没有找到使用的切入点。
function IsShowCom(isShow, Wrapped) { if (isShow === true) { return (Wrapped) => <Wrapped /> } return () => null;}// 如果你想转发ref,你得这么做function IsShowCom(isShow, Wrapped) { if (isShow === true) { return React.forwardRef((props, ref) => { return <Wrapped {...props} ref={ref} /> }); } return () => null;}// import IsShowCom from './isShowCom';function Business() { const One = IsShowCom( status === 0, <div> 123 </div> ); const Two = IsShowCom( bug === 1, <div> 456 </div> ); return ( <Fragment> <One/> <Two/> </Fragment> );}
注意⚠️
不要在render中使用高阶组件,因为高阶组件每次返回来的都是新的组件,会使子组件的状态丢失 。但是在无状态组件中,这样使用并没有什么问题,因为无状态组件本身就是随参数的变化而变化的,只是可能会产生性能问题。
总结
前端配置化的整体思路:
- 针对不同值进行不同处理的时候,思考一下,是不是可以将判断逻辑代码集中起来。
- 针对组件的显示/隐藏,可以将具体的隐藏逻辑封装为高阶组件,便于维护