JavaScript中if(array.length>0)与if(array?.length)的区别及假值疑问

先明确核心需求：你的代码需要确保$cart.coupons是存在的非空数组，否则调用find方法会直接报错。下面拆解两种写法的区别和问题：

1. 你的原始写法 #

$: hasActiveCoupons = $cart.coupons && $cart.coupons.length > 0;

if (hasActiveCoupons && $cart.coupons.find((coupon) => coupon.id === couponCode)) {
  // ...
}

存在的问题 #

• 只判断了$cart.coupons不是null/undefined，以及length > 0，但完全没校验是否为数组。如果$cart.coupons是字符串（比如'abc'）、自定义类数组对象（比如{ length: 3 }）这类非数组值，$cart.coupons.length > 0会返回true，但后续调用find方法时会直接抛出错误——因为只有数组原型上才有find方法。
• 同事提到的“返回假值”，大概率是指某些边界场景：比如$cart.coupons是数组但length为0，这时候hasActiveCoupons为false（这本身符合逻辑）；但如果$cart.coupons是一个带负数值length的对象（虽然数组的length不能为负，但自定义对象可以），$cart.coupons.length > 0也会返回false，这时候就不符合“判断非空数组”的预期。

2. 同事的修正写法 #

if ($cart.coupons?.length && $cart.coupons.find((coupon) => coupon.id === couponCode)) {
  // ...
}

优化点 #

• 使用可选链操作符?.：当$cart.coupons是null/undefined时，$cart.coupons?.length会直接返回undefined，在if判断中被转为false，避免了直接访问null/undefined的length属性导致的报错。
• 将length > 0简化为length：利用JS的布尔转换规则，只要length是真值（即非0、非NaN、非null/undefined）就会返回true。对于数组来说，length为0时是假值，大于0时是真值，这和length > 0的效果完全一致，但写法更简洁。

仍存在的不足 #

同事的写法依然没解决“判断是否为数组”的核心问题——如果$cart.coupons是字符串或类数组对象，依然会触发find方法报错。更严谨的写法应该补上数组类型校验：

if (Array.isArray($cart.coupons) && $cart.coupons.length && $cart.coupons.find(coupon => coupon.id === couponCode)) {
  // ...
}

或者结合可选链进一步优化空值处理：

if (Array.isArray($cart.coupons) && $cart.coupons?.length && $cart.coupons.find(coupon => coupon.id === couponCode)) {
  // ...
}

总结 #

• 你的写法：仅处理了null/undefined和数组长度，但缺失数组类型校验，存在报错风险；length >0的逻辑本身没问题，但面对非数组的类数组值时会出现错误判断。
• 同事的写法：用可选链优化了空值访问，简化了长度判断，但同样没做数组类型校验，仍有潜在报错可能。
• 最严谨的实现：先通过Array.isArray()确认是数组，再判断长度和执行find逻辑。

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