refactor(coin): coin 도메인 리팩토링 및 테스트 코드 작성

[hw9402]

trafficstars

close #92

리팩토링도 리팩토링이지만 테스트 코드를 중점적으로 봐주시면 감사하겠습니다! 양이 많기도 했고 시간도 조금 부족했어서 하드 코딩한 부분들이 좀 있을 겁니다..

진행한 작업

• 예외 처리를 클래스가 아닌 에러 코드를 주입하도록 변경했습니다
• controller 패키지를 presentation 패키지로 이름을 변경했습니다
• 서비스 레이어를 비즈니스 레이어와 구현 레이어로 분리했습니다
• 조건에 따른 예외 처리를 Validator에서 하도록 변경했습니다
• controller 및 service를 Command와 Query로 분리했습니다
• 테스트 코드를 작성했습니다

이외에

코드 리뷰 진행 중 Read-only Transaction에서 save메서드를 사용한 기존 코드로 인해 아래와 같은 에러가 떴던 부분을 수정했습니다!

java.sql.SQLException: Connection is read-only. Queries leading to data modification are not allowed

[qlido]

TradeWithoutTradeStatusAndCoinAccountId 가 정확하지만 한번에 읽히지 않는다는 단점이 있는거 같아요. 주식의 호가처럼 갯수와 가격을 전달하는 역할을 하는 VO의 이름을 다시 정하는 것도 코드 가독성에 많은 도움이 될 것 같아요.

[hw9402]

TradeWithoutTradeStatusAndCoinAccountId 가 정확하지만 한번에 읽히지 않는다는 단점이 있는거 같아요. 주식의 호가처럼 갯수와 가격을 전달하는 역할을 하는 VO의 이름을 다시 정하는 것도 코드 가독성에 많은 도움이 될 것 같아요.

다시 한 번 생각해보겠습니다!