리팩토링 인라인 함수처리 관련 질문입니다.

Question

해외 사이트에서, Android 리스트뷰의 아이템 toggle 기능을 어떤 분이 아래 처럼 구현해 놓은걸 보았는데요. 개인적으로 이해가 안되는 부분이 있어 설명해주실 수 있는분이 계신가 해서 글 올려봅니다. 아래 코드에서요selectView() 메서드 내부 로직을 toggleSelection() 메서드에 인라인 함수처리 해도 상관없는 것 아닌가요?굳이 왜 두개의 메서드로 나누어 놨을까요? 프로젝트도 예제 수준의 규모라서 개인적으로는 크게 확장성을 따질 필요가 없다고 생각이 들었습니다... 다른 분들의 의견도 한번 들어보고 싶습니다. // 리스트 아이템 토글 public void toggleSelection(int position) { selectView(position, !selectedItemIdList.get(position)); } // 리스트 아이템 선택 및 취소 public void selectView(int position, boolean value) { if (value) { selectedItemIdList.put(position, value); } else { selectedItemIdList.delete(position); } this.notifyDataSetChanged(); }

나도아빠다 · Accepted Answer

저도 자바스크립트 토글시 저렇게 쪼개는걸 선호합니다.toggle 할 수도 있지만 강제로 show를 해야할 경우, 강제로 hide해야할 경우도 있습니다.

리팩토링 인라인 함수처리 관련 질문입니다.

1개의 답변

#refactoring 목록