Проблема в том, что каждый из аргументов сравнивается со всеми ключами объекта. И если хотя бы с одним из ключей аргумент не совпадает (а так оно и будет), то ключ удаляется из объекта.
Нужно проходить по ключам объекта и проверять, есть ли такой ключ среди аргументов:
var arr = [{
name: 'Вася',
age: '21',
gender: 'male'
}];
function fun() {
for (var key in arr[0]) {
if (Array.prototype.indexOf.call(arguments, key) == -1) {
delete arr[0][key];
}
}
return arr;
}
console.log(fun('name', 'age'));
Так как меняется объект, являющийся первым элементом массив, то newArr
я из кода убрал. Если нужно в итоге получать копию массива с копиями объектов, то можно это сделать как-то так:
var arr = [{
name: 'Вася',
age: '21',
gender: 'male'
}];
function fun() {
var newArr = arr.slice();
for (var i = 0; i < newArr.length; i++)
newArr[i] = Object.assign({}, newArr[i]);
for (var key in newArr[0]) {
if (Array.prototype.indexOf.call(arguments, key) == -1) {
delete newArr[0][key];
}
}
return newArr;
}
console.log(fun('name', 'age'));
console.log(arr);
newArr
, кстати, тут вообще бесполезен: вы всё равно меняете исходный массив.