Tecvid,
ретурн -- это выход из ф-ции. То есть, если ты пишешь

f=function(){
if (foo) return bar
baz
}

То baz не отработает. Поэтому не везде их можно убирать.

это я понимаю, но не могу понять почему в случае проверки value === null без return не обходится, а вот без этой проверки с ней обходится) хотя по сути одно и та же функция, но допиленная чуточку) странно

Tecvid, если я правильно понял

sel=document.querySelector.bind(document)
function attr(selector, attrName, attrValue) {
  if (arguments.length > 2) {
    if (attrValue === null) {
      // удаляет атрибут
      return sel(selector).removeAttribute(attrName);
    }
    // задает значение атрибуту
    sel(selector).setAttribute(attrName, attrValue);// Выходим из ифа
  }
  // возвращает значение атрибута
  return sel(selector).getAttrubute(attrName);// синтаксическая ошибка getAttrUbute, но, сюда мы вообще не должны были ходить, поэтому return убирать там нельзя было
}

хмм я даж не заметил эту ошибку, оказывается вся тайна скрывалась в ней

attr(selector, attribute, value)

И в чем тут цимес?

attr(some_element,'data-huy-znaet-chto','Hello Word');
some_element.setAttribute('data-huy-znaet-chto','Hello Word');

На дохуя короче? Скаляры все равно кавычить.