[feature/calendar2] 커스텀뷰 캘린더 #7

SeojinSeojin · 2021-10-24T05:55:46Z

PR 개요

스샷

커스텀뷰를 쓴 이유

뎁쓰가 얕아졌다는 장점!! 로딩도 빨라졌다는 장점!!
- 원래는 캘린더액티비티(캘린더리사이클러뷰(캘린더알갱이리사이클러뷰)) 3겹이었지만
  - 이거 숨안쉬고 발음 가능하면 큰 칭찬 해드림
- 지금은 캘린더액티비티(캘린더뷰) 이렇게 2겹이 됨!ㅎㅎ
또한 캔버스로 그리면 렌더링을 안드가 하는게 아니구 skia라는 구글에서 만든 엔진이 대신 해준다고 주워들었음!
- 웹 캔버스도 직접 렌더링하는게 아니라 더 빠르다..라는 걸 주워들었셈

궁금한 점

CalendarItemView.kt에서
클래스 컨스트럭터에서 받아오는 변수들은 그 클래스 안에서 다 접근 가능한건데
내가 그 변수들도 클래스 내부 함수에서 파라미터로 넘겨줬는데
굳이 넘겨줄 필요가 없지 않나?라는 생각이 드네..
어떻게 해야할까?

변수명들 구리지 않아?
너무 길거나 짧다면 가감없이 말씀해주십쇼..
(xml에 있는 id들은 어짜피 바꿀거라 대충 지었셈!)

후세에 남길 말

짱치미 짱드로이드 여러분.. 이 레포에서 작업하려면 작업 쫌 해줘야 한다..
서진아 .. height, font size는 대강 뒀지만, 우리 꺼에다가 심을 때는 유틸에 있는 함수 써서 DP로 써주어야 한단다..
레이아웃 당연히 싹 갈아엎어야 하구..
캐츄 드로어블 가져오는 로직도 다시 심어야 한단다..
그리고 프라그먼트 띄우는 클릭리스너까지 뷰모델로 해보면 좋겠구나..
- 굳이 캘린더 알갱이 하나하나에 리스너를 달지 말고,
- 뷰모델에서 셀렉티드데이트쓰캐츄리스트라는 라이브데이터를 만들어서 거기다가 캐츄 배열만 넣어주어도 괜찮을 것 같구나..

참고자료

https://leveloper.tistory.com/174

* 버전 많이 바꿔두었습니다..

* 눌렀을 때 프라그먼트 뿅 튀어나오는 부분, * height랑 폰트사이즈 조정 은 우리 찐 레포에서 해야할듯!

l2hyunwoo

굳!! 나도 커스텀뷰 잘 몰라서 이번 기회에 코드 보게된 것 같네용! 퍼포먼스 문제만 없으면 주어진 코드 기반으로 만들 수 있을 것 같습니다.

l2hyunwoo · 2021-10-24T06:02:34Z

app/build.gradle

+    annotationProcessor 'com.github.bumptech.glide:compiler:4.12.0'
+    implementation "org.jetbrains.kotlinx:kotlinx-coroutines-core:1.4.3"
+    implementation "org.jetbrains.kotlinx:kotlinx-coroutines-android:1.5.0"
+    implementation 'androidx.lifecycle:lifecycle-runtime-ktx:2.4.0-rc01'


이거 벌써 rc 나왔구나 코루틴 사용하기 더 편해지겠누

l2hyunwoo · 2021-10-28T15:02:28Z

app/build.gradle

+        sourceCompatibility(JavaVersion.VERSION_1_8)
+        targetCompatibility(JavaVersion.VERSION_1_8)


Suggested change

sourceCompatibility(JavaVersion.VERSION_1_8)

targetCompatibility(JavaVersion.VERSION_1_8)

sourceCompatibility(JavaVersion.VERSION_11)

targetCompatibility(JavaVersion.VERSION_11)

버전 11로 올리기!

왁 큰일날뻔!! 감사형광등~

l2hyunwoo · 2021-10-28T15:02:44Z

