Android - ViewHolder Class는 Static과 Non-static 중에 무엇으로 정의하는 것이 좋을까?

이전 포스팅에서 우리는 ListView에서 새로운 아이템이 화면에 보여질 때 ConvertView라는 파라미터를 이용하여 성능을 개선해 보았고, 이보다 더 개선된 ConvertView에 ViewHolder pattern을 적용시켜 보았다. 이번 포스팅에서는 조금은 다른 주제로 ViewHolder를 다뤄보고자 한다. BaseAdapter를 상속한 CustomAdapter 내부에서 ViewHolder Class를 정의하려고 한다. 여기서 문제에 봉착한다. ViewHolder Class는 Static일까 Non-static일까? 정답이 있는걸까? 아래에서 알아보도록 하자.

 

 

1. Lint에서 아주 잘 설명해준다

 

• Reports any inner classes which may safely be made static. An inner class may be static if it doesn't reference its enclosing instance. A static inner class does not keep an implicit reference to its enclosing instance. This prevents a common cause of memory leaks and uses less memory per instance of the class.
• [번역] - static 으로 만들수 있는 내부 클래스를 발견했습니다. 내부 클래스가 자신의 바깥 클래스의 인스턴스를 참조하지 않는다면, 내부 클래스는 static으로 선언해도 됩니다. static 내부 클래스는 자신의 바깥 클래스 인스턴스의 임시적 참조를 유지하지 않습니다. 즉, static 내부 클래스로 선언하면 메모리 누수의 일반적인 원인을 예방할 수 있고, 클래스의 각 인스턴스당 더 적은 메모리를 사용하게 됩니다.

Non-static으로 Viewholder Class를 정의하였더니 Lint로 부터 해답을 얻게 되었다. 오류가 나지는 않는데 Static을 붙이는게 위와 같은 이유로 더 낫다고 한다. 그래서 Static 키워드를 붙여서 얼렁뚱땅 해결했다고 넘어가면 안된다. 책을 통해서 좀 더 확실하게 이해해보도록 하자.

 

 

2. 멤버 클래스는 되도록 static으로 만들라는데.. 확인해 보자

 

마침 집에 Effective Java라는 책이 있어서 책의 힘을 빌렸다. 책에서 다음과 같은 주제로 설명하고 있다. 인용글을 읽으면서 이해해보도록 하자.

 

Effective Java (조슈아 블로크 지음) - 멤버 클래스는 되도록 static으로 만들라 [p.146]

...
정적 멤버 클래스와 비정적 멤버 클래스의 구문상 차이는 단지 static이 붙어있고 없고 뿐이지만, 의미상 차이는 의외로 꽤 크다. 비정적 멤버 클래스의 인스턴스는 바깥 클래스의 인스턴스와 암묵적으로 연결된다. 그래서 비정적 멤버 클래스의 인스턴스 메서드에서 정규화된 this를 사용해 바깥 인스턴스의 메서드를 호출하거나 바깥 인스턴스의 참조를 가져올 수 있다. 정규화된 this란 클래스명.this 형태로 바깥 클래스의 이름을 명시하는 용법을 말한다.[JLS, 15.8.4] 따라서 개념상 중첩 클래스의 인스턴스가 바깥 인스턴스와 독립적으로 존재할 수 있다면 정적 멤버 클래스로 만들어야 한다. 비정적 멤버 클래스는 바깥 인스턴스 없이는 생성할 수 없기 때문이다.
비정적 멤버 클래스의 인스턴스와 바깥 인스턴스 사이의 관계는 멤버 클래스가 인스턴스화될 때 확립되며, 더 이상 변경할 수 없다. 이 관계는 바깥 클래스의 인스턴스 메서드에서 비정적 멤버 클래스의 생성자를 호출할 때 자동으로 만들어지는 게 보통이지만, 드물게는 직접 바깥 인스턴스의 클래스.new MemberClass(args)를 호출해 수동으로 만들기도 한다. 예상할 수 있듯, 이 관계 정보는 비정적 멤버 클래스의 인스턴스 안에 만들어져 메모리 공간을 차지하며, 생성 시간도 더 걸린다.
...
멤버 클래스에서 바깥 인스턴스에 접근할 일이 없다면 무조건 static을 붙여서 정적 멤버 클래스로 만들자. static을 생략하면 바깥 인스턴스로의 숨은 외부 참조를 갖게 된다. 앞서도 얘기했듯 이 참조를 저장하려면 시간과 공간이 소비된다. 더 심각한 문제는 가비지 컬렉션이 바깥 클래스의 인스턴스를 수거하지 못하는 메모리 누수가 생길 수 있다는 점이다(아이템 7). 참조가 눈에 보이지 않으니 문제의 원인을 찾기 어려워 때때로 심각한 상황을 초래하기도 한다.

 

위의 글을 요악해보면 다음과 같은 결론을 얻을 수 있다.

 

• 비정적 멤버 클래스의 인스턴스는 멤버 클래스가 인스턴스화 될 때 바깥 클래스의 인스턴스와 암묵적으로 연결된다. 바깥 인스턴스 없이는 멤버 클래스를 인스턴스화 할 수 없기 때문이다.
• 바깥 인스턴스에 접근할 일이 없으면 정적 멤버 클래스로 만들어야 한다. static을 생략하면 바깥 인스턴스로의 외부 참조를 갖게 되고,이 참조를 저장하기 위한 시간과 공간을 필요로 하게 된다. 즉, 메모리를 차지하고 생성시간이 느려지게 된다.

 

 

3. 다른 사람들의 생각은 어떨까?

 

• If you declare the viewholder as static you can reuse it in other adapters. Anyway, I do not recommend to do it, create a new separated class and use it from multiple places, it does make more sense. One class for one purpose. - Stackoverflow
• [번역] - ViewHolder를 정적으로 선언하면 다른 어댑터에서 재사용 할 수 있다. 하지만, 그것을 권장하지 않고 새롭게 분리 된 클래스를 만들고 여러 곳에서 사용하는 것이 더 합리적이라고 생각한다. 하나의 클래스는 하나의 목적을 위한 것이다.

-> 나랑 비슷한 의견을 가지고 있는 것 같다. Android의 List 형태의 View를 사용할 때 Adapter를 구현해주는데, 보통 Adapter 안에서 일회성 목적으로 정적 ViewHolder Class를 정의해 줄수도 있다. 하지만, Adapter 내에서 복잡한 레이아웃과 분기처리를 통해 다양한 ViewHolder를 필요로 한다면 정적 멤버 클래스 하나가 여러개의 ViewHolder를 가지는 것은 옳은 선택이 아닐 수도 있다. 어떻게 설계하느냐에 따라 달려있겠지만 하나의 목적은 다양한 곳에 재사용 될 가능성도 충분히 있다고 생각하고, 그렇게 설계할 수 있다면 내부에 멤버 클래스로 정의한 ViewHolder 클래스를 분리시켜 구성하는 것이 옳은 판단이라고 생각한다.