код запах - прохождение логический контроль аргумент функционирования

Question

Ниже приведен фрагмент кода, есть то, что мне не нравится:

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

Почему не вместо двух отдельных методов, как insertBefore и insertAfter? каковы плюсы и минусы этого и другого подхода?

Update:

Я получил это хороший 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); 
} 

который не намного лучше оригинала.