[2단계 - 행운의 로또 미션] 호프(김문희) 미션 제출합니다. #154

moonheekim0118 · 2022-03-05T12:46:46Z

데모페이지

안녕하세요 리뷰어님! 먼저 늦게 PR 올린 점 죄송합니다 ..😢 이것저것 해보려고 욕심내다보니까 시간이 늦어졌네요.
구현사항과 피드백 반영 사항 및 추가 사항은 아래와 같습니다.

Step 2 기능 구현

결과 확인하기 버튼을 누르면 당첨 통계, 수익률을 모달로 확인할 수 있다.
- 로또 당첨 금액은 고정되어 있는 것으로 가정한다.
다시 시작하기 버튼을 누르면 초기화 되서 다시 구매를 시작할 수 있다.

테스트 코드 추가

이전에 public 메서드는 모두 테스트코드를 작성하신다는 의견에 동의하여, public 메서드에 대한 테스트 코드를 작성했습니다.
단위테스트에 맞게, 파일을 여러개로 나누어서 테스트코드를 작성해보았습니다. (lottoUser, lotto (helper 함수), validation)

유의미한 클래스 이름

기존의 lottoData 역시 정확히 어떤 데이터를 가지고 있는지 표현하고 있다고 판단하지 않아서 lottoUser 로 바꾸어주었습니다.

그럼 좋은 피드백 미리 감사합니다 ! 좋은 주말 보내세요 👍 👍

