代码气味 - 将布尔控制参数传递给函数

Question

下面是代码片段，有一些我不喜欢的地方：

function insert(el, child, before){
  if ( before ) {
    el.insertBefore(child, el.childNodes[0]);
  } else {
    el.appendChild(child);
  }
}

为什么不使用insertBefore 和insertAfter 等两个单独的方法呢？这种方法和其他方法的优缺点是什么？

更新：

我得到了这个很好的article 解释我想要什么。

Answer 1

此函数的全部目的是避免在调用该函数的所有地方都放置if 语句。所以如果你有很多地方看起来像：

if (something) {
    foo.insertBefore(bar, foo.childNodes[0]));
} else {
    foo.appendChild(bar);
}

您可以将它们全部简化为：

insert(foo, bar, something);

用你的两种方法，它会变成：

if (something) {
    insertBefore(foo, bar);
} else {
    insertAfter(foo, bar);
}

这比原版好不了多少。