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

Feat/week06-xml-advanced #32

Open
wants to merge 9 commits into
base: develop-xml
Choose a base branch
from
Open

Conversation

0se0
Copy link
Member

@0se0 0se0 commented Jun 7, 2024

작업 내용

구현 화면

이전과 동일합니다.

Comment on lines 23 to 26
_binding = FragmentFollowerBinding.inflate(inflater, container, false)
return binding.root
return _binding?.root
}

Copy link

Choose a reason for hiding this comment

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

왜 바꾸신 것일까여?

Comment on lines 47 to 49
super.onDestroyView()
_binding?.rvFollower?.adapter = null
_binding = null
Copy link

Choose a reason for hiding this comment

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

followerAdapter 를 메모리 해제처리 해줘야합니다
즉 lateinit var로 선언하면 불가능해지겠죠?

Comment on lines +24 to +25
return _binding?.root
}
Copy link

Choose a reason for hiding this comment

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

Suggested change
return _binding?.root
}
return binding.root
}

Comment on lines 46 to 48
super.onDestroyView()
_binding?.rvRepo?.adapter = null
_binding = null
Copy link

Choose a reason for hiding this comment

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

여기도 _binding?.rvRepo?.adapter = null는 의미 없습니당

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants