动态访问函数对象属性

Question

我需要检查多个textareas的值是否等于这个对象的属性name：

function make_test(name, job, ID) {
  test = {};
  test.name = name;
  test.job = job;
  test.ID = ID;
  return test;
}
  new make_test("Paul", "manager", 1);  //doesn't work
  new make_test("John", "employee", 2); //doesn't work
  new make_test("Jan", "employee", 2);  //works

只有当 value 等于姓名 并且 文本区域的索引等于人员的 ID 时，它才应该是匹配的。例如，如果我在 textarea1 中输入“Paul”，它应该输出 paul 的工作：但它不应该在 textarea2 中输出它，这应该只输出具有 ID = 2 的人的工作。

问题：我的代码仅适用于最后一个声明的人（本例中为 Jan）。这就像其他人甚至不存在于对象中，除了最后一个。我该如何解决这个问题？

我确信答案很明显，但我不知道我做错了什么。

在这里演示：https://jsfiddle.net/Lau1989/hxcpstty/

感谢您的帮助

Answer 1

您需要将test 声明为局部变量，这样您的函数的所有调用都不会引用完全相同的全局变量。当你不声明你的变量时，它会变成一个隐含的全局变量，这可能会导致各种问题。如果您在 strict mode 中运行代码，解释器也会将此标记为错误（这通常很有帮助）。

您还需要将函数的返回结果分配给一个变量，以便您可以使用该变量引用新创建的对象。

function make_test(name, job, ID) {
    var test = {};
 // ^^^
    test.name = name;
    test.job = job;
    test.ID = ID;
    return test;
}
var o1 = make_test("Paul", "manager", 1);
var o2 = make_test("John", "employee", 2);
var o3 = make_test("Jan", "employee", 2);

console.log(o1.name);   // "Paul"
console.log(o3.job);    // "employee"

你也不需要new 在你的函数前面，因为你没有使用系统创建的对象——你只是创建自己的对象并返回它。它仍然可以与new 一起使用，但它很浪费，因为您没有使用系统将使用new 创建的对象。

如果您希望它是一个使用new 的实际构造函数并且可以继承原型，那么请执行以下操作：

function Make_test(name, job, ID) {
    this.name = name;
    this.job = job;
    this.ID = ID;
}

var o1 = new Make_test("Paul", "manager", 1);
var o2 = new Make_test("John", "employee", 2);
var o3 = new Make_test("Jan", "employee", 2);

注意，我在这里使用了大写的构造函数名称，因为按照惯例，构造函数的初始大写名称和常规函数的初始小写名称。

---

您也可以完全删除声明：

function make_test(name, job, ID) {
    return {name: name, job: job, ID: ID};
}

或者使用 ES6 语法：

function make_test(name, job, ID) {
    return {name, job, ID};
}