为什么这段代码在 codewars 的测试用例中会收到错误？ [复制]

Question

我不知道为什么，但我在运行以下代码时总是收到错误消息：

function list(names) {
  let str = '';
  if (!names) {
    return ''
  } else if (names.length == 2) {
    return names[0].name + ' & ' + names[1].name;
  } else if (names.length == 1) {
    return names[0].name;
  } else {
    for (i = 0; i < (names.length - 2); i++) {
      str += names[i].name + ', ';
    }

    str += names[names.length - 2].name + ' & ' + names[names.length - 1].name;
    return str;
  }
}

在codewars kata 中有测试用例。 错误是：

TypeError: Cannot read property 'name' of undefined
    at list
    at /home/codewarrior/index.js:36:19
    at /runner/frameworks/javascript/cw-2.js:152:11
    at Promise._execute

names 应该是一个可以为空的数组，并且数组元素是具有名称属性的对象。

Answer 1

作为替代解决方案，您可以使用 map、lastIndexOf 和 slice。

你必须遇到2个极端情况：

1) 一个长度为零的输入数组 那么你必须将return空字符串作为

 if (!names.length) return "";

2) 长度为 1 的输入数组，然后您只需将名称返回为：

if (names.length === 1) return names[0].name;

function list(names) {
  if (!names.length) return "";
  if (names.length === 1) return names[0].name;
  const namesStr = names.map((o) => o.name).join(", ");
  const index = namesStr.lastIndexOf(",");
  const result = namesStr.slice(0, index) + " &" + namesStr.slice(index + 1);
  return result;
}

const result = list([{ name: "Bart" }, { name: "Lisa" }, { name: "Maggie" }]);
console.log(result);

Answer 2

kata 提到您应该为空数组返回一个空字符串。我想你打算这样做

if (!names) {
    return ''
  }

...但是一个空数组仍然是真实的。也许那行应该是if (!names.length)。