Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Lee hyungjun/step.01 #5

Open
wants to merge 3 commits into
base: Step.01-Solution-Databinding-Start_use_local_value
Choose a base branch
from

Conversation

LEE-HYUNGJUN
Copy link

step01 완료

jinsu4755 and others added 3 commits December 31, 2020 14:03
그냥 어떻게 쓰는지 강의하려고 넣음
step01의 todo를 모두 처리했습니다.

Related to #3
@LEE-HYUNGJUN LEE-HYUNGJUN added the 형준 담당자: 형준 label Dec 31, 2020
@LEE-HYUNGJUN LEE-HYUNGJUN self-assigned this Dec 31, 2020
Copy link
Author

@LEE-HYUNGJUN LEE-HYUNGJUN left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

진수의 함수 매개변수와 제가 사용한 함수의 매개변수가 달랐습니당

val binding: ActivityMainBinding =
DataBindingUtil.setContentView(this, R.layout.activity_main)
initProfileImage(binding.imageProfileImage)
setProfileName(binding.textProfileName, DEFAULT_USER_NAME)
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

저같은 경우는 전역변수로 binding을 선언했고, 함수의 매개변수가 다른 것 말고는 진행 상황은 같았습니다

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

아주 좋아!
이건 내 개인적인 생각이었는데 binding value를 전역으로 선언할 필요가 있을까? 생각했었어.
Activity는 뷰에서 일어날 어떤 활동들? 그중에서도 생명주기와 연결이 강한 활동들이 일어나는 책임이라고 생각했기에
결국 binding value는 내가 뷰를 컨트롤 하면서 생기는 일이라 전역으로 쓰기보다는 범위를 줄여서 뷰에 관련된 일만 할 수 있도록 해야겠다 생각하고 각 뷰로 binding value를 쏴준거야

이 부분은 크게 정해진건 없으니 옳다고 생각하는 부분으로, 또는 Activity마다 유동적으로 적어주면 되는 부분이야~

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

무슨말인지 이해했어!! 사실 그렇게 까지 생각하지 않고, 전역변수로 쓰는 것이 더 편하다고만 생각했어서 먼저 전역변수로
선언해놓고 코드를 진행했던것 같아!! 참고할게~

Copy link
Member

@jinsu4755 jinsu4755 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

너무너무 고생했고 실전에서도 홧팅!

Comment on lines +35 to 37
dataBinding{
enabled = true
}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

이 부분은 아마 Gradle 빌드 돌리면 아마 워닝으로 뜰탠데 자세히 읽어보면

Suggested change
dataBinding{
enabled = true
}
buildFeatures{
dataBinding = true
}

다음과 같이 변경되었으니 적어달라고 뜰꺼야

val binding: ActivityMainBinding =
DataBindingUtil.setContentView(this, R.layout.activity_main)
initProfileImage(binding.imageProfileImage)
setProfileName(binding.textProfileName, DEFAULT_USER_NAME)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

아주 좋아!
이건 내 개인적인 생각이었는데 binding value를 전역으로 선언할 필요가 있을까? 생각했었어.
Activity는 뷰에서 일어날 어떤 활동들? 그중에서도 생명주기와 연결이 강한 활동들이 일어나는 책임이라고 생각했기에
결국 binding value는 내가 뷰를 컨트롤 하면서 생기는 일이라 전역으로 쓰기보다는 범위를 줄여서 뷰에 관련된 일만 할 수 있도록 해야겠다 생각하고 각 뷰로 binding value를 쏴준거야

이 부분은 크게 정해진건 없으니 옳다고 생각하는 부분으로, 또는 Activity마다 유동적으로 적어주면 되는 부분이야~

Comment on lines +6 to +10
<data>
<variable
name="main"
type="sotp.semina.practicegitanddatabinding.MainActivity" />
</data>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

이건 변수명에 관한건데 main이란 이름은 뭔가 개인적인 생각으로 많이 축약이다 생각이 들어!

바로 아래 타입이 적혀있어 큰 문제는 없지만 뷰가 복잡해지고 길어지고, variable이 많아지면 main이 엑티비티인지, 뷰모델인지, data class인지 판별하기 힘들 것 같다 생각했기 때문이야

@jinsu4755 jinsu4755 added the Good 과제 완료 label Jan 1, 2021
@jinsu4755 jinsu4755 mentioned this pull request Jan 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Good 과제 완료 형준 담당자: 형준
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants