JavaScript中.map内数组方法失效，求助修复括号验证函数

你的核心问题：用错了map方法 #

map的作用是遍历数组并转换每个元素，返回一个新数组，而你需要的是遍历过程中维护一个累积的状态（栈），所以用map完全不合适——你在每次map回调里都新建了let result = []，导致每次循环的栈都是独立的，之前的操作根本没法保留，这就是你用push或slice没效果的原因。

先纠正你代码里的基础错误 #

1. 逻辑或要用||，不是|（|是位运算，不是逻辑判断）
2. 数组不能用+=来添加元素，这会把数组转成字符串拼接，正确添加左括号应该用result.push(currentBrace)
3. slice(0, -2)是错误的：slice返回新数组，不会修改原数组；而且你需要的是取出栈顶元素（用pop），还要判断这个元素和当前右括号是否匹配，不是随便删除

正确的实现思路（用栈） #

1. 初始化一个空数组作为栈，用来存储遇到的左括号
2. 遍历每个括号字符：
   • 如果是左括号(/{/[，直接推入栈
   • 如果是右括号，先检查栈是否为空（空的话说明没有对应的左括号，直接返回false）；然后取出栈顶元素，判断是否和当前右括号匹配：
     • )对应(，]对应[，}对应{
     • 不匹配就返回false
3. 遍历结束后，检查栈是否为空——为空说明所有左括号都找到了匹配的右括号，返回true；否则还有未匹配的左括号，返回false

正确代码实现 #

function validBraces(braces) {
    const stack = [];
    // 用对象存右括号到左括号的映射，方便快速判断匹配
    const braceMap = {
        ')': '(',
        ']': '[',
        '}': '{'
    };

    for (const brace of braces) {
        // 如果是左括号，推入栈
        if (brace === '(' || brace === '{' || brace === '[') {
            stack.push(brace);
        } else {
            // 右括号的情况：栈为空 或 栈顶元素不匹配，直接返回false
            if (stack.length === 0 || stack.pop() !== braceMap[brace]) {
                return false;
            }
        }
    }
    // 遍历完后栈必须为空，才说明全部匹配
    return stack.length === 0;
}

// 测试示例
console.log(validBraces("(){}[]")); // true
console.log(validBraces("([{}])")); // true
console.log(validBraces("(}")); // false
console.log(validBraces("[(])")); // false
console.log(validBraces("[({})](]")); // false

关于map的补充说明 #

如果非要用map（完全不推荐，因为不符合它的设计意图），你需要把栈变量放到map外面，而且map最后还要做判断，但这样写很别扭，比如：

function validBraces(braces) {
    const stack = [];
    const braceMap = { ')': '(', ']': '[', '}': '{' };
    
    [...braces].map(brace => {
        if (['(', '{', '['].includes(brace)) {
            stack.push(brace);
        } else {
            if (stack.length && stack[stack.length - 1] === braceMap[brace]) {
                stack.pop();
            } else {
                // 标记错误，避免后续判断出错
                stack.push('error');
            }
        }
    });
    
    return stack.length === 0;
}

但这种写法完全违背了map的使用场景，map应该是纯函数（不修改外部变量），所以强烈建议用for循环或forEach来做这种有状态累积的遍历。

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