0

Javascript のループ内で複合代入を機能させようとしていますが、NaN がスローされます。Javascript はまだかなり新しいので、その理由がわかりません。私は本質的にこれをjQuery-Validationカスタムメソッドに変換しようとしています: https://github.com/pfwd/NHSNumber-Validation/blob/master/PHP/NHSValidation.class.php

これが私がこれまでに持っているものです

        // Taken from https://github.com/pfwd/NHSNumber-Validation
        var multipliers = {1:10, 2:9, 3:8, 4:7, 5:6, 6:5, 7:4, 8:3, 9:2};
        var currentSum, currentNumber, currentMultiplier = 0;

        //Get submitted NHS Number and remove whitespace
        var givenNumber = value.replace(/\s+/g, '');

        // Get length
        var numberLength = givenNumber.length;
        console.debug(givenNumber);
        console.debug(numberLength);

        // Number must be 10 digits in length
        if (numberLength !== 10) {
            return false;
        }

        // Check number
        var checkNumber = value.substring(9);
        console.debug(checkNumber);

        // Loop over each number in the string and calculate the current sum
        for (var i = 0; i <= 8; i++) {
            var minus = i-1;
            var plus = i+1;
            currentNumber = value.charAt(i);
            currentMultiplier = multipliers[plus];
            currentSum += (currentNumber * currentMultiplier);
            console.debug("i is " + i + " & current Num: " + currentNumber + " plus current multi: " + currentMultiplier + " plus " + currentSum);
        }

        var remainder = currentSum % 11;
        var total = 11 - remainder;
        console.debug(currentSum);

マイナス変数とプラス変数が必要かどうかはわかりませんが、NaNの問題を修正しようとしているときに試したものです。一般的なコンソール デバッグ行は次のようになります。

i is 0 & current Num: 1 plus current multi: 10 plus NaN

私も同じ NaN 結果でこれを試しました:

currentSum = currentSum + (currentNumber * currentMultiplier);
4

1 に答える 1

1 
var currentSum, currentNumber, currentMultiplier = 0;

currentMultiplier のみを初期化します。

そのはず

var currentSum, currentNumber, currentMultiplier;
currentSum =  currentNumber = currentMultiplier = 0;

デモ: http://jsfiddle.net/46dD5/

于 2013-07-24T13:16:53.163 に答える