fex-team/styleguide

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

oxUnd opened this issue · 6 comments

Singularity-zju commented

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

2betop commented

+1

miller commented

+1 原来GMU貌似是强制的

2betop commented

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

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

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

// good
if (isIE) doForIE();

// bad
if (isIE)
    doForIE();

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

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

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