app/build.gradle

    }
    kotlinOptions {
-        jvmTarget = '1.8'
+        jvmTarget = JavaVersion.VERSION_1_8


Suggested change

jvmTarget = JavaVersion.VERSION_1_8

jvmTarget = JavaVersion.VERSION_11.toString()

l2hyunwoo · 2021-10-28T15:04:50Z

app/src/main/java/com/teamcatchme/calendar_customview/CalendarItemView.kt

+class CalendarItemView @JvmOverloads
+constructor(
+    context: Context,
+    attrs: AttributeSet? = null,
+    defStyleAttr: Int = 0,
+    private val date: Int? = null,
+    private val catchuList: Array<Int> = arrayOf(),
+    private val isPrevious: Boolean = false
+) :


Suggested change

class CalendarItemView @JvmOverloads

constructor(

context: Context,

attrs: AttributeSet? = null,

defStyleAttr: Int = 0,

private val date: Int? = null,

private val catchuList: Array<Int> = arrayOf(),

private val isPrevious: Boolean = false

) :

class CalendarItemView @JvmOverloads constructor(

context: Context,

attrs: AttributeSet? = null,

defStyleAttr: Int = 0,

private val date: Int? = null,

private val catchuList: Array<Int> = arrayOf(),

private val isPrevious: Boolean = false

) : View(context, attrs, defStyleAttr) {

우선 생성자 형식은 저런데 뷰의 생성자에 저런 인자들이 들어가도 되나 나도 자세히 몰라서 확인해봐야할 듯

오키 일단 해보고 우리 리얼 레포에 PR날리기 전까지 나도 확인해보겠음!!
안 돌아가진 않는데 위험할 수는 있을 것 같아

l2hyunwoo · 2021-10-28T15:06:01Z

app/src/main/java/com/teamcatchme/calendar_customview/CalendarItemView.kt

+    private var paint: Paint = Paint()
+
+    private fun drawDateRect(canvas: Canvas, date: Int) {
+        paint.textSize = 36f


실제에서는 이건 dp로 맞춰줘야할 것 같습니다.

l2hyunwoo · 2021-10-28T15:08:30Z

app/src/main/java/com/teamcatchme/calendar_customview/CalendarItemView.kt

+        if (date != null) {
+            if (catchuList.isNotEmpty()) drawDateWithCatchuRect(canvas, date, catchuList)
+            else drawDateRect(canvas, date)
+        }


Suggested change

if (date != null) {

if (catchuList.isNotEmpty()) drawDateWithCatchuRect(canvas, date, catchuList)

else drawDateRect(canvas, date)

}

date?.run {

if (catchuList.isNotEmpty()) drawDateWithCatchuRect(canvas, it, catchuList)

else drawDateRect(canvas, it)

}

이런식으로 밀고갈 수 있을 것 같아

l2hyunwoo · 2021-10-28T15:08:44Z

app/src/main/java/com/teamcatchme/calendar_customview/CalendarView.kt

+        val firstDayOfMonth = dateCalendar.get(Calendar.DAY_OF_WEEK) - 1
+        val lastDateOfMonth = dateCalendar.getActualMaximum(Calendar.DATE)


변수명 굳

l2hyunwoo · 2021-10-28T15:11:40Z

app/src/main/java/com/teamcatchme/calendar_customview/CalendarView.kt

+            for (blank in (lastDateOfLastMonth - firstDayOfMonth)..lastDateOfLastMonth) {
+                addView(CalendarItemView(context = context, date = blank, isPrevious = true))
+            }


Suggested change

for (blank in (lastDateOfLastMonth - firstDayOfMonth)..lastDateOfLastMonth) {

addView(CalendarItemView(context = context, date = blank, isPrevious = true))

}

((lastDateOfLastMonth - firstDayOfMonth)..lastDateOfLastMonth).forEach { blank ->

addView(CalendarItemView(context = context, date = blank, isPrevious = true))

}

l2hyunwoo · 2021-10-28T15:15:57Z

app/src/main/java/com/teamcatchme/calendar_customview/CustomCalendarFragment.kt

+import com.teamcatchme.catchmesample.databinding.FragmentCustomCalendarBinding
+import java.util.*
+
+class CustomCalendarFragment(private val millsId: Long) : Fragment() {


프래그먼트 생성자는 무조건 빈 걸로 둬야한다고 하네요! 알아두시면 좋을 것 같습니다

그래서 정적 패토리 패턴을 쓰라고 했던거 ㅇㅇ

l2hyunwoo · 2021-10-28T15:18:27Z

app/src/main/java/com/teamcatchme/calendar_customview/utils.kt

+fun drawableToBitmap(drawable: Drawable): Bitmap? {
+    if (drawable is BitmapDrawable) {
+        return drawable.bitmap
+    }
+    val bitmap =
+        Bitmap.createBitmap(
+            drawable.intrinsicWidth,
+            drawable.intrinsicHeight,
+            Bitmap.Config.ARGB_8888
+        )
+    val canvas = Canvas(bitmap)
+    drawable.setBounds(0, 0, canvas.getWidth(), canvas.getHeight())
+    drawable.draw(canvas)
+    return bitmap
+}


Suggested change

fun drawableToBitmap(drawable: Drawable): Bitmap? {

if (drawable is BitmapDrawable) {

return drawable.bitmap

}

val bitmap =

Bitmap.createBitmap(

drawable.intrinsicWidth,

drawable.intrinsicHeight,

Bitmap.Config.ARGB_8888

)

val canvas = Canvas(bitmap)

drawable.setBounds(0, 0, canvas.getWidth(), canvas.getHeight())

drawable.draw(canvas)

return bitmap

}

fun Drawable.ToBitmap(): Bitmap? {

if (this is BitmapDrawable) {

return bitmap

}

val bitmap =

Bitmap.createBitmap(

intrinsicWidth,

intrinsicHeight,

Bitmap.Config.ARGB_8888

)

val canvas = Canvas(bitmap)

setBounds(0, 0, canvas.getWidth(), canvas.getHeight())

draw(canvas)

return bitmap

}

이런식으로도 변형 가능(이건 그냥 내 스타일이어서 취사 선택 해주시길 바람!)

- for -> forEach - null처리 바꾸기

- Drawable 확장함수로 바꾸기

* 어딜 프라그먼트한테 인자를 넘겨! 팩토리 패턴 써보기

l2hyunwoo

LGTM 👍🏻

l2hyunwoo · 2021-10-30T07:25:53Z

app/src/main/java/com/teamcatchme/calendar_customview/CustomCalendarFragment.kt

+            val args = Bundle()
+            args.putLong(MILLS_ID, millsId)
+            fragment.arguments = args


Suggested change

val args = Bundle()

args.putLong(MILLS_ID, millsId)

fragment.arguments = args

fragment.arguments = Bundle().also { putLong(MILLS_ID, millsId) }

3줄을 1줄로

l2hyunwoo · 2021-10-30T07:26:07Z

app/src/main/java/com/teamcatchme/calendar_customview/CustomCalendarFragment.kt

+        arguments?.let {
+            millsId = it.getLong(MILLS_ID)
+        }


SeojinSeojin added 4 commits October 18, 2021 02:49

[feature/calendar2] 바뀌어버린 안스를 위한 초기 세팅

d7a2254

* 버전 많이 바꿔두었습니다..

[feature/calendar2] 무한 뷰페이저 + 페이지 별 시간 계산하는 로직 구현

0f90dc1

[feature/calendar2] 커스텀뷰 캘린더 만들기

a207ee3

* 눌렀을 때 프라그먼트 뿅 튀어나오는 부분, * height랑 폰트사이즈 조정 은 우리 찐 레포에서 해야할듯!

[feat/calendar2] 월요일부터 시작하는 날에 한 줄 더 생기는 오류 해결

0460e15

SeojinSeojin requested a review from l2hyunwoo October 24, 2021 05:55

SeojinSeojin self-assigned this Oct 24, 2021

l2hyunwoo approved these changes Oct 28, 2021

View reviewed changes

SeojinSeojin added 4 commits October 29, 2021 01:30

[feature/calendar2] 버전 바꾸기

b8196fa

[feature/calendar2] 피드백 반영 (1/3)

537e18f

- for -> forEach - null처리 바꾸기

[feature/calendar2] 리뷰반영 (2/3)

0a93ca8

- Drawable 확장함수로 바꾸기

[feature/calendar2] 코드릐뷰 반영 (3/3)

66a42f1

* 어딜 프라그먼트한테 인자를 넘겨! 팩토리 패턴 써보기

l2hyunwoo approved these changes Oct 30, 2021

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[feature/calendar2] 커스텀뷰 캘린더 #7

[feature/calendar2] 커스텀뷰 캘린더 #7

SeojinSeojin commented Oct 24, 2021

l2hyunwoo left a comment

l2hyunwoo Oct 24, 2021

l2hyunwoo Oct 28, 2021

SeojinSeojin Oct 28, 2021

l2hyunwoo Oct 28, 2021

l2hyunwoo Oct 28, 2021

SeojinSeojin Oct 28, 2021

l2hyunwoo Oct 28, 2021

l2hyunwoo Oct 28, 2021

l2hyunwoo Oct 28, 2021

l2hyunwoo Oct 28, 2021

l2hyunwoo Oct 28, 2021

l2hyunwoo Oct 28, 2021

l2hyunwoo left a comment

l2hyunwoo Oct 30, 2021

l2hyunwoo Oct 30, 2021

		sourceCompatibility(JavaVersion.VERSION_1_8)
		targetCompatibility(JavaVersion.VERSION_1_8)

	jvmTarget = JavaVersion.VERSION_1_8
	jvmTarget = JavaVersion.VERSION_11.toString()

		val firstDayOfMonth = dateCalendar.get(Calendar.DAY_OF_WEEK) - 1
		val lastDateOfMonth = dateCalendar.getActualMaximum(Calendar.DATE)

[feature/calendar2] 커스텀뷰 캘린더 #7

Are you sure you want to change the base?

[feature/calendar2] 커스텀뷰 캘린더 #7

Conversation

SeojinSeojin commented Oct 24, 2021

PR 개요

궁금한 점

후세에 남길 말

참고자료

l2hyunwoo left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

l2hyunwoo left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment