[로또] 박세연 미션 제출합니다. by SEYEON-PARK · Pull Request #1 · gdgoc-skhu-missions/javascript-lotto

SEYEON-PARK · 2024-05-13T05:53:54Z

실행 과정
이번 페어 미션은 세 명이서 진행해서 지금까지 진행했던 거와는 느낌이 달랐습니다. 더욱 다양한 의견을 들을 수 있어서 좋았습니다. 하지만, 다양한 의견이 있었던 만큼 진행하는 시간도 더 걸릴 수밖에 없었습니다. 특히 이번에는 테스트 주도 개발(TDD)도 시도해보고자 했기에 더 시간이 오래 걸렸다고 생각합니다.

느낀 점
이번 미션은 TDD도 하고, 두 명의 페어와 함께 했기 때문에 시간이 많이 들었습니다. 3~4일 동안 매일 디스코드로 만나서 했어서 체력적으로 지치기도 했지만, 페어들과 함께 문제를 해결하는 과정이 재밌었습니다. 이번에는 메시지나 상수들도 다 파일에 선언해서 사용했는데, 이렇게 해본 건 처음이라 어색했습니다. 하지만, 실제 개발에서는 잘 사용되는 방법일 것 같아서 익숙해지면 좋겠다고 생각했습니다.

배운 점

TDD 시도해보는 방법
export와 export default를 언제 사용하는지, 그리고 import할 때의 차이점은 무엇인지
: 한 파일 안에 여러 개 export하면 import 할 때 {} 해야만 한다! 한 파일에 한 개 export하면 export default라고 적어주고, import할 때 {} 안 적어줘야 한다!
Object.values() 이용하면 객체를 배열처럼 이용할 수 있다.
: 이러면 forEach()도 이용 가능!
Number.isNaN()
: 개발자가 형변환해주고 매개변수에 대입해야 잘 작동한다.(Number.isNaN(Number(a)); 이런 식으로!)

궁금한 점

TDD를 한다고 했는데 이렇게 하면 되는 게 맞는지 궁금합니다.
기능별로 파일을 여러 개로 나누고, 함수도 짧게 만들고, 출력해야 할 메시지도 한 곳에 저장해두고 작업했는데 실제 업무에서도 이런 식으로 하나요?

khj0426

먼저 고생하셨습니다!

코드를 보면서 오히려 제가 더 배워가는 거 같아욤 !! 💯 전반적으로 큰 문제 없이 잘 읽혔습니다 👍 👍 👍 👍

첫번쨰 질문 <-
일단 제가 TDD를 많이 경험해 보지 못했기 떄문에 명확한 답은 드리기 힘들수도 있지만,
TDD는 코드를 작성하기 전 테스트 코드를 적는 것을 의미해요!

보통은 예상치 못한 버그를 잡기 위해 TDD를 사용하게 됩니다!

언제 TDD를 도입하고, 왜 TDD를 사용해야 하는지 알기 이전에 무엇을 테스트해야하는지부터 정하는 게 중요할 것 같아요!

팀마다 처한 상황이 다르고, 무엇을 테스트할지에 대한 기준도 서로 다르기 때문에,
팀의 논의가 꼭 필요할 것 같습니다..!

https://medium.com/@admin_1497/%EA%B0%9C%EB%B0%9C%EC%9E%90%EA%B0%80-%EC%98%AC%EB%B0%94%EB%A5%B8-tdd%EB%A5%BC-%EC%A0%81%EC%9A%A9%ED%95%98%EB%8A%94-%EB%B0%A9%EB%B2%95%EC%97%90-%EB%8C%80%ED%95%B4%EC%84%9C-667e7ba05fb6

두번째 질문<-(지극히 개인적인 의견입니다..!)
요것도 처한 상황에 따라 다를 것 같긴 합니다! 당장 요구사항이 많고 빠듯한 일정이라면,
어떻게든 요구사항을 만족시키고 기능을 구현하는 것이 최우선이라고 생각해요
(구현 후 한번씩 기술부채나 리팩토링을 할 수도 있겠죠?!)

https://blog.kmong.com/%EA%B3%A0%EA%B8%88%EB%A6%AC-%EC%8B%9C%EB%8C%80%EC%97%90-%EA%B8%B0%EC%88%A0-%EB%B6%80%EC%B1%84%EB%A5%BC-%ED%9A%A8%EA%B3%BC%EC%A0%81%EC%9C%BC%EB%A1%9C%EC%83%81%ED%99%98%ED%95%98%EB%8A%94-%EB%B2%95-front-end-%ED%8E%B8-aad53c07fa18

