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

[Java-2주차] 유호연 사다리 미션 제출합니다. #417

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

Conversation

yuho9
Copy link

@yuho9 yuho9 commented Jul 21, 2024

사다리 1단계

요구사항

입력

  • 참여할 사람 이름을 입력
    문자 형태로 사람이름을 모두 입력받음
    쉼표를 기준으로 입력받은 문자를 구분
    구분한 문자를 List에 저장
    오류처리 : 5자이하

  • 최대 사다리 높이 입력
    숫자를 입력받음
    오류처리 : 숫자가 아닌 문자를 입력 받았을 때 / 0이하의 수일때

사다리 생성

  • 0과 1을 랜덤으로 생성
  • 앞의 값이 0인지 1인지 판별
  • 0일 경우 1,0 모두 가능 / 1일 경우 0만 가능
  • 0일때는 " " / 1일때는 "-----" 배열에 저장
  • 사다리 높이만큼 생성
  • 가로 사다리들의 리스트 생성

출력

  • 참여한 사람 이름 출력
    5 - (이름의 길이) 만큼 공백을 포함하여 출력

  • 사다리 출력
    이름 아래에 세로 사다리가 오도록 공백 출력
    | 사이에 가로 사다리 출력

사다리 2단계

요구사항

  • 결과를 입력
    문자 형태로 사람이름을 모두 입력받음
    쉼표를 기준으로 입력받은 문자를 구분
    구분한 문자를 List에 저장

#결과 출력

  • all일때와
  • 참여자 한사람을 지정할때

실행결과 영상

https://drive.google.com/drive/folders/1NSHvgaqJSKyRHoz37W8AvLPZxcdsHQ0y?usp=drive_link

추가 구현

  • 이름 입력시 2개 이상 입력하고록 하였습니다
  • 결과 입력시 이름의 개수보다 많을경우 오류처리 하도록 하였습니다

에러처리

  • 이름 입력시 : 5자 초과, 2개 이상 입력
  • 높이 입력시 : 1이하의 숫자 입력시, 문자 입력시
  • 결과 입력시 : 5자초과, 이름의 개수보다 많을때 오류처리

신경쓴 부분

  • 저번에는 코드를 다 완성하고 기능 별로 커밋을 했는데 이번에는 기능을 하나 구현하면 바로 커밋 해보려고 해봤습니다.
  • 오류처리 할때 구현하려는 의도를 생각해서 제가 필요하다고 생각하는 방향으로 해봤습니다.(ex : 참여자는 2명이상이여야 합니다)

고민한 부분

  • tdd가 익숙치 않고 구현하는데도 계속 자잘한 문제가 생겨서 tdd를 제대로 하지 못했습니다...
  • 잘 짰다고 생각했는데 중간중간 문제가 뒤늦게 발견되어 시간이 많이 지체되었습니다...

구현 중 어려웠던 부분

  • 처음에 요구사항 분석시 잘못 분석해서 거의 완성했을 무렵에 코드를 다 수정해야했습니다.
  • 또한 1단계를 구현할때 근시안적으로 구현해서 2단계에서 많은 부분이 막혀 코드 전체를 수정했습니다.
  • 기능을 구현하고 바로 커밋하다보니 수정하거나 오류를 발견했을때 불편했습니다.
  • 구혀하는 것에 정신이 팔려 tdd는 시도해 보지 못했습니다.

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.

1 participant