Skip to content

Commit b7b5db7

Browse files
committed
Another bug is fixed in the bonus question editiong part
1 parent fe89c60 commit b7b5db7

File tree

1 file changed

+8
-4
lines changed

1 file changed

+8
-4
lines changed

solutions/javascript-test-3.js

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -428,18 +428,22 @@ const addUserSkill = (arr, name, skill) => {
428428
console.log(addUserSkill(users, 'Brook', 'Node'));
429429

430430
const editUser = (arr, name, newUser) => {
431+
let found = false;
431432
for (const user of arr) {
432433
if (user['name'] === name) {
433434
user.name = newUser.name;
434435
user.scores = newUser.scores;
435436
user.skills = newUser.skills;
436437
user.age = newUser.age;
437-
} else {
438+
found = true;
439+
break;
440+
}
441+
}
442+
if(!found) {
438443
return 'A user does not exist';
439444
}
440-
}
441445

442446
return arr;
443447
};
444-
// console.log(editUser(users, "Brook", user));
445-
// console.log(users);
448+
console.log(editUser(users, "Brook", newUser));
449+
console.log(users);

0 commit comments

Comments
 (0)