[#10] 버스 검색 기능 구현 #16
Merged
Conversation
This file contains 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
- 사용자가 입력한 버스 번호에 대한 데이터만 필터링하는 작업
Closed
3 tasks
| stopName: response[3], | ||
| romanizedStopName: response[4], | ||
| xCoordinate: response[5], | ||
| yCoordinate: String(response[6].dropLast(1)))) |
There was a problem hiding this comment.
이건 그냥 궁금한건데, 마지막에 dropLast(1)이 들어간 이유가 무엇인가요?
There was a problem hiding this comment.
파싱해서 결과 확인해보니, y좌표 뒤에 37.480651/r 이런식으로 개행문자인 /r이 포함되어 있더라구요.
그래서 제거해주고 값 저장하고 있습니다 :)
| @MainActor | ||
| private func apply(_ searchResponse: [[String]]) { | ||
| for response in searchResponse { | ||
| self.allBusStops.append(BusStopInfo(busNumber: response[0], |
There was a problem hiding this comment.
append 하기 전에 N/A 값이 있는지 확인하는 차원에서 열의 개수를 세는 것은 어떤가요? searchResponse의 count를 통해 알 수 있을 것 같아요
There was a problem hiding this comment.
csv 파일 자체에서 nil 값이 오는 건 고려하지 못했네요 🥲 !!
수정해서 올렸는데 확인 부탁드려요.
- model 자체를
Optional타입으로 변경했습니다. - csv 파일에서 빈 값이 넘어올 때
""빈 String으로 오는데,""대신nil값을 넣도록 했습니다.
Optional 타입으로 변경하고 빈 값일 경우 nil을 넣도록 한 이유는,
- 각 필드에 유효한 데이터가 있는지 명시적으로 확인하기 위해
- 빈 데이터일 경우에도 유연하게 대처하기 위해서 입니다.
(+ 추가로 busNumber와 busOrder는 추후 사용하기 편하도록 Int 타입으로 변환해서 갖고 있도록 수정했어요!)
|
변수 네이밍은 좋은 것 같습니다! |
…y values during CSV parsing - csv 파일 파싱하는 과정에서 빈 값을 처리하기 위해 모델 프로퍼티를 Optional Type으로 수정 - 빈 값으로 들어오는 경우도 유연하게 대처하기 위함
…rom search results - 사용자가 검색한 버스 번호에 대한 결과를 보여주기 위해 버스 번호만 뽑아내는 기능 추가
Closed
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
📝 작업 내용
💬 리뷰 요구사항