search

SeoulNative / naengboo

naengboo backend repo
2 stars 2 forks source link

Init project #3

Closed hoyeon94 closed 3 years ago

hoyeon94 commented 3 years ago

디렉터리 구조를 전체적으로 변경했습니다. 리뷰 부탁드립니다.

  1. database 디렉터리 구조 변경
    • 함수를 재사용성과 코드의 가독성을 높이기 위해, 각 모델 별로 data access object를 추가했습니다.
    • views/refrigerators.py, database/daos/ingredients_dao.py 를 참고해주세요.
  2. services 디렉터리 추가
livlikwav commented 3 years ago

저도 flask 안한지 꽤 되어서.. 제대로된 리뷰는 복습을 좀 하고 하도록 하겠습니다..

패키지가 추가되는 것 같은데, 관련해서 패키지 구조 설계를 issue에 문서화하는 것은 어떨까요?

flask-restx 관련해서 업데이트가 잘 되고 있는 boilerplate가 있는 것 같아 링크 추천드립니다. https://github.com/X1Zeth2X/flask-restx-boilerplate

do168 commented 3 years ago

/database 디렉토리를 살펴보았는데 model 와 dao 가 안에 포함돼있습니다. services 레이어처럼 보다 좀 더 독립적으로 구조화해야한다고 생각하는데 여러분들은 어떻게 생각하시나요?

hoyeon94 commented 3 years ago

/database 디렉토리를 살펴보았는데 model 와 dao 가 안에 포함돼있습니다. services 레이어처럼 보다 좀 더 독립적으로 구조화해야한다고 생각하는데 여러분들은 어떻게 생각하시나요?

우찬님 말씀대로 dao들은 database 디렉토리 내에 있을 필요는 없을 것 같습니다. model의 경우에는 스키마 변경 외에 접근할 일이 없고 DB의 설정값에 가깝다고 보면 될 것 같아서, 데이터베이스를 연결하고 초기화하는 database 디렉토리 내에 포함되어 있어도 괜찮을 것 같다고 생각하는데, 어찌 보시나요?

coloryourlife commented 3 years ago

model은 말씀하신 것처럼 database 디렉터리 안에 포함되어도 괜찮을 것 같습니다.