만국박람회 [STEP 2] Jiseong, Ari #118

leeari95 · 2021-12-10T06:34:04Z

안녕하세요. 찌루루!
STEP 2가 무사히 마무리되서 PR 보내드립니다.
저희가 STEP 3에서 진행해야 할 오토 레이아웃도 최대한 할 수 있는 만큼 시도해보았습니다.
추가적으로 부족한 점은 STEP 3에서 마저 채워나가도록 하겠습니다. ☺️
혹시 이 외에 저희가 놓친 부분이나 부족한 부분이 있다면 번거롭더라도 한번 더 짚어주신다면 감사하겠습니다. 🙏🏻

고민했던 것

파싱에 실패하였을 때 어떻게 에러처리를 할지 고민해보았습니다.
- 개발자에게만 보여줘야 할지, 사용자에게 보여줘야 할지 고민해보았고 앱을 사용하는 사용자가 에러를 보는 것이 적절하다는 판단에 얼럿을 띄우도록 구성하였습니다.
Navigation Bar를 MainViewController에서만 숨길 수 있는 방법을 고민해보았습니다.
- View Life Cycle을 활용하여 뷰가 보여질때 숨기고, 사라질때 다시 나타내도록 구현해보았습니다.
MainViewController에서 크기가 다른 Label의 Text부분을 어떻게 처리할지 고민해보았습니다.
- 스토리 보드에서 Label을 직접 추가하여 구현할 수도 있었지만, 스토리보드 의존성을 최대한 낮춰보기 위해 코드를 활용하여 Label에 텍스트를 추가하고 이후 특정 글자만 사이즈가 커질 수 있도록 구현했습니다.
파라미터로 전달하는 String을 변수로 따로 담아 enum타입으로 묶어두어 리팩토링을 진행했습니다.
셀의 재사용성을 높이기 위해 xib를 활용하여 커스텀 셀을 구현해보았습니다.
셀을 터치하는 시점에 해당 셀의 정보가 다음 화면으로 전달될 수 있도록 구현했습니다.
선택된 셀이 선택된 흔적이 남아있어 이를 없애기 위해 고민해보았습니다.
- delegate를 활용하여 셀이 눌렸을 때 시점에 deselect하도록 구성해보았습니다.
ResultType이 가독성이 좋고, 에러에 유연히 대처할 수 있도록 decode 메소드를 ResultType으로 구현해보았습니다.
- fail시 저희가 만든 에러의 description을 사용자에게 알려주고 success시 view를 구성하는 메소드를 타게끔 구성하였습니다.

궁금했던 것 / 조언을 얻고 싶은 점

xib를 활용하여 커스텀 셀을 구현하였는데 셀에 들어갈 요소들을 설계할 때 ios14부터 사용되는 contentConfiguration을 사용해야되나? 라는 의문점이 들었습니다.
저희 조는 cell의 커스텀 클래스를 활용해 UI요소들을 아울렛 변수로 연결해주고 들어갈 데이터를 설계해주는 방식으로 하였는데, 이 부분이 최신 ios 버전에선 적합하지 않은 것인지 궁금합니다.

private func configure(of cell: ItemTableViewCell, by item: ExpositionItem) {
      cell.itemImageView.image = UIImage(named: item.imageName)
      cell.itemNameLabel.text = item.name
      cell.itemShortDescriptionLabel.text = item.shortDescription
}

화면간 데이터 전달을 할 때 현업에서는 어떤 방식으로 하시는지가 궁금합니다.
저희는 prepare 메소드를 활용하여 데이터 전달을 하였는데, TableView의 Delegate 메소드에서 바로 데이터를 넘겨주기도 하는 것 같던데 어떤 것이 적절한지 잘 판단이 서질 않습니다.
셀을 터치하는 시점에 데이터를 NextViewController의 프로퍼티를 활용하여 전달해주고 있습니다. 이 부분을 프로퍼티를 활용하지 않는 다른 좋은 방법이 떠오르지가 않습니다.
화면에서 데이터를 주고받을 때 찌루루는 보통 어떤 방법으로 주고받는지 궁금합니다.
함수형 프로그래밍 방식으로 설계해보려 하였는데 저희가 짠 코드가 함수형 프로그래밍이라고 하기 적합한지 궁금합니다.
MainViewController 내부에 있는 setUpView() 메소드가 조금 방대하다고 느껴지는데 리팩토링을 하고 싶은데 좋은 방법이 떠오르지가 않습니다. 찌루루의 조언이 필요합니다!

