这篇文章没有对错之分,肯定也有不完善的地方,结合了自己日常开发和经验。可以让你书写代码更具严谨性,希望看完之后有所帮助。本文字数 4000+ ,看完本文大概需半小时。
/**
* @description xxxxxx
* @author chengfeng
* @since 19/05/21
*/
/**
* 拷贝数据
* @param {*} data 要拷贝的源数据
* @param {boolean} [isDeep=false] 是否深拷贝,默认浅拷贝
* @return {*} 返回拷贝后的数据
*/
/*业务代码注释*/
interface IState {
// 名字
name: string;
// 电话
phone: number;
// 地址
address: string;
}
import * as React from 'react';
import { Dropdown, Menu, Icon } from 'antd';
import Header from './Header';
import toast from 'common/toast';
import './index.less';
const handleCheck = () => {
onCancel && onCancel();
onClose && onClose();
};
下列关键字后必须有大括号(即使代码块的内容只有一行):if, else, for, while, do, switch, try, catch, finally, with。
// not good
if (condition) doSomething();
// good
if (condition) {
doSomething();
}
// bad
++ x;
y ++;
z = x?1:2;
// good
++x;
y++;
z = x ? 1 : 2;
// bad
if (condition){
}
while (condition){
}
function funcName(){
}
// good
if (condition) {
}
while (condition) {
}
function funcName() {
}
// bad
if(condition) {
}
while(condition) {
}
(function() {
})();
// good
if (condition) {
}
while (condition) {
}
(function () {
})();
// bad
var obj = {
a : 1,
b:2,
c :3
};
// good
var obj = {
a: 1,
b: 2,
c: 3
};
// bad
var obj = {
a: 1
, b: 2
, c: 3,
};
function test()
{
...
}
for (const key in object)
{
if (object.hasOwnProperty(key)) {
const element = object[key];
}
}
// good
var obj = {
a: 1,
b: 2,
c: 3,
};
function test() {
...
}
for (const key in object) {
if (object.hasOwnProperty(key)) {
const element = object[key];
}
}
// bad
if (condition) {
...
}
else {
...
}
try {
...
}
catch (e) {
...
}
finally {
...
}
// good
if (condition) {
...
} else {
...
}
try {
...
} catch (e) {
...
} finally {
...
}
// bad
const a = {
'b': 1
};
const a = {b: 1};
const a = {
b: 1,
c: 2
};
const arr = [1, 2, 3, 4,];
// good
const a = {
b: 1,
c: 2,
};
const arr = [1, 2, 3, 4];
interface IProps {
onClose?: () => void;
onOk?: (item: Record<string, any>) => void;
}
interface IProps {}
interface IState {}
// bad
function getLength(something: string | number): number {
return something.length;
}
// index.ts(2,22): error TS2339: Property 'length' does not exist on type 'string | number'.
// Property 'length' does not exist on type 'number'.
// bad
function getLength(something: string | number): number {
if ((<string>something).length) {
return (<string>something).length;
} else {
return something.toString().length;
}
}
// good
function getLength(something: string | number): number {
if (typeof something === 'string') {
return something.length;
} else {
return something.toString().length;
}
}
日常用到比较多的是四种,只读参数放第一位,必选参数第二位,可选参数次之,不确定参数放最后。
interface iProps {
readonly x: number;
readonly y: number;
name: string;
age: number;
height?: number;
[propName: string]: any;
}
用于定义一个 javascript 的对象,key 是字符串,value 是任意类型
const people:Record<string,any> = {
name: 'chengfeng',
age: 10
}
interface iPeople {
title: string;
name: string;
}
const people: Partial<Todo> = {
title: 'Delete inactive users',
};
定义的结构可以是接口iPeople的任意key
interface iPeople {
title: string;
name: string;
}
const people: Readonly<Todo> = {
title: 'todo list',
name: chenfeng;
};
title name属性就是只读的了
interface iPeople {
title?: string;
name?: string;
}
const people1: Props = { title: 'ts' }; // OK
const people22: Required<iPeople> = { title: 'ts' }; // Error: property 'name' missing
interface iPeople {
name: string;
age: number
}
type T = keyof iPeople // -> "name" | "age"
type Keys = "a" | "b"
type Obj = {
[p in Keys]: any
} // -> { a: any, b: any }
constructor (){
this.setState({ people: this.props.people })
}
state: IState = {
people: {},
};
render(){
{name}
}
render(){
{!!name || '--'}
}
// bad
const { list, totalCount } = await getPeopleList(keyword, page, pageSize);
list 可能是 null 或者 undefined
list.length 将直接导致前端报错
this.setState({
status: STATUS.READY,
apps: list,
total: totalCount,
page: page,
});
// good
const { list, totalCount } = await getPeopleList(keyword, page, pageSize);
this.setState({
status: STATUS.READY,
apps: list || [],
total: totalCount || 0,
page: page,
});
例如一些地方,不确定这个变量里面到底有什么,但自己觉得有,就疯狂的...,最明显的就是后端返回了一个对象给你,前端拿到之后判断都不判断直接data.dataList.forEach()
const data = await getPeopleList(keyword, page, pageSize);
data.dataList.forEach()
const data = await getPeopleList(keyword, page, pageSize);
if (data && data.dataList && Array.isArray(data.dataList) {
data.dataList.forEach()
}
let maxPrice = +form.maxPrice.value;
let maxPrice = Number(form.maxPrice.value);
let mobile = !!ua.match(/iPhone|iPad|Android|iPod|Windows Phone/);
js 中以下为假,其他情况为真
// bad
class Listing extends React.Component {
render() {
return <div>{this.props.hello}</div>;
}
}
// good
function Listing({ hello }) {
return <div>{hello}</div>;
}
// bad
this.props.app.openid;
this.state.time
// good
const { app } = this.props;
const { time } = this.state;
console.log(app.openid)
// bad
let searchContent = form.search.value;
// good
let searchContent = form.search.value.trim();
// bad
window.location.href = redirectUrl + '?a=10&b=20';
// good
window.location.href = redirectUrl + encodeURIComponent('?a=10&b=20');
// bad
import { withRouter, RouteComponentProps } from 'react-router-dom';
export interface IProps extends RouteComponentProps<any> {}
class App extends React.Component<IProps, AppStates> {}
export default withRouter(App);
// good
import { withRouter, RouteComponentProps } from 'react-router-dom';
class App extends React.Component<IProps & RouteComponentProps<{}>, AppStates> {}
export default withRouter(App);
// 目前
|- api
|- pageA.ts
|- pageB.ts
// 建议
|- api
|- pageA
|- index.js
|- aaa.js
|- bbb.js
|- pageB
|- index.js
|- aaa.js
|- bbb.js
|- ccc.js
function parse (str:string){
if (typeof(str) === 'string' ) {
}
}
getStudentList = async () => {
try {
this.setState({
loading: true,
isEmpty: false
});
await getStudentList({});
this.setState({
loading: false,
isEmpty: true
});
} catch (e) {
// TODO
console.log(e)
} finally {
// 失败之后的一些兜底操作
this.setState({
loading: false,
isEmpty: true
});
}
};
// 对象
this.setState({
})
// 函数,一般是用于在 setState 之前做一些操作
this.setState(
() => {
// TODO
console.log('')
return {
a:300
}
}
)
// 第二个参数,一般是用于在 setState 之后做一些操作
this.setState({
a:300
}, () => {
// TODO
})
// bad
func = async (name, value, status) => {
await this.setState({
name
});
// TODO
};
// good
func = (name, value, status) => {
this.setState(
{
name
},
() => {
// TODO
}
);
};
//badthis.state.dataAry.map((item, index) => { return <span key={index} />;});
//goodthis.state.dataAry.map(item => <span key={item.id} />);
//bad
const arr = [];
const key = '';
for (key in obj) {
arr.push(obj[key]);
}
//good
const arr = [];
const key = '';
for (key in obj) {
if (obj.hasOwnProperty(key)) {
arr.push(obj[key]);
}
}
/*
* Echart 用于代绘制图表,但当其自身发生错误时,可能影响到业务代码的执行
*/
// bad
const iniDom = document.getElementById('init-container');
const echartObj = echarts.init(iniDom);
this.setState(
{
echartObj
},
() => {
const { echartObj } = this.state;
// 更新图表
echartObj.setOption(CHART_CONFIG, true);
}
);
// good
try {
const iniDom = document.getElementById('init-container');
const echartObj = echarts.init(iniDom);
this.setState(
{
echartObj
},
() => {
const { echartObj } = this.state;
// 更新图表
echartObj.setOption(CHART_CONFIG, true);
}
);
} catch (error) {
// TODO
}
import { html2text } from 'xss';
render(){
<div
dangerouslySetInnerHTML={{
__html: html2text(htmlContent)
}}
/>
}
class MyComponent extends React.Component<iProps, iState> {
constructor(props) {
super(props);
this.inputRef = React.createRef();
}
render() {
return <input type="text" ref={this.inputRef} />;
}
componentDidMount() {
this.inputRef.current.focus();
}
}
// bad
if (type !== 0) {
// TODO
}
// good
const STATUS: Record<string, any> = {
READY: 0,
FETCHING: 1,
FAILED: 2
};
if (type === STATUS.READY) {
// TODO
}
// best
enum STATUS {
// 就绪
READY = 0,
// 请求中
FETCHING = 1,
// 请求失败
FAILED = 2,
}
import { Component, PureComponent } from 'react';
// bad
class Message extends Component {
render() {
return <span>{this.state.message}</span>;
}
}
// good
class Message extends PureComponent {
render() {
return <span>{this.state.message}</span>;
}
}
import { isReactPropsEqual, isReactStateEqual } from '@fe/common/lib/equal';
shouldComponentUpdate(nextProps:IProps, nextState:IState) {
if (isReactStateEqual(nextState,this.state) && isReactPropsEqual(nextProps,this.props)) {
return false;
}
return true;
}
很多小伙伴用了很久的ts,都不知道常用 Event 事件对象类型:
ClipboardEvent<T = Element> 剪贴板事件对象
DragEvent<T = Element> 拖拽事件对象
ChangeEvent<T = Element> Change 事件对象
KeyboardEvent<T = Element> 键盘事件对象
MouseEvent<T = Element> 鼠标事件对象
TouchEvent<T = Element> 触摸事件对象
WheelEvent<T = Element> 滚轮事件对象
AnimationEvent<T = Element> 动画事件对象
TransitionEvent<T = Element> 过渡事件对象
import { MouseEvent } from 'react';
interface IProps {
onClick(event: MouseEvent<HTMLDivElement>): void;
}
对于一些不需要控制 ui 的状态属性,我们可以直接绑到 this 上, 即私有属性,没有必要弄到 this.state 上,不然会触发渲染机制,造成性能浪费 例如:请求翻页数据的时候,我们都会有个变量。
// bad
state: IState = {
pageNo:1,
pageSize:10
};
// good
queryParams:Record<string,any> = {
pageNo:1,
pageSize:10
}
总结四句话。
我们在写组件或者函数的的时候,工具函数和业务逻辑抽离,表单校验和业务抽离、事件函数和业务抽离,ajax 和业务抽离。
例如有些页面是通过 location.href 跳转的,我们有些业务逻辑等都是放到 didmountMount,但是后期改需求,可能要用 react-router 进行跳转,可能要改的逻辑就会很多了,所以函数抽离出来,需求更新就少改一点代码。
如果还不确定如何划分函数的细粒度,我有个建议。使用过两次以上的代码,要抽离组件或者函数,两次的可以不用。
个人觉得 if else 嵌套深看起来也不会太难受,难受的是,项目迭代久之后,自己都忘记曾经写过这些代码,而且类型多或者不确定有什么类型,是否后期还会加的情况下,改起来就非常复杂了,而且很容易踩坑和背锅。
用配置取代 if嵌套,大概就是抽离一个 config.ts 出来,里面放一些配置。
例如你的业务代码里面,会根据不同 url 参数,代码会执行不同的逻辑.
/info?type=wechat&uid=123456&
const qsObj = qs(window.location.url)
const urlType = qsObj.type
// bad
if (urlType === 'wechat') {
doSomeThing()
} else if () {
doSomeThing()
} else if () {
doSomeThing()
} else if () {
doSomeThing()
}
// good
config.t
const urlTypeConfig: Record<string, typeItem> = {
'wechat': { // key 就是对应的type
name: 'wechat',
show: ['header', 'footer', 'wechat'] // 展示什么,可能是异步的
pession: ['admin'], // 权限是什么,可能是异步的
},
'zhifubao': { // key 就是对应的type
name: 'zhifubao',
show: ['header', 'footer', 'zhifubao'] // 展示什么,可能是异步的
pession: ['admin'], // 权限是什么,可能是异步的
},
}
// 业务逻辑
const qsObj = qs(window.location.url)
const urlType = qsObj.type
urlTypeConfig.forEach(item => {
if(urlType === item.type) {
doSomeThing(item.show)
}
})
发现团队一些小伙伴为了减少 render 函数里面的代码量,会把一些元素拆分到函数里面。
// bad
renderHeader = () => {
return (<div />)
}
renderBody = () => {
return (<div />)
}
renderFooter = () => {
return (<div />)
}
render(){
return(
<div>
renderHeader()
renderBody()
renderFooter()
</div>
)
}
更好的办法,是用函数式组件取代在当前组件里面写方法:
// good
function RenderHeader(props) = {
return (<div />)
}
function RenderBody(props) = {
return (<div />)
}
function RenderFooter(props) = {
return (<div />)
}
class Component extends React.Component<iProps, iState>{
render () {
return(
<div>
<RenderHeader />
<RenderBody />
<RenderFooter />
</div>
)
}
}
使用 a 标签打开一个新窗口过程中的安全问题。新页面中可以使用 window.opener 来控制原始页面。
如果新老页面同域,那么在新页面中可以任意操作原始页面。如果是不同域,新页面中依然可以通过 window.opener.location,访问到原始页面的 location 对象
在带有 target="_blank" 的 a 标签中,加上r el="noopener" 属性。如果使用window.open 的方式打开页面,将 opener 对象置为空。
var newWindow = window.open();
newWindow.opener = null;
clearSessioin = () => {
req.session.userName = undefined;
req.session.userName = void 0
}
在做一些前后端鉴权的时候,后端应该开启 domain、secure、httponly 严格模式,禁止前端操作 cookie,防止 csrf 攻击。
我们可以使用构建工具继承 husky、eslint、tslint、lint-stage、prettier 来规范代码。
作者:乘风gg https://juejin.im/post/5ce24f8ae51d45106477bd45#heading-0