1

jsコードをリファクタリングして、よりドライな慣習にしようとしていますが、このエラーが発生しません。これが私のコードです

function validate_field(e){
    console.log(typeof e);
    $(e).focusout(function(e){
        var price = $(e).val();
        var number = parseInt(price, 10);
        console.log(number)
        if (number < 0)
        {
            $(e).val(0);
        }
    })
}
$(function(){
    validate_field('#price');
})

スタックトレースによると、エラーはここのどこかにあります。ここでvar price = $(e).val(); 何が欠けていますか?

4

3 に答える 3

5

試す

   function validate_field(e){
        console.log(typeof e);
        $(e).focusout(function(ev){
-------------------------------^ here you are redeclaring the e previously passed as selector 
            var price = $(e).val();
            var number = parseInt(price, 10);
            console.log(number)
            if (number < 0)
            {
                $(e).val(0);
            }
        })
    }
    $(function(){
        validate_field('#price');
    })
于 2013-02-25T10:20:47.767 に答える
4

関数の変数でe引数を妨害しています。eそのはず :

function validate_field(s) { // here also ? I have set "s" as parameter, you can set any word you like
    console.log(typeof s); // noticed the difference here?
    $(s).focusout(function(e) { // noticed the difference here?
        var price = $(this).val();
        var number = parseInt(price, 10);
        console.log(number)
        if (number < 0)
        {
            $(e).val(0);
        }
    });
}

@dakaitが行ったように、マイナーな変更のためにイベント引数を変更することもできます。

于 2013-02-25T10:20:22.847 に答える
0

変数の再割り当てが問題を引き起こしています。

function validate_field(field){ // both variable should not be same e & 'e' of inside function
    console.log(typeof e);
    $(field).focusout(function(e){
        var price = $(field).val();
        var number = parseInt(price, 10);
        console.log(number)
        if (number < 0)
        {
            $(field).val(0);
        }
    })
}
$(function(){
    validate_field('#price');
})

また

スクリプト内の他の関数を呼び出さずに、以下のコードを直接使用できます

$(function(){
    $('#price').focusout(function(e){
            var price = $(this).val();
            var number = parseInt(price, 10);
            console.log(number)
            if (number < 0)
            {
                $(this).val(0);
            }
        })
});
于 2013-02-25T10:28:37.333 に答える