보통 홀더를 사용하는 어댑터가 이를 독점적으로 사용하기 때문에 (스코프를 제약하는 의미에서) 내부 클래스 형태로 많이 사용하는게 아닐까 싶습니다. 그렇다고 홀더를 외부 클래스로 분리한다고 해서 잘못된 코드라고 말하기는 어렵습니다. 홀더가 다른 클래스에서도 재사용된다면 그 때는 분리하는 방식을 취하는 것도 좋은 선택이니까요. 사실 이 부분은 코드를 어떻게 디자인 할지의 문제에 가까워서 그때 그때의 상황에 맞는 방법을 선택하는 것이 좋을 것 같습니다.