올바른 괄호

문제

괄호가 바르게 짝지어졌다는 것은 '(' 문자로 열렸으면 반드시 짝지어서 ')' 문자로 닫혀야 한다는 뜻입니다. 예를 들어
"()()" 또는 "(())()" 는 올바른 괄호입니다.
")()(" 또는 "(()(" 는 올바르지 않은 괄호입니다.
'(' 또는 ')' 로만 이루어진 문자열 s가 주어졌을 때, 문자열 s가 올바른 괄호이면 true를 return 하고, 올바르지 않은 괄호이면 false를 return 하는 solution 함수를 완성해 주세요.

제한사항

• 문자열 s의 길이 : 100,000 이하의 자연수
• 문자열 s는 '(' 또는 ')' 로만 이루어져 있습니다.

첫 시도

function solution(s) {
    let answer;
    const sArr = [...s]
    if(sArr[sArr.length - 1] === "(" || sArr[0] !== "(") {
        answer = false
        return answer
    } else if(sArr[sArr.length - 1] === ")") {
        let opPar = 0
        let cloPar = 0
        sArr.forEach((el) => {
            if(el === "(") {
                opPar++
            } else {
                cloPar++
            }
        })
        return answer = opPar === cloPar ? true : false
    }
    return answer
}

시나리오

먼저 주어진 배열의 첫번째 값이 ")"이거나 마지막 값이 "("면 바로 false를 리턴하고 모두 만족하면
배열안의 여는 괄호와 닫는 괄호의 갯수를 비교한다음
같으면 true 다르면 false를 리던하도록하였다.

문제 발생

테스트는 다 통과되었지만 제출 하면 런타임에러 5번과 11번?이 발생

문제 원인

찾아보니 반례가 존재하였다.
())(()이면 위의 조건에 모두 충족되지만 제대로 된 괄호가 아니다.
문제의 원인 패턴은 반복문이 돌아갈때 여는 괄호보다 닫는 괄호의의 갯수가 많아질 때 생긴다.

문제 해결

break;가 지원이 안되는 forEach에서 for문으로 바꾸고 여는 괄호와 닫는괄호의 갯수를 샐 때 닫는 괄호가 더 많이 세어지면 바로 false를 리턴하도록 하는 if문을 추가하였다.

최종코드

function solution(s) {
    let answer;
    const sArr = [...s]
    if(sArr[sArr.length - 1] === "(" || sArr[0] !== "(") {
        answer = false
        return answer
    } else if(sArr[sArr.length - 1] === ")") {
        let opPar = 0
        let cloPar = 0
        for(let idx = 0; idx < sArr.length; idx++) {
            if(sArr[idx] === "(") {
                opPar++
            } else {
                cloPar++
            } 
            if(opPar < cloPar) {
                answer = false
                break;
            }
        }
        return answer = opPar === cloPar ? true : false
    }
    return answer
}