이번 피드백도 잘부탁드립니다! 😊

- 테스트에 필요한 Asset 파일 추가

- 옵셔널이 아닌 에러나 데이터를 반환할 수 있도록 개선 - 에러처리에 따른 JSONError 타입 추가

- 파싱 실패시 사용자에게 보여줄 Alert을 extension으로 추가

- CategoryPrefix 타입 생성 - Exposition 타입의 연산 프로퍼티(numberFormatter) 추가 - Label의 접두어 Fontsize 변경 메소드 추가

- titleLabel에서 특정 구간에서 newLine을 할 수 있도록 수정

- 기존 커스텀 셀의 IBOutlet 이름을 수정 - 네비게이션 바의 타이틀을 수정 - 필요없는 주석 제거 - parsing() 메소드 구현 - UITableViewDataSource를 활용하여 TableView를 구성

- 네비게이션바 타이틀 설정 - 전달받은 데이터를 통해 UI 업데이트

- mainVC와 listVC에서 파싱할 때 파라미터로 전달했던 FileName을 새로 생성한 타입으로 대체

jae57

@leeari95 @yim2627
안녕하세요, 아리, 지성!
고민했던점 잘 읽어보았습니다! 구현의 이유들이 잘 나타나있어서 코드 이해에 도움이 많이 되었습니다!
설정이 쉽지 않았을텐데 xib 를 활용하여 재사용성을 높힌 점 매우 칭찬드립니다! 👏

궁금했던 것 / 조언을 얻고 싶은 점

xib를 활용하여 커스텀 셀을 구현하였는데 셀에 들어갈 요소들을 설계할 때 ios14부터 사용되는 contentConfiguration을 사용해야되나? 라는 의문점이 들었습니다.

iOS14 부터 사용가능한 contentConfiguration 은 커스텀 클래스가 아닌 기본 제공되는 셀을 쓸 때 사용가능한 걸로 알고 있습니다.
따라서 지금처럼 커스텀 클래스를 만든 상황이라면 어차피 사용할 수가 없네요!

화면간 데이터 전달을 할 때 현업에서는 어떤 방식으로 하시는지가 궁금합니다.
저희는 prepare 메소드를 활용하여 데이터 전달을 하였는데, TableView의 Delegate 메소드에서 바로 데이터를 넘겨주기도 하는 것 같던데 어떤 것이 적절한지 잘 판단이 서질 않습니다.

둘 다 가능하니 쓰고싶은 것 쓰시면 될 것 같아요!
한 화면에 Segue가 tableView만이 아니라 여러 곳에 다수 존재한다면 tableView에서의 이동은 tableViewDelegate에서 해주면 좀 더 명확해질 수 있겠네요!

셀을 터치하는 시점에 데이터를 NextViewController의 프로퍼티를 활용하여 전달해주고 있습니다.
이 부분을 프로퍼티를 활용하지 않는 다른 좋은 방법이 떠오르지가 않습니다.
화면에서 데이터를 주고받을 때 찌루루는 보통 어떤 방법으로 주고받는지 궁금합니다.

nextViewController에 setModel(_ model: Model) 과 같이 세터메서드를 하나만들어서 이용해보는 건 어떨까요?
저도 자주사용하는 방법입니다.
그러면 NextViewController의 프로퍼티는 setModel메서드 내에서만 접근하므로 private 으로 지정할 수 있겠어요!

함수형 프로그래밍 방식으로 설계해보려 하였는데 저희가 짠 코드가 함수형 프로그래밍이라고 하기 적합한지 궁금합니다.

우선 아리와 지성이 생각하기에 함수형 프로그래밍으로 짰다고 생각하는 곳이 어느 부분이고, 왜 그렇게 생각하셨는지 구체적으로 말씀해주시면 좀 더 이야기가 잘 될 것 같아요!

MainViewController 내부에 있는 setUpView() 메소드가 조금 방대하다고 느껴지는데 리팩토링을 하고 싶은데 좋은 방법이 떠오르지가 않습니다. 찌루루의 조언이 필요합니다!

제가 보기에는 딱 적절해보입니다! 이부분을 더 분리하기는 어려울 것 같아요! 이미 UI 구성하는 역할 한가지만 하고 있어서 충분해보입니다!

jae57 · 2021-12-13T07:40:40Z

Expo1900/Expo1900/Extension/UIViewController+extension.swift

