배열에서 특정 조건을 만족하면서 중복된 값을 찾기

어떤 배열 안에서 d로 나눠지는 값 중 중복된 값이 있다면 그 값의 갯수를 반환하는 함수를 코딩해보자.

내가 짠 코드의 문제점

1. 변수 선언을 하지 않고 사용했다.

이렇게 되면 전역 변수로 선언되어 의도치 않은 로직을 유발할 수 있으니 항상 주의하자.
=> let이나 const로 변수 선언해주기!

2. return answer = [...new Set(dupArr)].length;에서 굳이 answer 변수를 사용할 필요가 없다.

=> 반환할 값만 반환하면 되므로 return [...new Set(dupArr)].length;이라 하기.

3. 중복된 요소를 찾기 위한 방법의 효율성을 고려하자. 굳이 한쪽의 결과(true)만 필요한 삼항연산자를 사용하기는 것도 불필요하다.

=> filter 메소드를 활용해 더 효율적인 코드를 짜보자.

filter를 사용하여 원하는 조건을 만족하는 item들을 배열에 걸러 담고 Set을 사용해 중복값을 제거한 후 배열에 담아 배열의 길이를 구했다. 이전보다 훨씬 더 간결하고 효율적인 코드가 되었다.