안드로이드 프로젝트의 리뷰어들의 체크리스트

Memory Leaks

- context를 유지하는 코드 조각이 있는지?
- RxAndroid Code랑 연관이 있따면 RxCall이 생명주기의 끝에 없어지는지?
- coroutine을 사용하고 있다면, Jab이 런취되고 viewModel 스코프에 의해 적절히 해방이 되는지?
- 만일 CountDownTimer나 AsyncTask를 비디오/오디오 플레이어에서 사용하고 있다면, momory 리소스를 적절히 해방시키는지?
- ViewBinding을 사용하고 있다면 onDestroyView에서 null로 셋팅시키는지

Resource Annotations

fun showMessage(context: Context, val idRes: Int) {
  Toast.makeText(context, idRes, Toast.SHORT_LENGHT).show()
}

이렇게 하는것보단

fun showMessage(context: Context, @StringRes val idRes: Int) {
  Toast.makeText(context, idRes, Toast.SHORT_LENGHT).show()
}

이렇게 @StringRes를 넣어주는게 좋다.

 

Small details make the difference

- Not used imports;
- Not used resources such as drawables, strings, colors ..
- Commented Code;
- Not formatted code;
- Varaiable names, method names, file names, ...
- Code not following a style guid

https://betterprogramming.pub/code-review-on-android-projects-b7aae196cfda