-
Notifications
You must be signed in to change notification settings - Fork 77
[지하철 노선도 미션] 서주희 미션 제출합니다. #33
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
Open
maywngml
wants to merge
27
commits into
woowacourse:main
Choose a base branch
from
maywngml:maywngml
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Conversation
This file contains hidden or 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
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.
안녕하세요!! 우아한테크코스 프리코스 참가자 서주희입니다.
이번 미션은 지난 미션들과 다르게 프로젝트의 규모가 훨씬 커져서 프로젝트를 진행하면서 정말 많은 고민들을 했던 것 같습니다.
element 추가 방식
페이지를 어떻게 구성하면 효율적일지 또 어떤식으로 element를 추가하면 좋을지 고민을 많이 했었습니다. 결국에는 element를 생성하는 코드들을 공통 스크립트 파일에 작성해놓고 import하는 방식을 선택했습니다. 그런데 함수 하나로 element를 추가하려다보니 함수의 매개변수가 5개로 늘어나게 되었습니다. 이게 정말 효율적인 방식인지 아직도 고민이 됩니다. 더 나은 방법은 뭐가 있을지 앞으로 더 찾아보고 공부를 해야하는 부분인 것 같습니다.
localStorage 사용
사실 localStorage의 개념은 이론공부를 할때나 들었었지 실제로 사용해본 경험은 없었습니다. 항상 AWS나 MySQL등을 사용하여 DB를 관리했기 때문에 localStorage를 경험해볼 일이 없었습니다. 처음 사용해보는 것이어서 localStorage에 적응하는 데에도 시간이 조금 걸렸습니다. 처음엔 오류가 발생하거나 원하는대로 값이 저장되지 않아서 시간을 많이 소모했었습니다. 또한 각 노선의 구간들을 저장하는 방법을 생각하는 데에도 거의 3일의 시간이 걸렸습니다. 저는 lines를 키값으로 하여 노선의 이름들을 모두 저장한 후에 노선의 이름을 키값으로 만들어 구간을 배열로 저장하도록 했습니다. 생각나는 방법이 이것밖에 없어서 이렇게 진행했는데 더 효율적인 방법은 없는지 또 고민해봐야 할 것 같습니다.
프리코스를 진행하면서 느낀점
이번 프리코스를 진행하면서 정말 많은것들을 배울 수 있었습니다. 변수와 함수명을 통해 의도를 드러내서 주석 사용 줄이기, 코딩 컨벤션 지키기, 함수를 최대한 작게 분리하기 등 어찌보면 정말 기본적인 사항인데도 그동안 왜 크게 고려하지 않았을까 반성도 하게 됐습니다. 정말 이번에 피드백 주신 사항들을 지키면서 코드를 작성해보니 확실히 코드 가독성도 좋아지고 깔끔해지는걸 느낄 수 있었습니다. 그만큼 저의 부족함도 느낄 수 있었고 앞으로 이 부족함을 어떻게 채워야 할지 생각해볼 수 있는 계기가 되었던 것 같습니다. 조금은 고통스러웠던 3주였지만 그만큼 많은걸 얻을 수 있었던 것 같습니다. 함께 프리코스를 진행했던 다른 참가자분들도 정말 고생 많이하셨고 모두에게 좋은 결과가 있었으면 좋겠습니다.