Что делать перфекционисту в такой ситуации?

Question

[Владимир Грабко]

У меня код валидации

A.Plugin([
    "Validator"
]);
A.Middleware.AuthSave = function (e) {

    if (A.Validator.Len(e.login, 0)) {
        SetId("error", "Заполните поле Login");
    } else if (A.Validator.Len(e.email, 0)) {
        SetId("error", "Заполните поле Email");
    } else if (A.Validator.Len(e.pass, 0)) {
        SetId("error", "Заполните поле Pass");
    } else if (!A.Validator.IsLogin(e.login)) {
        SetId("error", "Login может содержать a-z . и -");
    } else if (!A.Validator.IsEmail(e.email)) {
        SetId("error", "Email не валиден");
    } else {
        ClearId("error");
        e.ok();
    }
};

Как можно исправить кучу этих if else ?

Comments

[A K]

switch - case

[Владимир Грабко]

Symphony: я думал об этом. Но там и не понял как в case засунуть выражения из if

[Nathan Stark]

case тут точно не подойдет

[Nathan Stark]

switch - case используется когда у одной переменой несколько значений, а тут ситуация когда разные переменые и разные значения

[Алексей Николаев]

Nathan Stark: switch(true) { ... case A.Validator.Len(e.pass, 0) :
Подойдет.

Answer 1

[Алексей]

Чисто как вариант - создать массив из записей типа { action: A.Validator.Len, field: e.login, param: 0, error: "Заполните поле Login" } и в цикле выполнять код типа if (!$action(field, param) { SetId("error", error); }

Comments

[#алгоптимизируй #отботизируй]

примерно так.

[Алексей]

Еще как вариант:

A.Validator.Len(e.login, 0) ? SetId("error", "Заполните поле Login") :
A.Validator.Len(e.email, 0) ? SetId("error", "Заполните поле Email") :
'A.Validator.Len(e.pass, 0) ? SetId("error", "Заполните поле Pass") :
.... : (ClearId("error"), e.ok());

(запятая - не опечатка)
или

SetId("error",
A.Validator.Len(e.login, 0) ? "Заполните поле Login" :
A.Validator.Len(e.email, 0) ? "Заполните поле Email" :
'A.Validator.Len(e.pass, 0) ? "Заполните поле Pass" :
.... : (ClearId("error"), e.ok(), "")
);

или как-то так.

Answer 2

[Алексей Зуев]

var conditions = [{
  method: 'Len',
  params: [e.login, 0],
  expected: true,
  msg: 'Заполните поле Login'
}, {
  method: 'Len',
  params: [e.email, 0],
  expected: true,
  msg: 'Заполните поле Email'
}, {
  method: 'Len',
  params: [e.pass, 0],
  expected: true,
  msg: 'Заполните поле Pass'
}, {
  method: 'IsLogin',
  params: [e.login],
  expected: false,
  msg: 'Login может содержать a-z . и -'
}, {
  method: 'IsEmail',
  params: [e.email],
  expected: false,
  msg: 'Login может содержать a-z . и -'
}];

if(!conditions.some(function(x) {
  if(A.Validator[x.method].apply(A.Validator, x.params) === x.expected) {
    SetId("error", x.msg);
    return false;
  }
  return true;
})) {
  ClearId("error");
  e.ok(e.login, e.email, e.pass);  
}

Не тестил

Comments

[Владимир Грабко]

Вы забыли ')' и вставку сообщения

[Владимир Грабко]

Обратно ошибка. Вы true с false местами перепутали.

[Алексей Зуев]

Владимир Грабко: бывает, надо было тестить)

[Владимир Грабко]

Алексей Зуев: ))

[Владимир Грабко]

Алексей Зуев: я когда написал про скобочку то уже всё сделал)

Answer 3

[Nathan Stark]

я бы ограничился просто короткой записю if (без фигурных скобок).Моем глазу так приятнее...

Answer 4

[Владимир Грабко]

Я вам очень благодарен. Теперь я в A.Validator добавил метод conditions который работает с массивом и при помощи кодогонерации вызывает функцию в случае успешного прохождения всех фильтров. (кодогенерация нужна так как в функцию отдаются все x.params[0]

Answer 5

[ThunderCat]

switch - case ?

Comments

[Владимир Грабко]

Судя по гуглению эта конструкция не может проверить кучу переменных в case

Answer 6

[Rou1997]

Вот здесь проблема решена:

A.Plugin([
    "Validator"
]);
A.Middleware.AuthSave = function (e) {

    if (A.Validator.Len(e.login, 0)) {
        SetId("error", "Заполните поле Login");
    }
    else if (A.Validator.Len(e.email, 0)) {
        SetId("error", "Заполните поле Email");
    }
    else if (A.Validator.Len(e.pass, 0)) {
        SetId("error", "Заполните поле Pass");
    }
    else if (!A.Validator.IsLogin(e.login)) {
        SetId("error", "Login может содержать a-z . и -");
    }
    else if (!A.Validator.IsEmail(e.email)) {
        SetId("error", "Email не валиден");
    }
    else {
        ClearId("error");
        e.ok();
    }
};

Найдите всего 1 отличие, оно и есть решающее.

Comments

[Владимир Грабко]

вы просто добавили переносы строк)

[Rou1997]

Владимир Грабко: Это повысило читабельность кода и облегчило добавление методом "копипасты" или удаление блоков else-if, больше никаких проблем в данном коде не было, так что теперь все в порядке.