箭头函数的参数是否加括号？

[YanagiEiichi]

@furybean @CarterLi 喷子龙，喷子李，你们怎么看？

callback((item) => {});

vs

callback(item => {});

@ElemeFE/owner

---

是否加括号	赞
总是加
只有一个时不加	👍 👍

[iqingting]

一个参数不加

[CarterLi]

一个参数不加

另外还有

callback(item => {
  return {};
});

vs

callback(item => ({}));

[YanagiEiichi]

👍 尽量直接使用表达式，因为可以省去一个 return。

[CarterLi]

@YanagiEiichi 不能完全不加吧，所以只有“总是加”和“只有一个时不加”两个选项

() => x
x => x
(a, b) => a + b

但是因为“只有一个参数”的callback非常常用，所以单独拎出来

另外没有参数时个人习惯写

()=> x // 右括号与等号间无空格

而非

() => x // 右括号与等号间有空格

有参数时都加空格

[YanagiEiichi]

主要是存在解构时必须加，比如

xxx.then(({ userId, userName }) => {
  // do something
});

然而它只有一个参数。。

[CarterLi]

另外标准规定，箭头函数不能作为构造函数使用（不能用 new 操作符），所以某些写法其实是错误的
例如

oneModule.controller('name', $scope => XXX)

然而 angular 中 controller 都是作为构造函数被 new 出来的
如果不经过 babel 翻译，就会出现 $scope => XXX is not a constructor 的 TypeError

相对的，class 的 constructor 成员函数必须使用 new 调用

[CarterLi]

题外话，建议推广一下 controllerAs 的写法，然后我们就可以写

module.controller('name', class NameCtrl {
  constructor() {}
  action1() {}
  action2() {}
});

然后

<div ng-controller="NameCtrl as name">
  <button ng-click="name.action1()">Action</button>
</div>

PS: 虽然还是 proposal，这个可以有：https://github.com/jeffmo/es-class-static-properties-and-fields

[YanagiEiichi]

确实，controller 应该是 class

[YanagiEiichi]

controller as 的写法就算了，我不推荐自己去写 ng-controller，controller 全部交给路由，其它需求交给 directive（directive 也应该是 class）