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.
@jae57
@lee_ari95 @yim2627
안녕하세요. 찌루루~
이번 만국박람회 프로젝트 리뷰 잘부탁드립니다. 😊 🙏🏻
JSON을 처음 다루는 것이라 어려운 부분이 있었지만, 생각보다 STEP 1 PR을 빠르게 보내드리게 되었습니다.
혹여나 저희가 놓친 부분이 있다면 번거롭더라도 한 번만 더 짚어주시면 감사하겠습니다!
고민했던 것
STEP 2 요구사항에 첨부되어있는 파일중 JSON 파일들을 살펴보고 예시 화면을 보면서 어떤 부분을 타입으로 설계해야 할지 고민해보았습니다.
왜 JSON 관련 타입을 만들 때 프로퍼티를 상수가 아니라 변수로 만드는지 고민해보았습니다.
이번 프로젝트에선 JSON 포맷의 데이터를 파싱하기만 하지, 저희의 인스턴스를 JSON으로 파싱하진 않기 때문에 모델 타입에 Codable를 채택하여 사용하지도 않을 Encodable까지 채택해줄 필요는 없다고 판단하여 Codable이 아닌 Decodable만 채택해주었습니다.
궁금했던 것 / 조언을 얻고 싶은 부분