Open csi111 opened 5 years ago
패키지명의 대문자
네이밍 통일
네이밍 혼란
미사용 OnPostClickListener
Databinding 활용
holder.binding.tvPostTitle.setText(postList.get(i).getTitle()); holder.binding.tvPostContent.setText(postList.get(i).getContent()); holder.binding.tvPostLikeCount.setText(postList.get(i).getLikeString());
ItemClick의 isCheck의 목적 불분명 public interface OnItemClickListener { void onItemClick(View v,int position,boolean isCheck); }
Util의 생성자 Util의 생성자는 private로 명시해주세요
과한 싱글턴 패턴 MinPriceAPI의 인스턴스가 싱글턴으로 이루어져야 할 이유?
그외 Unused 코드 import된 패키지나 코드상에 사용하지 않는 코드들은 그때그때마다 지워주시는게 좋을거라 생각이 됩니다.
확인했습니다! 수정하겠습니다~!
패키지명의 대문자
네이밍 통일
네이밍 혼란
미사용 OnPostClickListener
Databinding 활용
ItemClick의 isCheck의 목적 불분명 public interface OnItemClickListener { void onItemClick(View v,int position,boolean isCheck); }
Util의 생성자 Util의 생성자는 private로 명시해주세요
과한 싱글턴 패턴 MinPriceAPI의 인스턴스가 싱글턴으로 이루어져야 할 이유?
그외 Unused 코드 import된 패키지나 코드상에 사용하지 않는 코드들은 그때그때마다 지워주시는게 좋을거라 생각이 됩니다.