khj0426 · 2024-05-16T08:04:43Z

+  out_of_range_bonus: '[ERROR] 보너스 번호는 1부터 45 사이의 숫자여야 합니다.',
+  duplicate_bonus:
+    '[ERROR] 보너스 번호는 로또 번호랑 중복된 숫자를 입력하면 안됩니다.',
+});


너무 깔끔합니다.. 👍 에러 유형별로(구입금액 입력, 당첨번호 입력, 보너스번호 입력) 별로 한 깊이 더 객체가 만들어질 수도 있을 것 같아요!

또 요렇게 전역적으로 에러 상수를 관리할 수도 있지만 , 폴더 별로 관리할 수도 있습니다!
(폴더 별로 해당 폴더 안의 상수들을 관리하는 용도)

그래서 상황에 따라 적절한 방법을 골라서 사용하면 될 것 같아요!

khj0426 · 2024-05-16T08:12:23Z

+    prize: 5000,
+    message: '3개 일치 (5,000원) - ',
+  },
+  FOURTH: {


console.log(LOTTO_RESULT.FOURTH.bonus); LOTTO_RESULT.FOURTH.bonus = true; console.log(LOTTO_RESULT.FOURTH.bonus);

요 결과는 어떻게 나올까요?

처음에는 두 가지 때문에 안 바뀔 거라고 생각했었습니다.

const 때문에 안 될 것이다.

Object.freeze() 때문에 안 바뀔 것이다.

이렇게 생각했었는데 생각해보니 1번은 객체의 참조 주소만 바뀌지 않으면 가능할 것 같았습니다. 그리고 직접 해보니 바뀌는 걸 알 수 있었습니다.

그래서 왜 바뀌는지 궁금해서 찾아보니, 2번도 얕은 동결만 되고 있고, 깊은 동결이 되지 않아 객체 안의 값이 바뀔 수 있다는 걸 알 수 있었습니다! 감사합니다 :)

khj0426 · 2024-05-16T08:23:20Z

