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

6주차 세미나 심화과제 #13

Open
wants to merge 4 commits into
base: develop
Choose a base branch
from

Conversation

Eonji-sw
Copy link
Member

@Eonji-sw Eonji-sw commented Dec 29, 2023

📌𝘐𝘴𝘴𝘶𝘦𝘴

  • 코루틴 suspend 사용
  • BindingAdapter 사용

📎𝘞𝘰𝘳𝘬 𝘋𝘦𝘴𝘤𝘳𝘪𝘱𝘵𝘪𝘰𝘯

  • 코루틴의 경우 필수과제에 구현했었음
  • BindingAdapter의 경우 user image 불러오는 것에서 구현
  • 양방향 데이터 바인딩의 경우 TextInputLayout을 사용해서 아직 구현 안함

📷𝘚𝘤𝘳𝘦𝘦𝘯𝘴𝘩𝘰𝘵

💬𝘛𝘰 𝘙𝘦𝘷𝘪𝘦𝘸𝘦𝘳𝘴

@Eonji-sw Eonji-sw added the deepening 세미나 심화 과제 label Dec 29, 2023
Copy link

@sohyun127 sohyun127 left a comment

Choose a reason for hiding this comment

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

요거 한번 다시 수정해보면 좋을듯!! 빠이팅!!!

Glide.with(context)
.load(user.avatar).into(binding.ivItemImage)

loadImage(binding.ivItemImage, user.avatar)

Choose a reason for hiding this comment

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

옹?!!! 사실 이러면 DataBinding을 적용한게 아니라 loadImage 함수를 불러서 imageView에 img를 넣어준거라서, 한번 다시 수정해보면 좋을 것 같아요!!
BindnigAdapter는 잘 만들었어유!! 대신, 따로 BindingAdapter class 파일을 만들어서 관리하는 것이 가독성, 유지보수 측면에서 좋아보입니다!


//기존 뷰들에 없는 새로운 xml 속성을 연결하는 기능 메소드를 가지는 객체
//보통 static 메소드를 가진 class로 사용
//static 메소드를 가져야하기 때문에 class면 안됨 object로 명시

Choose a reason for hiding this comment

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

주석으로 보이는 공부 열정,,, 멋진 먼지 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
deepening 세미나 심화 과제
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants