Map function in JSON array

Question

I want to find an object from a JSON array. My code is working fine with the object found. But the issue is what if the employee id is not? My code console's "not found" two times. Kindly guide me on what the issue is with my code.

var E_ID

let empArray = [
 {

  "employee": {
    "employee_ID": 16,
    "employee_designation": "Node.js",
     "employee_FirstName": "John",
     "employee_lastName": "Doe",
  }
 },
 {
   "employee": {
    "employee_ID": 17,
    "employee_designation": "Full Stack",
    "employee_FirstName": "Samual",
    "employee_lastName": "Smith",
  },
 }
]

 function search_EmployeeByID(E_ID) {
  empArray.map(item => {
    if (item.employee.employee_ID == E_ID) {
         console.log(item.employee)
         return true
     
   }else {
          console.log("not found")
          return false
   }

 })

}

E_ID = parseInt(prompt("Enter Employee_ID to search for:"))
search_EmployeeByID(E_ID);`

find will loop over each item, until found, hence 2 times.. remove the comments, keep it simple const result = empArray.find(item => item.employee.employee_ID === E_ID), no function needed — Lawrence Cherone
– Lawrence Cherone, Commented Jan 9, 2023 at 22:00
Regarding changing find() to map(), the issue is the same... it will iterate over every entry — Phil
– Phil, Commented Jan 9, 2023 at 22:15

Barmar · Accepted Answer · 2023-01-09 22:00:16Z

2

The if statement should not be inside find(). find() will return the matching element, or null if it's not found. So test the return value.

function searchEmployeeByID(e_id) {
    let emp = empArray.find(e => e.employee.employee_ID == e_id);
    if (emp) (
        console.log(emp.employee);
        return true;
    } else {
        console.log("not found");
        return false;
    }
}

answered Jan 9, 2023 at 22:00

Barmar

789k57 gold badges554 silver badges669 bronze badges

Sign up to request clarification or add additional context in comments.

3 Comments

Muhammad Sohaib 3837-FBASBSSEF Over a year ago

Sorry, My task was by using a map(), find was used as I was learning to do it with find. I edited my code,

Barmar Over a year ago

I can't think of any reasonable way to do this with map(). That returns a new array of the results of all the callbacks, it can't be used to select a specific element of the array.

Barmar Over a year ago

More generally, you can't tell if the element is found during the iteration, you have to wait until the end.

Collectives™ on Stack Overflow

Map function in JSON array

1 Answer 1

3 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

3 Comments

Your Answer

Sign up or log in

Post as a guest

Related