add solution #2918
add solution #2918
Conversation
There was a problem hiding this comment.
Great job on implementing the task requirements! 🎉 Your code is well-structured and meets all the core requirements. The only minor suggestion is to optimize the getPreparedTodos function by directly returning the result of the map method, as it is already non-mutating. Keep up the excellent work and continue refining your skills! 😊
✨ Please rate this review in the Platform's notification center. I will be grateful for your feedback! ✨
| return todosFromServer.map(todo => { | ||
| return { | ||
| ...todo, | ||
| user: findUserById(todo.userId) || null, | ||
| }; | ||
| }); |
There was a problem hiding this comment.
The getPreparedTodos function uses map to transform todosFromServer. Since map is a non-mutating method, there is no need to create a copy of the array. You can directly return the result of map without wrapping it in another array. This aligns with the checklist item: "If you are using a non-mutating array method, you don't need to create a copy of the array."
DEMO LINK