search

Run-Us / Run-Us-server

Run Us 서버 레포지토리
2 stars 0 forks source link

[Feat/running ws controller] 작업 내역 중간 PR #12

Closed jeeheaG closed 5 days ago

jeeheaG commented 6 days ago

연관된 이슈를 적어주세요 📌

x

작업한 내용을 설명해주세요 ✔️

PM님의 전언에 따라 작업한 내용이 많진 않지만 정리해서 올립니다 테스트 작동도 못해봤다는 점 미리 말씀드립니다 . . .

일단 설계 좀 정리해놓고 개발해야할 것 같아서 따로 여기에 적어두고 했습니다. 개인 노션인데 참고로 같이 올립니다. (별거 없어요 사진에 있는게 다입니다) image

참고로 users경로가 들어간 건 러닝 전체에 영향을 끼친다기 보다는 사용자 개인에 한정된 액션이라 경로를 다르게 가져가려고 넣었습니다. 조정 가능합니다. 개발 사항은 아래와 같습니다.

완성된 코드가 아니라 엉성한 부분들이 있을 수 있어요. 양해부탁드립니다...

트러블 슈팅

x

리뷰어에게 하고 싶은 말을 적어주세요

  1. 엔티티 컬럼명 스네이크 케이스로 일일이 지정하신 것 같은데 아래 jpa 속성 안쓰시는 이유가 있으신가요? 자동으로 해줍니담 
    physical-strategy: org.hibernate.boot.model.naming.CamelCaseToUnderscoresNamingStrategy
  2. javadoc 작성 규칙이 있어야 할까요? 어느 단에는 꼭 쓰자 라던지...
  3. entity 패키지명 model로 하면 나중에 model 데이터클래스 만들어서 쓸 때 헷갈리지 않을까요? 원래 어떻게 쓰셨는지 궁금합니다
  4. 이건 저번 PR보다가 궁금해진건데 @ekgns33 equals 랑 hashcode 어노테이션 안쓰는 이유가 어떤건가요? 단순한 궁금증👀

감사합니닥... 급한 불 끄고 돌아올게요.. 아자아자 !

확인하기

ekgns33 commented 6 days ago
  1. 사용하면 편한데 컬럼명이 코드레벨이랑 다른게 있어서 컬럼표시하는편입니다.
  2. 원래는 비즈니스 로직쪽에 적는데 이번에는 public 메소드에는 기입하려고 노력중입니다.
  3. 사실 첫 단추를 잘못꿰서 ㅋㅋㅋㅋ 전 domain이라는 디렉토리명을 선호하는데 어떠신가여
  4. 별도의 로직이 없다면 어노테이션써도 상관없는데 저는 잘 안쓰네요(개인 기호)
jeeheaG commented 5 days ago

설명 감사합니다! domain이라는 이름 좋아요~~