나누어 떨어지는 숫자 배열

문제

array의 각 element 중 divisor로 나누어 떨어지는 값을 오름차순으로 정렬한 배열을 반환하는 함수, solution을 작성해주세요.
divisor로 나누어 떨어지는 element가 하나도 없다면 배열에 -1을 담아 반환하세요.

제한사항

• arr은 자연수를 담은 배열입니다.
• 정수 i, j에 대해 i ≠ j 이면 arr[i] ≠ arr[j] 입니다.
• divisor는 자연수입니다.
• array는 길이 1 이상인 배열입니다.

입출력 예

arr	divisor	return
[5, 9, 7, 10]	5	[5, 10]
[2, 36, 1, 3]	1	[1, 2, 3, 36]
[3, 2, 6]	10	[-1]

정답

function solution(arr, divisor) {
    const answer = [];

    arr.map((e) => {
        if(e % divisor === 0) {
            answer.push(e);
        }
    })

    if (answer.length === 0) {
        return [-1];
    }
    return answer.sort((a, b) => (a - b));
}

풀이

나눠진 수를 담을 answer변수를 생성하고
map메소드를 이용해 요소를 divisor나눈 나머지가 0인지 확인하고 맞다면 answer변수에 push합니다.
map이 종료되고 나면 answer의 length를 체크하여 나눠진 수가 있는지 확인하고 [-1] 혹은
정렬한 answer를 반환합니다.

리팩토링

function solution(arr, divisor) {
    const answer = arr.filter(e => e % divisor === 0);
    return answer.length === 0 ? [-1] : answer.sort((a, b) => a - b);
}

복잡한 문제가 아니기 때문에 성능에서는 큰 차이가 없지만
map대신 filter메소드를 사용하고 삼항연산자를 통해 return하여 전체적으로 구문이 짧아졌고 가독성 또한 높였습니다.