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

[로또 게임]박해원 과제 제출합니다. #16

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

haewonee
Copy link

@haewonee haewonee commented Jul 17, 2024

수정내역

  1. 피드백해주신 명확히 쓰여있지않은 변수명과 메서드명을 변경하였습니다!!!
  2. Input_Dangchum에서 원래 당첨 번호만 입력받도록 구성되어있었고, main에서 bonus입력을 받았었으나 보너스 입력을 Dangchum에서 하게끔 변경하였습니다!!
  3. 보너스 번호를 입력받을 때 scanner를 사용하였었는데 예전에 이를 사용하면 에러가 날 가능성이 있다는 말을 들은 적이 있어서 Console을 통해 입력받고 이 문자열을 정수형으로 바꿔서 사용하도록 고쳤습니다!!!
  4. Input_lotto_count에서 원래 illegal argument exception만 일으키고 다시 입력받지 않는 코드였는데, 에러 문구와 함께 그 자리에서 다시 구입금액을 입력받을 수 있도록 짰습니다!!!
    5.결합도를 낮추기 위해서 Lotto 클래스에 있었던 당첨 결과를 출력하는 함수를 새로운 Output_result 클래스를 생성하여 처리하였습니다!!
  5. 기존의 Earn 클래스에서 local_count와 value가 범위가 달라지면 예외가 발생할 수 있다는 피드백을 듣고 map자료구조를 사용하여 lotto_count[]와 value를 쌍으로 해서 sum을 구하려고 했으나 map에서 첫번째 key값만 따로 도출하는 방법을 찾지 못하여서 두개의 map을 생성하고 이 배열의 범위가 같을때에만 sum을 구하도록 짰습니다!!!

코드의 결합도를 낮추는데 중점을 잡아서 수정하였습니다 !! ! !

아직 강의 영상을 절반만을 시청하여서 MVC패턴을 사용하지 못하고 피드백만 반영했습니다 이번 프로젝트과제로 MVC사용법을 익힌 후 또다시 수정해서 올리겠습니다!!!

Copy link
Collaborator

@genius00hwan genius00hwan left a comment

Choose a reason for hiding this comment

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

사실 model과 view는 어느정도 나눠져 있는 설계입니다. model이 controller 역할도 하고 있는 경우가 있는데 그렇다면 model 이 뭔지 어떻게 분리를 해야하는지 생각해보세요

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