한주헌) Lv.7 구현 완료 (26.01.29 21:13 수정)#5
Open
TyrHanz wants to merge 15 commits into
Open
Conversation
ybin4548
reviewed
Jan 29, 2026
Comment on lines
+321
to
+338
| switch series { | ||
| case 1: | ||
| imageInfoImage.image = .harrypotter1 | ||
| case 2: | ||
| imageInfoImage.image = .harrypotter2 | ||
| case 3: | ||
| imageInfoImage.image = .harrypotter3 | ||
| case 4: | ||
| imageInfoImage.image = .harrypotter4 | ||
| case 5: | ||
| imageInfoImage.image = .harrypotter5 | ||
| case 6: | ||
| imageInfoImage.image = .harrypotter6 | ||
| case 7: | ||
| imageInfoImage.image = .harrypotter7 | ||
| default: | ||
| imageInfoImage.image = .none | ||
| } |
Collaborator
There was a problem hiding this comment.
switch문 대신 매개변수로 받는 series를 사용해보시는건 어떨까요 ?
imageInfoImage.image = UIImage(named:)
Collaborator
Author
There was a problem hiding this comment.
말씀해주신대로 바꿔보니 한줄로 코드가 끝나네요!
거기 다 책이 많아지면 case 도 추가로 작성해야하니 그것도 빡세네요 ㅎ
Comment on lines
+220
to
+225
| buttonBookCount.snp.makeConstraints{ | ||
| $0.top.equalTo(labelHeader.snp.bottom).offset(16) | ||
| $0.centerX.equalToSuperview() | ||
| $0.leading.greaterThanOrEqualToSuperview().offset(20) | ||
| $0.trailing.lessThanOrEqualToSuperview().inset(20) | ||
| } |
Collaborator
There was a problem hiding this comment.
버튼의 가로 세로 길이를 맞춰주는 속성이 추가되면 원이 더 예쁘게 보일 것 같습니다.
Collaborator
Author
There was a problem hiding this comment.
가로 세로 길이 픽스하겠습니다! ㅎㅎ
ybin4548
reviewed
Jan 29, 2026
Comment on lines
+149
to
+158
| let stackDedication = UIStackView() | ||
| stackDedication.axis = .vertical | ||
| stackDedication.alignment = .top | ||
| stackDedication.spacing = 8 | ||
|
|
||
| // 개요 스텍뷰 | ||
| let stackSummary = UIStackView() | ||
| stackSummary.axis = .vertical | ||
| stackSummary.alignment = .top | ||
| stackSummary.spacing = 8 |
Collaborator
There was a problem hiding this comment.
속성 정의가 같다면 묶어서 한번에 처리해보는 방법을 적용해보면 좋을 것 같습니다.
[].forEach
Collaborator
Author
There was a problem hiding this comment.
간이 배열로 담아서 쓸수있다는 걸 처음알았네요.. 감사합니다!
|
네이밍 조금만 고쳐봅시다. [파일명]
[파일명 및 클래스명]
|
Comment on lines
+297
to
+304
| imageInfoImage.image = UIImage(named: "harrypotter\(series)") | ||
| labelHeader.text = bookData[series-1].title | ||
| labelInfoHeader.text = bookData[series-1].title | ||
| labelInfoAuthor.text = bookData[series-1].author | ||
| labelInfoRelesed.text = convertDateText(bookData[series-1].release_date) | ||
| labelInfoPages.text = "\(bookData[series-1].pages)" | ||
| labelInfoDedication.text = bookData[series-1].dedication | ||
| viewInfoSummry.setLabelText(bookData[series-1].summary) |
There was a problem hiding this comment.
series라는 offset을 전달받지 말고 Book 타입을 받아서 쓰면 더 좋을 것 같습니다.
series - 1이라는 연산이 반복되어서 들어가고, -1 연산을 안넣는 실수하기에도 딱 좋은!
Collaborator
Author
There was a problem hiding this comment.
감사합니다! Book타입으로 전달받아서 출력하는 형식으로 바꿔보겠습니다!
TyrHanz
changed the title
- 메인 뷰 컨트롤러 기능 분할 (ViewControlller -> MainView - MainViewController)
- 코드컨밴션에 맞게 코드 정리 진행
devBambu
approved these changes
Feb 2, 2026
Collaborator
There was a problem hiding this comment.
loadBooks에서 비동기 작업이 일어나는 게 아니니 클로저 없이도 사용가능할 것으로 생각됩니다..!
클로저가 없으면 사용부에서 dataService.loadBooks를 호출하면 결과로 (Result<[Book], Error>)가 바로 나올 것 같은데, 클로저를 사용하지 않아도 함수 내부에서 바로 처리할 수 있지 않을까요?
func loadBooks() {
let result = dataService.loadBooks
switch result {
case .success(let books):
case .failure(let error):
}
}- MVC 패턴에 맞게 뷰에 선언된 데이터들 뷰컨트롤러로 이관 및 기능 수정 - 클로져를 통한 버튼 이벤트 추가
…racker into Han_Develop
- 개요 및 메인 버튼 이벤트 관련 클로저 설정코드에서 강한 참조가 발생하는 이슈 해결
- 전반적인 코드컨벤션이 지켜지지 않는 코드 수정 - 앱 맨 처음 실행 시 개요 버튼이 더보기로 설정되는 이슈 수정 - 모델에 선언된 값들 중 수정되지 않는 파라미터들 상수로 변경 - 스크롤뷰 하단 잘리는 이슈 및 스크롤 위치 이슈 수정 - ViewController에서 View 할당 시, ViewDidLoad가 아닌 loadView에서 진행하게 변경
- 버튼명 변경
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
디자인 코드를 직접 짜는게 아직 어색해서 가시성이 많이 안좋습니다..
구현 중 하나 이슈사항이 있습니다.
책 상세뷰어를 스택뷰로 구현해야했는데 문제를 자세히 분석하지않아 UIView에 디자인을 배치하여 구현 하였습니다.
추후 해당 부분은 수정하여 구현하겠습니다.