-  #validate(numbers) {
-    if (numbers.length !== 6) {
-      throw new Error("[ERROR] 로또 번호는 6개여야 합니다.");
+  changeStringToArray(inputNumbers) {


map메소드를 사용한다면 어떻게 바꿀 수 있을까요?

changeStringToArray는 범용적으로 쓸 수 있는 함수일 것 같은데요!
요 함수가 꼭 이곳에 위치해야 할까요?

사용하는 입장에서 문자열 -> 배열로 바꾸는 함수를 사용하려면 lotto객체를 만들어야만 사용할 수 있는데,
해당 방법이 바람직할까요 ?!!

네! 참고하여 수정하였습니다~

khj0426 · 2024-05-16T14:38:43Z

+import LottoGenerator from './LottoGenerator.js';
+
+class Game {
+  constructor() {


이렇게 사용하면 사용하는 바깥에서 맘대로 멤버변수를 제어할 수 있게 됩니당.
예를 들어 Game클래스로 만든 인스턴스를 사용하는 바깥에서는

const game = new Game(); game.purchase = 6; console.log(game.purchase);

다음과 같이 맘대로 멤버변수에 접근하고 수정할 수 있어요!

Game의 규칙이 잘 지켜지도록, 바깥에서 조작할 수 없도록 하려면 어떻게 하면 좋을까요?!

앞에 #을 붙이면 될 것 같아요! 제가 생각한 게 맞을까요?

맞습니당 👍

khj0426 · 2024-05-16T14:46:51Z

+import { Random } from '@woowacourse/mission-utils';
+import { LOTTO } from './constants/Constants.js';
+
+const LottoGenerator = {


요부분은 클래스로 구현하지 않고 객체로 구현한 이유가 있을까욤?!

이 부분은 다른 팀원분이 작성해주신 부분이라 잘 모르겠습니다..! 물어보고 이유가 있으면 추가해서 댓글 달아놓을게요!

물어보니 큰 이유는 없는 것 같아요! 클래스로 구현하는 게 더 통일성 있고 좋을까요?

khj0426 · 2024-05-16T15:00:25Z

+import { LOTTO_RESULT } from './constants/Constants.js';
+
+class Statistics {
+  #results = { 3: 0, 4: 0, 5: 0, '5+1': 0, 6: 0 };


요부분은 상수로 따로 뺴시지 않은 이유가 궁금합니당

생각을 못 했는데 상수로 빼면 좋을 것 같아 그렇게 수정했습니다!

khj0426 · 2024-05-16T15:05:22Z

+    this.winningNumber = winningNumber;
+    this.bonusNumber = bonusNumber;
+    // this.#statistics = statistics;
+  }


요부분도 외부에서 멤버변수를 조작할 수 있을 것 같은데 어떻게 방지할 수 있을까요?!

const statistics = new Statistics( this.lottos, this.winningNumbers, this.bonusNumber ); statistics.bonusNumber = 'Hi!'

외부에서 멤버 변수를 조작할 수 없도록 구현 수정했습니다!(#을 이용했어요)

khj0426 · 2024-05-16T15:05:52Z

+  }
+
+  calculateRank(matchNumber, lotto) {
+    if (matchNumber >= 3) {


요기서 관리하는 3,5도 의미있는 숫자 같아요!

네! 상수로 선언했습니다!

HarrySeop and others added 30 commits May 7, 2024 23:41

Docs: 문서 양식 불러오기

85b8cfc

Docs: 구현할 기능 목록 최신화

9a1f949

Feat: InputView 구현

8e4001d

Docs: 가이드 수정

0f29029

Merge branch 'main' of https://github.com/SEYEON-PARK/javascript-lotto

8b2cb69

Modify: 함수 선언 변경

c2e80f1

Add: Constant 사용을 위한 기본 구조 선언

5a05a28

Feat: 로또 구입 금액 유효성 검사

52f8f92

Add: TDD 구현

c293d69

Modify: 구매 금액 TDD 완성

26850b9

Modify: 페어 Prettier 설정 통일 + 구현 목록 업데이트

37f41aa

Feat: 로또 당첨 번호 입력 유효성 검사 및 TDD 구현

4015299

Modify: Vanila JS 방식으로 구조 변경

9bb0fec

Modify: 구입금액 관련 상수화 완료

168e69d

Modify: 로또 당첨 금액 관련 상수화 완료

6be323f

Feat: 로또 번호 생성기 완성

1f0c886

Feat: 보너스 번호 입력 구현

f8ef6b8

Add: 보너스 번호 유효성 검사 추가 및 TDD 구현

4b61c3f

Modify: 진행상황 업데이트

0265fef

Modify: 로또 유효성 검사 수정

a115b5b

Modify: 로또 유효성 검사 추가 수정

eefc62d

Modify: 로또 유효성 검사 완료

e01e156

Modify: 보너스 유효성 검사 완료, 진행상황 업데이트

f12bb7e

Add: 로또 클래스 함수 추가

113ebe4

Feat: 출력문 구현 중, 오타 수정

facbad1

Modify: 모듈 방식 수정...

33dab06

Modify: eslint 관련 코드 수정, 코드 최신화

bc71e2d

Add: 로또 출력 메시지 구현

ab54c9d

Add: 당첨번호 입력, 보너스 번호 입력 구현 완료, 모듈 형식 변경

6758765

Delete: 타입 검사 콘솔 제거

cb8d3ed

HarrySeop and others added 7 commits May 11, 2024 00:39

Add: 출력문 추가

dbe6fa5

Feat: 당첨 개수 반환 함수 추가

d92a17a

Modify: export문 추가

5729a3d

ADD : 맞은 개수 확인 코드 추가

73c6d28

Modify: 함수 이름 변경

b299347

Feat: 당첨 내역 출력 코드 작성

ad8c2a6

Feat: 수익률 기능 추가

4e5d2cb

khj0426 approved these changes May 16, 2024

View reviewed changes

SEYEON-PARK added 4 commits May 20, 2024 14:39

Modify : map 메소드 활용해보기!

fdb4c79

Modify : changeStringToArray() 함수 위치 바꾸기!

e33735f

Modify : 상수를 이용하여 객체 정의하기

7621a95

Modify : 클래스 외부에서 멤버 변수를 수정할 수 없도록 코드 수정

d3df6b4

Conversation

SEYEON-PARK commented May 13, 2024

Uh oh!

khj0426 left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants