Conversation
| protocol DetailDelegate: AnyObject { | ||
| func dataBind(nickname: String) // 다음화면에서 이전화면으로 Data | ||
| } | ||
| class DetailViewController_Closer: UIViewController { |
| private func setUpButton() { | ||
| versionInfoView.updateRecordButton.addTarget(self, action: #selector(updateRecordButtonTapped), for: .touchUpInside) | ||
|
|
||
| appScoreView.allReviewButton.addTarget(self, action: #selector(allReviewButtonTapped), for: .touchUpInside) |
Johyerin
left a comment
Johyerin
left a comment
There was a problem hiding this comment.
고생하셨씁니다!! 여러 뷰를 컴포넌트화 하여 처리하신 점 굳굳 남은 과제들도 화이팅🔥🔥
There was a problem hiding this comment.
.DS_Store 파일은 당장은 문제가 되지 않더라도 추후에 깃에서 충돌을 유발하게 될 수도 있기 때문에 올라가지 않게 주의해주시는 게 좋습니다!
| scrollView.snp.makeConstraints { | ||
| $0.top.equalToSuperview() | ||
| $0.bottom.equalToSuperview() | ||
| $0.leading.equalToSuperview() | ||
| $0.trailing.equalToSuperview() | ||
| } |
There was a problem hiding this comment.
$0.edges.equalToSuperview()로 적어줄 수 있을 것 같아용
| } | ||
| contentView.snp.makeConstraints { | ||
| $0.edges.equalToSuperview() | ||
| $0.width.equalTo(scrollView.snp.width) |
There was a problem hiding this comment.
contentView는 scrollView에 속하기 때문에 equalToSuperview()로 적어줘도 좋을 것 같습니닷~!
| } | ||
| titleView.snp.makeConstraints { | ||
| $0.height.equalTo(150) | ||
| $0.leading.trailing.equalToSuperview() |
There was a problem hiding this comment.
$0.horizontalEdge.equalToSuperview() 로도 적어줄 수 있을 것 같아욥!
| import SnapKit | ||
| import UIKit | ||
| import Then |
There was a problem hiding this comment.
퍼스트 파티 프레임워크인 UIKit을 가장 먼저 임포트해주면 더 보기에 좋을 것 같다는 의견 남깁니당
| $0.top.equalToSuperview().offset(20) | ||
| $0.leading.equalToSuperview().offset(20) |
There was a problem hiding this comment.
superview를 기준으로 잡을 때에는 inset 사용을 추천드립니다!
| setLayout() | ||
| } | ||
|
|
||
| private func setUI() { |
| $0.delegate = self | ||
| $0.dataSource = self |
There was a problem hiding this comment.
delegate나 dataSource를 채택하는 부분은 따로 메소드화 시켜도 좋을 것 같아요 !
지금은 tableview 하나만 채택하면 되지만, 나중에 데이터 전달이 많아지게 될 경우에는 한 메소드에서 관리하는게 훨씬 편합니당
|
|
||
| private func setLayout() { | ||
| chartTableView.snp.makeConstraints { | ||
| $0.leading.trailing.top.bottom.equalToSuperview() |
There was a problem hiding this comment.
요고는
$0.edges.equalToSuperview()
로도 쓸 수 있습니다
|
|
||
| class ChartCell: UITableViewCell { | ||
|
|
||
| private lazy var iconImageView = UIImageView() |
| func tableView(_ tableView: UITableView, heightForHeaderInSection section: Int) -> CGFloat { | ||
| let sectionData = sectionList[section] | ||
| let headerHeight: CGFloat = sectionData.title.isEmpty ? 0 : | ||
| sectionData.subtitle.isEmpty ? 60 : 75 | ||
| return headerHeight | ||
| } |
sozohoy
left a comment
sozohoy
left a comment
There was a problem hiding this comment.
수고하셨습니다! 가현님이 이미 코드리뷰를 잘 남겨주셔서..
또한, 해당 과제와 상관없는 커밋이 같이 껴서 들어오는 이유에 대해서 알아보세요!
5 participants
구현 완료 항목
Simulator.Screen.Recording.-.iPhone.16.Pro.-.2024-11-01.at.20.51.23.mp4