https://school.programmers.co.kr/learn/courses/30/lessons/42586?language=javascript
🔊 문제 설명
괄호가 바르게 짝지어졌다는 것은
'(' 문자로 열렸으면 반드시 짝지어서 ')' 문자로 닫혀야 한다는 뜻입니다. 예를 들어
"()()" 또는 "(())()" 는 올바른 괄호입니다.
")()(" 또는 "(()(" 는 올바르지 않은 괄호입니다.
'(' 또는 ')' 로만 이루어진 문자열 s가 주어졌을 때, 문자열 s가 올바른 괄호이면 true를 return 하고, 올바르지 않은 괄호이면 false를 return 하는 solution 함수를 완성해 주세요.
👩💻 나의 첫번째 풀이
function solution(s){
let stack = [];
for(let i = 0; i < s.length; i++){
let bracket = s.charAt(i);
if(bracket === '('){
stack.push(bracket);
}else {
if(stack.length === 0)
return false;
stack.pop();
}
}
return stack.length === 0;
}
효율성에서 계속 0점이였다. 질문하기에서 stack에( 라는 문자 하나만 누적되는 형태라면, 굳이 stack 배열을 쓰지 않고 int 변수 하나로 + 혹은 - 시키는 것이 유리하다는 것을 알 수 있었다!
또한, 괄호는 ( 와 ) 가 하나의 짝이므로 ( 혹은 ) 하나의 문자만이 들어온 경우는 무조건 false를 반환하므로 애초에 이를 판단하여 효율성을 통과할 수 있었다.
수정하다보니 문자열을 순회하는 과정에서 charAt을 통해 순회하지 않고, for of 구문을 사용하였고, 삼항 연산자를 사용하여 코드의 길이를 많이 줄일 수 있었다.
s의 길이가 1인 경우를 판별하지 않아도 효율성을 통과하지만,
1인 경우를 판별하는 경우가 4.69ms로 조금 더 빠른 것을 알 수 있다!
✍️ 수정된 풀이
function solution(s){
let stackCnt = 0;
for(let bracket of s){
stackCnt += bracket === '(' ? 1 : -1;
if(stackCnt === -1) return false;
}
return stackCnt === 0;
}같은 코드인데 변수명 바꾸면 효율성이 통과가 안된다거나 줄바꿈으로 안된다거나.. 돌아갔던 코드가 다시 돌리니까 통과가 안되거나 하는 오류가 있다.
심지어
function solution(s){
if(s.length === 1) return false;
let stackCnt = 0;
for(let bracket of s){
stackCnt += bracket === '(' ? 1 : -1;
if(stackCnt === -1) return false;
}
return stackCnt === 0;
}if(s.length === 1) return false;
요거 하나만 추가했을 뿐인데 안됨
문제가 약간 이상하다는 것도 알 수 있다 ~~^^
