Code Review Practice - gpeegpee/learn-java GitHub Wiki

  • @Test를 지우고 @Ignore 붙이는 경우

  • ArrayList생성후 List로 assign하지 않고 구현타입으로 참조하는 경우

  • DBhelper static method에서 thread-safe 고려하지 않은 경우

  • constant int로 정의하는 것보다 @intdef / enum 사용 고려

  • magic number는 constant로 정의해서 쓰자

  • Test로 scope을 확장하는 경우 VisibleForTesting 사용

  • Lint suppresss 보다는 RequiresApi 사용가능한 방향으로 변경

  • Kotlin에서는 isNullOrEmpty()로 Null/Empty 둘다 가능

  • Data Binding으로 findViewById 대체