commit 8734269 Author: moonheekim0118 <hellomooneekim@gmail.com> Date: Sun Feb 27 20:48:31 2022 +0900 refactor: Model 네이밍 수정 commit 5dc8c6e Author: moonheekim0118 <hellomooneekim@gmail.com> Date: Sun Feb 27 20:45:15 2022 +0900 test: 로또 번호 테스트 코드 추가 commit 318d7ca Author: moonheekim0118 <hellomooneekim@gmail.com> Date: Sun Feb 27 13:05:09 2022 +0900 refactor: View 컴포넌트 추상화 commit 717b0a9 Author: moonheekim0118 <hellomooneekim@gmail.com> Date: Sat Feb 26 23:35:39 2022 +0900 refactor: Model에서 불필요한 메서드 유틸로 이동 commit 48b01ea Author: moonheekim0118 <hellomooneekim@gmail.com> Date: Sat Feb 26 23:20:46 2022 +0900 refactor: 메서드 이름 수정 commit 343facd Author: moonheekim0118 <hellomooneekim@gmail.com> Date: Sat Feb 26 23:18:12 2022 +0900 refactor: 상수 및 선택자 함수 변경 - CLASS_SELECTOR -> CLASS_NAME 으로 변경 - ID_SELECTOR 에 # 추가 - querySelector 유틸 함수에 # 제거 commit 2a36fcf Author: moonheekim0118 <hellomooneekim@gmail.com> Date: Wed Feb 23 18:35:37 2022 +0900 docs: 코드 구조도 추가 commit 4334860 Author: moonheekim0118 <hellomooneekim@gmail.com> Date: Wed Feb 23 16:20:41 2022 +0900 docs: 기능목록 수정 commit 478cd75 Author: gytks4 <gytks4@naver.com> Date: Wed Feb 23 15:39:13 2022 +0900 docs: 기능 구현 완료 체크 Co-authored-by: moonheekim0118 <hellomooneekim@gmail.com> commit 5c416fa Author: gytks4 <gytks4@naver.com> Date: Wed Feb 23 15:37:25 2022 +0900 feat: 버튼에 포인터 추가 Co-authored-by: moonheekim0118 <hellomooneekim@gmail.com> commit 0380381 Author: gytks4 <gytks4@naver.com> Date: Wed Feb 23 15:36:44 2022 +0900 test: beforeEach 삭제 Co-authored-by: moonheekim0118 <hellomooneekim@gmail.com> commit db08655 Author: gytks4 <gytks4@naver.com> Date: Wed Feb 23 15:30:43 2022 +0900 refactor: 접근자 변경 Co-authored-by: moonheekim0118 <hellomooneekim@gmail.com> commit f7e69dd Author: gytks4 <gytks4@naver.com> Date: Wed Feb 23 15:17:48 2022 +0900 refactor: showLottoListSection 추상화 Co-authored-by: moonheekim0118 <hellomooneekim@gmail.com> commit d73a064 Author: gytks4 <gytks4@naver.com> Date: Wed Feb 23 15:06:44 2022 +0900 refactor: 웹 접근성 향상 Co-authored-by: moonheekim0118 <hellomooneekim@gmail.com> commit 6338ccd Author: gytks4 <gytks4@naver.com> Date: Wed Feb 23 14:57:05 2022 +0900 feat: 금액이 1000원으로 나누어떨어지지 않으면 에러 생성 Co-authored-by: moonheekim0118 <hellomooneekim@gmail.com> commit 0af2f16 Author: gytks4 <gytks4@naver.com> Date: Wed Feb 23 14:30:22 2022 +0900 fix: 로또 리스트가 쌓이는 문제 해결 Co-authored-by: moonheekim0118 <hellomooneekim@gmail.com> commit ea70e0e Author: gytks4 <gytks4@naver.com> Date: Wed Feb 23 14:25:20 2022 +0900 feat: 금액 입력 전, lotto-list-section 숨기기 Co-authored-by: moonheekim0118 <hellomooneekim@gmail.com> commit d549f26 Author: gytks4 <gytks4@naver.com> Date: Wed Feb 23 14:13:17 2022 +0900 chore: 티켓 이미지를 이모지로 대체 Co-authored-by: moonheekim0118 <hellomooneekim@gmail.com> commit aea694e Author: gytks4 <gytks4@naver.com> Date: Tue Feb 22 23:23:26 2022 +0900 feat: 토글 버튼 구현 Co-authored-by: moonheekim0118 <hellomooneekim@gmail.com> commit 98445e9 Author: gytks4 <gytks4@naver.com> Date: Tue Feb 22 22:52:21 2022 +0900 feat: 금액 입력 시, 로또 리스트 보여주기 Co-authored-by: moonheekim0118 <hellomooneekim@gmail.com> commit 4519e56 Author: gytks4 <gytks4@naver.com> Date: Tue Feb 22 22:32:09 2022 +0900 feat: 금액 입력 시, 총 로또 개수 보여주기 Co-authored-by: moonheekim0118 <hellomooneekim@gmail.com> commit 428deba Author: gytks4 <gytks4@naver.com> Date: Tue Feb 22 22:01:14 2022 +0900 feat: 로또 구입 금액 입력 시, 로또 발급 Co-authored-by: moonheekim0118 <hellomooneekim@gmail.com> commit 5308e8c Author: gytks4 <gytks4@naver.com> Date: Tue Feb 22 20:58:46 2022 +0900 chore: MVC 패턴 세팅 Co-authored-by: moonheekim0118 <hellomooneekim@gmail.com> commit 9d38759 Author: gytks4 <gytks4@naver.com> Date: Tue Feb 22 20:31:21 2022 +0900 chore: 이미지 로드를 위한 웹팩 설정 Co-authored-by: moonheekim0118 hellomooneekim@gmail.com commit 277e881 Author: gytks4 <gytks4@naver.com> Date: Tue Feb 22 20:30:56 2022 +0900 feat: 마크업 CSS 스타일링 Co-authored-by: moonheekim0118 hellomooneekim@gmail.com commit 730a2dc Author: gytks4 <gytks4@naver.com> Date: Tue Feb 22 20:30:22 2022 +0900 feat: HTML 마크업 Co-authored-by: moonheekim0118 hellomooneekim@gmail.com commit 33ed15b Author: gytks4 <gytks4@naver.com> Date: Tue Feb 22 15:03:00 2022 +0900 chore: 초기 환경 세팅 Co-authored-by: moonheekim0118 hellomooneekim@gmail.com commit ebe5495 Author: gytks4 <gytks4@naver.com> Date: Tue Feb 22 14:51:47 2022 +0900 docs: README.md 기능 목록 작성 Co-authored-by: moonheekim0118 hellomooneekim@gmail.com commit fd736d0 Author: gytks4 <gytks4@naver.com> Date: Tue Feb 22 14:51:16 2022 +0900 docs: TODO.md 작성 Co-authored-by: moonheekim0118 hellomooneekim@gmail.com

- 생성자로부터 container 엘리먼트 받아오도록 구현 - hide/show 메서드 구현

lsw1164

안녕하세요 호프님, 역할이 잘 분리돼있고, 공통 기능도 적절하게 모은 거 같습니다.
테스트도 분리하니까 더 보기 좋네요 👍

lsw1164 · 2022-03-05T19:19:30Z

src/js/core/View.js

+  hide() {
+    this.container.classList.add(CLASS_NAME.HIDE);
+  }
+
+  show() {
+    this.container.classList.remove(CLASS_NAME.HIDE);
+  }


