Skip to content

Conversation

@include42
Copy link

@include42 include42 commented Apr 10, 2020

안녕하세요. 카프카입니다.

치킨집 미션 제출합니다.

마지막에 시간이 부족하여, 미구현된 부분을 todo로 체크해 두었습니다.
기본적인 테스트는 구현하였으나, MVC 구조가 아직 미완성인 상태입니다.
몇몇 기능 역시 작동하지 않으나, 기본적인 기능 수행은 되는 상태입니다.

감사합니다.

README 기능/구현 요구사항 작성
Money 금액 계산을 위해 쓰이며, 곱셈 연산을 지원하는 금액 클래스 구현
MoneyTest Money 클래스의 생성, 예외, 연산 및 getter 테스트
MoneyException 금액과 관련된 예외처리 발생 시 호출되는 예외 클래스 구현
Money 내부의 값 형식을 정수로 변경, getter의 반환형식 변경, 뺄셈 연산 추가
MoneyTest 테스트 코드의 입력 형식 int로 변경, 뺄셈 테스트코드 추가
MenuException 메뉴와 관련된 예외처리 발생 시 호출되는 예외 클래스
Menu 생성자를 package 내에서만 호출하도록 변경, price를 Money 클래스로 변경
MenuRepository 번호를 받아 적절한 메뉴를 반환하는 valueOf 메서드 추가
domain.menu 메뉴와 관련된 클래스를 모아두는 패키지
domain.table 테이블과 관련된 클래스를 모아두는 패키지
domain.money 금액 연산과 관련된 클래스를 모아두는 패키지
ALL java 파일을 적절한 패키지로 이동
README Menu 및 MenuRepository에 대한 구현/테스트 사항 추가
README 예외처리 관련 사항 추가
MoneyException 삭제
IllegalMoneyException 기존 MoneyException의 역할 담당
MenuException 삭제
IllegalMenuException 기존 MenuException의 역할 담당
Menu toString, 생성, validate 체크 기능 구현
MenuTest 생성, 예외처리, toString 테스트 코드 구현
ALL 예외처리 이름 변경에 따른 리팩토링
MenuRepositoryTest MenuRepository의 valueOf 작동 및 예외 테스트 구현
README Customer 클래스에 대한 구현/예외/테스트 내용 추가
README Customer 클래스가 add를 실행할 경우에 대한 예외처리 추가
Customer 메뉴의 리스트를 감싸고 금액계산, 메뉴추가를 수행하는 클래스 구현
CustomerTest Customer의 생성, add, calculateMoney, 예외처리 테스트
Menu price의 getter 구현
MoneyTest equals 연산에 대한 테스트 구현
MenuRepositoryTest menus 메서드에 대한 테스트 구현
MoneyTest todo 작성
README Customer에 대한 기획 변경, 추가된 테스트 코드 반영
Customer 기존의 List를 Menu와 그 개수의 맵으로 수정.
Money 음수가 아니라면 예외가 발생하지 않도록 수정
ALL 변경사항에 맞춘 리팩토링
README Table, TableRepository, Store에 대한 구현 요구사항 추가
Table order, calculate 추가
TableTest 테이블 클래스의 생성, 예외, 기능수행 테스트 구현
Payment 현금/카드 등 지불수단에 대한 Enum 클래스 구현
Menu isChicken 메서드 추가
Customer Payment를 받아 이에 따른 할인 적용
Table 만약 치킨 메뉴가 10개 이상일 시 할인 적용
Table 주문 개수가 0 이하이거나 100 이상일 경우 IllegalMenuException 발생
TableTest 예외처리에 대한 테스트 구현
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