if else 语句 javascript 返回 false 语句

Question

有人可以帮助我编写这段代码吗？我正在尝试编写一个函数，该函数从对象中获取名称并返回名称标签： “嗨！我是[名称]，来自[国家/地区]。”

我试过这段代码

const GUEST_LIST = {
    Randy: "Germany",
    Karla: "France",
    Wendy: "Japan",
    Norman: "England",
    Sam: "Argentina"
}

function greeting(name) {
  var x = Object.keys(GUEST_LIST);
  const array = Object.keys(GUEST_LIST)
    .map(function(key) {
        return GUEST_LIST[key];
    });
  
  
  var txt ="";
  for (let i in x)
    {
      if (name === x[i])
        {
          txt = "Hi I'm "+x[i] +", and I'm from " +array[i];  
        }
      else
        {
          txt = "Hi! I'm a guest";
        }
      
    }
     return txt;
}
console.log(greeting("Randy"))

但它总是返回“嗨！我是客人”，除非我输入Sam，

Answer 1

您的问题是，x即使您从数组中找到了与传递到函数中的名称相匹配的名称，for 循环仍将继续循环数组中的其他名称。这意味着在 for 循环的进一步迭代中，您的代码块将运行并覆盖先前设置else的值。txt这就是“Sam”起作用的原因，因为这是数组中的姓氏x，因此txt不会被 for 循环的进一步迭代覆盖。

关于代码需要注意的另一件事是，for...in不应使用循环来迭代 array。它可能会导致访问不需要的值，因为它不仅循环遍历数组的索引，还循环遍历其他属性。

话虽如此，您对代码进行了过度设计。目前，您的对象存储键值对。键是名称，值是国家/地区。对于对象中的每个键，您可以使用括号表示法访问它：

console.log(GUEST_LIST["Randy"]); // Germany

考虑到这个想法，您的name变量可以用作对象的键，然后可以使用该键来获取国家/地区。如果 GUEST_LIST 对象不包含您的密钥（即：尝试从对象访问密钥返回的值是假的），那么您可以返回默认"Hi! I'm a guest"文本：

const GUEST_LIST = {
  Randy: "Germany",
  Karla: "France",
  Wendy: "Japan",
  Norman: "England",
  Sam: "Argentina"
};

function greeting(name) {
  const country = GUEST_LIST[name];
  if(country)
    return "Hi I'm "+name +", and I'm from " +country; 
  else
    return "Hi! I'm a guest";
}
console.log(greeting("Randy"));