+
+extension UIViewController {
+ func showAlert(message: String) {
+ let cancelAction = UIAlertAction(title: "OK", style: .default)


title 은 OK인데 변수명은 cancelAction인 이유가 따로 있나요?

잘못된 네이밍이였습니다 okAction으로 수정하였습니다.

jae57 · 2021-12-13T08:21:39Z

Expo1900/JSONParesTests/JSONParesTests.swift

@@ -0,0 +1,21 @@
+import XCTest
+
+class JSONParesTests: XCTestCase {


JSONParseTests 오타인거 같아요!

jae57 · 2021-12-13T08:24:18Z

Expo1900/Expo1900/Controller/ItemListViewController.swift

+
+class ItemListViewController: UIViewController {
+ private var items = [ExpositionItem]()
+ @IBOutlet private weak var itemListTableVIew: UITableView!


i 가 대문자네요!

jae57 · 2021-12-13T08:28:27Z

Expo1900/Expo1900/Controller/ItemListViewController.swift

+ return cell
+ }
+
+ private func configure(of cell: ItemTableViewCell, by item: ExpositionItem) {


이 메서드도 ItemTableViewCell 내에 선언된다면 cell 의 프로퍼티들을 외부에서 알지못해도 UI 설정이 가능하겠네요!

네 뷰컨에서 굳이 해줄 필요없이 커스텀 셀 클래스 내부에서 해줘도 되겠단 생각이들어 코드를 옮긴 후 뷰컨에서 호출해주는 방식을 사용하였습니다 😁

jae57 · 2021-12-13T08:30:24Z

Expo1900/Expo1900/Controller/ItemListViewController.swift

+ }
+}
+
+extension ItemListViewController {


전반적으로 extension으로 영역을 잘 나눠주셔서 가독성이 좋네요!

jae57 · 2021-12-13T08:31:19Z

Expo1900/JSONParesTests/JSONParesTests.swift

+ func test_Decode_파리만국박람회() {
+ let exposition = JSONParse<Exposition>.decode(fileName: "exposition_universelle_1900")
+
+ XCTAssertEqual(exposition?.visitors, 48130300)


테스트 쪽은 모두 에러가 나고 있어요! 해결해볼 수 있을까요?

Result Type으로 바꾸면서 테스트함수를 미처 보지 못한것같습니다 🥲
수정하였습니다.

- ItemDetailViewController에 setModel() 메소드 추가 - ItemListViewController에서 데이터 전달 시 메소드를 통해 전달할 수 있도록 수정 - 오타 수정시 적용되지 않았던 IBOutlet 연결 부분 재연결해주어 수정

leeari95 · 2021-12-14T03:04:13Z

찌루루! 피드백 감사합니다!
말씀해주셨던 부분 모두 개선해보고 저희가 추가적으로 개선해보았습니다.

확인해주시고 머지 부탁드리겠습니다. 😁 🙏🏻

개선 사항

컴파일에러 발생했던 테스트 코드 에러 수정하기
오타수정
화면간 데이터 전달 시 메소드를 활용하도록 수정
스토리보드 분할
배열의 인덱스 안전하게 조회하도록 개선
파일 분리
guard문 중에 의미없는 return문을 fatalError()로 수정

- IBOutlet 프로퍼티에 접근제어 추가 - configure 메소드를 private으로 수정하고 setModel로 데이터를 넘겨받도록 수정

jae57 · 2021-12-15T03:56:47Z

@leeari95 @yim2627
고생하셨습니다! Step2 핵심 구현사항 무리없이 잘 진행해주셔서 머지하겠습니다!

yim2627 and others added 28 commits December 8, 2021 15:49

✨ feat: JSON Parse 타입 생성 #1

f20ea39

✅ test: JSONPares 타입에 대한 Test 코드 생성 #1

343b60a

- 테스트에 필요한 Asset 파일 추가

✨ feat: 각 ViewController 생성 후 VC 클래스 생성 #2 #3 #4

76796fd

♻️ refactor: JSONParse 타입의 decode 메소드 반환타입을 Result로 수정 #1

33dd802

- 옵셔널이 아닌 에러나 데이터를 반환할 수 있도록 개선 - 에러처리에 따른 JSONError 타입 추가

✨ feat: Main 화면 Storyboard UI 구성 및 아울렛변수 연결 #2

cc88de1

✨ feat: JSON파일을 파싱하여 메인화면에 적용될 수 있도록 구현 #2

0d95abe

- 파싱 실패시 사용자에게 보여줄 Alert을 extension으로 추가

✨ feat: NavigationBar Hidden 처리 #2

0cbbe28

♻️ refactor: 사용하지 않는 Delegate 프로토콜을 삭제 후 코드 내부 수정 #2

6cd32b4

✨ feat: Main 화면의 Label 포맷 섫정 #2

0d1d99d

- CategoryPrefix 타입 생성 - Exposition 타입의 연산 프로퍼티(numberFormatter) 추가 - Label의 접두어 Fontsize 변경 메소드 추가

♻️ refactor: 메인 화면의 오토 레이아웃을 수정 #2

4e7bacc

- titleLabel에서 특정 구간에서 newLine을 할 수 있도록 수정

✨ feat: 테이블 뷰, 커스텀 셀 레이아웃 설정 #3

5f0ccf7

✨ feat: Cell 눌렀을 때 화면 전환 구현 #3

e34a404

✨ feat: ItemDetailViewController 화면 구성과 IBOutlet 연결 #4

a9d8570

✨ feat: 화면 전환시 데이터 전달 구현 #4

b0f42af

- 네비게이션바 타이틀 설정 - 전달받은 데이터를 통해 UI 업데이트

♻️ refactor: FileName 타입을 새로 생성하여 리팩토링 #5

e665f92

- mainVC와 listVC에서 파싱할 때 파라미터로 전달했던 FileName을 새로 생성한 타입으로 대체

✨ feat: 네비게이션 바 타이틀 수정 메소드 분리 #3

27ce704

♻️ refactor: cell의 UI요소들을 설정해주는 부분을 configure 메소드로 분리 #5

5d3a462

♻️ refactor: 접근제어자 추가 #5

6c13b78

♻️ refactor: extension으로 ViewController를 분리 #5

b544f71

♻️ refactor: 테이블뷰 셀 등록부분을 메소드로 분리 #5

7a8652c

✨ feat: 메인 VC의 title을 설정 #2

7182e7c

♻️ refactor: 얼럿을 HIG에 맞게 수정 #5

d0bf64c

✨ feat: 데이터전달 실패할 시 얼럿을 띄우도록 추가 #5

4049dcb

✨ feat: cell selectState 초기화 구문 추가 #3

ad3ce8f

📝 docs: README.md 내부에 STEP 2 내용 추가

e3d103d

📝 docs: README 사진 안보이는 부분 수정

ecee086

📝 docs: README.md 목차 수정

a2f8b95

jae57 reviewed Dec 13, 2021

View reviewed changes

♻️ refactor: Storyboard 분할 #6

7edf47f

leeari95 and others added 6 commits December 14, 2021 11:15

✅ test: 컴파일에러 발생했던 테스트코드 전반적으로 수정 #6

9a778ae

✨ feat: 배열 안전하게 조회하는 subscript 구현 #6

a7bc4ba

🚚 style: 파일과 타입들의 네이밍 오타를 수정 #6

4c8e05d

♻️ refactor: 에러처리 방법 변경 및 인덴트 수정 #6

2afd3f5

🚚 style: JSONError 타입 파일 분리 #6

bcae35e

leeari95 requested a review from jae57 December 14, 2021 03:04

♻️ refactor: 셀 내부 프로퍼티를 접근하지않고 UI를 구성하도록 개선 #6

ce627a2

- IBOutlet 프로퍼티에 접근제어 추가 - configure 메소드를 private으로 수정하고 setModel로 데이터를 넘겨받도록 수정

jae57 merged commit 1170bb5 into yagom-academy:4-leeari95 Dec 15, 2021

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

만국박람회 [STEP 2] Jiseong, Ari #118

만국박람회 [STEP 2] Jiseong, Ari #118

leeari95 commented Dec 10, 2021

jae57 left a comment

jae57 Dec 13, 2021

yim2627 Dec 14, 2021

jae57 Dec 13, 2021

jae57 Dec 13, 2021

jae57 Dec 13, 2021

yim2627 Dec 14, 2021

jae57 Dec 13, 2021

jae57 Dec 13, 2021

yim2627 Dec 14, 2021

leeari95 commented Dec 14, 2021

jae57 commented Dec 15, 2021

		@@ -0,0 +1,21 @@
		import XCTest

		class JSONParesTests: XCTestCase {

만국박람회 [STEP 2] Jiseong, Ari #118

만국박람회 [STEP 2] Jiseong, Ari #118

Conversation

leeari95 commented Dec 10, 2021

고민했던 것

궁금했던 것 / 조언을 얻고 싶은 점

jae57 left a comment

Choose a reason for hiding this comment

궁금했던 것 / 조언을 얻고 싶은 점

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

leeari95 commented Dec 14, 2021

개선 사항

jae57 commented Dec 15, 2021