ExtendScript中switch语句多条件匹配实现问题咨询

首先得说，你当前的写法是有问题的，原因出在switch的匹配逻辑上：switch是拿searchFor和每个case后面的值做严格相等比较的。你的第二个case写的是myFound[i].parent.fillColor == app.activeDocument.swatches.item(14)，这个表达式会被计算成true或false，然后和searchFor（也就是那个正则字符串）做比较，显然永远不会匹配上，所以这个逻辑根本不会触发。

那正确的实现思路应该是怎样的？因为你需要的是同时满足两个条件：

1. searchFor等于"^201\\d$"
2. 当前找到的元素的父元素填充色是指定的第14个色板

针对这个需求，我给你两种方案：

方案一：改用if判断（更直观推荐） #

既然是多条件组合判断，if其实比switch更适合这种场景，代码可读性更高：

function changeGrep(searchFor){
    app.findGrepPreferences.findWhat = searchFor;
    var myDoc = app.activeDocument;
    // 提前缓存色板，避免循环里重复查找，提升性能
    var targetSwatch = myDoc.swatches.item(14);
    var newSwatch = myDoc.swatches.item(3);
    var myFound = myDoc.findGrep();
    
    for(var i=0; i<myFound.length; i++){
        if(searchFor === "^201\\d$" && myFound[i].parent.fillColor === targetSwatch){
            myFound[i].parent.fillColor = newSwatch;
        }
        // 其他条件判断可以继续加else if
    }
}

方案二：调整switch写法（如果一定要保留switch） #

如果你的代码里还有其他基于searchFor的case要处理，非要用switch的话，可以把switch的判断值改成true，然后每个case写完整的条件表达式：

function changeGrep(searchFor){
    app.findGrepPreferences.findWhat = searchFor;
    var myDoc = app.activeDocument;
    var targetSwatch = myDoc.swatches.item(14);
    var newSwatch = myDoc.swatches.item(3);
    var myFound = myDoc.findGrep();
    
    for(var i=0; i<myFound.length; i++){
        switch(true){
            case (searchFor === "^201\\d$" && myFound[i].parent.fillColor === targetSwatch):
                myFound[i].parent.fillColor = newSwatch;
                break;
            // 其他case也写完整的条件表达式
            // case (searchFor === "其他正则" && 其他条件):
            //     // 逻辑
            //     break;
        }
    }
}

另外还有个小优化点：你在循环里多次调用app.activeDocument，最好提前把它赋值给变量myDoc，还有色板也提前缓存，减少对DOM的重复访问，脚本运行会更高效。

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