function lookUpProfile(firstName, prop){
var result;
for (var i = 0; i < contacts.length; i++) {
if (contacts[i].firstName === firstName)
if (contacts[i].hasOwnProperty(prop)) {
return contacts[i][prop];
} else if (contacts[i].hasOwnProperty(prop) === false){
return “No such property”;
}else {
return "No such contact";
}
}
// Only change code above this line
}
// Change these values to test your function
lookUpProfile(“Akira”, “likes”);
In the above, every condition passes except for “No such contact”; - I’ve been at this for ages now and finally decided to just ask for help.
you need to open some curly braces after your first if statement.
you don’t need to check for the hasOwnProperty(prop) === false. When you reach that point, you already tested on the if the same thing. If it is not true, you fall on the else statement anyway. No need to test again.
finally, the return "No such contact";, needs to be done outside the for loop. Only after looping through all the names, you can be sure that the contact does not exist. If the name exists, the function returns something else and finishes execution before reaching this point.
So, now you can try implementing those things on your own. Of you’re totally confused here’s your code with those fixes:
Spoiler
function lookUpProfile(firstName, prop){
var result;
for (var i = 0; i < contacts.length; i++) {
if (contacts[i].firstName === firstName){ // <-- here you need to open curly braces
if (contacts[i].hasOwnProperty(prop)) {
return contacts[i][prop];
} else{ // <- you dont need to check again the value of contacts[i].hasOwnProperty(prop),
// if not true we do the else, which means it was false
return "No such property";
}}} //Here close all ifs and the for loop
return "No such contact"; //only after looping through everything and not finding anything, you can safely
//say there is no such contact.
}
Why can’t the return "No such property" be inside an else statement. Why doesn’t is work? The second if statement has an else statement but no else statement for the first one.