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

[#12] 버스 루트 별 정류장 띄우기 #23

Merged
merged 5 commits into from
Oct 19, 2024

Conversation

Minkyeong-Choi
Copy link
Collaborator

📝 작업 내용

  • 207번 버튼을 누르면 207번에 해당하는 노선이 뜹니다. (참고로 472번은 위치가 멀어서 뜨지 않습니다.)
  • 첫 화면이 기본 설정 좌표로 뜨는 오류, 사용자 위치로 돌아오는 버튼의 오류를 수정하였습니다.

💬 리뷰 요구사항(선택)

  • 이전의 지도를 띄웠던 코드를 합치는 과정에서 BusStop 구조체를 생성하게 되었습니다. 구조체의 이름이 적절한가요?
  • 이번 Pr에서 지도가 사용자의 현재 위치에 따라 업데이트되지 않고, 자유롭게 사용할 수 있게 하는 기능은 구현되지 않았습니다.

@Minkyeong-Choi Minkyeong-Choi self-assigned this Oct 17, 2024
@Minkyeong-Choi Minkyeong-Choi linked an issue Oct 17, 2024 that may be closed by this pull request
3 tasks
Comment on lines 92 to 96
struct BusStop: Identifiable {
let id = UUID()
let name: String
let latitude: String
let longitude: String
Copy link
Collaborator

Choose a reason for hiding this comment

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

기존에 사용하던 BusStopInfo와 차이점이 Identifiableid 뿐인 것 같은데, 따로 선언해서 사용하는 이유가 있나요 !
없다면 기존 구조체에 추가해서 사용하는 것도 좋을 것 같아요 :)

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

수정했습니다! 확인 부탁드려요

@@ -75,23 +75,16 @@ final class BusStopSearchViewModel: ObservableObject {
}

func getNameAndCoordinates(busNum: String) {
Copy link
Collaborator

Choose a reason for hiding this comment

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

민~ 혹시 이 함수가, 사용자가 입력한 번호에 해당하는 정류장을 뽑아오는 함수인건가요~?
맞다면, BusSearchViewModel에 있는 searchBusStops 함수와 동일해 보이네요!
사용자가 입력한 버스 번호에 해당하는 정보는, BusSearchViewModel에 있는 filteredBusStops 변수에 접근해서 사용하면 돼요 :)

Copy link
Collaborator

@dbqls200 dbqls200 left a comment

Choose a reason for hiding this comment

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

수고하셨습니다!

@Minkyeong-Choi Minkyeong-Choi merged commit 15ddcef into dev Oct 19, 2024
@Minkyeong-Choi Minkyeong-Choi deleted the fix/12-showing-bus-stop-location branch October 19, 2024 04:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

지도 위에 정류장 위치 표시하기
2 participants