$.each 和 jQuery.inArray 函数组合的奇怪输出

Question

我的完整代码：

jQuery.extend({

combinationCheck: function (p1position) {

    var Combination = [1, 2, 3, 4, 5, 6, 7, 8];
    Combination[0] = [1, 2, 3];
    Combination[1] = [4, 5, 6];
    Combination[2] = [7, 8, 9];
    Combination[3] = [1, 4, 7];
    Combination[4] = [2, 5, 8];
    Combination[5] = [4, 6, 8];
    Combination[6] = [1, 5, 9];
    Combination[7] = [3, 5, 7];


    $.each(p1position, function (index, value) {

        var num = value;

        if ($.inArray(String(value), Combination[1]) != '-1') {
            alert("there");
        }
        else {
            alert("not there");
        }

    });
});

所以它有效。如果我将 num 设置为 5，它会发出警报“is There”，如果我将 num 设置为 8 --> ，则会发出警报。 “不在那里”。 但问题是我有另一个数组。

p1position = [1,5];

并遍历数组..

$.each(p1position,function(index,value){
    var num = value;
//then call the jQuery.inArray function as written above, it always return not there. even though 5 is in the Combination[1] array.
});

我对花几个小时试图解决这个问题感到非常困惑。

Answer 1

您所询问的代码的具体问题是您在检查之前将值转换为字符串：

if ($.inArray(String(value), Combination[1]) != '-1') {
//            ^^^^^^^^^^^^^

inArray 执行 === （严格相等) 检查，然后"1" !== 1。该行应为：

if ($.inArray(value, Combination[1]) !== -1) {

更改：

1. 不要将 值 转换为字符串。

2. 将结果与 -1（数字）而不是 "-1"（字符串）进行比较。 inArray 返回一个数字。

3. 使用 !== 而不是 != （这主要是风格问题，如果您愿意，可以使用 !=） .

---

不过，该代码还存在一些其他问题。

1. 您缺少 }，因此您所说的完整代码无法解析。

2. 每次调用 combinationCheck 时，您都会重新创建Combination。如果您的目标是创建 Tic-Tac-Toe 游戏，您将需要能够在检查之间保留 Combination 状态。

这是一组相当少的修复：

(function() {
    var Combination = [
            [1, 2, 3],
            [4, 5, 6],
            [7, 8, 9],
            [1, 4, 7],
            [2, 5, 8],
            [4, 6, 8],
            [1, 5, 9],
            [3, 5, 7]
        ];

    jQuery.extend({

        combinationCheck: function (p1position) {

            $.each(p1position, function (index, value) {

                if ($.inArray(value, Combination[1]) !== -1) {
                    alert(value + " is there");
                }
                else {
                    alert(value + " is NOT there");
                }

            });
        }
    });

})();

...其中给出：

jQuery.combinationCheck([1, 5]);

...报告未找到 1，但找到 5。

实时副本