fex-team / styleguide

文档与源码编写风格
Creative Commons Attribution 4.0 International
2.54k stars 846 forks source link

if, for等强制{}会不会更好点 #9

Open oxUnd opened 10 years ago

oxUnd commented 10 years ago

https://github.com/fex-team/styleguide/blob/master/javascript.md#%E5%9D%97%E7%8A%B6%E4%BB%A3%E7%A0%81

Singularity-zju commented 10 years ago

必须顶 if的强制大括号,不然以后加个逻辑很容易忽略。 而且现在我记得Sublime这种编辑器里你直接if后面回车就自动加上大括号了。

2betop commented 10 years ago

+1

miller commented 10 years ago

+1 原来GMU貌似是强制的

2betop commented 10 years ago

想起来了,主要是考虑到这个case,独立一行可读性更强。

var noop = function() {};
var emptyObject = {};
techird commented 10 years ago

很多时候我条件判断的时候只做一件事,我觉得还是不要强制的好。我建议如果 if / for 的执行只有一条语句的话,不写大括号需要强制写在一行,如果是多行,强制加大括号。

// good
if (isIE) doForIE();

// bad
if (isIE)
    doForIE();

// good
if (isIE) {
    doForIE();
}

// good
if (isIE) {
   doForIE();
   maskChrome();
}
2betop commented 10 years ago

如果只有一行我也倾向于 if (isIE) doForIE();这种写法,超过多行才强制用大括号。