if 语句未返回预期结果 Javascript

Question

在了解了一些对象构造函数后，我编写了一个小脚本，我想用这段代码更进一步，但 if statement 的行为与预期不符......我只想知道为什么?我知道这种脚本在 Javascript 中是不正常的......但我相信=== 正在评估为true，因此无论我输入什么，都会在警报框中打印dave，而不是遵循if statement 条件。

var dave, jason, nick, get;

get = prompt('Input your Name', 'Name Here')


var Person = function (name, birthYear, job) {
    this.name = name;
    this.birthYear = birthYear;
    this.job = job;
};

dave = new Person('Dave', 1976, 'Showman');
jason = new Person('Jason', 1987, 'Novice');
nick = new Person('Nick', 1993, 'Cable-tech');


function printObj(object) {
    var output = '';
    for (var property in object) {
        output += property + ': ' + object[property] + '\n';
    }
    alert(output);
}

if (get.toLowerCase === 'dave'.toLowerCase) {
    printObj(dave);
} else if (get.toLowerCase === 'jason'.toLowerCase) {
    printObj(jason);
} else if (get.toLowerCase === 'nick'.toLowerCase) {
    printObj(nick);
} else {
    alert('Not a defined object')
}

比较明显我可能做错了...但这就是在 javascript on freenode 中向我解释的方式，因为起初这是我的问题，这仍然是我想。我认为我只是在做一些明显混乱的事情，感谢我收到的任何见解，MDN 和 W3 只能解释这么多。

Answer 1

get.toLowerCase() 是一个方法，应该用左括号和右括号调用。目前的写法是检查方法是否等于不同字符串实例上的相同方法。

if (get.toLowerCase() === 'dave') {
    printObj(dave);
} else if (get.toLowerCase() === 'jason') {
    printObj(jason);
} else if (get.toLowerCase() === 'nick') {
    printObj(nick);
} else {
    alert('Not a defined object')
}

由于您要与小写字符串进行比较，因此您无需在 'dave'、'jason' 或 'nick' 上调用 toLowerCase()。我想您可能想与变量dave、jason 和nick 中存储的值进行比较。

if (get.toLowerCase() === dave.name.toLowerCase()) {
    printObj(dave);
} else if (get.toLowerCase() === jason.name.toLowerCase()) {
    printObj(jason);
} else if (get.toLowerCase() === nick.name.toLowerCase()) {
    printObj(nick);
} else {
    alert('Not a defined object')
}

最后，您不应将变量命名为get。 get 在 ES6 中是保留的，所以当浏览器开始原生支持它时，你的代码会做一些非常奇怪的事情。我建议类似userInput。

var dave, jason, nick, userInput;

userInput = prompt('Input your Name', 'Name Here')

//...other code...

if (userInput.toLowerCase() === dave.name.toLowerCase()) {
    printObj(dave);
} else if (userInput.toLowerCase() === jason.name.toLowerCase()) {
    printObj(jason);
} else if (userInput.toLowerCase() === nick.name.toLowerCase()) {
    printObj(nick);
} else {
    alert('Not a defined object')
}

Answer 2

您以错误的方式使用.toLowerCase() 方法。另外，您正在使用get

您的代码：

var dave, jason, nick, get;

get = prompt('Input your Name', 'Name Here')

// other code

if (get.toLowerCase === 'dave'.toLowerCase) {
    printObj(dave);
} else if (get.toLowerCase === 'jason'.toLowerCase) {
    printObj(jason);
} else if (get.toLowerCase === 'nick'.toLowerCase) {
    printObj(nick);
} else {
    alert('Not a defined object')
}

正如您在代码中看到的那样，您的 .toLowerCase 方法错过了 ()。并且请不要使用get 作为变量名，因为get 是关键字。

试试这个：

var dave, jason, nick, inputValue;

inputValue = prompt('Input your Name', 'Name Here')

// other code

if (inputValue.toLowerCase() === 'dave'.toLowerCase()) {
    printObj(dave);
} else if (inputValue.toLowerCase() === 'jason'.toLowerCase()) {
    printObj(jason);
} else if (inputValue.toLowerCase() === 'nick'.toLowerCase()) {
    printObj(nick);
} else {
    alert('Not a defined object')
}

此外，如果您要比较自己定义的比较，则无需添加.toLowerCase()。您实际上想将输入值（这是一种未知格式，例如大写或小写）与您要比较的小写字符串值进行比较。

使用这个：

if (get.toLowerCase === 'dave')

而不是

if (get.toLowerCase === 'dave'.toLowerCase())