---
# System prepended metadata

title: review

---

# review

1.  통합 
    1. config 부분.
        1. 현재 docker container를 실행할 때 환경변수를 지정하여 config를 완성하고 있는데 .env파일을 사용하는게 적절한지 환경변수를 직접 불러오는게 적절한지 고민입니다.
2.  채팅.
    1. 채팅 로그를 어떤식으로 남기는게 좋을지. ([https://github.com/STOVE-Milk/steam-clone/blob/feature/back-chat/src/backend/chat/chatServer.go#L238](https://github.com/STOVE-Milk/steam-clone/blob/feature/back-chat/src/backend/chat/chatServer.go#L238))
        1. 웹 소켓 클라이언트와 룸을 총괄적으로 다루는 chatServer.go에서 현재 채팅 로그를 mongo DB에 남기는 중입니다. 이런 식으로 진행한다면 채팅이 넘어가는 로직에서 에러가 발생한다고 해도 로그가 남을 듯 한데 맞는 방법인지 궁금합니다.
    2. 레디스 룸 관리
        1. 레디스를 통해 룸을 효율적으로 관리하고자 하는데 레디스가 가져야할 데이터의 범위를 짐작하기가 어렵습니다. 어떠한 기준을 가지고 레디스에 값을 저장해야할 지 알고싶습니다. 
3.  상점
    1. controller와 repository 활용이 적절한지.
        1. 라우팅되는 부분에서는 반환되는 response객체에 값을 대입하는 부분만 보여주려 노력하였고 로직은 controller 부분에서 처리하고자 노력하였습니다. controller를 사용하는 구조에서 제가 생각한 개념이 적절하지 않은 부분이 있는지 궁금합니다.
            1. routing([https://github.com/STOVE-Milk/steam-clone/blob/feature/back-store/src/backend/store/server.go](https://github.com/STOVE-Milk/steam-clone/blob/feature/back-store/src/backend/store/server.go))
            2. controller([https://github.com/STOVE-Milk/steam-clone/blob/feature/back-store/src/backend/store/controller/game.go](https://github.com/STOVE-Milk/steam-clone/blob/feature/back-store/src/backend/store/controller/game.go))
            3. repository([https://github.com/STOVE-Milk/steam-clone/blob/feature/back-store/src/backend/store/repository/repository.go](https://github.com/STOVE-Milk/steam-clone/blob/feature/back-store/src/backend/store/repository/repository.go))
        2. routing, controller, repository에서 목표가 같은 함수 이름을 동일하게 지정하였습니다. 함수이름이 혼란이 생기기 쉬울 듯 하지만 또한 일관성있어 보기 편할듯 하기도 합니다. 모듈마다 함수이름을 같게하는게 적절한지, 아니라면 어떠한 방식이 좋을지 궁금합니다.