Linq: 집합이 비어 있지 않은지 결정하기 위해 Count() > 0을 절대 사용하지 마세요

작은 쓰레기 같은 놈들 · 게시됨 2017. 10. 12. 오후 1:25:01

Linq 이전에는 보통 집합이 비어 있지 않은지, 즉 집합이 원소를 포함하는지 판단하기 위해 다음과 같은 방법을 사용했습니다:

Length 또는 Count 속성을 사용하면 위 글씨에 문제가 없습니다.

하지만 Linq 시대에는 Enumerable.Count 확장 메서드가 Length와 Count 속성을 "통합"하여 비무효성을 판단하는 다음과 같은 방식으로 작성되었습니다:

로그인이 보이네요.

이 글쓰기는 괜찮고 잘 작동하지만, 심각한 성능 문제를 일으킬 수 있습니다.

반드시 가능한 것은 아니며, 위의 메서드가 배열, 리스트, 컬렉션 형태로 전달되면 문제가 발생하지 <T>않는다는 점을 유의하세요<T>.

그렇다면 언제 문제가 생길까요? 다음 방법들을 살펴보겠습니다:

로그인이 보이네요.

호출 때:

로그인이 보이네요.

실행 속도는 꽤 느릴 것이고, 제 컴퓨터가 소스 코드를 실행하는 데 약 70초가 걸렸습니다. Count() > 0.

분석해 보면 GetNums 코드 5행의 yield return가 int를 실행한다는 것을 알 수 있습니다. 맥스밸류, 꼭 필요한가요?

사실, 한 원소를 반환하는 한 집합이 비어 있지 않으므로 모든 원소를 반환할 필요가 없다고 결론 내릴 수 있습니다.

그렇다면 어떻게 판단해야 할까요? Enumerable.any 확장 메서드를 사용할 수 있습니다:

SomeAction 메서드를 다음과 같이 수정하세요:

로그인이 보이네요.

다시 부르면 실행 시간이 거의 없다는 것을 알 수 있습니다.

요약하자면, Count() > 0은 예일 리턴에 직면할 때 필연적으로 성능 문제를 겪게 됩니다.

[출처] Linq: 집합이 비어 있지 않은지 결정하기 위해 Count() > 0을 절대 사용하지 마세요

관련 게시물

본 섹션