Profile Lookup question about return

JavaScript
1

Here is my code

//Setup
var contacts = [
    {
        "firstName": "Akira",
        "lastName": "Laine",
        "number": "0543236543",
        "likes": ["Pizza", "Coding", "Brownie Points"]
    },
    {
        "firstName": "Harry",
        "lastName": "Potter",
        "number": "0994372684",
        "likes": ["Hogwarts", "Magic", "Hagrid"]
    },
    {
        "firstName": "Sherlock",
        "lastName": "Holmes",
        "number": "0487345643",
        "likes": ["Intriguing Cases", "Violin"]
    },
    {
        "firstName": "Kristian",
        "lastName": "Vos",
        "number": "unknown",
        "likes": ["JavaScript", "Gaming", "Foxes"]
    }
];


function lookUpProfile(name, prop){
// Only change code below this line
for (var i = 0; i < contacts.length; i++) {
  if (name === contacts[i]["firstName"]) {
    if (contacts[i].hasOwnProperty(prop)) {
      return contacts[i][prop];
    }
    else {
      return "No such property";
    }
  }
  else {
    return "No such contact";
  }
}
// Only change code above this line
}

// Change these values to test your function
console.log(lookUpProfile("Akira", "lastName"));

I know that the correct response would be to return “No such contact” as a command out of the “If else” loop, but I don’t understand why… Doesn’t it make more logical sense to create an “else” statement to account for if name !== firstName?

2

If you do it inside the loop, then you will only ever check the first contact in the list. If name doesn’t match firstName, it will return “No such contact” immediately.

2 Likes