공통 기능이군요 👍

lsw1164 · 2022-03-05T19:29:43Z

src/js/utils/dom.js

 export const replaceHTML = (element, template) => {
-  removeChildren(element);
+  element.replaceChildren();


lsw1164 · 2022-03-05T19:31:14Z

src/js/utils/dom.js

 export const $ = (selector, target = document) => target.querySelector(selector);

-const removeChildren = parent => {
-  while (parent.firstChild) {
-    parent.firstChild.remove();
-  }
-};
+export const $$ = (selector, target = document) => target.querySelectorAll(selector);


특정 element부터 탐색 좋네요 👍

lsw1164

호프님, 이번 미션도 수고 많으셨습니다. Approve 하겠습니다 :)

moonheekim0118 added 29 commits February 28, 2022 23:34

feat: 당첨번호 입력 퍼블리싱

3590f77

feat: 당첨번호 입력 뷰 구현

ff13bff

feat: 당첨 번호 검증 구현

db95650

test: 당첨 번호 검증 테스트 작성

3a4d755

feat: 모달 토글 기능 구현

ae0c658

docs: 기능 목록 수정

16c5ce0

feat: 결과 확인하기 버튼을 누르면 당첨 통계, 수익률을 모달로 확인 구현

6333c5e

style : 마크업 수정

20ca128

feat: 다시 시작하기 버튼을 누르면 초기화 되서 다시 구매를 시작 구현

c1d01ce

refactor: 뷰 Core 수정

eb696ac

- 생성자로부터 container 엘리먼트 받아오도록 구현 - hide/show 메서드 구현

refactor: 모델 수정, helper 폴더 생성

7fb00d4

refactor: uknown에러 메시지 추가

e0f16ac

refactor: import문에서 확장자명 제거

2876f18

refactor: html template 생성 메서드 추가

f05318c

feat: 금액 입력후 인풋과 버튼에 disable 속성 추가

634c84b

refactor: 숫자 상수화

20cffbc

refactor: 다시시작하기 버튼 눌렀을 때, 당첨번호 Input 비워지도록 구현

718afea

refactor: 자동으로 탭 움직이도록 수정

2a1ea54

refactor: 불필요한 CSS 제거

0430202

test: lottoUser 모델 테스트 분리

c5936c1

test: validation 함수 테스트 추가

5cab422

test: lotto helper 함수 테스트 추가

b0daf94

chore: 불필요한 파일 제거

ad3e005

feat: 추상클래스 인스턴스화하지 못하도록 구현

8119208

refactor: 반복해서 쓰이는 Element 변수화

5470dc7

refactor: removeChildren -> replaceChildren으로 교체

afb5e1e

chore: 배포 설정

535ecf6

Merge branch 'moonheekim0118' into step2-moonheekim0118

745b191

pocojang requested a review from lsw1164 March 5, 2022 13:19

pocojang assigned lsw1164 Mar 5, 2022

lsw1164 requested changes Mar 5, 2022

View reviewed changes

moonheekim0118 added 3 commits March 7, 2022 02:54

chore: EOL 제거

e81b403

refactor: for문 -> forEach로 수정

44bfd50

refactor: 수익률 계산 함수 수정

128cab5

lsw1164 approved these changes Mar 7, 2022

View reviewed changes

lsw1164 merged commit 41bd8b3 into woowacourse:moonheekim0118 Mar 7, 2022

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[2단계 - 행운의 로또 미션] 호프(김문희) 미션 제출합니다. #154

[2단계 - 행운의 로또 미션] 호프(김문희) 미션 제출합니다. #154

moonheekim0118 commented Mar 5, 2022

lsw1164 left a comment

lsw1164 Mar 5, 2022

lsw1164 Mar 5, 2022

lsw1164 Mar 5, 2022

lsw1164 left a comment

[2단계 - 행운의 로또 미션] 호프(김문희) 미션 제출합니다. #154

[2단계 - 행운의 로또 미션] 호프(김문희) 미션 제출합니다. #154

Conversation

moonheekim0118 commented Mar 5, 2022

데모페이지

Step 2 기능 구현

테스트 코드 추가

유의미한 클래스 이름

lsw1164 left a comment

Choose a reason for hiding this comment

lsw1164 Mar 5, 2022

Choose a reason for hiding this comment

lsw1164 Mar 5, 2022

Choose a reason for hiding this comment

lsw1164 Mar 5, 2022

Choose a reason for hiding this comment

lsw1164 left a comment

Choose a reason for hiding this comment