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

[Refactor] feature-1 에 대한 리뷰사항 반영 #30

Merged
merged 2 commits into from
Apr 8, 2022

Conversation

Jinsujin
Copy link
Collaborator

@Jinsujin Jinsujin commented Apr 8, 2022

💡 Issue

관련 Issue

📝 Tasks

🔹 작업

  • 외부에서 사용하지 않는 변수를 private 접근제어자로 변경
  • Cell Identifier 에 대한 하드코딩을 Cell 내부같은 한곳에서 관리하기

🔹 고민과 해결

  • TableView Cell 에서 사용된 하드코딩된 값을 한곳에서 관리할 수 있도록 모았습니다
  • 클래스 이름을 String(describing:) 으로 가져올 수 있다는 것을 알았습니다

@Jinsujin Jinsujin self-assigned this Apr 8, 2022
@SangHwi-Back SangHwi-Back merged commit b4a5e68 into SangHwi-Back:develop-iOS Apr 8, 2022
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