AI-Funding / AI_Funding_BE

0 stars 1 forks source link

Feature/#16거래내역 DTO 추가 #31

Closed JeonHyerim86 closed 2 years ago

JeonHyerim86 commented 2 years ago

거래 내역 관련 dto를 추가하였습니다.

kwYoohae commented 2 years ago

현재 저희가 정의한 Json 형식은 다음과 같습니다.

{
    "trade_history" : [
        {
            "stock_name" : "종목 이름",
            "trade_date" : "거래 날짜",
            "total_price" : "거래 금액",
            "trade_type" : "거래 종류",
            "trade_amount" : "거래 수량",
            "current_price" : "단가",
            "trade_price" : "매수/도 가"
        }
    ]
}

하지만 DTO의 반환에서는 설정을 안해주어서 그냥 카멜케이스로 반환받게 됩니다.

그래서 Json으로 반환할때는 변환해주는 애너테이션이라던지 방법을 찾아서 하시면 좋을것 같습니다 😄

HoSeong0731 commented 2 years ago

또 코드 중복이 되더라도 Request와 Response를 1:1 매칭을 시킬 건지 N:N 매칭을 시킬 건지는 논의가 필요할 것 같네요 1 : 1매칭 코드 중복이 될 수 있으나, 1:1 대응이기 때문에 어떤 코드와 대입되는지 비교적 알기 쉽다.

다 : 다 매칭 코드 중복을 없앨 수 있으나, 해당 1 : N 관계인지 N : 1 관계인지에 따라 네이밍에 신경써줘야 한다.

정도의 장단점이 있을 것 같습니다!

HoSeong0731 commented 2 years ago

확인해보시고 이상 없는 것 같으면 Merge 부탁드립니다 :)

kwYoohae commented 2 years ago

@HoSeong0731 현재 해당 내용에 대해서 수정커밋이 없어서 머지를 못하는 상황입니다.

@JeonHyerim86 수정사항 반영 부탁드립니다.

HoSeong0731 commented 2 years ago

@JeonHyerim86 혜림님도 재리뷰 필요할 때는 https://github.com/AI-Funding/AI_Funding_BE/pull/34 해당 PR의 코멘트들 참고해주시면 감사합니다 :)