Почему не работает скрипт в функции?

Question

[Влад Бабин]

Пишу скрипт для проверки на заполнение полей при входе/авторизации
вот та часть которая работает:

$('#name').blur(function(){
            $('#name').removeClass();
            var nameLngth = $('#name').val().length;
            if(nameLngth <= 1){
                $('#auth').addClass('notValid');
                $('#auth').addClass('non');
                $('#auth').html(text);
                $('.bg').addClass('notValid');
                $('#name').addClass('notValid');
            } else {
                $('#name').removeClass('notValid');
                $('.bg').removeClass('notValid');
                $('#auth').removeClass('notValid');
                $('#auth').removeClass('non');
                $('#auth').html('<h2>Вход</h2');
                $('#name').addClass('valid');
                $('.bg').addClass('valid');
                $('#auth').addClass('valid');
            }
        });

но таких полей 5 штук, и получиться что этот весь код нужно будет продублировать 5 раз
вот я и решил написать ф-цию, и в ф-ции этот же скрипт не работает....подскажите где ошибка.

function isEmptyAuth(nameClass, text){
        $(nameClass).blur(function(){
            $(nameClass).removeClass();
            var nameLngth = $(nameClass).val().length;
            if(nameLngth <= 1){
                $('#auth').addClass('notValid');
                $('#auth').addClass('non');
                $('#auth').html(text);
                $('.bg').addClass('notValid');
                $(nameClass).addClass('notValid');
            } else {
                $(nameClass).removeClass('notValid');
                $('.bg').removeClass('notValid');
                $('#auth').removeClass('notValid');
                $('#auth').removeClass('non');
                $('#auth').html('<h2>Вход</h2');
                $(nameClass).addClass('valid');
                $('.bg').addClass('valid');
                $('#auth').addClass('valid');
            }
        });
    }

    isEmptyAuth('\'#name\'', 'ведите пожалуйста ваше имя');

Answer 1

[Влад Бабин]

нужно написать обертку:

$(function() {
    function isEmptyAuth(nameClass, text){
        $(nameClass).blur(function(){
            $(nameClass).removeClass();
            var nameLngth = $(nameClass).val().length;
            if(nameLngth <= 1){
                $('#auth').addClass('notValid');
                $('#auth').addClass('non');
                $('#auth').html(text);
                $('.bg').addClass('notValid');
                $(nameClass).addClass('notValid');
            } else {
                $(nameClass).removeClass('notValid');
                $('.bg').removeClass('notValid');
                $('#auth').removeClass('notValid');
                $('#auth').removeClass('non');
                $('#auth').html('<h2>Вход</h2');
                $(nameClass).addClass('valid');
                $('.bg').addClass('valid');
                $('#auth').addClass('valid');
            }
        });
    }

    isEmptyAuth('\'#name\'', 'ведите пожалуйста ваше имя');
});

Answer 2

[Николай Чуприк]

Зачем ещё кавычки внутри строки в вызове функции. Надо так:

isEmptyAuth('#name', 'ведите пожалуйста ваше имя');

Comments

[Влад Бабин]

ну как зачем? потому в jQuery запросы пишуться $('запрос') тоесть в скобках
P.S я уже пробовал и без скобок... не помогло

[Влад Бабин]

вроде же все правильно, не понимаю почему не работает

[Николай Чуприк]

Может nameClass не видна внутри blur? Если заменить на this?

[Влад Бабин]

Николай Чуприк: видна, я пробовал через alert выводил - выводит '#name'

[Николай Чуприк]

Так, а как именно не работает. Что трассировка говорит?

[Влад Бабин]

Николай Чуприк: ничего.... такое ощущение что обходит эти запросы в ф-ции, ошибок тоже не пишет

[Николай Чуприк]

А дык, елы палы!

У вас обработчик события вешается на #name только после вызова isEmptyAuth. До это никакой обработчик по blur на поле ввода не висит! Вот и не заходит в функцию.

[Николай Чуприк]

В изначальном коде можно же просто в селекторе перечислить все пять полей:
$('#name1, #name2, ...').blur(...)

Только дальше уже использовать $(this) вместо $('#name')

Ну и строка лишняя после else
$(nameClass).removeClass('notValid');

[Влад Бабин]

Николай Чуприк: спасибо, это я потом уберу, но ф-ция не работает....

[Леонид Князев]

Кто скзаал, что не работает https://jsfiddle.net/kLknxke0/?

[Влад Бабин]

Леонид Князев: ха! действительно! нужно было обертку $(function() {});
СПАСИБО!))
я не мог додуматься до такого =)