有人可以解释我的方法有什么问题吗？

Question

我试图尝试这个问题。 https://leetcode.com/problems/valid-parentheses/ 。 但是，如果在第二个 for 循环中我可以比较 s.charAt(i) 和 stack.pop()，我感到很困惑。

基本上，我的方法是这样的。 将整个字符串压入堆栈，然后使用 stack.charAt(i) 遍历堆栈的前半部分，并使用 stack.pop() 将其与后半部分进行比较。 我只是想知道我的代码可能出了什么问题，因为我在期望一个真值时得到一个假值。 我只是想了解我的概念是否有缺陷？

class Solution {
    public boolean isValid(String s) {
        Stack<Character> stack = new Stack<>();
        
        boolean done = false;
        
        if(s.length() < 2)
            return true;
        
        for(int i = 0; i < s.length(); i++){
            stack.push(s.charAt(i));
        }
        
        for(int i = 0; i < s.length()/2; i++){
            if(s.charAt(i) == stack.pop())
                done = true;
        }
        
        return done;   
    }
}

Answer 1

虽然您的代码可能适用于“ {([])} ”和“ (()) ”等字符串，但您假设这些字符串是对称的。 诸如“ {()[]} ”之类的有效输入不会传入您的代码，因为它不是对称的。 修改您的代码以解决不对称问题。 提示：当字符是结束字符如“)”“}”和“]”时，可能会弹出 2 个元素。

Answer 2

您应该检查字符串中的括号是否匹配：对于'('有一个匹配')' 。 但是，请注意字符'('不等于字符')' 。

您的代码正在检查字符串是否匹配字符串的后半部分是字符串前半部分的模式，例如{[))[{ 。 换句话说，您正在检查输入是否为回文。

您应该采取的方法是仅将起始括号存储在堆栈中：

    for(int i = 0; i < s.length(); i++){
        if s.charAt(i) is an open bracket:
            stack.push(s.charAt(i));
        else:
            "pop" a character from stack
            if it's '(' make sure that s.charAt(i) is ')'
            if it's '[' make sure that s.charAt(i) is ']'
            if it's '{' make sure that s.charAt(i) is '}'
    }

Answer 3

您的代码首先迭代到字符串的中间并填充堆栈，然后从中间迭代到末尾并弹出堆栈。 换句话说，您隐含地假设有效字符串的前半部分是各种类型的左括号，第二部分是它们对应的右括号（例如([])或{[()]} ）。 然而，这些并不是唯一有效的字符串 - 没有任何限制表明右括号后不能跟左括号，只要配对匹配 - 例如， (()())是有效的细绳。

不要对字符串中的 position 进行假设，您应该遍历字符串，对于每个字符，如果它是一个左括号，则将其推入堆栈，如果它是一个右括号，则弹出堆栈并比较两者：

private static final Map<Character, Character> PARENS = new HashMap<>();
static {
    PARENS.put('(', ')');
    PARENS.put('[', ']');
    PARENS.put('{', '}');
}
public static boolean isValid(String s) {
    Stack<Character> stack = new Stack<>();

    for (int i = 0; i < s.length(); ++i) {
        char ch = s.charAt(i);
        if (PARENS.containsKey(ch)) {
            stack.push(ch);
        } else {
            if (stack.isEmpty()) {
                return false;
            }
            char match = stack.pop();
            if (ch != PARENS.get(match)) {
                return false;
            }
        }
    }

    return stack.isEmpty();
}