Basic JavaScript - Profile Lookup

I cannot understand why the return “No such contact” is outside the for loop? I mean it should be the else part of first if statement where some name is not matched to firstName. But why that return is outside the loop

 // Setup
const 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(let 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";
            }
        }
                 return "No such contact";

  }

  // Only change code above this line
}

lookUpProfile("Akira", "likes");

Your browser information:

User Agent is: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/114.0.0.0 Safari/537.36

Challenge: Basic JavaScript - Profile Lookup

Link to the challenge:

Thats essentially what its doing. The inner part of the if statement only runs if it finds a contact, and then it looks for the property. If there is a property then inner if returns the contact with the property, or if there is no property then it returns the no property message. However, if it doesn’t find a contact then it runs the contact message

Whenever i feel confused about my logic, i draw out flowchart / use pseudo-code to make my mind clearer.

An add-on:

if you place return inside the for-loop:

You are not scanning for whole array. Your function immediately stops when the first arrayitem do not matched. Doing so, it is like “break” in a for-loop, which you dont want.

This topic was automatically closed 182 days after the last reply. New replies are no longer allowed.