自定义isEmpty函数判断JavaScript空对象存在哪些问题？是否存在该函数不适用的场景？

嘿，很高兴你自己动手尝试解决这个问题！你的这个isEmpty函数在很多常规场景下确实能正常工作，但它确实存在几个容易踩坑的边界情况，咱们一步步拆解：

你的函数存在的问题 #

• 原型链可枚举属性会干扰结果
  for...in循环会遍历对象自身以及原型链上所有可枚举的属性。如果项目里有人修改了Object.prototype（比如添加了自定义的可枚举属性），那你的函数会把任何空对象误判为非空。举个实际的例子：

  // 假设某个第三方库或团队成员添加了这个原型属性
  Object.prototype.globalProp = 'some value';
  console.log(isEmpty({})); // 会返回false，因为遍历到了globalProp
  

• 无法处理非对象类型的输入
  如果传入的不是对象（比如null、undefined、字符串、数字），for...in循环会直接抛出TypeError。比如调用isEmpty(null)，浏览器会报错，而实际开发中我们通常希望函数能优雅处理这类异常输入。

需要使用更严谨方案的场景 #

当你需要让函数更健壮，覆盖以下场景时，就需要用到那些看起来“复杂”的方案：

1. 过滤原型链属性，只检查对象自身属性
   这时候可以给你的循环加上hasOwnProperty判断，确保只遍历对象自己的属性：

   function isEmpty(obj) {
     for(var key in obj) {
       if(obj.hasOwnProperty(key)) { // 仅检查对象自身的可枚举属性
         return false;
       }
     }
     return true;
   }
   

   这样哪怕原型链被修改，也不会影响判断结果。

2. 需要更简洁的现代写法
   ES5及以上支持的Object.keys(obj).length === 0也是个不错的选择，它只返回对象自身的可枚举属性，代码更简洁，也能避免原型链问题。不过要注意，如果你需要兼容IE8及以下的老浏览器，这个方法就不适用了。

3. 需要检测不可枚举属性的极端场景
   如果对象存在不可枚举的自身属性（这种情况在业务开发中很少见，但在一些框架或库的内部逻辑中可能出现），for...in（哪怕加了hasOwnProperty）和Object.keys都检测不到。这时候可以用Object.getOwnPropertyNames(obj).length === 0，甚至要考虑Symbol类型的属性，用Reflect.ownKeys(obj).length === 0来做全面检查。

4. 需要处理非对象输入
   严谨的函数通常会先判断输入类型，比如先检查obj是否是对象且不为null：

   function isEmpty(obj) {
     if (obj === null || typeof obj !== 'object') {
       return true; // 或者根据需求返回false，取决于你对非空对象的定义
     }
     for(var key in obj) {
       if(obj.hasOwnProperty(key)) {
         return false;
       }
     }
     return true;
   }
   

总的来说，你的基础版本在原型链干净、输入都是标准空对象的简单场景下没问题，但在需要覆盖边界情况、保证函数健壮性的场景下，那些“复杂”的方案其实是必要的。

内容的提问来源于stack exchange，提问作者NoobScript