这是 Javascript 闭包用例吗？

Question

我可能有初学者 Javascript 问题：

var countries = [
    "Bangladesh", "Germany", "Pakistan"];


function testexistence(arr, input) {

    for (var i = 0; i < arr.length; i++) {
        if (arr[i] != input) {
            alert("not exist");
            arr.push(input);
            break;
        } else {
            alert("already exist ");
        }
    }

}

testexistence(countries, "UK");
testexistence(countries, "Pakistan");
testexistence(countries, "UK");

我期望的是：当我再次为“UK”调用该函数时，它显示“已经存在”；但这并没有发生。我不想玩“原型”或定义自己的。我只需要一种解决方案。

我的代码中有一个用例，我必须在数组中插入一个新值，并且在接下来的循环中我必须检查该值；但我最终要插入一个现有值...

为什么我最终要插入现有值以及为什么此检查 (arr[i] != input) 失败？

还请解释一下，为什么上面的代码不能正常工作

Answer 1

您需要搜索整个数组才能确定它不存在。

function testexistence(arr, input) {
    for (var i = 0; i < arr.length; i++) {
        if (arr[i] === input) {
            alert("already exists");
            return; // halt the search by returning
        }
    }

    // If we're here, we never returned inside the loop, so it wasn't found.
    arr.push(input);
    alert("did not exist, now it does");
}

---

我可能会将您的函数命名为 addUnique 或其他名称，而不是 testexistence。

Answer 2

试试：

function testexistence(arr, input) {
    if (!~arr.indexOf(input)) {
        arr.push(input);
    }
}

演示： http://jsfiddle.net/L9NhU/

请注意，Array.indexOf 在旧版浏览器中不可用，因此您可以使用 polyfill（或保持当前循环）。这是它的 MDN 文档，其中包括一个 polyfill：https://developer.mozilla.org/en-US/docs/JavaScript/Reference/Global_Objects/Array/indexOf

Answer 3

首先，它绝不是闭包。

不管怎样，这里是the one-liner you wanted，对Ian's answer的修改

function testexistence(arr, input) {
  (!~arr.indexOf(input)) && arr.push(input);
}

我们使用了一些东西：

• Array.indexOf 在数组中搜索您传递的第一个匹配项，如果存在则返回从零开始的值，如果不存在则返回 -1。
• !~ 是这里的一个特例，我们在这里测试-1。值~x 等于-(x+1)，这使得-1 成为0（假）和所有其他非零（真）。将! 添加到组合中会使-1 成为真值而其他值是假的。
• && 评估其两侧。如果左边是“真”，则评估右边，否则不评估。它也被称为“守卫操作员”

Answer 4

你需要尝试这样的事情

var countries = ["london", "germany", "france"];


function testexistence(arr, input) {
   var isExists = false;

    for (var i = 0; i < arr.length; i++) {
        if (arr[i] == input) {
            isExists = true;
        }         
    }

    if(!isExists)
    {
        alert("Not Exists");
        arr.push(input);
    }
    else
    {
        alert("Exists");
    }
}

testexistence(countries, "UK");
testexistence(countries, "london");
testexistence(countries, "UK");

Answer 5

你可以用这个代替你的：

function testexistence(arr, input) {

    for (var i = 0; i < arr.length; i++) {
        if (arr[i] == input) {
            alert("already exist ");
            return;
        }
    }

    //if the if part would not work, you pass to here
    alert("not exist");
    arr.push(item);
}