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] #27 - 책리스트api연결 #28

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

Conversation

zzongha
Copy link
Contributor

@zzongha zzongha commented Dec 22, 2022

🔥Pull requests

⛳️ 작업한 브랜치

👷 작업한 내용

책리스트 api를 연결해보았습니다.

🚨참고 사항

미완성이지만,, 서버가 연결되긴 했어요..

📸 스크린샷

기능 스크린샷
GIF

📟 관련 이슈

  • Resolved: #이슈번호

@zzongha zzongha added 종화 🫶🏻 안뇽 나는 종화야 Feat 기능 추가 및 구현 labels Dec 22, 2022
@zzongha zzongha requested review from devxsby and 6uohul December 22, 2022 13:43
@zzongha zzongha self-assigned this Dec 22, 2022
Copy link
Contributor

@devxsby devxsby left a comment

Choose a reason for hiding this comment

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

고생하셨어용 대박~

@@ -0,0 +1,41 @@
//
// File.swift
Copy link
Contributor

Choose a reason for hiding this comment

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

주석 이름 체크해주세영~


import Foundation

// MARK: - Welcome
Copy link
Contributor

Choose a reason for hiding this comment

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

이것도..

//

import Foundation
import UIKit
Copy link
Contributor

Choose a reason for hiding this comment

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

요건 필요 없지않아여?

Comment on lines +226 to +233
case .success(let result):
let status = result.statusCode
if status >= 200 && status < 300 {
do {
self.bookListResponse = try result.map(BookListResponseDTO?.self)!
guard let pickArray = self.bookListResponse?.data.pick else { return }
guard let topicArray = self.bookListResponse?.data.topic else { return }
self.hotBookContainerView.topicViewArray = topicArray
Copy link
Contributor

Choose a reason for hiding this comment

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

오 곤듀~ 잘했다 ~ 👍🏻

Comment on lines +218 to +220
var pickArray: [BookListDataArray] = []
var topicArray: [BookListDataArray] = []
var bookListResponse: BookListResponseDTO?
Copy link
Contributor

Choose a reason for hiding this comment

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

맨 위에 선언해주면 더 잘보일거같아여~

Comment on lines +246 to +250
override func viewWillAppear(_ animated: Bool) {
super.viewWillAppear(animated)

bookListAPI()
}
Copy link
Contributor

Choose a reason for hiding this comment

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

life cycle 관련 코드는 위로 올려주세요..~

@@ -10,6 +10,8 @@ import SnapKit
import Then

class HotBookCVC: UICollectionViewCell {

static let identifier = "HotBookCVC"
Copy link
Contributor

Choose a reason for hiding this comment

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

여기 이거 선언하지말구 제가 util에 만들어놓은 classname으루 쓰면 될 것 같아여

Comment on lines +47 to +56
DispatchQueue.global().async { [weak self] in
let url = URL(string: model.cover)
if let data = try? Data(contentsOf: url!) {
if let image = UIImage(data: data) {
DispatchQueue.main.async {
self?.bookSampleImageView.image = image
}
}
}
}
Copy link
Contributor

Choose a reason for hiding this comment

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

대박. 이 코드 설명좀 해주세여 ~

Comment on lines +57 to +63
// let url = URL(string: model.cover)
// do {
// let data = try Data(contentsOf: url!)
// bookSampleImageView.image = UIImage(data: data)
// } catch (let error) {
// print("err")
// }
Copy link
Contributor

Choose a reason for hiding this comment

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

안쓰는 코드 지워주쎄용

Comment on lines +64 to +65
bookNameLabel.text = model.name
bookIntroLabel.text = model.pickDescription
Copy link
Contributor

Choose a reason for hiding this comment

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

야무지네요☃️

Copy link
Contributor

@6uohul 6uohul left a comment

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
Feat 기능 추가 및 구현 종화 🫶🏻 안뇽 나는 종화야
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

3 participants