baseline code review

[choi4624]

코드 못읽는 분들을 위한 도움 되는 글

https://mangkyu.tistory.com/125 생성자 기반 의존성 주입 https://yuricoding.tistory.com/77 대체 왜 폴더가 model controller service repository 이모양이죠 https://javaengine.tistory.com/entry/JPA-%EC%82%AC%EC%9A%A9%EB%B2%95-JpaRepository jpaRepository

https://mangkyu.tistory.com/150 의존성 주입은 왜 스프링에서 잘 써야 하나요

자바 상속

** 참고

Testing Method를 만들고 그에 맞추어서 서비스를 만들 것인지 메소드를 만들고 그에 맞추어 테스팅 메소드를 할 것인지 등도 나중에 정해야 합니다.

1. 코드에서 보여준 기본적인 객체들이랑 속성에 대해서 어떤 생각이 드나요?
2. 클래스들에 들어가 있는 값들은 이거 보다는 이거가 낫지 않을까요?
3. 여기에서 각 클래스 별로 어떤 역할을 해야 할 것 같나요?
4. 웹 디자인은 어떻게 하면 좋을까요?

[jookbooin]

Table 클래스

boolean on : 손님 있는지 없는지 확인용으로 추가 해야할 것 같습니다. 테이블별 List \

나 List\ menuId , Map (menuID , 수량 )등등 있으면 좋을 것 같습니다. -> Receipt 클래스를 대체할 수 있지 않을까요 Table클래스 내부에는 순수 table 정보만 두고 Service쪽에서 배정을 하는 로직을 구현하면 되지 않을까 싶습니다.

enum userClass

AppUser에 관리자인지 고객인지 구별하는 변수 int adminCheck ; 하나 두고 (0 ,1 같이 관리자 , 고객 구별) enum userClass에 유저 등급 정보만 두는게 어떨까요

Menu클래스

• kCal ,weight 은 굳이 필요 없을 것 같습니다.

Worker

Worker -> admin 으로 변경하는 것이 훨씬 파악하기 쉬워 보입니다.

[jookbooin]

Table 자동 배정

Table 자동 배정할 떄 큐를 통해 구현한다면 순서배정 담당하는 클래스 하나 필요하지 않을까요? TableQueue

1. seats 별 자료구조 : List or Queue 인터페이스
   ...

   
   관리자 -> 테이블 사용 (테이블 사용 on , off) 될때 해당 seats 범위 별로 해당 큐에 들어오고 나가는 형식으로 구성하면되지 않을까 싶습니다
   
   

   빈으로 등록하면 Reservation( book ) 대기리스트도 함께 보관할 수 있지 않을까요

   choi4624 commented 1 year ago

   baseline 코드는 망했어. 이제 다른 코드로 바뀐다.

   • © Githubissues.
   • Githubissues is a development platform for aggregating issues.

[choi4624]

baseline 코드는 망했어. 이제 다른 코드로 바뀐다.