JS 使 forEach 匹配更有效

Question

variants是 objectsselectedOptions的对象，是 option1、option2、option3 的对象。下面forEach通过搜索variants来找到匹配项。是否有更有效的方法，使用数组方法或类似方法来执行以下操作：Object.values(variants).forEach(variant => {    if (variant.options.option1 === selectedOptions.option1 && variant.options.option2 === selectedOptions.option2 && variant.options.option3 === selectedOptions.option3) {        selectedVariant = variant.gid;    }});

Answer 1

您在这里要做的一件事是遍历所有变体，selectedVariant每次找到匹配项时都会覆盖。如果应该只有一个匹配，当不再需要它时，您仍然会访问所有其他变体。

更有效的是：

selectedVariant = Object.values(variants).find(variant => (
    variant.options.option1 === selectedOptions.option1 && 
    variant.options.option2 === selectedOptions.option2 &&
    variant.options.option3 === selectedOptions.option3
    )).gid;

这样，您在找到匹配项的那一刻就停止了。

老实说，将一个变体设置为等于另一个变体的 gid 看起来是错误的。将分配给的变量命名为selectedVariantGid，或分配整个变体，然后在.gid需要时使用该属性。清晰的命名很重要。