Sonarqube空数组使用告警的解决方案合理性咨询

Sonarqube抛出该告警的根本原因是你定义的productsData仅被初始化为空数组，后续没有任何修改赋值逻辑，Sonarqube检测到该变量永远为空，调用concat属于无意义的冗余操作。

方案合理性评估 #

你提出的return [...productsData, ...resultArray]方案是否合理，取决于你的业务场景：

• 如果productsData仅为临时占位的空数组，后续开发中会新增填充它的业务逻辑：该方案完全合理，相比concat写法可读性更高，后续你给productsData赋值后无需修改这行代码即可正常运行，也能直接消除Sonarqube告警。
• 如果productsData全程都不会有数据，永远是空数组：该方案可以消除告警，但仍存在冗余的空数组展开操作，不属于最优解。

更优处理方案 #

场景1：productsData永远为空，不需要拼接其他数据 #

直接返回resultArray即可，完全没有冗余操作，性能最优：

return resultArray;

场景2：需要保留和原有concat逻辑完全一致的行为 #

注意原有写法productsData.concat(...resultArray)和你修改后的展开写法存在逻辑差异：如果resultArray的子元素包含数组，concat会自动拆解一层子数组，而直接展开resultArray会保留子数组结构。如果要保留原有逻辑，同时兼容后续productsData的赋值，可以写成：

return [...productsData, ...[].concat(...resultArray)];

场景3：无需兼容原有concat的自动拆包逻辑，仅需要拼接两个数组 #

你提出的展开写法就足够简洁，也符合现代JS的编码习惯：

return [...productsData, ...resultArray];

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