[2단계 - 웹 기반 로또 게임] 비비빙 미션 제출합니다.#455
Open
binggwa wants to merge 107 commits intowoowacourse:binggwafrom
Open
Conversation
Co-authored-by: binggwa <qudrhks317@gmail.com>
Co-authored-by: binggwa <qudrhks317@gmail.com>
Co-Authored-By: binggwa <199538015+binggwa@users.noreply.github.com>
Co-Authored-By: binggwa <199538015+binggwa@users.noreply.github.com>
Co-Authored-By: binggwa <199538015+binggwa@users.noreply.github.com>
Co-Authored-By: binggwa <199538015+binggwa@users.noreply.github.com>
Co-Authored-By: binggwa <199538015+binggwa@users.noreply.github.com>
Co-Authored-By: binggwa <199538015+binggwa@users.noreply.github.com>
Co-Authored-By: binggwa <199538015+binggwa@users.noreply.github.com>
Co-Authored-By: binggwa <199538015+binggwa@users.noreply.github.com>
Co-Authored-By: binggwa <199538015+binggwa@users.noreply.github.com>
Co-Authored-By: binggwa <199538015+binggwa@users.noreply.github.com>
Co-Authored-By: binggwa <199538015+binggwa@users.noreply.github.com>
Co-Authored-By: binggwa <199538015+binggwa@users.noreply.github.com>
Co-Authored-By: binggwa <199538015+binggwa@users.noreply.github.com>
Co-Authored-By: binggwa <199538015+binggwa@users.noreply.github.com>
Co-Authored-By: binggwa <199538015+binggwa@users.noreply.github.com>
Co-Authored-By: binggwa <199538015+binggwa@users.noreply.github.com>
Co-Authored-By: binggwa <199538015+binggwa@users.noreply.github.com>
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.
학습 목표
이번 미션을 통해 다음과 같은 학습 경험들을 쌓는 것을 목표로 합니다.
제출 전 체크 리스트
리뷰 요청 & 논의하고 싶은 내용
1) 이번 단계에서 가장 많이 고민했던 문제와 해결 과정에서 배운 점
1단계 미션에서 작성한 Domain 로직을 단 1도 수정하지 않고 진행하는 것이 주 목표였고, 실제로 매직넘버를 조금 분리한 것 외에 핵심적인 도메인 로직은 하나도 변경하지 않고 웹 UI를 만들어서 연결할 수 있었다. 관심사 분리가 잘 되었던 것 같아 다행이었다. 또, Web에서도 MVC 패턴을 어떻게 적용하는 지에 대해 고민했었는데, 처음에는 일단 WebApp 컨트롤러에 모든 로직을 담아 일단 돌아가는 코드를 만들고, 잘 돌아가는 것을 확인한 뒤에 LottoWebView를 따로 만들어 View의 역할을 담당하는 로직을 분리했다. 수업을 통한 이전 기수들의 리뷰를 통해 View가 담당할 규칙에 대해 다음과 같은 사실을 배울 수 있었다.
해당 리뷰들을 통해서 Web 환경에서의 View 분리에 대해 더 잘 알게 되었다고 생각한다.
2) 이번 리뷰를 통해 논의하고 싶은 부분
이번 단계에서 고민했던 문제는 리팩토링 수준과 figma 시안의 준수정도였습니다.
1단계에서의 초기 피드백 덕분에 도메인 로직을 수정하지 않고 2단계를 수월하게 진행할 수 있었던 것 같습니다!! 리뷰 항상 감사드리고 2단계도 잘 부탁드립니다! 😊
✅ 리뷰어 체크 포인트
1단계
2단계