3

Здравствуйте.

Подскажите, как сделать, чтобы не дублировать этот код:

var showError = function(el) {
  el.parentNode.classList.remove('error');
  el.parentNode.classList.add('success');
  el.nextElementSibling.InnerHTML = element.dataset.error;
}
var showSuccess = function(el) {
  el.parentNode.classList.remove('error');
  el.parentNode.classList.add('success');
  el.nextElementSibling.InnerHTML = '';
}
12
  • var a=b=function(){...}
    – DNS
    27 мар 2017 в 19:18
  • @DNS, функции разные действия выполняют
    – Grundy
    27 мар 2017 в 19:18
  • йопте, var a=b=function(el,msg){...}
    – DNS
    27 мар 2017 в 19:19
  • @DNS, а что поменялось-то? у тебя a и b - одна и та же функция, соответственно выполняют одни и те же действия. А требуется выполнять разные
    – Grundy
    27 мар 2017 в 19:21
  • Разные? Одни и те же для разного параметра..
    – DNS
    27 мар 2017 в 19:22

1 ответ 1

4
function getClassSwitcher(classOff, classOn, getErrorMsg) {
  return function(el) {
    el.parentNode.classList.remove(classOff);
    el.parentNode.classList.add(classOn);
    el.nextElementSibling.innerHTML = getErrorMsg();
  };
}

const showError = getClassSwitcher('success', 'error', () => element.dataset.error);
const showSuccess = getClassSwitcher('error', 'success', () => '');

Вариант попроще:

function toggleClasses(el, on, off) {
  el.classList.remove(off);
  el.classList.add(on);
}

const showError = function(el) {
  toggleClasses(el.parentNode, 'error', 'success');
  el.nextElementSibling.innerHTML = element.dataset.error;
}
const showSuccess = function(el) {
  toggleClasses(el.parentNode, 'success', 'error');
  el.nextElementSibling.innerHTML = '';
}
10
  • зачем msg делать функцией?
    – Grundy
    27 мар 2017 в 19:25
  • Чтобы взять актуальное значение свойства, а не его копию на момент объявления функции
    – vp_arth
    27 мар 2017 в 19:25
  • @vp_arth, быстро убери функцию с msg) Она там явно ни к чему
    – Yuri
    27 мар 2017 в 19:26
  • 1
    @Yuri, нет, в его варианте к чему, иначе в момент вызова showError - неизвестно значение атрибута
    – Grundy
    27 мар 2017 в 19:27
  • 1
    Блин, вы же не знаете, логику появления этого dataset.error. Не умеете вы рефакторить) Ничего не должно поменяться.
    – vp_arth
    27 мар 2017 в 19:27

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.