你这个问题的关关键原因是你的判断 == "" 只能匹配变量值是 "" 的情况，但实际上 undefined 和 null 也应该作为无属性的情况处理，这里可以假设只有字符串值的情况出现，直接用 !contacts[i][prop] 来判断。

另外，你的代码里有些地方可以优化，我注释出来了

function lookUp(firstName, prop) {
    // Only change code below this line
    var name = false;
    var pro = false;

    // [J.Fan> 这里为什么是 i < 4，对于一个已知数组数据，
    // 应该用 contacts.length 来取其长度以适应数据的变化
    for (var i = 0; i < 4; i++) {
        if ((contacts[i].firstName == firstName) && (contacts[i][prop] !== "")) {
            name = true;
            pro = true;

            // 这里既然已经 return 了，就不会执行到下面对 name 和 pro 的判断
            // 所以上面两句赋值可以不要
            return contacts[i][prop];
        } else {
            // 再次判断 contacts[i].firstName == firstName
            // 可以考虑单独把这个条件提出来作为外层 if，内层再进行 contacts[i][prop] 的判断
            if (contacts[i].firstName == firstName)
                name = true;
            if (contacts[i][prop] !== "")
                pro = true;
        }
    }
    if (!name)
        return "No such contact";
    if (!pro)
        return "No such property";
    // Only change code above this line
}

如果我来写，我会想到用数组的 filter 方法先按 firstName 把对象找出来再来取其属性

// es6 语句
function lookUp2(firstName, prop) {
    // filter 过程出来是一个数组，取其第 0 个元素就是找到的第一个对象
    // 如果没有对象被找到，返回 []，其第 0 个元素是 undefined，不会引发异常
    const contact = contacts.filter(c => c.firstName === firstName)[0];
    if (!contact) {
        return "No such contact";
    }
    
    // 当 contact[prop] 是 undefined、null或 "" 等可以判 false 的值时
    // 通过 || 运算符，取第二个操作数
    return contact[prop] || "No sch property";
}

这里用 filter 是一个全遍历的过程，对于数组数据量大的时候，效率会比较低。实际用 find() 更合适，找到即返回该对象，否则返回 undefined。不过 find() 还在试验阶段，所以当前我没有使用它。如果使用它，代码会是这样

// es6
function lookUp3(firstName, prop) {
    // find 返回的是找到的对象本身或 undefined，而不是数组，所以不需要再取 [0]
    const contact = contacts.find(c => c.firstName === firstName);
    if (!contact) {
        return "No such contact";
    }
    return contact[prop] || "No sch property";
}

因为Akira的确是没有address这个属性的啊

if((contacts[i].firstName==firstName) && && (contacts[i][prop]!==""))又正确。

这个本来就正确啊..因为undefined本来就不等于""

建议:
把 (contacts[i][prop]!=="") 改成(contacts[i][prop])

修改：

`
if(contacts[i][prop]){
   pro = true;
}
`

离开了看见