[프로그래머스] lv1. 문자열 내 p와 y의 개수

lv1. 문자열 내 p와 y의 개수

문제

대문자와 소문자가 섞여있는 문자열 s가 주어집니다. s에 'p'의 개수와 'y'의 개수를 비교해 같으면 True, 다르면 False를 return 하는 solution를 완성하세요. 'p', 'y' 모두 하나도 없는 경우는 항상 True를 리턴합니다. 단, 개수를 비교할 때 대문자와 소문자는 구별하지 않습니다.

예를 들어 s가 "pPoooyY"면 true를 return하고 "Pyy"라면 false를 return합니다.

제한사항
문자열 s의 길이 : 50 이하의 자연수
문자열 s는 알파벳으로만 이루어져 있습니다.

 

기존 코드

function solution(s){
    // p와 y를 찾는 정규표현식
    let regexrP = /p/ig;
    let regexrY = /y/ig;
    
    // s에 포함된 p, y의 개수
    let countP = s.match(regexrP).length;
    let countY = s.match(regexrY).length;
    
    return countP === countY || countP + countY < 1 ? true : false;
}

코드 실행은 통과, 제출 후 채점하기에서는 런타임 오류 발생하였다.

 

원인 : TypeError: Cannot read property 'length' of null

➡️ 들어온 인자가 "p"나 "y"를 포함하지 않는 경우 'match()' 메서드는 'null'을 반환한다. 'null'의 length를 참조하려고하면 에러가 발생한다.

수정 코드

'match()' 메서드 수행 전에 'null' 체크를 수행하는 로직을 추가해주었다.

function solution(s){
    // p와 y를 찾는 정규표현식
    let regexrP = /p/ig;
    let regexrY = /y/ig;
    
    // s에 포함된 p, y의 개수, 'match()' 메서드 'null' 체크 추가
    let countP = s.match(regexrP) ? s.match(regexrP).length : 0;
    let countY = s.match(regexrY) ? s.match(regexrY).length : 0;
    
    return countP === countY || countP + countY < 1 ? true : false;
}