JS 写法的选择

auroraccc · 2018-12-20T04:27:52Z

写法一： const foo = (data) => { if(data.length ele + 1) } 写法二： const bar = (data) => { if(data.length ele + 1) } 这两种写法哪种更好？或者其他更好的写法？

推荐关注

› Meteor

› JSLint - a JavaScript code quality tool

› jsFiddle

› D3.js

› WebStorm

推荐书目

› JavaScript 权威指南第 5 版

› Closure: The Definitive Guide

This topic created in 2727 days ago, the information mentioned may be changed or developed.

写法一：

const foo = (data) => {
	if(data.length < 3) data = [1,2,3]
	return data.map(ele => ele + 1)
}

写法二：

const bar = (data) => {
	if(data.length < 3) return bar([1,2,3])
	return data.map(ele => ele + 1)
}

这两种写法哪种更好？或者其他更好的写法？

写法

Data

ele

return

15 replies • 2018-12-20 14:18:10 +08:00

akatquas

Dec 20, 2018 via iPhone

我选第一种，效率和空间都略好一点

weixiangzhe

Dec 20, 2018 via iPhone

2，不要直接改参数变量会有不可预计的东西

Athrob

Dec 20, 2018 via Android

个人感觉写法一比较好。写法二 bar 改成 bar2，函数里面也得改。

Cbdy

Dec 20, 2018

```
function bar(data: number[]) {
let res = [2, 3, 4]
if (data.length > 3) {
res = data.map(e => e + 1)
}
return res
}
```

V2exUser

Dec 20, 2018 via Android

第一种，清晰

Mutoo

Dec 20, 2018

2 有 stack overflow 的风险（例如不小心写成 return bar([1,2]) 时，data.length < 3 永远成立）

dixeran

Dec 20, 2018 via Android

所以综合考虑 2L 和 3L，用方法二然后改成 callee?

Justin13

Dec 20, 2018 via Android

第二个，能纯尽量纯。
或者
```js
const foo = (data) => {
const newData = data.length < 3 ? [1,2,3] : data;
return newData.map(ele => ele + 1)
}
```

yuanfnadi

Dec 20, 2018

const foo = data => {
if (data.length < 3) {
return [2, 3, 4];
}
return data.map(ele => ele + 1);
};

whileFalse

Dec 20, 2018

第一种。
因为第二种无法扩展到有多个 if 的情况。

@Mutoo 这位仁兄说的问题我倒觉得无所谓。同是 bug，静默错误未必比直接卡死好多少。

otakustay

Dec 20, 2018

你不让我写三元的话，肯定选第 2 种，if 分支多起来也是不同分支不同 return。选择不去改变任何一个变量 /参数的值是很好的实践

wly19960911

Dec 20, 2018

2 的写法究竟是哪里来的。function 减少对外界变量的依赖，这个封装的基础。

const foo = (data) => {
if(data.length < 3) return [1,2,3]
return data.map(ele => ele + 1)
}

这样最好

cuzfinal

Dec 20, 2018

选 2，不要改参数

yikyo

Dec 20, 2018

第一种 ESLint 检测过不去。不能修改参数值。

jorneyr

Dec 20, 2018

只看得懂 1