Node代码中最常见的几种废代码

uselesscode.md

三元表达式

var foo = bar === 'xxx' ? true : false;
// 更好的代码是：
var foo = bar === 'xxx';

两个和三个等号的判断，都是得到布尔值。

回调函数的传递

Topic.count(query,function(err,count) {
  if(err) return cb(err);
  return cb(err,count);
});

// 更好的代码是：
Topic.count(query, cb);

假值赋值

if (!foo) foo = 'bar';
// 如下更好
foo = foo || 'bar';

if判断

if (flag == true) {
  // then...
}
// better:
if (flag) {
  // then...
}

第一种很容易理解。请问第二种该如何理解？

因为回调写多余了.

把间接调用改成直接调用呗。。。

插入。。

还有

if (!foo) foo = 'bar' 

不如

foo = foo || 'bar' 

标题党啊

if (flag === true) {
    // then...
}