function isDuplicate(arr, propName){ let tempObj = {}; arr.forEach(item => { if (tempObj[item.propName]) { return true; } tempObj[item.propName] = true; }); return false; } let arr = [{names:1,aee:123},{names:1,aee:15456423},{names:2,aee:4658456}]; if(isDuplicate(arr, 'names')){ console.log('names属性存在重复'); }else{ console.log('names属性不存在重复'); } 代码优化

在上面的代码中，使用了 `arr.forEach` 方法来遍历数组，但是在判断重复时使用 `item.propName`，这样会导致判断的属性名永远为字符串 "propName"，而不是实际传入的属性名。

正确的写法应该是使用 `item[propName]` 来获取属性值，这样才能动态地获取传入的属性名。

以下是修改后的代码：

function isDuplicate(arr, propName){
  let tempObj = {};
  arr.forEach(item => {
    if (tempObj[item[propName]]) {
      return true;
    }
    tempObj[item[propName]] = true;
  });
  return false;
}

let arr = [{names:1,aee:123},{names:1,aee:15456423},{names:2,aee:4658456}];
if(isDuplicate(arr, 'names')){
  console.log('names属性存在重复');
}else{
  console.log('names属性不存在重复');
}

输出结果为：names属性存在重复